Verify class names, package names, and module names

src/main/kotlin/fixie/generator/module/FixieModule.kt

@@ -122,8 +122,19 @@ class FixieModule(
 
 		for (className in allClassNames) {
 			if (allClassNames.count { it == className } > 1) {
-				throw IllegalArgumentException("Duplicate class name $className")
+				throw InvalidConfigException("Duplicate class name $className")
 			}
+			if (!className.matches(Regex("[A-Z_](\$[A-Z_]|[\\w_])*"))) {
+				throw InvalidConfigException("Invalid class name $className")
+			}
+		}
+
+		if (!packageName.matches(Regex("^[a-z][a-z0-9_]*(\\.[a-z0-9A-Z_]+)*[a-z0-9A-Z_]*\$"))) {
+			throw InvalidConfigException("Invalid package name $packageName")
+		}
+
+		if (!moduleName.matches(Regex("[a-z0-9A-Z_-]+"))) {
+			throw InvalidConfigException("Invalid module name $moduleName")
 		}
 	}
 }

src/test/kotlin/fixie/generator/parser/TestParser.kt

@@ -230,7 +230,7 @@ class TestParser {
 		val modules = parseModule(
 			"""{
             packageName = 'kit.%TYPE%.%OVERFLOW%',
-            moduleName = 'module.kit.%TYPE%.%OVERFLOW%',
+            moduleName = 'module-kit-%TYPE%-%OVERFLOW%',
             numbers = [{
                 className = 'Hello',
                 internalType = '%TYPE%',
@@ -318,4 +318,134 @@ class TestParser {
 			"Expected $error to contain 'you must use all variations in the moduleName to avoid this'"
 		)
 	}
+
+	@Test
+	fun testDuplicateClassName() {
+		assertEquals(
+			"Duplicate class name Hello",
+			assertThrows<InvalidConfigException> {
+				parseModule(
+					"""{
+            packageName = 'hello',
+            moduleName = 'hello',
+            numbers = [{
+                className = 'Hello',
+                internalType = 'Short',
+                oneValue = 50,
+                checkOverflow = false
+            }],
+            displacements = [{
+				className = 'Hello',
+				number = 'Hello',
+				oneUnit = 'Meter',
+				displayUnit = 'Meter',
+				createNumberExtensions = false
+			}]
+        }"""
+				)
+			}.message
+		)
+	}
+
+	@Test
+	fun testInvalidModuleName() {
+		assertEquals(
+			"Invalid module name hel lo",
+			assertThrows<InvalidConfigException> {
+				parseModule(
+					"""{
+            packageName = 'hello',
+            moduleName = 'hel lo'
+        }"""
+				)
+			}.message
+		)
+	}
+
+	@Test
+	fun testInvalidPackageName() {
+		assertEquals(
+			"Invalid package name hel lo",
+			assertThrows<InvalidConfigException> {
+				parseModule(
+					"""{
+            packageName = 'hel lo',
+            moduleName = 'hello'
+        }"""
+				)
+			}.message
+		)
+	}
+
+	@Test
+	fun testInvalidClassName() {
+		assertEquals(
+			"Invalid class name Hel lo",
+			assertThrows<InvalidConfigException> {
+				parseModule(
+					"""{
+            packageName = 'hello',
+            moduleName = 'hello',
+			numbers = [{
+                className = 'Hel lo',
+                internalType = 'Short',
+                oneValue = 50,
+                checkOverflow = false
+            }],
+        }"""
+				)
+			}.message
+		)
+	}
+
+	@Test
+	fun testEmptyModuleName() {
+		assertEquals(
+			"Invalid module name ",
+			assertThrows<InvalidConfigException> {
+				parseModule(
+					"""{
+            packageName = 'hello',
+            moduleName = ''
+        }"""
+				)
+			}.message
+		)
+	}
+
+	@Test
+	fun testEmptyPackageName() {
+		assertEquals(
+			"Invalid package name ",
+			assertThrows<InvalidConfigException> {
+				parseModule(
+					"""{
+            packageName = '',
+            moduleName = 'hello'
+        }"""
+				)
+			}.message
+		)
+	}
+
+	@Test
+	fun testEmptyClassName() {
+		assertEquals(
+			"Invalid class name ",
+			assertThrows<InvalidConfigException> {
+				parseModule(
+					"""{
+            packageName = 'hello',
+            moduleName = 'hello',
+			numbers = [{
+                className = '',
+                internalType = 'Short',
+                oneValue = 50,
+                checkOverflow = false
+            }],
+        }"""
+				)
+			}.message
+		)
+	}
 }