Bug 1979165 [wpt PR 53961] - Revert "[FedCM] Adding disconnect to reset test state.", a=testonly

marco-c · marco-c · commit cb606c9400f8 · 2025-08-04T15:45:26.000Z
Automatic update from web-platform-tests Revert "[FedCM] Adding disconnect to reset test state." (#53961) This reverts commit 35632c9ba41725c5fabdb14ea66b00af64f83796. Reason for revert: Seeing new failure type in associated tests on `WebKit Win10` builder - https://ci.chromium.org/ui/test/chromium/ninja%3A%2F%2F%3Aheadless_shell_wpt%2Fexternal%2Fwpt%2Ffedcm%2Ffedcm-authz%2Ffedcm-disclosure-text-shown.https.html?q=V%3Abuilder%3DWebKit%2520Win10+V%3Aos%3DWindows-10-19045+V%3Atest_suite%3Dheadless_shell_wpt_tests. Bug: 41482163 Original change's description: > [FedCM] Adding disconnect to reset test state. > > Test results are affected because of previously run tests. > Disconnect helps in resetting the test state for each test case. > > Bug: 41482163 > Change-Id: Ib97c2ff69ba0d01f886947c0969190c778bb7986 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6780242 > Commit-Queue: Christian Biesinger <cbiesingerchromium.org> > Commit-Queue: Jaimukund Bhan <bhanjaimukundgmail.com> > Reviewed-by: Nicolás Peña <npmchromium.org> > Reviewed-by: Christian Biesinger <cbiesingerchromium.org> > Cr-Commit-Position: refs/heads/main{#1491540} Bug: 41482163 No-Presubmit: true No-Tree-Checks: true No-Try: true Change-Id: Icd5c5ef42e1c541d191f34f122e7337847c96d34 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6779935 Commit-Queue: Rubber Stamper <rubber-stamperappspot.gserviceaccount.com> Auto-Submit: Paul Adedeji <pauladedejigoogle.com> Bot-Commit: Rubber Stamper <rubber-stamperappspot.gserviceaccount.com> Cr-Commit-Position: refs/heads/main{#1491632} Co-authored-by: Paul Adedeji <pauladedejigoogle.com> -- wpt-commits: 9320b1f724632c52929a3fdb11bdaf65eafc7611 wpt-pr: 53961 UltraBlame original commit: 6c9b0b7d81c7d60664f2004ec9c24593460fc3e4
diff --git a/testing/web-platform/tests/fedcm/fedcm-multi-idp/fedcm-too-many-idps.https.html b/testing/web-platform/tests/fedcm/fedcm-multi-idp/fedcm-too-many-idps.https.html
@@ -0,0 +1,32 @@
+<!DOCTYPE html>
+<title>Federated Credential Management API: too many IDPs requested.</title>
+<link rel="help" href="https://fedidcg.github.io/FedCM">
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script src="/resources/testdriver.js"></script>
+<script src="/resources/testdriver-vendor.js"></script>
+
+<body>
+
+<script type="module">
+import {fedcm_test,
+        manifest_origin,
+        default_manifest_path} from '../support/fedcm-helper.sub.js';
+
+fedcm_test(async t => {
+  let providers = [];
+  for (let i = 0; i < 11; i++) {
+    providers.push({
+      configURL: `${manifest_origin}${default_manifest_path}?i=${i}`,
+      clientId: '1',
+      nonce: '2'
+    });
+  }
+  assert_throws_js(TypeError, navigator.credentials.get({
+    identity: {
+      providers: providers,
+    }
+  }));
+}, "Multi IDP FedCM call fails when too many IDPs are requested.");
+
+</script>
diff --git a/testing/web-platform/tests/fedcm/support/fedcm-helper.sub.js b/testing/web-platform/tests/fedcm/support/fedcm-helper.sub.js
@@ -129,6 +129,14 @@ export function request_options_with_two_idps(mediation = 'required') {
 export function fedcm_test(test_func, test_name) {
   promise_test(async t => {
     assert_implements(window.IdentityCredential, "FedCM is not supported");
+
+    try {
+      await navigator.credentials.preventSilentAccess();
+    } catch (ex) {
+      
+      
+    }
+
     
     try {
       await test_driver.set_fedcm_delay_enabled(false);
