Closed Bug 1761817 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 33389 - [basic-card] Migrate WPTs that use basic-card by name

Categories

(Testing :: web-platform-tests, task, P4)

Product:
Testing
Component:
web-platform-tests
Type:
task

Tracking

(firefox100 fixed)

Status:
RESOLVED FIXED
Milestone:
100 Branch
Tracking Flags:
Tracking Status
firefox100 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 33389 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/33389
Details from upstream follow.

Nick Burris <nburris@chromium.org> wrote:

[basic-card] Migrate WPTs that use basic-card by name

With basic card now disabled and soon to be removed, migrate some web
platform tests to use a URL payment method; these currently use the
basic-card payment method by name but don't depend on a valid payment
method. These tests assert errors before the payment method would be
loaded, so they can just use a placeholder payment method URL.

Bug: 1209835
Change-Id: I70ae36d0e510e460ad6e9fa349aa86cf7db2b069
Reviewed-on: https://chromium-review.googlesource.com/3555253
WPT-Export-Revision: c1fff217dcd5fe687219163f0831732aa344f74d

CI Results

Ran 0 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 4 tests and 21 subtests

Status Summary

Firefox

OK : 3
FAIL : 23
TIMEOUT: 2

Chrome

OK : 4
PASS : 20
FAIL : 4

Safari

OK : 1
PASS : 4
CRASH : 1
FAIL : 17
ERROR : 2
NOTRUN : 2

Links

GitHub PR Head
GitHub PR Base

Details

Firefox-only Failures

/secure-payment-confirmation/constructor.https.html
Valid payment method data does not throw exceptions.: FAIL
The timeout field is optional.: FAIL
Providing both payee name and payee origin is valid.: FAIL

New Tests That Don't Pass

/feature-policy/reporting/payment-reporting.https.html
PaymentRequest Report Format: FAIL (Chrome: PASS, Safari: NOTRUN)
/permissions-policy/reporting/payment-report-only.https.html: TIMEOUT (Chrome: OK, Safari: CRASH)
PaymentRequest report only mode: TIMEOUT (Chrome: FAIL)
/permissions-policy/reporting/payment-reporting.https.html
PaymentRequest Report Format: FAIL (Chrome: PASS, Safari: NOTRUN)
/secure-payment-confirmation/constructor.https.html
Valid payment method data does not throw exceptions.: FAIL (Chrome: PASS, Safari: PASS)
The timeout field is optional.: FAIL (Chrome: PASS, Safari: PASS)
Extra payment method not allowed afterward.: FAIL (Chrome: PASS, Safari: FAIL)
Extra payment method not allowed beforehand.: FAIL (Chrome: PASS, Safari: FAIL)
The credentialIds field is required.: FAIL (Chrome: PASS, Safari: FAIL)
The challenge field is required.: FAIL (Chrome: PASS, Safari: FAIL)
Empty credentialIds field throws exception.: FAIL (Chrome: PASS, Safari: FAIL)
Null challenge field throws exception.: FAIL (Chrome: PASS, Safari: FAIL)
Empty challenge field throws exception.: FAIL (Chrome: PASS, Safari: FAIL)
Timeout longer than 1 hour throws exception.: FAIL (Chrome: PASS, Safari: FAIL)
Large credentialIds value throws exception.: FAIL (Chrome: PASS, Safari: FAIL)
Large challenge value throws exception.: FAIL (Chrome: PASS, Safari: FAIL)
Instrument field is required.: FAIL (Chrome: PASS, Safari: FAIL)
Instrument display name is required.: FAIL (Chrome: PASS, Safari: FAIL)
Instrument icon is required.: FAIL (Chrome: PASS, Safari: FAIL)
Omitting both payee origin and payee name throws exception.: FAIL (Chrome: PASS, Safari: FAIL)
Payee name without payee origin is valid.: FAIL (Chrome: FAIL, Safari: PASS)
Providing both payee name and payee origin is valid.: FAIL (Chrome: PASS, Safari: PASS)
Empty payee name throws exception.: FAIL (Chrome: FAIL, Safari: FAIL)
Empty payee origin throws exception.: FAIL (Chrome: PASS, Safari: FAIL)
rpId is required.: FAIL (Chrome: FAIL, Safari: FAIL)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2ea3bea6191a
[wpt PR 33389] - [basic-card] Migrate WPTs that use basic-card by name, a=testonly

https://hg.mozilla.org/mozilla-central/rev/2ea3bea6191a

Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox100: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 100 Branch
You need to log in before you can comment on or make changes to this bug.