fix: move into separate test

styfle · styfle · commit 24e2703c44e6 · 2023-07-12T12:19:55.000-04:00
diff --git a/packages/react-dom/src/__tests__/ReactDOMFloat-test.js b/packages/react-dom/src/__tests__/ReactDOMFloat-test.js
@@ -3585,13 +3585,6 @@ body {
         imageSizes: 'makes no sense',
       });
 
-      ReactDOM.preload('rp', {
-        as: 'image',
-        imageSrcSet: 'rpsrcset',
-        imageSizes: 'rpsizes',
-        referrerPolicy: 'no-referrer',
-      });
-
       if (isClient) {
         // Will key off href in absense of imageSrcSet
         ReactDOM.preload('client', {as: 'image'});
@@ -3641,19 +3634,31 @@ body {
             imagesizes="foosizes"
           />
           <link rel="preload" as="somethingelse" href="bar" />
+        </head>
+        <body>hello</body>
+      </html>,
+    );
+
+    const root = ReactDOMClient.hydrateRoot(document, <App />);
+    await waitForAll([]);
+    expect(getMeaningfulChildren(document)).toEqual(
+      <html>
+        <head>
+          <link rel="preload" as="image" href="foo" />
+          <link rel="preload" as="image" imagesrcset="fooset" />
           <link
             rel="preload"
             as="image"
-            imagesrcset="rpsrcset"
-            imagesizes="rpsizes"
-            referrerpolicy="no-referrer"
+            imagesrcset="fooset"
+            imagesizes="foosizes"
           />
+          <link rel="preload" as="somethingelse" href="bar" />
         </head>
         <body>hello</body>
       </html>,
     );
 
-    const root = ReactDOMClient.hydrateRoot(document, <App />);
+    root.render(<App isClient={true} />);
     await waitForAll([]);
     expect(getMeaningfulChildren(document)).toEqual(
       <html>
@@ -3667,46 +3672,98 @@ body {
             imagesizes="foosizes"
           />
           <link rel="preload" as="somethingelse" href="bar" />
+          <link rel="preload" as="image" href="client" />
+          <link rel="preload" as="image" imagesrcset="clientset" />
+          <link
+            rel="preload"
+            as="image"
+            imagesrcset="clientset"
+            imagesizes="clientsizes"
+          />
+        </head>
+        <body>hello</body>
+      </html>,
+    );
+  });
+
+  it('should handle referrerPolicy on image preload', async () => {
+    function App({isClient}) {
+      ReactDOM.preload('/server', {
+        as: 'image',
+        imageSrcSet: '/server',
+        imageSizes: '100vw',
+        referrerPolicy: 'no-referrer',
+      });
+
+      if (isClient) {
+        ReactDOM.preload('/client', {
+          as: 'image',
+          imageSrcSet: '/client',
+          imageSizes: '100vw',
+          referrerPolicy: 'no-referrer',
+        });
+      }
+
+      return (
+        <html>
+          <body>hello</body>
+        </html>
+      );
+    }
+
+    await act(() => {
+      renderToPipeableStream(<App />).pipe(writable);
+    });
+    expect(getMeaningfulChildren(document)).toEqual(
+      <html>
+        <head>
           <link
             rel="preload"
             as="image"
-            imagesrcset="rpsrcset"
-            imagesizes="rpsizes"
+            imagesrcset="/server"
+            imagesizes="100vw"
             referrerpolicy="no-referrer"
           />
         </head>
         <body>hello</body>
       </html>,
     );
 
-    root.render(<App isClient={true} />);
+    const root = ReactDOMClient.hydrateRoot(document, <App />);
     await waitForAll([]);
     expect(getMeaningfulChildren(document)).toEqual(
       <html>
         <head>
-          <link rel="preload" as="image" href="foo" />
-          <link rel="preload" as="image" imagesrcset="fooset" />
           <link
             rel="preload"
             as="image"
-            imagesrcset="fooset"
-            imagesizes="foosizes"
+            imagesrcset="/server"
+            imagesizes="100vw"
+            referrerpolicy="no-referrer"
           />
-          <link rel="preload" as="somethingelse" href="bar" />
+        </head>
+        <body>hello</body>
+      </html>,
+    );
+
+    root.render(<App isClient={true} />);
+    await waitForAll([]);
+    expect(getMeaningfulChildren(document)).toEqual(
+      <html>
+        <head>
           <link
             rel="preload"
             as="image"
-            imagesrcset="rpsrcset"
-            imagesizes="rpsizes"
+            imagesrcset="/server"
+            imagesizes="100vw"
             referrerpolicy="no-referrer"
           />
-          <link rel="preload" as="image" href="client" />
-          <link rel="preload" as="image" imagesrcset="clientset" />
           <link
             rel="preload"
             as="image"
-            imagesrcset="clientset"
-            imagesizes="clientsizes"
+            imagesrcset="/client"
+            imagesizes="100vw"
+            referrerpolicy="no-referrer"
           />
         </head>
         <body>hello</body>
