Closed Bug 1381799 Opened 7 years ago Closed 6 years ago

Intermittent toolkit/components/extensions/test/xpcshell/test_ext_alarms.js | test_alarm_clear_non_matching_name - [test_alarm_clear_non_matching_name : 243] alarm was not removed - Expected: 1, Actual: 0 - false == true

Categories

(WebExtensions :: Untriaged, defect)

Product:
WebExtensions
Component:
Untriaged
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox63 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla63
Tracking Flags:
Tracking Status
firefox63 --- fixed

People

(Reporter: aryx, Assigned: robwu)

References

Details

(Keywords: intermittent-failure, Whiteboard: [stockwell unknown])

Attachments

(1 file)

Bug 1381799 - Fix intermittent test_alarm_clear_non_matching_name
46 bytes, text/x-phabricator-request
aswan
: review+
Details | Review 
https://treeherder.mozilla.org/logviewer.html#?job_id=115141713&repo=mozilla-inbound

[task 2017-07-18T08:28:31.120087Z] 08:28:31     INFO -  TEST-PASS | xpcshell.ini:toolkit/components/extensions/test/xpcshell/test_ext_alarms.js | test_alarm_clear_non_matching_name - [test_alarm_clear_non_matching_name : 251] alarm was not cleared - true == true
[task 2017-07-18T08:28:31.120472Z] 08:28:31  WARNING -  TEST-UNEXPECTED-FAIL | xpcshell.ini:toolkit/components/extensions/test/xpcshell/test_ext_alarms.js | test_alarm_clear_non_matching_name - [test_alarm_clear_non_matching_name : 243] alarm was not removed - Expected: 1, Actual: 0 - false == true
[task 2017-07-18T08:28:31.120736Z] 08:28:31     INFO -  resource://testing-common/ExtensionXPCShellUtils.jsm:handleResult:243
[task 2017-07-18T08:28:31.121038Z] 08:28:31     INFO -  resource://gre/modules/ExtensionUtils.jsm:runSafeSyncWithoutClone:61
[task 2017-07-18T08:28:31.121396Z] 08:28:31     INFO -  resource://gre/modules/ExtensionUtils.jsm:emit/promises<:256
[task 2017-07-18T08:28:31.121748Z] 08:28:31     INFO -  resource://gre/modules/ExtensionUtils.jsm:emit:255
[task 2017-07-18T08:28:31.122103Z] 08:28:31     INFO -  resource://gre/modules/Extension.jsm:receiveMessage:874
[task 2017-07-18T08:28:31.122517Z] 08:28:31     INFO -  /storage/sdcard/tests/xpc/head.js:_do_main:221
[task 2017-07-18T08:28:31.122865Z] 08:28:31     INFO -  /storage/sdcard/tests/xpc/head.js:_execute_test:550
[task 2017-07-18T08:28:31.123125Z] 08:28:31     INFO -  -e:null:1
[task 2017-07-18T08:28:31.123510Z] 08:28:31     INFO -  exiting test
status-firefox57: --- → wontfix
Priority: -- → P5
Summary: Intermittent xpcshell.ini:toolkit/components/extensions/test/xpcshell/test_ext_alarms.js | test_alarm_clear_non_matching_name - [test_alarm_clear_non_matching_name : 243] alarm was not removed - Expected: 1, Actual: 0 - false == true → Intermittent toolkit/components/extensions/test/xpcshell/test_ext_alarms.js | test_alarm_clear_non_matching_name - [test_alarm_clear_non_matching_name : 243] alarm was not removed - Expected: 1, Actual: 0 - false == true
Hi,

This bug has failed 32 times in the last 7 days.

It seems it only fails on android-4-3-armv7-api16 debug build type.

Here is a recent log of the failure:
https://treeherder.mozilla.org/logviewer.html#?job_id=180572440&repo=mozilla-inbound&lineNumber=2072

:ddurst: Can you please take a look at this bug?
Flags: needinfo?(ddurst)
Product: Toolkit → WebExtensions
This only fails on Android ARM devices, which are known to be slow.

If the test runs really slowly, then the 2 second timer from the test can be reached and clear the alarm before the test has a chance of checking its existence, and cause this reported error.
Assignee: nobody → rob
Status: NEW → ASSIGNED
Flags: needinfo?(ddurst)
The failure from the bug could happen if the test runs so slow that the
timer expires before the existence of the alarm is checked.
To avoid that failure, extend the lifetime of the alarm from 2 to 20
seconds.
Comment on attachment 9005221 [details]
Bug 1381799 - Fix intermittent test_alarm_clear_non_matching_name

Andrew Swan [:aswan] has approved the revision.
Attachment #9005221 - Flags: review+
Pushed by rob@robwu.nl:
https://hg.mozilla.org/integration/autoland/rev/037dab36696a
Fix intermittent test_alarm_clear_non_matching_name r=aswan
https://hg.mozilla.org/mozilla-central/rev/037dab36696a
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox63: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Flags: qe-verify-
Status: REOPENED → RESOLVED
Closed: 6 years ago → 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: