Extend test coverage for using TOML version catalog

See #82

Author: jjohannes

samples/versions-in-catalog/gradle/libs.versions.toml

@@ -4,5 +4,4 @@ com_fasterxml_jackson_databind = "2.12.5"
 org_slf4j = "1.7.28"
 org_slf4j_simple = "1.7.28"
 
-org_junit_jupiter_api = "5.7.2"
-org_junit_jupiter_engine = "5.7.2"
+org-junit-jupiter-api = "5.7.2"

src/test/groovy/org/gradlex/javamodule/dependencies/test/CustomizationTest.groovy

@@ -84,8 +84,8 @@ class CustomizationTest extends Specification {
         given:
         settingsFile << '''
             dependencyResolutionManagement.versionCatalogs.create("modules") {
-                version("org_apache_xmlbeans", "5.0.1")
-                version("com_fasterxml_jackson_databind", "2.12.5")
+                version("org.apache.xmlbeans", "5.0.1")
+                version("com.fasterxml.jackson.databind", "2.12.5")
             }
         '''
         appBuildFile << '''
@@ -136,4 +136,42 @@ class CustomizationTest extends Specification {
         then:
         result.output.contains('[jakarta.mail-2.0.1.jar, jakarta.servlet-api-6.0.0.jar, jakarta.inject-api-1.0.5.jar, jakarta.activation-2.0.1.jar]')
     }
+
+    def "can use toml catalog with '_' for '.'"() {
+        given:
+        file('gradle/libs.versions.toml') << '''
+            [versions]
+            org_apache_xmlbeans = "5.0.1"
+        '''.stripIndent()
+        appModuleInfoFile << '''
+            module org.gradlex.test.app { 
+                requires org.apache.xmlbeans;
+            }
+        '''
+
+        when:
+        def result = build()
+
+        then:
+        !result.output.contains('[WARN]')
+    }
+
+    def "can use toml catalog with '-' for '.'"() {
+        given:
+        file('gradle/libs.versions.toml') << '''
+            [versions]
+            org-apache-xmlbeans = "5.0.1"
+        '''.stripIndent()
+        appModuleInfoFile << '''
+            module org.gradlex.test.app { 
+                requires org.apache.xmlbeans;
+            }
+        '''
+
+        when:
+        def result = build()
+
+        then:
+        !result.output.contains('[WARN]')
+    }
 }

src/test/groovy/org/gradlex/javamodule/dependencies/test/WarningsTest.groovy

@@ -8,34 +8,6 @@ class WarningsTest extends Specification {
     @Delegate
     GradleBuild build = new GradleBuild()
 
-    def "warning for version missing in catalog can be disabled"() {
-        given:
-        file('gradle/libs.versions.toml') << '''
-            [versions]
-            org_apache_xmlbeans = "5.0.1"
-        '''.stripIndent()
-        appBuildFile << '''
-            javaModuleDependencies.warnForMissingVersions.set(false)
-            dependencies.constraints {
-                javaModuleDependencies {
-                    implementation(gav("com.fasterxml.jackson.databind", "2.12.5"))
-                }
-            }
-        '''
-        appModuleInfoFile << '''
-            module org.gradlex.test.app { 
-                requires com.fasterxml.jackson.databind;
-                requires org.apache.xmlbeans;
-            }
-        '''
-
-        when:
-        def result = build()
-
-        then:
-        !result.output.contains('[WARN]')
-    }
-
     def "prints warning for missing mapping"() {
         given:
         appModuleInfoFile << '''