test: remove non-cross browser assertion in the cookie test (puppetee…

…r#12527)
getflywheel · Jun 3, 2024 · d8654f1 · d8654f1
1 parent 26d800c
commit d8654f1
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 8 deletions.
diff --git a/test/TestExpectations.json b/test/TestExpectations.json
@@ -1184,13 +1184,6 @@
     "expectations": ["FAIL"],
     "comment": "TODO: add a comment explaining why this expectation is required (include links to issues)"
   },
-  {
-    "testIdPattern": "[cookies.spec] Cookie specs Page.setCookie should set cookies from a frame",
-    "platforms": ["darwin", "linux", "win32"],
-    "parameters": ["firefox", "webDriverBiDi"],
-    "expectations": ["FAIL"],
-    "comment": "TODO: add a comment explaining why this expectation is required (include links to issues)"
-  },
   {
     "testIdPattern": "[cookies.spec] Cookie specs Page.setCookie should set multiple cookies",
     "platforms": ["darwin", "linux", "win32"],

diff --git a/test/src/cookies.spec.ts b/test/src/cookies.spec.ts
@@ -463,7 +463,6 @@ describe('Cookie specs', () => {
       expect(await page.evaluate('document.cookie')).toBe(
         'localhost-cookie=best'
       );
-      expect(await page.frames()[1]!.evaluate('document.cookie')).toBe('');
 
       await expectCookieEquals(await page.cookies(), [
         {