CtrlK
BlogDocsLog inGet started
Tessl Logo

migrate-junit-source-to-tabletest

Convert JUnit 5 @MethodSource/@CsvSource/@ValueSource parameterized tests to @TableTest (JDK8)

84

Quality

80%

Does it follow best practices?

Impact

Pending

No eval scenarios have been run

SecuritybySnyk

Passed

No known issues

Optimize this skill with Tessl

npx tessl skill review --optimize ./.claude/skills/migrate-junit-source-to-tabletest/SKILL.md
SKILL.md
Quality
Evals
Security

Goal: Migrate JUnit 5 parameterized tests using @MethodSource/@CsvSource/@ValueSource to @TableTest with minimal churn and passing tests.

Process (do in this order):

  1. Locate targets via Grep (no agent subprocess). Search for: "@ParameterizedTest", "@MethodSource", "@CsvSource", "@ValueSource".
  2. Read all matching files up front (parallel is OK).
  3. Convert eligible tests to @TableTest.
  4. Write each modified file once in full using Write (no incremental per-test edits).
  5. Run module tests once and verify "BUILD SUCCESSFUL". If failed, inspect JUnit XML report.

Import: import org.tabletest.junit.TableTest;

JDK 8 rules:

  • No text blocks.
  • @TableTest must use String[] annotation array syntax:
    @TableTest({
     "a | b",
     "1 | 2"
    })

Table formatting rules (mandatory):

  • Always include a header row (parameter names).
  • Always add a "scenario" column; using common sense for naming; scenario is NOT a method parameter.
  • Use '|' as delimiter.
  • Align columns with spaces so pipes line up vertically.
  • Prefer single quotes for strings requiring quotes (e.g., 'a|b', '[]', '{}', ' ').
  • Use value sets ({a, b, c}) instead of matrix-style repetition when only one dimension varies across otherwise-identical rows.

Conversions: A) @CsvSource

  • Remove @ParameterizedTest and @CsvSource.
  • If delimiter is '|': rows map directly to @TableTest.
  • If delimiter is ',' (default): replace ',' with '|' in rows.

B) @ValueSource

  • Keep single-parameter tests on @ValueSource (and @NullSource when null cases are needed).
  • Otherwise convert to @TableTest with header from parameter name.
  • Each value becomes one row.
  • Add "scenario" column using common sense for name.

C) @MethodSource (convert only if values are representable as strings)

  • Convert when argument values are primitives, strings, enums, booleans, nulls, and simple collection literals supported by TableTest:
    • Array: [a, b, ...]
    • List: [a, b, ...]
    • Set: {a, b, ...}
    • Map: [k: v, ...]
  • @TableTest and @MethodSource may be combined on the same @ParameterizedTest when most cases are tabular but a few cases require programmatic setup.
  • In combined mode, keep table-friendly cases in @TableTest, and put only non-tabular/complex cases in @MethodSource.
  • If @TableTest is not viable for the test at all, use @MethodSource only.
  • For @MethodSource, name the arguments method <testMethodName>Arguments (camelCase, e.g. testMethodArguments) and return Stream<Arguments> using Stream.of(...) and arguments(...) with static import.
  • Blank cell = null (non-primitive).
  • '' = empty string.
  • For String params that start with '[' or '{', quote to avoid collection parsing (prefer '[]'/'{}').

D) @TypeConverter

  • Use @TypeConverter for symbolic constants used by migrated table rows (e.g. Long.MAX_VALUE, DDSpanId.MAX).
  • Prefer explicit one-case-one-return mappings.
  • Prefer shared converter utilities (e.g. in utils/test-utils) when reuse across modules is likely.

Scenario handling:

  • If MethodSource includes a leading description string OR @ParameterizedTest(name=...) uses {0}, convert that to a scenario column and remove that parameter from method signature.

Cleanup:

  • Delete now-unused @MethodSource provider methods and unused imports.

Mixed eligibility:

  • Prefer combining @TableTest + @MethodSource on one @ParameterizedTest when only some cases are complex.

Do NOT convert when:

  • Most rows require complex builders/mocks.

Test command (exact): ./gradlew :path:to:module:test --rerun-tasks 2>&1 | tail -20

  • If BUILD FAILED: cat path/to/module/build/test-results/test/TEST-*.xml
Repository
DataDog/dd-trace-java
Last updated
Created

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.