From 0c8b159da25a96f12b7bd3de48e49bd611ea4e19 Mon Sep 17 00:00:00 2001
From: hond <hond@microsoft.com>
Date: Fri, 9 Oct 2020 11:18:24 +0800
Subject: [PATCH] [LG] Redo the default fallback of namespace

---
 libraries/botbuilder-lg/src/templates.ts      | 13 +++++----
 libraries/botbuilder-lg/tests/lg.test.js      | 27 +++++++++++++++++++
 .../injectionTest/injectWithoutNamespace.lg   |  4 +++
 3 files changed, 37 insertions(+), 7 deletions(-)
 create mode 100644 libraries/botbuilder-lg/tests/testData/examples/injectionTest/injectWithoutNamespace.lg

diff --git a/libraries/botbuilder-lg/src/templates.ts b/libraries/botbuilder-lg/src/templates.ts
index 79ef081a3e..3ee32702fc 100644
--- a/libraries/botbuilder-lg/src/templates.ts
+++ b/libraries/botbuilder-lg/src/templates.ts
@@ -27,7 +27,7 @@ import { AnalyzerResult } from './analyzerResult';
 import { TemplateErrors } from './templateErrors';
 import { TemplateExtensions } from './templateExtensions';
 import { EvaluationOptions, LGLineBreakStyle } from './evaluationOptions';
-import { isAbsolute, basename } from 'path';
+import { basename } from 'path';
 import { StaticChecker } from './staticChecker';
 import { LGResource } from './lgResource';
 import { CustomizedMemory } from './customizedMemory';
@@ -549,7 +549,10 @@ export class Templates implements Iterable<Template> {
             const globalFuncs = curTemplates.getGlobalFunctionTable(curTemplates.options);
             for (const templateName of globalFuncs) {
                 if (curTemplates.items.find((u) => u.name === templateName) !== undefined) {
-                    const newGlobalName = `${curTemplates.namespace}.${templateName}`;
+                    const prefix =
+                        !curTemplates.namespace || !curTemplates.namespace.trim() ? '' : curTemplates.namespace + '.';
+
+                    const newGlobalName = prefix + templateName;
                     Expression.functions.add(
                         newGlobalName,
                         new ExpressionEvaluator(
@@ -606,11 +609,7 @@ export class Templates implements Iterable<Template> {
     private extractNamespace(options: string[]): string {
         let result = this.extractOptionByKey(this.namespaceKey, options);
         if (!result) {
-            if (isAbsolute(this.source)) {
-                result = basename(this.source).split('.')[0];
-            } else {
-                throw new Error('namespace is required or the id should be an absoulte path!"');
-            }
+            result = basename(this.id || '').split('.')[0];
         }
 
         return result;
diff --git a/libraries/botbuilder-lg/tests/lg.test.js b/libraries/botbuilder-lg/tests/lg.test.js
index d2dfd7f8ff..9a118e624a 100644
--- a/libraries/botbuilder-lg/tests/lg.test.js
+++ b/libraries/botbuilder-lg/tests/lg.test.js
@@ -57,6 +57,7 @@ describe('LG', function() {
         MemoryAccess: Templates.parseFile(GetExampleFilePath('MemoryAccess.lg')),
         ReExecute: Templates.parseFile(GetExampleFilePath('ReExecute.lg')),
         inject: Templates.parseFile(GetExampleFilePath('./injectionTest/inject.lg')),
+        injectWithoutNamespace: Templates.parseFile(GetExampleFilePath('./injectionTest/injectWithoutNamespace.lg')),
         StrictModeTrue: Templates.parseFile(GetExampleFilePath('./EvaluationOptions/StrictModeTrue.lg')),
         a1: Templates.parseFile(GetExampleFilePath('./EvaluationOptions/a1.lg')),
         a2: Templates.parseFile(GetExampleFilePath('./EvaluationOptions/a2.lg')),
@@ -1413,4 +1414,30 @@ describe('LG', function() {
         var {value: evaled, error} = Expression.parse('common.sumFourNumbers(i, j, k, l)').tryEvaluate(scope2);
         assert.strictEqual(10, evaled);
     });
+
+    it('TestInjectLGWithoutNamespace', function() {
+        const lgPath = GetExampleFilePath('./injectionTest/injectWithoutNamespace.lg');
+        let resource = new LGResource('myId', lgPath, fs.readFileSync(lgPath, 'utf-8'));
+        Templates.parseResource(resource);
+
+        var { value: evaled, error } = Expression.parse('myId.greeting()').tryEvaluate({ name: 'Alice' });
+        assert(error === undefined);
+        assert.strictEqual('hi Alice', evaled);
+
+        // using the fuileName parsed from Id as the namespace
+        resource = new LGResource('./path/myNewId.lg', lgPath, fs.readFileSync(lgPath, 'utf-8'));
+        Templates.parseResource(resource);
+
+        var { value: evaled, error } = Expression.parse('myNewId.greeting()').tryEvaluate({ name: 'Alice' });
+        assert(error === undefined);
+        assert.strictEqual('hi Alice', evaled);
+
+        // With empty id
+        resource = new LGResource('', lgPath, fs.readFileSync(lgPath, 'utf-8'));
+        Templates.parseResource(resource);
+
+        var { value: evaled, error } = Expression.parse('greeting()').tryEvaluate({ name: 'Alice' });
+        assert(error === undefined);
+        assert.strictEqual('hi Alice', evaled);
+    });
 });
diff --git a/libraries/botbuilder-lg/tests/testData/examples/injectionTest/injectWithoutNamespace.lg b/libraries/botbuilder-lg/tests/testData/examples/injectionTest/injectWithoutNamespace.lg
new file mode 100644
index 0000000000..7b06bbf9d3
--- /dev/null
+++ b/libraries/botbuilder-lg/tests/testData/examples/injectionTest/injectWithoutNamespace.lg
@@ -0,0 +1,4 @@
+> !# @Exports = greeting
+
+# greeting
+- hi ${name}