test: update webstorage wpt

anonrig · targos · commit 40cdf756e6af · 2025-02-11T08:46:55.000+01:00
PR-URL: #56963 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Filip Skokan <panva.ip@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Matthew Aitken <maitken033380023@gmail.com> Reviewed-By: Jason Zhang <xzha4350@gmail.com>
diff --git a/test/fixtures/wpt/README.md b/test/fixtures/wpt/README.md
@@ -36,7 +36,7 @@ Last update:
 - WebCryptoAPI: https://github.com/web-platform-tests/wpt/tree/3e3374efde/WebCryptoAPI
 - webidl/ecmascript-binding/es-exceptions: https://github.com/web-platform-tests/wpt/tree/a370aad338/webidl/ecmascript-binding/es-exceptions
 - webmessaging/broadcastchannel: https://github.com/web-platform-tests/wpt/tree/6495c91853/webmessaging/broadcastchannel
-- webstorage: https://github.com/web-platform-tests/wpt/tree/9dafa89214/webstorage
+- webstorage: https://github.com/web-platform-tests/wpt/tree/1291340aaa/webstorage
 
 [Web Platform Tests]: https://github.com/web-platform-tests/wpt
 [`git node wpt`]: https://github.com/nodejs/node-core-utils/blob/main/docs/git-node.md#git-node-wpt
diff --git a/test/fixtures/wpt/versions.json b/test/fixtures/wpt/versions.json
@@ -104,7 +104,7 @@
     "path": "webmessaging/broadcastchannel"
   },
   "webstorage": {
-    "commit": "9dafa892146c4b5b1f604a39b3cf8677f8f70d44",
+    "commit": "1291340aaaa6e73db43b412e47401eca3830c556",
     "path": "webstorage"
   }
 }
diff --git a/test/fixtures/wpt/webstorage/localstorage-about-blank-3P-iframe-opens-3P-window.partitioned.html b/test/fixtures/wpt/webstorage/localstorage-about-blank-3P-iframe-opens-3P-window.partitioned.html
@@ -57,7 +57,7 @@
       for (let id in ids) {
         assert_true(id !== undefined, "id is not undefined");
       }
-      // Note: we use assert_true, rather than assert_equals because we're
+      // Note: we use assert_true, rather than assert_equals becuase we're
       // setting random numbers as IDs - this would mean expectations
       // files wouldn't work as intended.
       assert_true(crossSiteIframeAboutBlankID !== crossSiteIframeID,
diff --git a/test/fixtures/wpt/webstorage/localstorage-basic-partitioned.sub.html b/test/fixtures/wpt/webstorage/localstorage-basic-partitioned.sub.html
@@ -38,7 +38,7 @@
       assert_true(typeof sameSiteID === "string");
 
       if (location.origin !== altOrigin) {
-        crossSiteWindow = window.open(`${altOrigin}/webstorage/localstorage-basic-partitioned.tentative.sub.html`, "", "noopener=false");
+        crossSiteWindow = window.open(`${altOrigin}/webstorage/localstorage-basic-partitioned.sub.html`, "", "noopener=false");
         t.add_cleanup(() => crossSiteWindow.close());
       }
     }
diff --git a/test/fixtures/wpt/webstorage/localstorage-cross-origin-iframe.https.window.js b/test/fixtures/wpt/webstorage/localstorage-cross-origin-iframe.https.window.js
diff --git a/test/fixtures/wpt/webstorage/sessionStorage-basic-partitioned.sub.html b/test/fixtures/wpt/webstorage/sessionStorage-basic-partitioned.sub.html
@@ -47,7 +47,7 @@
       // new cross-site window that contains our shared-iframe to repeat
       // the process in a cross-site environment.
       if (location.origin !== altOrigin) {
-        crossSiteWindow = window.open(`${altOrigin}/webstorage/sessionStorage-basic-partitioned.tentative.sub.html`, "", "noopener=false");
+        crossSiteWindow = window.open(`${altOrigin}/webstorage/sessionStorage-basic-partitioned.sub.html`, "", "noopener=false");
         t.add_cleanup(() => crossSiteWindow.close());
       }
     }
diff --git a/test/wpt/status/webstorage.json b/test/wpt/status/webstorage.json
@@ -8,6 +8,9 @@
   "localstorage-cross-origin-iframe.tentative.https.window.js": {
     "skip": "iframes are not supported in Node.js."
   },
+  "localstorage-cross-origin-iframe.https.window.js": {
+    "skip": "iframes are not supported in Node.js."
+  },
   "storage_local_window_open.window.js": {
     "skip": "window.open() is not supported in Node.js."
   },

Original file line number	Diff line number	Diff line change
`@@ -104,7 +104,7 @@`
`104`	`104`	`"path": "webmessaging/broadcastchannel"`
`105`	`105`	`},`
`106`	`106`	`"webstorage": {`
`107`		`- "commit": "9dafa892146c4b5b1f604a39b3cf8677f8f70d44",`
	`107`	`+ "commit": "1291340aaaa6e73db43b412e47401eca3830c556",`
`108`	`108`	`"path": "webstorage"`
`109`	`109`	`}`
`110`	`110`	`}`
Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@`
`57`	`57`	`for (let id in ids) {`
`58`	`58`	`assert_true(id !== undefined, "id is not undefined");`
`59`	`59`	`}`
`60`		`- // Note: we use assert_true, rather than assert_equals because we're`
	`60`	`+ // Note: we use assert_true, rather than assert_equals becuase we're`
`61`	`61`	`// setting random numbers as IDs - this would mean expectations`
`62`	`62`	`// files wouldn't work as intended.`
`63`	`63`	`assert_true(crossSiteIframeAboutBlankID !== crossSiteIframeID,`
Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@`
`38`	`38`	`assert_true(typeof sameSiteID === "string");`
`39`	`39`
`40`	`40`	`if (location.origin !== altOrigin) {`
`41`		- crossSiteWindow = window.open(`${altOrigin}/webstorage/localstorage-basic-partitioned.tentative.sub.html`, "", "noopener=false");
	`41`	+ crossSiteWindow = window.open(`${altOrigin}/webstorage/localstorage-basic-partitioned.sub.html`, "", "noopener=false");
`42`	`42`	`t.add_cleanup(() => crossSiteWindow.close());`
`43`	`43`	`}`
`44`	`44`	`}`
Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@`
`47`	`47`	`// new cross-site window that contains our shared-iframe to repeat`
`48`	`48`	`// the process in a cross-site environment.`
`49`	`49`	`if (location.origin !== altOrigin) {`
`50`		- crossSiteWindow = window.open(`${altOrigin}/webstorage/sessionStorage-basic-partitioned.tentative.sub.html`, "", "noopener=false");
	`50`	+ crossSiteWindow = window.open(`${altOrigin}/webstorage/sessionStorage-basic-partitioned.sub.html`, "", "noopener=false");
`51`	`51`	`t.add_cleanup(() => crossSiteWindow.close());`
`52`	`52`	`}`
`53`	`53`	`}`