Skip to content

Commit c3bf2eb

Browse files
Minor skills cleanup
1 parent 5a07425 commit c3bf2eb

2 files changed

Lines changed: 14 additions & 6 deletions

File tree

  • .claude/skills

.claude/skills/migrate-groovy-to-java/SKILL.md

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,13 @@ TableTest usage
4040

4141
JDK 8 rules:
4242
- No text blocks.
43-
- @TableTest must use String[] annotation array syntax: `@TableTest({ "a | b", "1 | 2" })`
43+
- @TableTest must use String[] annotation array syntax:
44+
```
45+
@TableTest({
46+
"a | b",
47+
"1 | 2"
48+
})
49+
```
4450

4551
Spock `where:`@TableTest:
4652
- First row = header (column names = method parameters).

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

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,13 +16,17 @@ Dependency:
1616
- Groovy: testImplementation libs.tabletest
1717
- Kotlin: testImplementation(libs.tabletest)
1818

19-
Import:
20-
- Ensure: import org.tabletest.junit.TableTest;
19+
Import: `import org.tabletest.junit.TableTest;`
2120

2221
JDK 8 rules:
2322
- No text blocks.
2423
- @TableTest must use String[] annotation array syntax:
25-
@TableTest({ "a | b", "1 | 2" })
24+
```
25+
@TableTest({
26+
"a | b",
27+
"1 | 2"
28+
})
29+
```
2630

2731
Table formatting rules (mandatory):
2832
- Always include a header row (parameter names).
@@ -64,11 +68,9 @@ Cleanup:
6468

6569
Mixed eligibility:
6670
- Prefer combining `@TableTest` + `@MethodSource` on one `@ParameterizedTest` when only some cases are complex.
67-
- Use `@MethodSource` only when tabular representation is not practical for the test.
6871

6972
Do NOT convert when:
7073
- Most rows require complex builders/mocks.
71-
- Parameters are arrays (String[], int[]) — keep @MethodSource (or refactor to List to convert).
7274

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

0 commit comments

Comments
 (0)