Merge lp://staging/~laney/autopilot-qt/no-hardcode-xpathselect into lp://staging/autopilot-qt/wily

Proposed by Christopher Lee
Status: Needs review
Proposed branch: lp://staging/~laney/autopilot-qt/no-hardcode-xpathselect
Merge into: lp://staging/autopilot-qt/wily
Diff against target: 13 lines (+1/-2)
1 file modified
debian/control (+1/-2)
To merge this branch: bzr merge lp://staging/~laney/autopilot-qt/no-hardcode-xpathselect
Reviewer Review Type Date Requested Status
Max Brustkern (community) Approve
Marco Trevisan (Treviño) (community) Approve
PS Jenkins bot continuous-integration Pending
Review via email: mp+268987@code.staging.launchpad.net

This proposal supersedes a proposal from 2015-08-24.

Description of the change

Don't hardcode the xpathselect dependency - it should be auto detected.

Note that I can't verify this because the tests fail

FAIL! : tst_Introspection::test_introspect(/) Compared values are not the same
   Actual (first_object.state.value(firstResultPropertyName)): QVariant(QVariantList,<value not representable as string>)
   Expected (firstResultPropertyValue) : QVariant(QVariantList,<value not representable as string>)
   Loc: [../../../tests/unittests/tst_introspection.cpp(265)]
FAIL! : tst_Introspection::test_introspect(//QWidget[id=6]) Compared values are not the same
   Actual (resultList.count()): 0
   Expected (resultCount) : 1
   Loc: [../../../tests/unittests/tst_introspection.cpp(259)]
FAIL! : tst_Introspection::test_introspect(//QPushButton[id=9]) Compared values are not the same
   Actual (resultList.count()): 0
   Expected (resultCount) : 1
   Loc: [../../../tests/unittests/tst_introspection.cpp(259)]

...please could you help me understand why and fix this?

Then branch off this branch and upload to silo 019.

To post a comment you must log in.
Revision history for this message
Iain Lane (laney) wrote : Posted in a previous version of this proposal

If you try to fix this, please test against the xpathselect in silo 019 currently.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal

FAILED: Continuous integration, rev:93
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~laney/autopilot-qt/no-hardcode-xpathselect/+merge/268943/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/autopilot-qt-ci/30/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/autopilot-qt-wily-amd64-ci/1/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/autopilot-qt-wily-armhf-ci/1/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/autopilot-qt-wily-i386-ci/1/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/autopilot-qt-ci/30/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Christopher Lee (veebers) wrote :

Please see this branch for test fixes: https://code.launchpad.net/~canonical-platform-qa/autopilot-qt/wily_fix-unittests/+merge/268990

Not sure if it would sense to just merge that into this branch?

Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

+1

review: Approve
Revision history for this message
Max Brustkern (nuclearbob) wrote :

LGTM

review: Approve

Unmerged revisions

93. By Iain Lane

Drop hardcoded xpathselect dep, it's not dlopened.

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