Intermittent browser_notifications_2.js | Main action button is disabled - false == true - JS frame :: chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_notifications_2.js :: test_empty_password/< :: line 36 etc.

REOPENED
Unassigned

Status

()

Toolkit
Password Manager
P3
normal
REOPENED
2 years ago
a year ago

People

(Reporter: philor, Unassigned)

Tracking

({intermittent-failure, leave-open})

Trunk
intermittent-failure, leave-open
Points:
---

Firefox Tracking Flags

(firefox49 affected)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

Blocks: 1217134
Flags: needinfo?(gasolin)

Comment 1

2 years ago
dup of bug 1275100
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Flags: needinfo?(gasolin)
Resolution: --- → DUPLICATE
Duplicate of bug: 1275100
This is about an intermittent failure. Bug 1275100 is about the fact that the test will fail on merge day because of the signon.rememberSignons.visibilityToggle pref only being true on Nightly: https://mxr.mozilla.org/mozilla-central/source/modules/libpref/init/all.js?rev=cfda20b697dd#4196
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---

Comment 3

2 years ago
18 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 9
* try: 6
* fx-team: 2
* mozilla-central: 1

Platform breakdown:
* linux64: 14
* linux32: 3
* osx-10-10: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1272849&startday=2016-05-23&endday=2016-05-29&tree=all

Comment 4

2 years ago
6 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 3
* fx-team: 3

Platform breakdown:
* linux64: 6

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1272849&startday=2016-05-30&endday=2016-06-05&tree=all

Comment 5

2 years ago
14 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 8
* fx-team: 3
* mozilla-central: 2
* mozilla-aurora: 1

Platform breakdown:
* linux64: 9
* linux32: 5

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1272849&startday=2016-06-06&endday=2016-06-12&tree=all

Comment 6

2 years ago
9 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 3
* autoland: 3
* fx-team: 2
* mozilla-central: 1

Platform breakdown:
* linux64: 9

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1272849&startday=2016-06-27&endday=2016-07-03&tree=all

Comment 7

2 years ago
7 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* fx-team: 3
* mozilla-inbound: 2
* try: 1
* autoland: 1

Platform breakdown:
* linux64: 5
* windows7-32: 1
* linux32: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1272849&startday=2016-07-11&endday=2016-07-17&tree=all
Can you work on this? This is marked as a dependency of one of your fixed bugs.
Flags: needinfo?(gasolin)

Updated

2 years ago
Assignee: nobody → gasolin
Flags: needinfo?(gasolin)

Comment 9

2 years ago
The bug seems related to bug 1227652
See Also: → bug 1227652

Comment 10

2 years ago
5 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* autoland: 3
* try: 2

Platform breakdown:
* linux64: 5

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1272849&startday=2016-07-18&endday=2016-07-24&tree=all

Comment 11

2 years ago
6 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* try: 2
* mozilla-central: 2
* mozilla-inbound: 1
* fx-team: 1

Platform breakdown:
* linux64: 6

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1272849&startday=2016-07-25&endday=2016-07-31&tree=all

Comment 12

2 years ago
8 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* autoland: 3
* try: 2
* mozilla-inbound: 2
* fx-team: 1

Platform breakdown:
* linux64: 8

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1272849&startday=2016-08-01&endday=2016-08-07&tree=all
Comment hidden (mozreview-request)

Comment 14

2 years ago
Since pyGTK error on linux is not resolved anytime soon, I'd like to disable the test on linux to avoid intermittent
Comment on attachment 8779190 [details]
Bug 1272849 - skip browser_notification_2 test on linux to avoid intermittent;

https://reviewboard.mozilla.org/r/70224/#review67510

The pyGTK issue is harmless (as the bug summary says) and isn't what's causing the orange. The issue is the "Main action button is disabled - false == true…" from the description.
Attachment #8779190 - Flags: review?(MattN+bmo) → review-
8 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* try: 5
* autoland: 2
* mozilla-inbound: 1

