Merge lp://staging/~nskaggs/ubuntu-sdk-tutorials/update-currency-tests into lp://staging/ubuntu-sdk-tutorials

Proposed by Nicholas Skaggs
Status: Merged
Merged at revision: 137
Proposed branch: lp://staging/~nskaggs/ubuntu-sdk-tutorials/update-currency-tests
Merge into: lp://staging/ubuntu-sdk-tutorials
Diff against target: 781 lines (+167/-447)
14 files modified
getting-started/CurrencyConverter/Main.qml (+0/-2)
getting-started/CurrencyConverter/Makefile (+6/-6)
getting-started/CurrencyConverter/tests/acceptance/currencyconverter/__init__.py (+76/-2)
getting-started/CurrencyConverter/tests/acceptance/currencyconverter/tests/__init__.py (+37/-43)
getting-started/CurrencyConverter/tests/acceptance/currencyconverter/tests/test_currencyconverter.py (+29/-74)
getting-started/CurrencyConverter/tests/acceptance/run (+12/-0)
getting-started/CurrencyConverter/tests/autopilot/CurrencyConverter/emulators/__init__.py (+0/-17)
getting-started/CurrencyConverter/tests/autopilot/CurrencyConverter/emulators/main_window.py (+0/-44)
getting-started/CurrencyConverter/tests/autopilot/CurrencyConverter/emulators/ubuntusdk.py (+0/-165)
getting-started/CurrencyConverter/tests/autopilot/CurrencyConverter/test_main.py (+0/-25)
getting-started/CurrencyConverter/tests/autopilot/run (+0/-12)
getting-started/CurrencyConverter/tests/integration/tst_currencyconverter.qml (+6/-6)
getting-started/CurrencyConverter/tests/unit/tst_convert.qml (+1/-1)
getting-started/CurrencyConverter/tests/unit/tst_hellocomponent.qml (+0/-50)
To merge this branch: bzr merge lp://staging/~nskaggs/ubuntu-sdk-tutorials/update-currency-tests
Reviewer Review Type Date Requested Status
Daniel Holbach (community) Needs Fixing
Review via email: mp+252776@code.staging.launchpad.net

Description of the change

update tests for currency converter

To post a comment you must log in.
Revision history for this message
Daniel Holbach (dholbach) wrote :

daniel@daydream:/tmp/ubuntu-sdk-tutorials/getting-started/CurrencyConverter$ sudo apt-get install python3-autopilot autopilot-desktop[sudo] password for daniel:
Reading package lists... Done
Building dependency tree
Reading state information... Done
autopilot-desktop is already the newest version.
python3-autopilot is already the newest version.
0 to upgrade, 0 to newly install, 0 to remove and 0 not to upgrade.
daniel@daydream:/tmp/ubuntu-sdk-tutorials/getting-started/CurrencyConverter$ make test
qmltestrunner -input tests/unit
file:///tmp/ubuntu-sdk-tutorials/getting-started/CurrencyConverter/tests/unit/tst_convert.qml:19:5: CurrencyConverter is not a type
         CurrencyConverter {
         ^
********* Start testing of qmltestrunner *********
Config: Using QtTest library 5.4.0, Qt 5.4.0 (x86_64-little_endian-lp64 shared (dynamic) release build; by GCC 4.9.2)
QWARN : qmltestrunner::tst_convert::compile()
  /tmp/ubuntu-sdk-tutorials/getting-started/CurrencyConverter/tests/unit/tst_convert.qml produced 1 error(s):
    /tmp/ubuntu-sdk-tutorials/getting-started/CurrencyConverter/tests/unit/tst_convert.qml:19,5: CurrencyConverter is not a type
  Working directory: /tmp/ubuntu-sdk-tutorials/getting-started/CurrencyConverter
  View: QQuickView, import paths:
    '/home/daniel/.local/share'
    '/usr/share'
    '/usr/local/share'
    '/usr/share/gnome'
    '/usr/share/ubuntu'
    '/tmp/ubuntu-sdk-tutorials/getting-started/CurrencyConverter'
    '/usr/lib/x86_64-linux-gnu/qt5/bin'
    '/usr/lib/x86_64-linux-gnu/qt5/qml'
  Plugin paths:
    '.'

FAIL! : qmltestrunner::tst_convert::compile() CurrencyConverter is not a type
   Loc: [/tmp/ubuntu-sdk-tutorials/getting-started/CurrencyConverter/tests/unit/tst_convert.qml(19)]
Totals: 0 passed, 1 failed, 0 skipped, 0 blacklisted
********* Finished testing of qmltestrunner *********
Makefile:32: recipe for target 'test' failed
make: *** [test] Error 1
daniel@daydream:/tmp/ubuntu-sdk-tutorials/getting-started/CurrencyConverter$

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

make test works now. Merging with dholbach's blessing.

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