test: update snapshot for test cases

rrweb-io · Sep 15, 2022 · 23addd0 · 23addd0
1 parent cf47344
commit 23addd0
Show file tree

Hide file tree

Showing 3 changed files with 57 additions and 36 deletions.
diff --git a/packages/rrweb/src/replay/index.ts b/packages/rrweb/src/replay/index.ts
@@ -1655,12 +1655,6 @@ export class Replayer {
                     skipChild: true,
                     hackCss: true,
                     cache: this.cache,
-                    afterAppend: (node: Node | RRNode, id: number) => {
-                      for (const plugin of this.config.plugins || []) {
-                        if (plugin.onBuild)
-                          plugin.onBuild(node, { id, replayer: this });
-                      }
-                    },
                   });
                   const siblingNode = target.nextSibling;
                   const parentNode = target.parentNode;

diff --git a/packages/rrweb/test/__snapshots__/record.test.ts.snap b/packages/rrweb/test/__snapshots__/record.test.ts.snap
@@ -173,9 +173,12 @@ exports[`record captures CORS stylesheets that are still loading 1`] = `
     \\"type\\": 3,
     \\"data\\": {
       \\"source\\": 0,
+      \\"texts\\": [],
+      \\"attributes\\": [],
+      \\"removes\\": [],
       \\"adds\\": [
         {
-          \\"parentId\\": 9,
+          \\"parentId\\": 4,
           \\"nextId\\": null,
           \\"node\\": {
             \\"type\\": 2,
@@ -188,10 +191,7 @@ exports[`record captures CORS stylesheets that are still loading 1`] = `
             \\"id\\": 9
           }
         }
-      ],
-      \\"removes\\": [],
-      \\"texts\\": [],
-      \\"attributes\\": []
+      ]
     }
   }
 ]"
@@ -2007,7 +2007,19 @@ exports[`record captures stylesheets in iframes that are still loading 1`] = `
                     \\"type\\": 2,
                     \\"tagName\\": \\"head\\",
                     \\"attributes\\": {},
-                    \\"childNodes\\": [],
+                    \\"childNodes\\": [
+                      {
+                        \\"type\\": 2,
+                        \\"tagName\\": \\"link\\",
+                        \\"attributes\\": {
+                          \\"rel\\": \\"stylesheet\\",
+                          \\"href\\": \\"blob:null\\"
+                        },
+                        \\"childNodes\\": [],
+                        \\"rootId\\": 10,
+                        \\"id\\": 13
+                      }
+                    ],
                     \\"rootId\\": 10,
                     \\"id\\": 12
                   },
@@ -2039,25 +2051,17 @@ exports[`record captures stylesheets in iframes that are still loading 1`] = `
     \\"type\\": 3,
     \\"data\\": {
       \\"source\\": 0,
-      \\"adds\\": [
+      \\"adds\\": [],
+      \\"removes\\": [],
+      \\"texts\\": [],
+      \\"attributes\\": [
         {
-          \\"parentId\\": 13,
-          \\"nextId\\": null,
-          \\"node\\": {
-            \\"type\\": 2,
-            \\"tagName\\": \\"link\\",
-            \\"attributes\\": {
-              \\"_cssText\\": \\"body { color: pink; }\\"
-            },
-            \\"childNodes\\": [],
-            \\"rootId\\": 10,
-            \\"id\\": 13
+          \\"id\\": 13,
+          \\"attributes\\": {
+            \\"_cssText\\": \\"body { color: pink; }\\"
           }
         }
-      ],
-      \\"removes\\": [],
-      \\"texts\\": [],
-      \\"attributes\\": []
+      ]
     }
   }
 ]"
@@ -2286,24 +2290,42 @@ exports[`record captures stylesheets that are still loading 1`] = `
     \\"type\\": 3,
     \\"data\\": {
       \\"source\\": 0,
+      \\"texts\\": [],
+      \\"attributes\\": [],
+      \\"removes\\": [],
       \\"adds\\": [
         {
-          \\"parentId\\": 9,
+          \\"parentId\\": 4,
           \\"nextId\\": null,
           \\"node\\": {
             \\"type\\": 2,
             \\"tagName\\": \\"link\\",
             \\"attributes\\": {
-              \\"_cssText\\": \\"body { color: pink; }\\"
+              \\"rel\\": \\"stylesheet\\",
+              \\"href\\": \\"blob:null\\"
             },
             \\"childNodes\\": [],
             \\"id\\": 9
           }
         }
-      ],
+      ]
+    }
+  },
+  {
+    \\"type\\": 3,
+    \\"data\\": {
+      \\"source\\": 0,
+      \\"adds\\": [],
       \\"removes\\": [],
       \\"texts\\": [],
-      \\"attributes\\": []
+      \\"attributes\\": [
+        {
+          \\"id\\": 9,
+          \\"attributes\\": {
+            \\"_cssText\\": \\"body { color: pink; }\\"
+          }
+        }
+      ]
     }
   }
 ]"

diff --git a/packages/rrweb/test/record.test.ts b/packages/rrweb/test/record.test.ts
@@ -626,8 +626,11 @@ describe('record', function (this: ISuite) {
 
     // `blob:` URLs are not available immediately, so we need to wait for the browser to load them
     await waitForRAF(ctx.page);
-
-    assertSnapshot(ctx.events);
+    // 'blob' URL is different in every execution so we need to remove it from the snapshot.
+    const filteredEvents = JSON.parse(
+      JSON.stringify(ctx.events).replace(/blob\:[\w\d-/]+"/, 'blob:null"'),
+    );
+    assertSnapshot(filteredEvents);
   });
 
   it('captures stylesheets in iframes that are still loading', async () => {
@@ -659,8 +662,10 @@ describe('record', function (this: ISuite) {
 
     // `blob:` URLs are not available immediately, so we need to wait for the browser to load them
     await waitForRAF(ctx.page);
-
-    assertSnapshot(ctx.events);
+    const filteredEvents = JSON.parse(
+      JSON.stringify(ctx.events).replace(/blob\:[\w\d-/]+"/, 'blob:null"'),
+    );
+    assertSnapshot(filteredEvents);
   });
 
   it('captures CORS stylesheets that are still loading', async () => {