Skip to content

Document CTest add_test()#128

Merged
ryanmrichard merged 20 commits intoCMakePP:masterfrom
AutonomicPerfectionist:document-ctest-add-test
Sep 19, 2022
Merged

Document CTest add_test()#128
ryanmrichard merged 20 commits intoCMakePP:masterfrom
AutonomicPerfectionist:document-ctest-add-test

Conversation

@AutonomicPerfectionist
Copy link
Contributor

@AutonomicPerfectionist AutonomicPerfectionist commented Sep 12, 2022

Is this pull request associated with an issue(s)?
Fixes #123

Description
This PR adds explicit support for documenting CTest's add_test() command. This allows regular tests to be documented in addition to the CMakeTest tests that were already supported.

TODOs

  • Add unit tests
  • Update documentation

@AutonomicPerfectionist AutonomicPerfectionist changed the title Document ctest add test Document CTest add_test() Sep 12, 2022
@codecov
Copy link

codecov bot commented Sep 12, 2022

Codecov Report

Base: 96.05% // Head: 96.18% // Increases project coverage by +0.13% 🎉

Coverage data is based on head (bcc53b8) compared to base (f5e67ea).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #128      +/-   ##
==========================================
+ Coverage   96.05%   96.18%   +0.13%     
==========================================
  Files           7        7              
  Lines         837      866      +29     
==========================================
+ Hits          804      833      +29     
  Misses         33       33              
Flag Coverage Δ
unittests 96.18% <100.00%> (+0.13%) ⬆️

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

Impacted Files Coverage Δ
src/cminx/aggregator.py 92.88% <100.00%> (+0.68%) ⬆️
src/cminx/config.py 100.00% <100.00%> (ø)
src/cminx/documentation_types.py 98.70% <100.00%> (+0.06%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@AutonomicPerfectionist AutonomicPerfectionist marked this pull request as ready for review September 17, 2022 03:24
Copy link
Collaborator

@ryanmrichard ryanmrichard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@ryanmrichard ryanmrichard merged commit 7184b41 into CMakePP:master Sep 19, 2022
@ryanmrichard
Copy link
Collaborator

🚀 [bumpr] Bumped! New version:v1.0.12 Changes:v1.0.11...v1.0.12

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.

[Question] Is it possible to document bare add_test calls

2 participants