Closed Bug 1010415 Opened 11 years ago Closed 10 years ago

Intermittent notification_resend_test.js | mozChromeNotifications: Sending one notification, remove from tray, resend

Categories

(Firefox OS Graveyard :: Gaia::System, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
2.1 S4 (12sep)

People

(Reporter: RyanVM, Assigned: gerard-majax)

References

Details

(Keywords: intermittent-failure, Whiteboard: [systemsfe])

Attachments

(2 files, 1 obsolete file)

Attached image test screenshot
https://tbpl.mozilla.org/php/getParsedLog.php?id=39665712&tree=B2g-Inbound b2g_ubuntu64_vm b2g-inbound opt test gaia-integration on 2014-05-14 09:19:40 PDT for push bd5ec747c11e slave: tst-linux64-spot-617 09:35:01 INFO - TEST-START | mozChromeNotifications: Sending one notification, remove from tray, resend 09:35:33 INFO - Screenshot: 09:35:33 INFO - TEST-UNEXPECTED-FAIL | /builds/slave/test/gaia/apps/system/test/marionette/notification_resend_test.js | mozChromeNotifications: Sending one notification, remove from tray, resend 09:35:33 INFO - TEST-END | mozChromeNotifications: Sending one notification, remove from tray, resend
Blocks: 960072
Disabling this test so we can turn on Gij on TBPL: https://github.com/mozilla-b2g/gaia/commit/a12ea45a5aca843105ea3188df146d48e84cf673 Adding leave-open to prevent auto-closing of this bug by sheriffs. Here is a recent run failure of this test: https://tbpl.mozilla.org/php/getParsedLog.php?id=43553081&tree=B2g-Inbound Ni? on Alex for awareness.
Flags: needinfo?(lissyx+mozillians)
Keywords: leave-open
Originally I had thought that the "Error: Polling socket recv() timeout!" error was framework related, but I've seen this same test fail multiple times with that, and other tests don't seem to fail so much. So we should investigate to see what that means.
I don't understand, is it an intermittent test or a permanent fail ?!
Flags: needinfo?(lissyx+mozillians)
(In reply to Alexandre LISSY :gerard-majax from comment #13) > I don't understand, is it an intermittent test or a permanent fail ?! This is an intermittent test. We currently have way too many intermittent tests and need to get down to about 1% to re-enable tests on TBPL. In order to do so we will be disabling a lot of tests unfortunately. After we are back up and running on TBPL I will be more than happy to help people debug these issues. Regarding this specific test, I'm still not sure if it's an issue with the test, or a problem with the harness. We'll need to do a bit of investigation to figure that out.
Depends on: 1037371
(In reply to Kevin Grandon :kgrandon from comment #14) > (In reply to Alexandre LISSY :gerard-majax from comment #13) > > I don't understand, is it an intermittent test or a permanent fail ?! > > This is an intermittent test. We currently have way too many intermittent > tests and need to get down to about 1% to re-enable tests on TBPL. In order > to do so we will be disabling a lot of tests unfortunately. After we are > back up and running on TBPL I will be more than happy to help people debug > these issues. > > Regarding this specific test, I'm still not sure if it's an issue with the > test, or a problem with the harness. We'll need to do a bit of investigation > to figure that out. Thanks. I spent some time looking at the test, I found some weirdness. I have no idea if it could explain the issue, but I filed bug 1037371 and I have a PR that makes use of as much as possible of addEventListener() inside executeAsyncScript() instead of mixing executeScript() and executeAsyncScript(). We'll see if it helps. Meanwhile, I was not able to reproduce the issue locally.
See Also: → 1037716
Blocks: 1035939
No longer blocks: 960072
Attached file [Gaia PR] re-enable (obsolete) —
Let's see if this test is still intermittent in tbpl.
Michael, it seems we now have a permafail. At least if we reproduce locally, we can investigate ... :) https://tbpl.mozilla.org/?rev=99315d2bd079b7d14ae7b40b4edb6d5fbe45f4a4&tree=Gaia-Try&showall=1
1) mozChromeNotifications: Sending one notification, remove from tray, resend: JavaScriptError: (17) NotFoundError: Node was not found Remote Stack: execute_async_script @undefined, line undefined inline javascript, line 72 src: " nodes = container.removeChild(nodes[0]).querySelectorAll(selector);" at Error.MarionetteError (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/marionette-client/lib/marionette/error.js:67:13) at Object.Client._handleCallback (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/marionette-client/lib/marionette/client.js:476:19) at /home/alex/codaz/Mozilla/b2g/gaia/node_modules/marionette-client/lib/marionette/client.js:510:21 at TcpSync.send (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/marionette-client/lib/marionette/drivers/tcp-sync.js:153:10) at Object.send (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/marionette-client/lib/marionette/client.js:457:36) at Object.Client._sendCommand (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/marionette-client/lib/marionette/client.js:503:19) at Object._executeScript (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/marionette-client/lib/marionette/client.js:1468:19) at Object.executeAsyncScript (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/marionette-client/lib/marionette/client.js:1249:19) at Context.<anonymous> (/home/alex/codaz/Mozilla/b2g/gaia/apps/system/test/marionette/notification_resend_test.js:192:20) at Test.Runnable.run (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/mocha/lib/runnable.js:196:15) at Runner.runTest (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/mocha/lib/runner.js:373:10) at /home/alex/codaz/Mozilla/b2g/gaia/node_modules/mocha/lib/runner.js:451:12 at next (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/mocha/lib/runner.js:298:14) at /home/alex/codaz/Mozilla/b2g/gaia/node_modules/mocha/lib/runner.js:308:7 at next (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/mocha/lib/runner.js:246:23) at /home/alex/codaz/Mozilla/b2g/gaia/node_modules/mocha/lib/runner.js:270:7 at done (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/mocha/lib/runnable.js:185:5) at /home/alex/codaz/Mozilla/b2g/gaia/node_modules/mocha/lib/runnable.js:199:9 at Object.executeHook (/home/alex/codaz/Mozilla/b2g/gaia/node_modules/marionette-client/lib/marionette/client.js:369:18) at process._tickCallback (node.js:419:13)
Attached file Fix and re-enable test
Michael, this simple change does fix the test locally :)
Assignee: nobody → lissyx+mozillians
Attachment #8476054 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #8485534 - Flags: review?(mhenretty)
Blocks: 1048138
So the gaia try with ~50 retriggers expose no failure at all of the notification test, but there are other failures.
Flags: needinfo?(mhenretty)
Attachment #8485534 - Flags: review?(mhenretty) → review+
Flags: needinfo?(mhenretty)
master: https://github.com/mozilla-b2g/gaia/commit/c5ad0797cc83b9a6e6519e19fa8f401c021979bb I'm going to look into some of those other notification errors.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [systemsfe]
Target Milestone: --- → 2.1 S4 (12sep)
Removing leave-open keyword from resolved bugs, per :sylvestre.
Keywords: leave-open
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: