fix(1632): erase const enums after inlining

Author: a-tarasyuk

internal/transformers/tstransforms/runtimesyntax.go

@@ -309,6 +309,10 @@ func (tx *RuntimeSyntaxTransformer) addVarForDeclaration(statements []*ast.State
 }
 
 func (tx *RuntimeSyntaxTransformer) visitEnumDeclaration(node *ast.EnumDeclaration) *ast.Node {
+	if !tx.shouldEmitEnumDeclaration(node.AsEnumDeclaration()) {
+		return tx.Factory().NewNotEmittedStatement()
+	}
+
 	statements := []*ast.Statement{}
 
 	// If needed, we should emit a variable declaration for the enum:
@@ -1130,6 +1134,10 @@ func (tx *RuntimeSyntaxTransformer) evaluateEntity(node *ast.Node, location *ast
 	return result
 }
 
+func (tx *RuntimeSyntaxTransformer) shouldEmitEnumDeclaration(node *ast.EnumDeclaration) bool {
+	return !ast.IsEnumConst(node.AsNode()) || tx.compilerOptions.ShouldPreserveConstEnums()
+}
+
 func getInnermostModuleDeclarationFromDottedModule(moduleDeclaration *ast.ModuleDeclaration) *ast.ModuleDeclaration {
 	for moduleDeclaration.Body != nil && moduleDeclaration.Body.Kind == ast.KindModuleDeclaration {
 		moduleDeclaration = moduleDeclaration.Body.AsModuleDeclaration()

internal/transformers/tstransforms/runtimesyntax_test.go

@@ -200,11 +200,7 @@ var E;
     E[E["B"] = 1] = "B";
 })(E || (E = {}));`},
 
-		{title: "const enum", input: "const enum E {A, B}", output: `var E;
-(function (E) {
-    E[E["A"] = 0] = "A";
-    E[E["B"] = 1] = "B";
-})(E || (E = {}));`},
+		{title: "const enum", input: "const enum E {A, B}", output: ""},
 
 		{title: "merged enum", input: "enum E {A} enum E {B=A}", output: `var E;
 (function (E) {

testdata/baselines/reference/compiler/preserveConstEnums(preserveconstenums=false).js

@@ -0,0 +1,16 @@
+//// [tests/cases/compiler/preserveConstEnums.ts] ////
+
+//// [preserveConstEnums.ts]
+const enum A {
+    B = 1
+}
+
+const a = A.B
+
+
+//// [preserveConstEnums.js]
+var A;
+(function (A) {
+    A[A["B"] = 1] = "B";
+})(A || (A = {}));
+const a = 1 /* A.B */;

testdata/baselines/reference/compiler/preserveConstEnums(preserveconstenums=false).symbols

@@ -0,0 +1,16 @@
+//// [tests/cases/compiler/preserveConstEnums.ts] ////
+
+=== preserveConstEnums.ts ===
+const enum A {
+>A : Symbol(A, Decl(preserveConstEnums.ts, 0, 0))
+
+    B = 1
+>B : Symbol(A.B, Decl(preserveConstEnums.ts, 0, 14))
+}
+
+const a = A.B
+>a : Symbol(a, Decl(preserveConstEnums.ts, 4, 5))
+>A.B : Symbol(A.B, Decl(preserveConstEnums.ts, 0, 14))
+>A : Symbol(A, Decl(preserveConstEnums.ts, 0, 0))
+>B : Symbol(A.B, Decl(preserveConstEnums.ts, 0, 14))
+

testdata/baselines/reference/compiler/preserveConstEnums(preserveconstenums=false).types

@@ -0,0 +1,17 @@
+//// [tests/cases/compiler/preserveConstEnums.ts] ////
+
+=== preserveConstEnums.ts ===
+const enum A {
+>A : A
+
+    B = 1
+>B : A.B
+>1 : 1
+}
+
+const a = A.B
+>a : A.B
+>A.B : A
+>A : typeof A
+>B : A
+

testdata/baselines/reference/compiler/preserveConstEnums(preserveconstenums=true).js

@@ -0,0 +1,16 @@
+//// [tests/cases/compiler/preserveConstEnums.ts] ////
+
+//// [preserveConstEnums.ts]
+const enum A {
+    B = 1
+}
+
+const a = A.B
+
+
+//// [preserveConstEnums.js]
+var A;
+(function (A) {
+    A[A["B"] = 1] = "B";
+})(A || (A = {}));
+const a = 1 /* A.B */;

testdata/baselines/reference/compiler/preserveConstEnums(preserveconstenums=true).symbols

@@ -0,0 +1,16 @@
+//// [tests/cases/compiler/preserveConstEnums.ts] ////
+
+=== preserveConstEnums.ts ===
+const enum A {
+>A : Symbol(A, Decl(preserveConstEnums.ts, 0, 0))
+
+    B = 1
+>B : Symbol(A.B, Decl(preserveConstEnums.ts, 0, 14))
+}
+
+const a = A.B
+>a : Symbol(a, Decl(preserveConstEnums.ts, 4, 5))
+>A.B : Symbol(A.B, Decl(preserveConstEnums.ts, 0, 14))
+>A : Symbol(A, Decl(preserveConstEnums.ts, 0, 0))
+>B : Symbol(A.B, Decl(preserveConstEnums.ts, 0, 14))
+

testdata/baselines/reference/compiler/preserveConstEnums(preserveconstenums=true).types

@@ -0,0 +1,17 @@
+//// [tests/cases/compiler/preserveConstEnums.ts] ////
+
+=== preserveConstEnums.ts ===
+const enum A {
+>A : A
+
+    B = 1
+>B : A.B
+>1 : 1
+}
+
+const a = A.B
+>a : A.B
+>A.B : A
+>A : typeof A
+>B : A
+

testdata/tests/cases/compiler/preserveConstEnums.ts

@@ -0,0 +1,7 @@
+// @preserveConstEnums: true,false
+
+const enum A {
+    B = 1
+}
+
+const a = A.B