add more testcases

babel · Oct 13, 2021 · 4130dc5 · 4130dc5
1 parent 2515eea
commit 4130dc5
Showing 1 changed file with 12 additions and 8 deletions.
diff --git a/...-plugin-bugfix-webkit-id-destructuring-collision-in-function-expression/test/util.test.js b/...-plugin-bugfix-webkit-id-destructuring-collision-in-function-expression/test/util.test.js
@@ -26,26 +26,30 @@ describe("shouldTransform", () => {
     "(function a({a}) {})",
     "(function a(...a) {})",
     "(function a([a = 1]) {})",
+    "(function a(b, { a: [,...a] }) {})",
   ];
 
   const negativeCases = [
     "(function () {})",
     "(function a() {})",
     "(function a(a) {})",
     "(function a() { var a })",
+    "(function b([a]) { var a })",
+    "(function b([a]) { function a() {} })",
     "(function a(x = a) {})",
     "(function a() { var { a } = {}; })",
+    "(function b([a]) { var { a } = {}; })",
+    "(function a({ [a]: b }) {})",
   ];
 
-  describe("default parser options", () => {
-    test.each(positiveCases)("shouldTransform(%p) should return 'a'", input => {
+  describe("the following cases should be transformed", () => {
+    test.each(positiveCases)("%p", input => {
       expect(shouldTransform(getPath(input))).toBe("a");
     });
-    test.each(negativeCases)(
-      "shouldTransform(in %p) should return false",
-      input => {
-        expect(shouldTransform(getPath(input))).toBe(false);
-      },
-    );
+  });
+  describe("the following cases should not be transformed", () => {
+    test.each(negativeCases)("%p", input => {
+      expect(shouldTransform(getPath(input))).toBe(false);
+    });
   });
 });