Skip to content

[WIP] Replace JUnit testing framework with TestNG#505

Closed
Copilot wants to merge 1 commit intomasterfrom
copilot/replace-junit-with-testng
Closed

[WIP] Replace JUnit testing framework with TestNG#505
Copilot wants to merge 1 commit intomasterfrom
copilot/replace-junit-with-testng

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 23, 2026

Migration Plan: Replace JUnit with TestNG

  • Update pom.xml dependency
    • Remove JUnit 4.13.2 dependency
    • Add TestNG dependency
    • Update maven-surefire-plugin if needed
  • Update all test files to use TestNG
    • Replace JUnit imports with TestNG imports
    • Replace @test annotations (org.junit.Test → org.testng.annotations.Test)
    • Replace @before with @BeforeMethod
    • Replace Assert.* with TestNG Assert.*
    • Update TestUtil helper class
  • Verify all tests still pass
    • Run mvn test to ensure all tests work with TestNG
  • Clean up: ensure no JUnit references remain
Original prompt

This section details on the original issue you should resolve

<issue_title>[feature] replace JUnit testing with TestNG</issue_title>
<issue_description>need to remove JUnit totally and replace with TestNG: https://github.com/testng-team/testng</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@nomeguy nomeguy marked this pull request as ready for review January 23, 2026 07:04
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.08%. Comparing base (aa90a03) to head (76e6a24).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #505   +/-   ##
=======================================
  Coverage   68.08%   68.08%           
=======================================
  Files          63       63           
  Lines        3083     3083           
  Branches      574      574           
=======================================
  Hits         2099     2099           
  Misses        801      801           
  Partials      183      183           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@nomeguy nomeguy closed this Jan 23, 2026
Copilot AI requested a review from nomeguy January 23, 2026 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feature] replace JUnit testing with TestNG

4 participants