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.

Sign up for GitHub

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

Jump to bottom

Feature/only important integration tests 4 #847

Draft
wants to merge 14 commits into
base: main
Choose a base branch
from
Draft

Feature/only important integration tests 4 #847

wants to merge 14 commits into from
+1,696 −1,488

Conversation

mgovers
Copy link
Member

@mgovers mgovers commented Nov 28, 2024 •
edited
Loading

  • Splits math solver into separate test files per solver type
    • migrate PF solvers
    • migrate SE solvers
    • migrate SC solver
    • migrate other tests
    • delete test_math_solver.cpp
  • Formats test case templates more nicely

Relates to #486
Continuation of:

@mgovers mgovers added improvement Improvement on internal implementation do-not-merge This should not be merged labels Nov 28, 2024
@mgovers mgovers self-assigned this Nov 28, 2024
@mgovers mgovers removed the do-not-merge This should not be merged label Nov 28, 2024
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Nov 28, 2024

Quality Gate Passed Quality Gate passed

Issues
14 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@mgovers
sonar cloud
426a238 
Signed-off-by: Martijn Govers <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mgovers mgovers

Labels
improvement Improvement on internal implementation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mgovers