Cordova Ubuntu Tests

Merge lp://qastaging/~vrruiz/cordova-ubuntu-tests/ap-better-reporting into lp://qastaging/cordova-ubuntu-tests

  1. ap-better-reporting
  2. Merge into trunk
Proposed by Víctor R. Ruiz
Status: Merged
Approved by: Robert Bruce Park
Approved revision: 29
Merged at revision: 25
Proposed branch: lp://qastaging/~vrruiz/cordova-ubuntu-tests/ap-better-reporting
Merge into: lp://qastaging/cordova-ubuntu-tests
Diff against target: 806 lines (+159/-91)
26 files modified
tests/autopilot/cordova_ubuntu/tests/__init__.py (+30/-12)
tests/autopilot/cordova_ubuntu/tests/test_mobilespec.py (+37/-26)
tests/autopilot/tools/qml/main-local.qml (+1/-1)
www/autotest/html/JUnitXmlReporter.js (+5/-5)
www/autotest/pages/accelerometer.html (+2/-2)
www/autotest/pages/all.html (+3/-3)
www/autotest/pages/battery.html (+2/-2)
www/autotest/pages/bridge.html (+2/-2)
www/autotest/pages/camera.html (+2/-2)
www/autotest/pages/capture.html (+2/-2)
www/autotest/pages/compass.html (+2/-2)
www/autotest/pages/contacts.html (+2/-2)
www/autotest/pages/datauri.html (+2/-2)
www/autotest/pages/device.html (+2/-2)
www/autotest/pages/file.html (+4/-4)
www/autotest/pages/filetransfer.html (+4/-4)
www/autotest/pages/geolocation.html (+2/-2)
www/autotest/pages/globalization.html (+2/-2)
www/autotest/pages/media.html (+2/-2)
www/autotest/pages/network.html (+2/-2)
www/autotest/pages/notification.html (+2/-2)
www/autotest/pages/platform.html (+2/-2)
www/autotest/pages/splashscreen.html (+2/-2)
www/autotest/pages/storage.html (+2/-2)
www/autotest/pages/whitelist.html (+2/-2)
www/autotest/test-runner.js (+39/-0)
To merge this branch: bzr merge lp://qastaging/~vrruiz/cordova-ubuntu-tests/ap-better-reporting
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Robert Bruce Park (community) Approve
Review via email: mp+191177@code.qastaging.launchpad.net

Commit message

Skip unsupported API tests.

Description of the change

Skip unsupported API tests.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:24
http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-ci/16/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-saucy-amd64-ci/16
    SUCCESS: http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-saucy-armhf-ci/16

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/cordova-ubuntu-tests-ci/16/rebuild

review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:25
http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-ci/18/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-saucy-amd64-ci/18/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-saucy-armhf-ci/18/console

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/cordova-ubuntu-tests-ci/18/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:26
http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-ci/19/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-saucy-amd64-ci/19/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-saucy-armhf-ci/19/console

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/cordova-ubuntu-tests-ci/19/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:27
http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-ci/20/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-saucy-amd64-ci/20/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-saucy-armhf-ci/20/console

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/cordova-ubuntu-tests-ci/20/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:28
http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-ci/21/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-saucy-amd64-ci/21
    SUCCESS: http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-saucy-armhf-ci/21

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/cordova-ubuntu-tests-ci/21/rebuild

review: Approve (continuous-integration)
Revision history for this message
Robert Bruce Park (robru) wrote :

What is the purpose of get_www_dir_path? Unlike get_{cordova_container,persistent}_path, it returns the same value regardless of what happens in the if statement, making it useless. Do you intend to do something about the www dir not existing? It should be guaranteed to exist due to the dependencies of the deb package. Please drop that function and reference self.WWW_DIR_PATH wherever you would call self.get_www_dir_path() from.

review: Needs Fixing
Revision history for this message
Víctor R. Ruiz (vrruiz) wrote :

Robert: To be able to run the tests on the development branch, wherever the developer is working.

Revision history for this message
Víctor R. Ruiz (vrruiz) wrote :

Robert: Ok, checked carefully and that method is not used anymore. Removed.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Autolanding.
Unapproved changes made after approval.
http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-autolanding/11/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-saucy-amd64-autolanding/11
    SUCCESS: http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-saucy-armhf-autolanding/11

review: Needs Fixing (continuous-integration)
Revision history for this message
Robert Bruce Park (robru) wrote :

Thanks Victor! Looks good now ;-)

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

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