Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

**DO-NOT-MERGE - Mutation tests setup #100

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
Open

**DO-NOT-MERGE - Mutation tests setup #100

wants to merge 19 commits into from

Conversation

hemantt
Copy link
Collaborator

@hemantt hemantt commented Feb 13, 2023

Before creating a pull request make sure that:

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:

JIRA link (if applicable)

Change description

Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[ ] No

@hemantt hemantt changed the title Mutation tests setup **DO-NOT-MERGE - Mutation tests setup Feb 14, 2023
paulridout and others added 2 commits March 13, 2023 16:53
- Changed mutation (pitest) test settings in main build.gradle file.  Moved pitest task inside subprojects section.  Classes to scan are now defined in module build.gradle files.
- Added pitest section to module build.gradle files to specify classes and test classes to scan.
- Removed testImplementations from module build.gradle files that weren't required.  These were causing issues with mutation test class scanning where the same package path was in more than one module.
- Removed pitest apply plugin command from module build.gradle files.  This is now being applied in main build.gradle subprojects section.
- Tidied up LoggingContextTest class.  Changed getNextLoggingIdTest() as it was failing during mutation test due to multiple threads changing logging id.
@hemantt hemantt requested review from paulridout and kiran-yenigala-hmcts and removed request for paulridout March 14, 2023 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants