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

refactor: add missing tests from simulation mode implementation #585

Merged
merged 7 commits into from
Apr 29, 2024

Conversation

vbreuss
Copy link
Member

@vbreuss vbreuss commented Apr 29, 2024

Add tests for some edge cases from the implementation of the simulation mode that were previously forgotten.

@vbreuss vbreuss added the refactor A change or improvement without functional impact label Apr 29, 2024
@vbreuss vbreuss self-assigned this Apr 29, 2024
@vbreuss vbreuss changed the title refactor: add missing tests from path simulation implementation refactor: add missing tests from simulation mode implementation Apr 29, 2024
Copy link

Stryker.NET mutation tested the changes in the pull request:
Mutation testing badge

@vbreuss vbreuss force-pushed the topic/add-missing-tests branch from 2933430 to 4aee39d Compare April 29, 2024 14:27
@vbreuss vbreuss force-pushed the topic/add-missing-tests branch from 9f93141 to 9b3c1c1 Compare April 29, 2024 15:45
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@vbreuss vbreuss marked this pull request as ready for review April 29, 2024 16:06
@vbreuss vbreuss merged commit bf27a22 into main Apr 29, 2024
12 checks passed
@vbreuss vbreuss deleted the topic/add-missing-tests branch April 29, 2024 16:10
Copy link

github-actions bot commented May 5, 2024

This is addressed in release v3.2.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor A change or improvement without functional impact state: released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant