fix: do not initialize abstract property / accessor (#15344)

babel · Jan 12, 2023 · 7f13bf9 · 7f13bf9
1 parent f7e231f
commit 7f13bf9
Show file tree

Hide file tree

Showing 10 changed files with 16 additions and 7 deletions.
diff --git a/packages/babel-plugin-transform-typescript/src/index.ts b/packages/babel-plugin-transform-typescript/src/index.ts
@@ -160,6 +160,8 @@ export default declare((api, opts: Options) => {
             path.remove();
           }
         }
+      } else if (node.abstract) {
+        path.remove();
       } else if (!process.env.BABEL_8_BREAKING) {
         if (
           !allowDeclareFields &&

diff --git a/...lugin-transform-typescript/test/fixtures/class/abstract-allowDeclareFields-false/input.ts b/...lugin-transform-typescript/test/fixtures/class/abstract-allowDeclareFields-false/input.ts
@@ -0,0 +1,3 @@
+abstract class A {
+    abstract prop: string;
+}
diff --git a/...n-transform-typescript/test/fixtures/class/abstract-allowDeclareFields-false/options.json b/...n-transform-typescript/test/fixtures/class/abstract-allowDeclareFields-false/options.json
@@ -0,0 +1,3 @@
+{
+  "plugins": [["transform-typescript", { "allowDeclareFields": false }]]
+}
diff --git a/...ugin-transform-typescript/test/fixtures/class/abstract-allowDeclareFields-false/output.js b/...ugin-transform-typescript/test/fixtures/class/abstract-allowDeclareFields-false/output.js
@@ -0,0 +1 @@
+class A {}
diff --git a/...plugin-transform-typescript/test/fixtures/class/abstract-allowDeclareFields-true/input.ts b/...plugin-transform-typescript/test/fixtures/class/abstract-allowDeclareFields-true/input.ts
@@ -0,0 +1,3 @@
+abstract class A {
+    abstract prop: string;
+}
diff --git a/...in-transform-typescript/test/fixtures/class/abstract-allowDeclareFields-true/options.json b/...in-transform-typescript/test/fixtures/class/abstract-allowDeclareFields-true/options.json
@@ -0,0 +1,3 @@
+{
+  "plugins": [["transform-typescript", { "allowDeclareFields": true }]]
+}
diff --git a/...lugin-transform-typescript/test/fixtures/class/abstract-allowDeclareFields-true/output.js b/...lugin-transform-typescript/test/fixtures/class/abstract-allowDeclareFields-true/output.js
@@ -0,0 +1 @@
+class A {}
diff --git a/...ugin-transform-typescript/test/fixtures/class/accessor-allowDeclareFields-false/output.js b/...ugin-transform-typescript/test/fixtures/class/accessor-allowDeclareFields-false/output.js
@@ -4,11 +4,8 @@ class Foo {
   accessor #prop3 = 1;
   accessor [prop4] = 1;
   accessor prop5 = 1;
-  accessor prop6;
   accessor #p;
   accessor a;
   accessor aa;
-  accessor #s;
-  accessor f = 1;
   accessor g;
 }
diff --git a/...ansform-typescript/test/fixtures/class/accessor-allowDeclareFields-true-babel-7/output.js b/...ansform-typescript/test/fixtures/class/accessor-allowDeclareFields-true-babel-7/output.js
@@ -4,5 +4,4 @@ class Foo {
   accessor #prop3 = 1;
   accessor [prop4] = 1;
   accessor prop5 = 1;
-  accessor f = 1;
 }
diff --git a/...lugin-transform-typescript/test/fixtures/class/accessor-allowDeclareFields-true/output.js b/...lugin-transform-typescript/test/fixtures/class/accessor-allowDeclareFields-true/output.js
@@ -4,11 +4,8 @@ class Foo {
   accessor #prop3 = 1;
   accessor [prop4] = 1;
   accessor prop5 = 1;
-  accessor prop6;
   accessor #p;
   accessor a;
   accessor aa;
-  accessor #s;
-  accessor f = 1;
   accessor g;
 }