Closed Bug 1451143 Opened 2 years ago Closed 2 years ago

Intermittent toolkit/components/payments/test/browser/browser_card_edit.js | Check updated cc-name - "J. Smith" == "A. Nonymous" -

Categories

(Firefox :: WebPayments UI, defect, P1)

defect

Tracking

()

RESOLVED FIXED
Firefox 61
Tracking Status
firefox61 --- fixed

People

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

References

Details

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

Attachments

(1 file)

Filed by: apavel [at] mozilla.com

https://treeherder.mozilla.org/logviewer.html#?job_id=171707606&repo=autoland

https://queue.taskcluster.net/v1/task/NgeztiisR7uFzf3-TRJKXA/runs/0/artifacts/public/logs/live_backing.log

[task 2018-04-03T20:21:13.390Z] 20:21:13     INFO - Console message: [JavaScript Error: "Content Security Policy: The page’s settings blocked the loading of a resource at self (“default-src”)." {file: "resource://payments/paymentRequest.xhtml" line: 0}]
[task 2018-04-03T20:21:13.391Z] 20:21:13     INFO - TEST-PASS | toolkit/components/payments/test/browser/browser_card_edit.js | Field #cc-number shouldn't be disabled - true == true - 
[task 2018-04-03T20:21:13.392Z] 20:21:13     INFO - TEST-PASS | toolkit/components/payments/test/browser/browser_card_edit.js | Field #cc-name shouldn't be disabled - true == true - 
[task 2018-04-03T20:21:13.393Z] 20:21:13     INFO - TEST-PASS | toolkit/components/payments/test/browser/browser_card_edit.js | Field #cc-exp-month shouldn't be disabled - true == true - 
[task 2018-04-03T20:21:13.394Z] 20:21:13     INFO - TEST-PASS | toolkit/components/payments/test/browser/browser_card_edit.js | Field #cc-exp-year shouldn't be disabled - true == true - 
[task 2018-04-03T20:21:13.397Z] 20:21:13     INFO - TEST-PASS | toolkit/components/payments/test/browser/browser_card_edit.js | Check there is one card - 1 == 1 - 
[task 2018-04-03T20:21:13.400Z] 20:21:13     INFO - TEST-PASS | toolkit/components/payments/test/browser/browser_card_edit.js | Check cc-number - "************1111" == "************1111" - 
[task 2018-04-03T20:21:13.401Z] 20:21:13     INFO - TEST-PASS | toolkit/components/payments/test/browser/browser_card_edit.js | Check cc-name - "J. Smith" == "J. Smith" - 
[task 2018-04-03T20:21:13.403Z] 20:21:13     INFO - TEST-PASS | toolkit/components/payments/test/browser/browser_card_edit.js | Check cc-exp-month - 11 == 11 - 
[task 2018-04-03T20:21:13.407Z] 20:21:13     INFO - TEST-PASS | toolkit/components/payments/test/browser/browser_card_edit.js | Check cc-exp-year - 2018 == 2018 - 
[task 2018-04-03T20:21:13.408Z] 20:21:13     INFO - Buffered messages logged at 20:21:10
[task 2018-04-03T20:21:13.409Z] 20:21:13     INFO - Leaving test bound test_add_link
[task 2018-04-03T20:21:13.410Z] 20:21:13     INFO - Entering test bound test_edit_link
[task 2018-04-03T20:21:13.411Z] 20:21:13     INFO - TEST-PASS | toolkit/components/payments/test/browser/browser_card_edit.js | Should have one payment request - 
[task 2018-04-03T20:21:13.412Z] 20:21:13     INFO - TEST-PASS | toolkit/components/payments/test/browser/browser_card_edit.js | Got a payment request with an ID - 
[task 2018-04-03T20:21:13.413Z] 20:21:13     INFO - Buffered messages logged at 20:21:12
[task 2018-04-03T20:21:13.414Z] 20:21:13     INFO - Console message: [JavaScript Error: "Content Security Policy: The page’s settings blocked the loading of a resource at self (“default-src”). Source: width: 500px." {file: "resource://payments/paymentRequest.xhtml" line: 0 column: 0 source: "width: 500px"}]
[task 2018-04-03T20:21:13.415Z] 20:21:13     INFO - Console message: [JavaScript Error: "Content Security Policy: The page’s settings blocked the loading of a resource at self (“default-src”)." {file: "resource://payments/paymentRequest.xhtml" line: 0}]
[task 2018-04-03T20:21:13.416Z] 20:21:13     INFO - Console message: [JavaScript Error: "Content Security Policy: The page’s settings blocked the loading of a resource at self (“default-src”). Source: width: 500px." {file: "resource://payments/paymentRequest.xhtml" line: 0 column: 0 source: "width: 500px"}]
[task 2018-04-03T20:21:13.417Z] 20:21:13     INFO - Console message: [JavaScript Error: "Content Security Policy: The page’s settings blocked the loading of a resource at self (“default-src”)." {file: "resource://payments/paymentRequest.xhtml" line: 0}]
[task 2018-04-03T20:21:13.418Z] 20:21:13     INFO - TEST-PASS | toolkit/components/payments/test/browser/browser_card_edit.js | Edit link text - "Edit" == "Edit" - 
[task 2018-04-03T20:21:13.419Z] 20:21:13     INFO - overwriting field values
[task 2018-04-03T20:21:13.420Z] 20:21:13     INFO - TEST-PASS | toolkit/components/payments/test/browser/browser_card_edit.js | Field #cc-name shouldn't be disabled - true == true - 
[task 2018-04-03T20:21:13.421Z] 20:21:13     INFO - TEST-PASS | toolkit/components/payments/test/browser/browser_card_edit.js | Field #cc-exp-month shouldn't be disabled - true == true - 
[task 2018-04-03T20:21:13.422Z] 20:21:13     INFO - TEST-PASS | toolkit/components/payments/test/browser/browser_card_edit.js | Field #cc-exp-year shouldn't be disabled - true == true - 
[task 2018-04-03T20:21:13.423Z] 20:21:13     INFO - TEST-PASS | toolkit/components/payments/test/browser/browser_card_edit.js | cc-number field should be disabled - true == true - 
[task 2018-04-03T20:21:13.424Z] 20:21:13     INFO - TEST-PASS | toolkit/components/payments/test/browser/browser_card_edit.js | Check there is still one card - 1 == 1 - 
[task 2018-04-03T20:21:13.425Z] 20:21:13     INFO - TEST-PASS | toolkit/components/payments/test/browser/browser_card_edit.js | Card number should be masked and unmodified. - "************1111" == "************1111" - 
[task 2018-04-03T20:21:13.426Z] 20:21:13     INFO - Buffered messages finished
[task 2018-04-03T20:21:13.427Z] 20:21:13     INFO - TEST-UNEXPECTED-FAIL | toolkit/components/payments/test/browser/browser_card_edit.js | Check updated cc-name - "J. Smith" == "A. Nonymous" - 
[task 2018-04-03T20:21:13.428Z] 20:21:13     INFO - Stack trace:
[task 2018-04-03T20:21:13.429Z] 20:21:13     INFO - resource://testing-common/content-task.js line 50 > eval:check:46
[task 2018-04-03T20:21:13.430Z] 20:21:13     INFO - Not taking screenshot here: see the one that was previously logged
[task 2018-04-03T20:21:13.431Z] 20:21:13     INFO - TEST-UNEXPECTED-FAIL | toolkit/components/payments/test/browser/browser_card_edit.js | Check updated cc-exp-month - 11 == 3 - 
[task 2018-04-03T20:21:13.431Z] 20:21:13     INFO - Stack trace:
[task 2018-04-03T20:21:13.433Z] 20:21:13     INFO - resource://testing-common/content-task.js line 50 > eval:check:46
[task 2018-04-03T20:21:13.434Z] 20:21:13     INFO - Not taking screenshot here: see the one that was previously logged
[task 2018-04-03T20:21:13.435Z] 20:21:13     INFO - TEST-UNEXPECTED-FAIL | toolkit/components/payments/test/browser/browser_card_edit.js | Check updated cc-exp-year - 2018 == 2019 - 
[task 2018-04-03T20:21:13.435Z] 20:21:13     INFO - Stack trace:
[task 2018-04-03T20:21:13.436Z] 20:21:13     INFO - resource://testing-common/content-task.js line 50 > eval:check:46
[task 2018-04-03T20:21:13.438Z] 20:21:13     INFO - Console message: [JavaScript Error: "NS_ERROR_NOT_INITIALIZED: " {file: "chrome://payments/content/paymentDialogWrapper.js" line: 277}]
[task 2018-04-03T20:21:13.439Z] 20:21:13     INFO - Console message: [JavaScript Error: "Content Security Policy: The page’s settings blocked the loading of a resource at self (“default-src”)." {file: "resource://payments/paymentRequest.xhtml" line: 0}]
[task 2018-04-03T20:21:14.373Z] 20:21:14     INFO - GECKO(3641) | --DOMWINDOW == 11 (0xd15b5400) [pid = 3641] [serial = 14] [outer = (nil)] [url = chrome://payments/content/paymentDialogWrapper.xhtml?requestId=4455e5ec-6629-4bc2-8ac1-18deaf9e061e]
[task 2018-04-03T20:21:14.374Z] 20:21:14     INFO - GECKO(3641) | --DOCSHELL 0xcd68c000 == 5 [pid = 3641] [id = {0ac3d150-61e7-4512-8525-220245bb6df1}]
[task 2018-04-03T20:21:14.391Z] 20:21:14     INFO - GECKO(3641) | --DOCSHELL 0xcc464400 == 4 [pid = 3641] [id = {6b263c8d-a2f5-4fd2-b897-a9156772992f}]
[task 2018-04-03T20:21:15.323Z] 20:21:15     INFO - GECKO(3641) | --DOMWINDOW == 10 (0xcd6d6c00) [pid = 3641] [serial = 16] [outer = (nil)] [url = about:blank]
[task 2018-04-03T20:21:15.404Z] 20:21:15     INFO - Leaving test bound test_edit_link
[task 2018-04-03T20:21:15.446Z] 20:21:15     INFO - GECKO(3641) | MEMORY STAT vsizeMaxContiguous not supported in this build configuration.
[task 2018-04-03T20:21:15.451Z] 20:21:15     INFO - GECKO(3641) | MEMORY STAT | vsize 879MB | residentFast 257MB | heapAllocated 74MB
[task 2018-04-03T20:21:15.452Z] 20:21:15     INFO - TEST-OK | toolkit/components/payments/test/browser/browser_card_edit.js | took 15699ms
[task 2018-04-03T20:21:15.490Z] 20:21:15     INFO - GECKO(3641) | ++DOCSHELL 0xe3282c00 == 2 [pid = 3756] [id = {d62c145d-7d84-4fe1-832d-558b7cd582cd}]
[task 2018-04-03T20:21:15.492Z] 20:21:15     INFO - GECKO(3641) | ++DOMWINDOW == 7 (0xe3274300) [pid = 3756] [serial = 7] [outer = (nil)]
[task 2018-04-03T20:21:15.526Z] 20:21:15     INFO - GECKO(3641) | --DOMWINDOW == 3 (0xe7d5e800) [pid = 3689] [serial = 3] [outer = (nil)] [url = about:blank]
[task 2018-04-03T20:21:15.528Z] 20:21:15     INFO - GECKO(3641) | --DOMWINDOW == 2 (0xe7d67c00) [pid = 3689] [serial = 4] [outer = (nil)] [url = about:blank]
[task 2018-04-03T20:21:15.585Z] 20:21:15     INFO - GECKO(3641) | ++DOMWINDOW == 8 (0xe328cc00) [pid = 3756] [serial = 8] [outer = 0xe3274300]
[task 2018-04-03T20:21:15.661Z] 20:21:15     INFO - checking window state
[task 2018-04-03T20:21:15.723Z] 20:21:15     INFO - GECKO(3641) | --DOMWINDOW == 9 (0xdbbb2100) [pid = 3641] [serial = 15] [outer = (nil)] [url = chrome://payments/content/paymentDialogWrapper.xhtml?requestId=42f650dc-1414-4773-a8ec-4ee0e582f46d]
Whiteboard: [webpayments] [triage]
Priority: P5 → P3
Whiteboard: [webpayments] [triage] → [webpayments-reserve]
Product: Toolkit → Firefox
Version: unspecified → Trunk
This is failing due to the following lines:
https://searchfox.org/mozilla-central/rev/795575287259a370d00518098472eaa5b362bfa7/browser/components/payments/test/browser/browser_card_edit.js#105-108

This condition just checks that the number of cards on file has not changed, not that the card has actually been updated yet. Updating this condition to also check that the updated value is returned will fix this. I'll attach a patch shortly.
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Whiteboard: [webpayments-reserve] → [webpayments]
Comment on attachment 8972582 [details]
Bug 1451143 - Check that the card is updated before proceeding with test.

https://reviewboard.mozilla.org/r/241146/#review247096

Thanks
Attachment #8972582 - Flags: review?(MattN+bmo) → review+
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/18718bb58de4
Check that the card is updated before proceeding with test. r=MattN
https://hg.mozilla.org/mozilla-central/rev/18718bb58de4
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
You need to log in before you can comment on or make changes to this bug.