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

Add code coverage workflow #20

Merged
merged 61 commits into from
Sep 11, 2024
Merged

Add code coverage workflow #20

merged 61 commits into from
Sep 11, 2024

Conversation

sbaldu
Copy link
Collaborator

@sbaldu sbaldu commented Oct 19, 2023

Measures code coverage of the tests:

@sbaldu sbaldu marked this pull request as draft October 19, 2023 10:50
@sbaldu sbaldu changed the title Add draft of code_coverage.yml Create code coverage workflow Oct 19, 2023
@sbaldu sbaldu force-pushed the main branch 2 times, most recently from e2c1b29 to 7938389 Compare November 4, 2023 11:11
@sbaldu sbaldu linked an issue Nov 18, 2023 that may be closed by this pull request
@Grufoony Grufoony added workflow Regarding the CI/CD pipeline workflows repo Regarding the repository security Regrding the security of the code labels Nov 20, 2023
sbaldu and others added 9 commits December 20, 2023 16:10
* Refactor test files and update CMake workflow

* Update test command in README.md
* Create benchmark.yml file

* Change name of the workflow

* Add checkout of submodules

Fix typo in benchmark of street
* Remove def constructor of Street and add pair parameter to id constructor

* Implement addition of nodes in `Graph::addStreet`

* Rewrite base case of addStreets

* Fix old definition of Street constructors

* Update tests of Street for new constructors

* Fix methods for building Graph

* Add tests for addStreet

* Formatting

* Initialize `maxSpeed` in first Street constructor

* Fix typo in Street tests

Update benchmarks to new `Street` structure and methods
* Add nodes to nodeSet when using `addStreet` (#65)

* Remove def constructor of Street and add pair parameter to id constructor

* Implement addition of nodes in `Graph::addStreet`

* Rewrite base case of addStreets

* Fix old definition of Street constructors

* Update tests of Street for new constructors

* Fix methods for building Graph

* Add tests for addStreet

* Formatting

* Initialize `maxSpeed` in first Street constructor

* Fix typo in Street tests

* Remove reference in variadic methods template constraint

* Fix typo

* Update declaration of base case

* Add tests of addStreets

* Fix typo

* Add type trait specialization for const Nodes and Streets
* Add insert_and_expand functions to SparseMatrix

* Refactor insert_and_expand function in
SparseMatrix.hpp

* still not working

* Fix reshape bug and update unit tests

* Fix matrix expansion for row vectors

* Fix indentation in SparseMatrix.hpp

* Fix matrix reshape and insert_and_expand functions

* Fix indentation in SparseMatrix.hpp
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@Grufoony
Copy link
Collaborator

@sbaldu seems like you have to generate a token even if the repository is public. Please have a look at:

@Grufoony
Copy link
Collaborator

Moreover, we can push the output report from the coverage to the gh pages branch and set up the badge in the project Readme

Copy link

codecov bot commented Sep 10, 2024

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@Grufoony
Copy link
Collaborator

@sbaldu I guess you should also add the badge to the Readme by yourself: https://docs.codecov.com/docs/status-badges

@sbaldu sbaldu marked this pull request as ready for review September 11, 2024 09:06
@Grufoony Grufoony changed the title Create code coverage workflow Add code coverage workflow Sep 11, 2024
@sbaldu sbaldu merged commit 93efdf4 into main Sep 11, 2024
24 checks passed
@Grufoony Grufoony deleted the feature_code_coverage branch September 11, 2024 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
repo Regarding the repository security Regrding the security of the code workflow Regarding the CI/CD pipeline workflows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add code coverage tool
2 participants