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

Migrate to Catch v3.0.1 #82

Merged
merged 14 commits into from
Jun 4, 2022
Merged

Migrate to Catch v3.0.1 #82

merged 14 commits into from
Jun 4, 2022

Conversation

phbasler
Copy link
Owner

Closes #78
Migrate to the new Catch version

@codecov-commenter
Copy link

Codecov Report

Merging #82 (b8bde90) into develop (af9aeaa) will increase coverage by 0.76%.
The diff coverage is 80.00%.

@@             Coverage Diff             @@
##           develop      #82      +/-   ##
===========================================
+ Coverage    80.02%   80.79%   +0.76%     
===========================================
  Files           39       39              
  Lines        14194    14195       +1     
===========================================
+ Hits         11359    11469     +110     
+ Misses        2835     2726     -109     
Flag Coverage Δ
unittests 80.79% <80.00%> (+0.76%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...nclude/BSMPT/minimizer/LibCMAES/MinimizeLibCMAES.h 100.00% <ø> (ø)
include/BSMPT/minimizer/LibNLOPT/MinimizeNLOPT.h 90.90% <0.00%> (-9.10%) ⬇️
include/BSMPT/minimizer/Minimizer.h 50.00% <ø> (ø)
include/BSMPT/models/ClassPotentialOrigin.h 90.00% <ø> (ø)
src/minimizer/Minimizer.cpp 61.47% <45.45%> (+0.15%) ⬆️
src/minimizer/MinimizeGSL.cpp 90.07% <85.71%> (ø)
src/minimizer/LibNLOPT/MinimizeNLOPT.cpp 72.09% <92.30%> (ø)
include/BSMPT/minimizer/MinimizeGSL.h 100.00% <100.00%> (ø)
src/minimizer/LibCMAES/MinimizeLibCMAES.cpp 100.00% <100.00%> (ø)
src/models/ClassPotentialOrigin.cpp 54.43% <100.00%> (+4.27%) ⬆️
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9bb0922...b8bde90. Read the comment docs.

@phbasler phbasler merged commit d61fb64 into develop Jun 4, 2022
@phbasler phbasler deleted the CatchMigration branch June 4, 2022 15:18
guikos11 pushed a commit to guikos11/BSMPT that referenced this pull request Jul 28, 2024
* Migrated to Catch2WithMain

* Migrated event listener

* Added missing file

* Fixed catch2 links

* Removed temp required

* Fix exclude

* Code cov fault fix (phbasler#83)

* Turning vectorization off if coverage

* Run CI for this test

* Only allow vectorization if coverage flags are not set

* Migrated to Catch2WithMain

* Migrated event listener

* Added missing file

* Fixed catch2 links

* Removed temp required

* Fix exclude
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.

2 participants