Lualib omit when unused (#280)

hazzard993 · Perryvw · commit 9eb59627e06d · 2018-11-11T09:34:46.000+01:00
* lualib inline omit header when no features are used

* Tests to enforce no lualib text when unused, unless using always
diff --git a/src/Transpiler.ts b/src/Transpiler.ts
@@ -233,7 +233,7 @@ export abstract class LuaTranspiler {
         }
 
         // Inline lualib features
-        if (this.options.luaLibImport === LuaLibImportKind.Inline) {
+        if (this.options.luaLibImport === LuaLibImportKind.Inline && this.luaLibFeatureSet.size > 0) {
             result += "\n" + "-- Lua Library Imports\n";
             for (const feature of this.luaLibFeatureSet) {
                 const featureFile = path.resolve(__dirname, `../dist/lualib/${feature}.lua`);
diff --git a/test/unit/modules.spec.ts b/test/unit/modules.spec.ts
@@ -23,15 +23,6 @@ export class LuaModuleTests {
         Expect(lua.startsWith(`require("lualib_bundle")`));
     }
 
-    @Test("lualibRequireNoUses")
-    public lualibRequireNoUses(): void {
-        // Transpile
-        const lua = util.transpileString(``, { luaLibImport: LuaLibImportKind.Require, luaTarget: LuaTarget.LuaJIT });
-
-        // Assert
-        Expect(lua).toBe(``);
-    }
-
     @Test("lualibRequireAlways")
     public lualibRequireAlways(): void {
         // Transpile
@@ -63,4 +54,16 @@ export class LuaModuleTests {
 
         Expect(result).toBe(3);
     }
+
+    @TestCase(LuaLibImportKind.Inline)
+    @TestCase(LuaLibImportKind.None)
+    @TestCase(LuaLibImportKind.Require)
+    @Test("LuaLib no uses? No code")
+    public lualibNoUsesNoCode(impKind: LuaLibImportKind): void {
+        // Transpile
+        const lua = util.transpileString(``, { luaLibImport: impKind });
+
+        // Assert
+        Expect(lua).toBe(``);
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -233,7 +233,7 @@ export abstract class LuaTranspiler {`
`233`	`233`	`}`
`234`	`234`
`235`	`235`	`// Inline lualib features`
`236`		`- if (this.options.luaLibImport === LuaLibImportKind.Inline) {`
	`236`	`+ if (this.options.luaLibImport === LuaLibImportKind.Inline && this.luaLibFeatureSet.size > 0) {`
`237`	`237`	`result += "\n" + "-- Lua Library Imports\n";`
`238`	`238`	`for (const feature of this.luaLibFeatureSet) {`
`239`	`239`	const featureFile = path.resolve(__dirname, `../dist/lualib/${feature}.lua`);