Fix test formatting and add tabletest to java_deps

Author: sarahchen6

.claude/skills/migrate-junit-source-to-tabletest/SKILL.md

@@ -12,9 +12,8 @@ Process (do in this order):
 5) Run module tests once and verify "BUILD SUCCESSFUL". If failed, inspect JUnit XML report.
 
 Dependency:
-- If missing, add:
-  - Groovy: testImplementation libs.tabletest
-  - Kotlin: testImplementation(libs.tabletest)
+- @TableTest should be in `gradle/java_deps.gradle` as `testImplementation libs.tabletest` where Java modules inherit it via `gradle/java.gradle`.
+- As a fallback, add module-specific `testImplementation(libs.tabletest)` or `testImplementation libs.tabletest`.
 
 Import: `import org.tabletest.junit.TableTest;`
 

dd-trace-api/build.gradle.kts

@@ -75,6 +75,5 @@ dependencies {
   api(libs.slf4j)
   testImplementation(libs.guava)
   testImplementation(libs.bundles.mockito)
-  testImplementation(libs.tabletest)
   testImplementation(project(":utils:test-utils"))
 }

dd-trace-api/src/test/java/datadog/trace/api/DDTraceIdTest.java

@@ -159,18 +159,18 @@ void failParsingIllegal128BitIdHexadecimalStringRepresentation(String hexId) {
   }
 
   @TableTest({
-    "scenario             | hexId               | start | length | lowerCaseOnly",
-    "null string          |                     | 0     | 0      | true         ",
-    "empty string         | ''                  | 0     | 0      | true         ",
-    "out of bound length  | '123456789abcdef0'  | 0     | 17     | true         ",
-    "out of bound end     | '123456789abcdef0'  | 7     | 10     | true         ",
-    "out of bound start   | '123456789abcdef0'  | 17    | 0      | true         ",
-    "invalid minus one    | '-1'                | 0     | 1      | true         ",
-    "invalid minus a      | '-a'                | 0     | 1      | true         ",
-    "invalid character    | '123abcg'           | 0     | 7      | true         ",
-    "invalid upper case A | 'A'                 | 0     | 1      | true         ",
-    "invalid upper case   | '123ABC'            | 0     | 6      | true         ",
-    "too long             | '111111111111111111111111111111111' | 0 | 33 | true"
+    "scenario             | hexId                               | start | length | lowerCaseOnly",
+    "null string          |                                     | 0     | 0      | true         ",
+    "empty string         | ''                                  | 0     | 0      | true         ",
+    "out of bound length  | '123456789abcdef0'                  | 0     | 17     | true         ",
+    "out of bound end     | '123456789abcdef0'                  | 7     | 10     | true         ",
+    "out of bound start   | '123456789abcdef0'                  | 17    | 0      | true         ",
+    "invalid minus one    | '-1'                                | 0     | 1      | true         ",
+    "invalid minus a      | '-a'                                | 0     | 1      | true         ",
+    "invalid character    | '123abcg'                           | 0     | 7      | true         ",
+    "invalid upper case A | 'A'                                 | 0     | 1      | true         ",
+    "invalid upper case   | '123ABC'                            | 0     | 6      | true         ",
+    "too long             | '111111111111111111111111111111111' | 0     | 33     | true         "
   })
   @ParameterizedTest
   void failParsingIllegal128BitIdHexadecimalStringRepresentationFromPartialString(

gradle/java_deps.gradle

@@ -3,6 +3,7 @@ apply plugin: 'groovy'
 dependencies {
   testImplementation libs.bundles.spock
   testImplementation libs.bundles.groovy
+  testImplementation libs.tabletest
 
   testImplementation libs.bundles.test.logging