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

test: Jacoco reporting #84

Merged
merged 7 commits into from
Mar 15, 2024
Merged

test: Jacoco reporting #84

merged 7 commits into from
Mar 15, 2024

Conversation

Jack-Golding
Copy link
Contributor

Implemented jacoco reporting for the integration tests
Additionally added more integration tests for all relevant endpoints

Implemented jacoco reporting and added intergration tests

WIP
coverage currently sitting at 21%
Added jacoco test results to github actions
@Jack-Golding Jack-Golding marked this pull request as ready for review March 14, 2024 14:50
Copy link

github-actions bot commented Mar 14, 2024

Unit Test Results

34 tests  +34   34 ✔️ +34   58s ⏱️ +58s
  4 suites +  4     0 💤 ±  0 
  4 files   +  4     0 ±  0 

Results for commit 0df0c1c. ± Comparison against base commit 0052b1b.

♻️ This comment has been updated with latest results.

These have been bootstrapped from the mod-agreements templating service and contain funcitons very specific to that use case
Added back in starting values code, refactored into a singe method
@Jack-Golding Jack-Golding merged commit 003ba99 into master Mar 15, 2024
5 checks passed
@Jack-Golding Jack-Golding deleted the jacoco-reporting branch March 15, 2024 16:47
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.

1 participant