diff --git a/.github/skills/code-review/SKILL.md b/.github/skills/code-review/SKILL.md index 1524fd707f..0d8680a500 100644 --- a/.github/skills/code-review/SKILL.md +++ b/.github/skills/code-review/SKILL.md @@ -26,12 +26,14 @@ Deeper references, when a review needs them: `org.junit.jupiter` imports. Two styles coexist and a new test must match the style already in the file it joins: -- **JUnit 3 style** — around 117 classes extend `XWorkTestCase`, which extends +- **JUnit 3 style** — classes extending `XWorkTestCase`, which extends `junit.framework.TestCase`. Test methods must be named `testXxx()`. A Jupiter `@Test` annotation added to one of these **silently never runs** — it does not fail, it is simply not collected. Flag this as blocking whenever you see `org.junit.jupiter` in a diff. -- **JUnit 4 style** — around 212 classes use `import org.junit.Test`. +- **JUnit 4 style** — classes using `import org.junit.Test`. + +Both styles are widespread and neither is being migrated away from. AssertJ assertions and Mockito mocks are both available and widely used. Introducing JUnit 5 is a build-infrastructure change that needs its own `WW-` diff --git a/CLAUDE.md b/CLAUDE.md index 98e8f53826..7d113577f9 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -79,10 +79,13 @@ Run with `mvn test -DskipAssembly`. **Tests are JUnit 4 — there is no JUnit 5 anywhere in this repo.** `parent/pom.xml` declares `junit:junit:4.13.2`; there are zero `org.junit.jupiter` imports. Two styles coexist: -- **JUnit 3 style** — ~114 classes extend `XWorkTestCase` (which extends `junit.framework.TestCase`). +- **JUnit 3 style** — classes extending `XWorkTestCase` (which extends `junit.framework.TestCase`). Methods must be named `testXxx()`. A Jupiter `@Test` annotation added to one of these **silently never runs** — it does not fail, it is simply not collected. -- **JUnit 4 style** — ~210 classes use `import org.junit.Test`. +- **JUnit 4 style** — classes using `import org.junit.Test`. + +Both styles are widespread and neither is being migrated away from; `grep -rl` for either marker +gives the current split. Before adding a test, open the target file and match the style already there. AssertJ assertions and Mockito mocks are both available and widely used. Introducing Jupiter is a build-infrastructure change