Fix

fisker · May 7, 2024 · bfde04f · bfde04f
1 parent 60c57d2
commit bfde04f
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 8 deletions.
diff --git a/test/prefer-structured-clone.mjs b/test/prefer-structured-clone.mjs
@@ -29,6 +29,7 @@ test.snapshot({
 		'JSON.parse(JSON.stringify(foo))',
 		'JSON.parse(JSON.stringify(foo),)',
 		'JSON.parse(JSON.stringify(foo,))',
+		'JSON.parse(JSON.stringify(foo,),)',
 		'JSON.parse( ((JSON.stringify)) (foo))',
 		'(( JSON.parse)) (JSON.stringify(foo))',
 		'JSON.parse(JSON.stringify( ((foo)) ))',

diff --git a/test/snapshots/prefer-structured-clone.mjs.md b/test/snapshots/prefer-structured-clone.mjs.md
@@ -39,7 +39,7 @@ Generated by [AVA](https://avajs.dev).
     ␊
     --------------------------------------------------------------------------------␊
     Suggestion 1/1: Switch to \`structuredClone(…)\`.␊
-      1 | structuredClone(foo)␊
+      1 | structuredClone(foo,)␊
     `
 
 ## invalid(3): JSON.parse(JSON.stringify(foo,))
@@ -57,11 +57,30 @@ Generated by [AVA](https://avajs.dev).
         | ^^^^^^^^^^^^^^^^^^^^^^^^^ Prefer \`structuredClone(…)\` over \`JSON.parse(JSON.stringify(…))\` to create a deep clone.␊
     ␊
     --------------------------------------------------------------------------------␊
+    Suggestion 1/1: Switch to \`structuredClone(…)\`.␊
+      1 | structuredClone(foo)␊
+    `
+
+## invalid(4): JSON.parse(JSON.stringify(foo,),)
+
+> Input
+
+    `␊
+      1 | JSON.parse(JSON.stringify(foo,),)␊
+    `
+
+> Error 1/1
+
+    `␊
+    > 1 | JSON.parse(JSON.stringify(foo,),)␊
+        | ^^^^^^^^^^^^^^^^^^^^^^^^^ Prefer \`structuredClone(…)\` over \`JSON.parse(JSON.stringify(…))\` to create a deep clone.␊
+    ␊
+    --------------------------------------------------------------------------------␊
     Suggestion 1/1: Switch to \`structuredClone(…)\`.␊
       1 | structuredClone(foo,)␊
     `
 
-## invalid(4): JSON.parse( ((JSON.stringify)) (foo))
+## invalid(5): JSON.parse( ((JSON.stringify)) (foo))
 
 > Input
 
@@ -80,7 +99,7 @@ Generated by [AVA](https://avajs.dev).
       1 | structuredClone(  foo)␊
     `
 
-## invalid(5): (( JSON.parse)) (JSON.stringify(foo))
+## invalid(6): (( JSON.parse)) (JSON.stringify(foo))
 
 > Input
 
@@ -99,7 +118,7 @@ Generated by [AVA](https://avajs.dev).
       1 | (( structuredClone)) (foo)␊
     `
 
-## invalid(6): JSON.parse(JSON.stringify( ((foo)) ))
+## invalid(7): JSON.parse(JSON.stringify( ((foo)) ))
 
 > Input
 
@@ -118,7 +137,7 @@ Generated by [AVA](https://avajs.dev).
       1 | structuredClone( ((foo)) )␊
     `
 
-## invalid(7): function foo() { return JSON .parse( JSON. stringify( bar, ), ); }
+## invalid(8): function foo() { return JSON .parse( JSON. stringify( bar, ), ); }
 
 > Input
 
@@ -156,9 +175,9 @@ Generated by [AVA](https://avajs.dev).
       1 | function foo() {␊
       2 | 	return structuredClone(␊
       3 | 			␊
-      4 | 					bar,␊
-      5 | 				)␊
-      6 | 		;␊
+      4 | 					bar␊
+      5 | 				,␊
+      6 | 		);␊
       7 | }␊
     `
 

diff --git a/test/snapshots/prefer-structured-clone.mjs.snap b/test/snapshots/prefer-structured-clone.mjs.snap