Merge lp://staging/~mzanetti/ubuntu-sdk-tutorials/add-qmltests into lp://staging/ubuntu-sdk-tutorials

Proposed by Michael Zanetti
Status: Merged
Merged at revision: 55
Proposed branch: lp://staging/~mzanetti/ubuntu-sdk-tutorials/add-qmltests
Merge into: lp://staging/ubuntu-sdk-tutorials
Diff against target: 218 lines (+192/-0)
4 files modified
getting-started/CurrencyConverter/CurrencyConverter.qml (+1/-0)
getting-started/CurrencyConverter/tests/qmltests/helpers.js (+14/-0)
getting-started/CurrencyConverter/tests/qmltests/tst_CurrencyConverter.qml (+111/-0)
getting-started/CurrencyConverter/tests/qmltests/tst_convert.qml (+66/-0)
To merge this branch: bzr merge lp://staging/~mzanetti/ubuntu-sdk-tutorials/add-qmltests
Reviewer Review Type Date Requested Status
David Planella Approve
Nicholas Skaggs Approve
Juhapekka Piiroinen Pending
Review via email: mp+171907@code.staging.launchpad.net

Commit message

added qmltest tutorial

To post a comment you must log in.
56. By Michael Zanetti

indentation

Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

I asked Michael to help out in showing us an example for using the qmltestrunner. He agreed and promptly produced this lovely example to go with the currencyConverter tutorial. I'll add an article for it as well and we should then be complete on testing qml apps :-)

review: Approve
57. By Michael Zanetti

fix comments

58. By Michael Zanetti

improve comments

Revision history for this message
Michael Zanetti (mzanetti) wrote :

Btw, It might make sense to package helpers.js up somewhere. Maybe even wrapping it in a UbuntuTestCase (see lp:unity8 for an example, search for UnityTestCase) and shipping that as part of some package like ubuntu-ui-toolkit-utils or whatever so people don't need to copy that file all over the place. This file can be extended over time with other useful features related to testing Ubuntu Touch QML apps.

Revision history for this message
David Planella (dpm) wrote :

This is really awesome, thanks Michael!

review: Approve
Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

I agree, helper.js can be added to https://code.launchpad.net/ubuntu-ui-toolkit

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches