Modernize test infrastructure: JUnit 4 migration and Maven structure#214
Merged
Modernize test infrastructure: JUnit 4 migration and Maven structure#214
Conversation
Phase 1 of test infrastructure modernization: - Move production code from src/cc/ to src/main/java/cc/ - Move test code to src/test/java/ with updated package names - Move test resources to src/test/resources/ - Update pom.xml for new directory structure - Update CI/CD to use Java 17 with test artifact collection - Remove Ant build (build.xml) and bundled JUnit jars - Remove unused test imports from production code - Disable parallel test execution (some tests capture System.out) All 195 tests pass. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Migrate 57 test files from JUnit 3 to JUnit 4 style: - Remove TestCase inheritance and JUnit 3 boilerplate - Add @test annotations to all test methods - Add @before annotations where setUp() methods exist - Update imports to use org.junit.* and static assertions - Create TESTING.md with comprehensive testing guide - Update README.md with Maven build instructions - Update .gitignore for Maven project structure All 194 tests pass after migration. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
src/main/java,src/test/java)@TestannotationsChanges
Build System
src/main/java,src/test/java)src/test/java/cc/mallet/{module}/(same package as production code)Test Migration (JUnit 3 → JUnit 4)
extends TestCasefrom all test classessuite(),main(), constructors)@Testannotations to all test methods@Beforeannotations wheresetUp()methods existorg.junit.*and static assertionsDocumentation
TESTING.mdwith comprehensive testing guideREADME.mdwith Maven build instructions.gitignorefor Maven project structureTest plan
mvn clean test(194 tests pass)mvn jacoco:report(16% instruction coverage)🤖 Generated with Claude Code