From 7cb7ff1af970c1e759e92e5e7bee10a13ffe8db5 Mon Sep 17 00:00:00 2001
From: Andreas Hager <andy@mazebert.com>
Date: Sat, 30 Sep 2023 08:35:00 +0200
Subject: [PATCH] #278 template names are properly normalized

---
 .../java/gg/jte/compiler/TemplateCompiler.java   |  2 ++
 .../main/java/gg/jte/compiler/module/Module.java | 12 ++++++++++++
 .../java/gg/jte/TemplateEngine_ModulesTest.java  | 16 ++++++++++++++++
 3 files changed, 30 insertions(+)

diff --git a/jte/src/main/java/gg/jte/compiler/TemplateCompiler.java b/jte/src/main/java/gg/jte/compiler/TemplateCompiler.java
index a8665ba9..a8f4e1cd 100644
--- a/jte/src/main/java/gg/jte/compiler/TemplateCompiler.java
+++ b/jte/src/main/java/gg/jte/compiler/TemplateCompiler.java
@@ -284,6 +284,8 @@ public ClassInfo generateTemplateCall(String name, LinkedHashSet<ClassDefinition
         Module templateModule = module.resolve(name);
         CodeResolver codeResolver = templateModule.getCodeResolver();
 
+        name = templateModule.normalize(name);
+
         templateDependencies.add(new TemplateDependency(name, codeResolver.getLastModified(name)));
         ClassInfo classInfo = new ClassInfo(name, config.packageName);
 
diff --git a/jte/src/main/java/gg/jte/compiler/module/Module.java b/jte/src/main/java/gg/jte/compiler/module/Module.java
index db58d687..0ca20feb 100644
--- a/jte/src/main/java/gg/jte/compiler/module/Module.java
+++ b/jte/src/main/java/gg/jte/compiler/module/Module.java
@@ -44,6 +44,18 @@ public Module resolve(String name) {
       return result;
    }
 
+   public String normalize(String name) {
+      if (isRoot()) {
+         return name;
+      } else {
+         if (name.startsWith(alias + "/")) {
+            return name;
+         } else {
+            return alias + "/" + name;
+         }
+      }
+   }
+
    public boolean isRoot() {
       return alias.isEmpty();
    }
diff --git a/jte/src/test/java/gg/jte/TemplateEngine_ModulesTest.java b/jte/src/test/java/gg/jte/TemplateEngine_ModulesTest.java
index a7f8c2a3..095053ec 100644
--- a/jte/src/test/java/gg/jte/TemplateEngine_ModulesTest.java
+++ b/jte/src/test/java/gg/jte/TemplateEngine_ModulesTest.java
@@ -4,6 +4,7 @@
 import static org.assertj.core.api.Assertions.catchThrowable;
 
 import java.nio.file.Path;
+import java.util.List;
 
 import org.junit.jupiter.api.Test;
 
@@ -70,5 +71,20 @@ <p>line chart (checkout)</p>
               line chart (core)""");
    }
 
+   @Test
+   void emptyTopLevelModule_generateAll() {
+      DirectoryCodeResolver codeResolver = new DirectoryCodeResolver(Path.of("src/test/modules/empty-top-level-module"));
+      TemplateEngine templateEngine = TemplateEngine.create(codeResolver, ContentType.Html);
+
+      List<String> templates = templateEngine.generateAll();
+
+      assertThat(templates).containsExactlyInAnyOrder(
+              "gg/jte/generated/ondemand/apexcharts/JtelinechartGenerated.java",
+              "gg/jte/generated/ondemand/checkout/JtepageGenerated.java",
+              "gg/jte/generated/ondemand/core/JtelayoutGenerated.java",
+              "gg/jte/generated/ondemand/core/component/JtecssGenerated.java"
+      );
+   }
+
    // TODO adjust precompileAll() and generateAll() to iterate over all module files as well!
 }