Platform breakdown:
* linux64: 7
* linux32: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1272849&startday=2016-08-08&endday=2016-08-14&tree=all
Can you please spend some time on this?
Flags: needinfo?(gasolin)
The origin browser_notification_2 related tests are fully skipped on linux, 
in patch bug 1217134 I separated browser_notification tests to browser_notification_1 and browser_notification_2, then removed the skip sentence because I can run test successfully both at local and treeherder.

Though now the browser_notification_2 still shows intermittent issues, I think a practical way is to add skipped="linux" back if we can't simply find the root cause which make test fail on linux.
Flags: needinfo?(gasolin) → needinfo?(MattN+bmo)
25 automation job failures were associated with this bug yesterday.

Repository breakdown:
* mozilla-inbound: 25

Platform breakdown:
* linux64: 22
* linux32: 3

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1272849&startday=2016-08-18&endday=2016-08-18&tree=all
49 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 41
* try: 2
* mozilla-central: 2
* mozilla-beta: 1
* mozilla-aurora: 1
* fx-team: 1
* autoland: 1

Platform breakdown:
* linux64: 41
* linux32: 7
* windows7-32: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1272849&startday=2016-08-15&endday=2016-08-21&tree=all
(In reply to Fred Lin [:gasolin] from comment #18)
> Though now the browser_notification_2 still shows intermittent issues, I
> think a practical way is to add skipped="linux" back if we can't simply find
> the root cause which make test fail on linux.

Fine with me though it's not 100% Linux it will at least reduce the volume.
Flags: needinfo?(MattN+bmo)
7 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 3
* mozilla-aurora: 2
* autoland: 2

Platform breakdown:
* linux64: 7

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1272849&startday=2016-08-22&endday=2016-08-28&tree=all
Comment on attachment 8779190 [details]
Bug 1272849 - skip browser_notification_2 test on linux to avoid intermittent;

Ask for review again based on comment 18
Attachment #8779190 - Flags: review- → review?(MattN+bmo)
Comment on attachment 8779190 [details]
Bug 1272849 - skip browser_notification_2 test on linux to avoid intermittent;

https://reviewboard.mozilla.org/r/70224/#review73222

::: toolkit/components/passwordmgr/test/browser/browser.ini:41
(Diff revision 1)
>    subtst_notifications_11_popup.html
>  [browser_username_select_dialog.js]
>  support-files =
>    subtst_notifications_change_p.html
>  [browser_DOMFormHasPassword.js]
>  [browser_DOMInputPasswordAdded.js]

Nit: You have a typo ("testt") in the commit message.

Thanks
Attachment #8779190 - Flags: review?(MattN+bmo) → review+
Comment hidden (mozreview-request)
fixed commit message, thanks!
Keywords: checkin-needed
Fred, could you leave a comment after the skip-if statement and refer to this bug?

Given that this bug is not really being fixed, we should leave it open as well.
Flags: needinfo?(gasolin)
Keywords: leave-open
Comment hidden (mozreview-request)
Thanks for remind, added a comment after skip-if sentence
Flags: needinfo?(gasolin)

Comment 30

a year ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/fx-team/rev/ada110cf62d8
Skip browser_notification_2 test on linux to avoid intermittent. r=MattN
Keywords: checkin-needed

Comment 32

a year ago
Bulk assigning P3 to all open intermittent bugs without a priority set in Firefox components per bug 1298978.
Priority: -- → P3
5 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-central: 2
* autoland: 2
* mozilla-inbound: 1

Platform breakdown:
* osx-10-10: 3
* windowsxp: 2

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1272849&startday=2016-10-03&endday=2016-10-09&tree=all

Updated

a year ago
Assignee: gasolin → nobody
5 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* autoland: 3
* mozilla-inbound: 1
* mozilla-central: 1

Platform breakdown:
* windows8-64: 2
* windowsxp: 1
* windows7-32-vm: 1
* osx-10-10: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1272849&startday=2016-10-24&endday=2016-10-30&tree=all
You need to log in before you can comment on or make changes to this bug.