Language configuration is now case sensitive

casid · Oct 6, 2023 · 1888f90 · 1888f90
1 parent a059cd3
commit 1888f90
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/jte-models/src/main/java/gg/jte/models/generator/ModelConfig.java b/jte-models/src/main/java/gg/jte/models/generator/ModelConfig.java
@@ -22,7 +22,7 @@ public String implementationAnnotation() {
     public Language language() {
         String configuredLanguage = map.getOrDefault("language", "Java");
         try {
-            return Language.valueOf(configuredLanguage.toUpperCase());
+            return Language.valueOf(configuredLanguage);
         } catch (IllegalArgumentException ex) {
             String supportedValues = Arrays.toString(Language.values());
             throw new IllegalArgumentException(

diff --git a/jte-models/src/test/java/gg/jte/models/generator/TestModelConfig.java b/jte-models/src/test/java/gg/jte/models/generator/TestModelConfig.java
@@ -52,9 +52,10 @@ public void languageConfigurationSupportsKotlin() {
     }
 
     @Test
-    public void languageConfigurationIsCaseInsensitive() {
-        var modelConfig = new ModelConfig(Map.of("language", "jAvA"));
-        assertEquals(modelConfig.language(), Language.JAVA);
+    public void languageConfigurationIsCaseSensitive() {
+        assertThrows(IllegalArgumentException.class, () -> {
+            new ModelConfig(Map.of("language", "jAvA")).language();
+        });
     }
 
     @Test