Closed Bug 1665786 Opened 5 years ago Closed 4 years ago

Intermittent security/manager/ssl/tests/mochitest/browser/browser_certificateManager.js | Expected override to have field (Not Stored) - "" == "(Not Stored)" - JS frame :: browser_certificateManager.js

Categories

(Core :: Security: PSM, defect, P5)

defect

Tracking

()

RESOLVED FIXED
88 Branch
Tracking Status
firefox88 --- fixed

People

(Reporter: intermittent-bug-filer, Assigned: florian)

References

(Regression)

Details

(Keywords: intermittent-failure, regression)

Attachments

(1 file)

Filed by: ncsoregi [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=316005755&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/Fh3YUv9KR9WelFg2BxRRpQ/runs/0/artifacts/public/logs/live_backing.log


[task 2020-09-17T22:54:35.565Z] 22:54:35     INFO - TEST-PASS | security/manager/ssl/tests/mochitest/browser/browser_certificateManager.js | Expected override to be example.com:9999 but got example.com:9999 - "example.com:9999" == "example.com:9999" - 
[task 2020-09-17T22:54:35.566Z] 22:54:35     INFO - Buffered messages finished
[task 2020-09-17T22:54:35.567Z] 22:54:35     INFO - TEST-UNEXPECTED-FAIL | security/manager/ssl/tests/mochitest/browser/browser_certificateManager.js | Expected override to have field (Not Stored) - "" == "(Not Stored)" - JS frame :: chrome://mochitests/content/browser/security/manager/ssl/tests/mochitest/browser/browser_certificateManager.js :: checkServerCertificates/< :: line 36
[task 2020-09-17T22:54:35.568Z] 22:54:35     INFO - Stack trace:
[task 2020-09-17T22:54:35.569Z] 22:54:35     INFO - chrome://mochitests/content/browser/security/manager/ssl/tests/mochitest/browser/browser_certificateManager.js:checkServerCertificates/<:36
[task 2020-09-17T22:54:35.570Z] 22:54:35     INFO - chrome://mochitests/content/browser/security/manager/ssl/tests/mochitest/browser/browser_certificateManager.js:checkServerCertificates:24
[task 2020-09-17T22:54:35.570Z] 22:54:35     INFO - chrome://mochitests/content/browser/security/manager/ssl/tests/mochitest/browser/browser_certificateManager.js:test_cert_manager_server_tab:174
[task 2020-09-17T22:54:35.571Z] 22:54:35     INFO - Not taking screenshot here: see the one that was previously logged
[task 2020-09-17T22:54:35.572Z] 22:54:35     INFO - TEST-UNEXPECTED-FAIL | security/manager/ssl/tests/mochitest/browser/browser_certificateManager.js | Expected override to be Temporary - "" == "Temporary" - JS frame :: chrome://mochitests/content/browser/security/manager/ssl/tests/mochitest/browser/browser_certificateManager.js :: checkServerCertificates/< :: line 42
[task 2020-09-17T22:54:35.573Z] 22:54:35     INFO - Stack trace:
[task 2020-09-17T22:54:35.573Z] 22:54:35     INFO - chrome://mochitests/content/browser/security/manager/ssl/tests/mochitest/browser/browser_certificateManager.js:checkServerCertificates/<:42
[task 2020-09-17T22:54:35.573Z] 22:54:35     INFO - chrome://mochitests/content/browser/security/manager/ssl/tests/mochitest/browser/browser_certificateManager.js:checkServerCertificates:24
[task 2020-09-17T22:54:35.573Z] 22:54:35     INFO - chrome://mochitests/content/browser/security/manager/ssl/tests/mochitest/browser/browser_certificateManager.js:test_cert_manager_server_tab:174
[task 2020-09-17T22:54:35.574Z] 22:54:35     INFO - Leaving test bound test_cert_manager_server_tab
[task 2020-09-17T22:54:35.574Z] 22:54:35     INFO - GECKO(14248) | MEMORY STAT | vsize 3116MB | residentFast 409MB | heapAllocated 149MB
[task 2020-09-17T22:54:35.575Z] 22:54:35     INFO - TEST-OK | security/manager/ssl/tests/mochitest/browser/browser_certificateManager.js | took 2905ms```
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → INCOMPLETE
Status: RESOLVED → REOPENED
Resolution: INCOMPLETE → ---
Status: REOPENED → RESOLVED
Closed: 5 years ago5 years ago
Resolution: --- → INCOMPLETE
Status: RESOLVED → REOPENED
Resolution: INCOMPLETE → ---

This becomes a permafail on some platforms (Linux on try, and my local mac build) if I fix 1596165.

The problem is that the fix from bug 1664808 waits for 100ms (using the implicit 100ms initial timer from TestUtils.waitForCondition) instead of waiting for something that guarantees strings will have been translated.

Assignee: nobody → florian
Blocks: 1596165
Regressed by: 1664808
Has Regression Range: --- → yes
Keywords: regression
Pushed by fqueze@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4c19157577ef browser_certificateManager.js should wait for the next refresh driver tick instead of relying on the implicit initial waitForCondition timer to wait for strings to be localized, r=keeler.
Status: REOPENED → RESOLVED
Closed: 5 years ago4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: