Merge lp:~osomon/webbrowser-app/increase-test-timeout into lp:webbrowser-app

Proposed by Olivier Tilloy
Status: Rejected
Rejected by: Olivier Tilloy
Proposed branch: lp:~osomon/webbrowser-app/increase-test-timeout
Merge into: lp:webbrowser-app
Diff against target: 30 lines (+3/-3)
1 file modified
tests/unittests/qml/tst_UbuntuWebView02.qml (+3/-3)
To merge this branch: bzr merge lp:~osomon/webbrowser-app/increase-test-timeout
Reviewer Review Type Date Requested Status
Olivier Tilloy Disapprove
system-apps-ci-bot continuous-integration Needs Fixing
Review via email: mp+299352@code.launchpad.net

Commit message

Increase significantly the timeout for a bunch of flaky tests,
in the hope that it will make them more robust.

To post a comment you must log in.
Revision history for this message
system-apps-ci-bot (system-apps-ci-bot) wrote :

FAILED: Continuous integration, rev:1501
https://jenkins.canonical.com/system-apps/job/lp-webbrowser-app-ci/566/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/system-apps/job/build/865/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-0-fetch/865
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-1-sourcepkg/release=vivid+overlay/785
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-1-sourcepkg/release=xenial+overlay/785
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-1-sourcepkg/release=yakkety/785
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/787/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/787/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=yakkety/787
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=yakkety/787/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/787
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/787/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/787
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/787/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=yakkety/787
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=yakkety/787/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/787
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/787/artifact/output/*zip*/output.zip
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/787/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=yakkety/787
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=yakkety/787/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-apps/job/lp-webbrowser-app-ci/566/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Olivier Tilloy (osomon) wrote :

This obviously doesn’t address the issue, the problem is probably not a too short timeout.

review: Disapprove

Unmerged revisions

1501. By Olivier Tilloy

Increase significantly the timeout for a bunch of flaky tests,
in the hope that it will make them more robust.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/unittests/qml/tst_UbuntuWebView02.qml'
2--- tests/unittests/qml/tst_UbuntuWebView02.qml 2015-08-27 13:09:40 +0000
3+++ tests/unittests/qml/tst_UbuntuWebView02.qml 2016-07-06 20:16:39 +0000
4@@ -87,7 +87,7 @@
5
6 function test_no_contextual_actions() {
7 webview.loadHtml(root.htmlWithHyperlink, "file:///")
8- tryCompare(webview, "loading", false)
9+ tryCompare(webview, "loading", false, 20000)
10 rightClickWebview()
11 compare(getContextMenu(), null)
12 }
13@@ -95,7 +95,7 @@
14 function test_contextual_actions() {
15 webview.contextualActions = actionList
16 webview.loadHtml(root.htmlWithHyperlink, "file:///")
17- tryCompare(webview, "loading", false)
18+ tryCompare(webview, "loading", false, 20000)
19 rightClickWebview()
20 compare(getContextMenu().actions, actionList)
21 compare(webview.contextualData.href, "http://example.org/")
22@@ -108,7 +108,7 @@
23 action1.enabled = false
24 action2.enabled = false
25 webview.loadHtml(root.htmlWithHyperlink, "file:///")
26- tryCompare(webview, "loading", false)
27+ tryCompare(webview, "loading", false, 20000)
28 rightClickWebview()
29 compare(getContextMenu(), null)
30 action1.enabled = true

Subscribers

People subscribed via source and target branches

to status/vote changes: