Skip to content

Code Coverage#368

Open
mewilker wants to merge 284 commits into
softwareconstruction240:mainfrom
mewilker:code-coverage
Open

Code Coverage#368
mewilker wants to merge 284 commits into
softwareconstruction240:mainfrom
mewilker:code-coverage

Conversation

@mewilker
Copy link
Copy Markdown
Contributor

@mewilker mewilker commented Jun 4, 2026

Warning

Do not merge this PR until we are ready for the code coverage requirement.
This PR should be merged with softwareconstruction240/autograder#643

Resolves #355

This PR changes the specs of the project. 80% line coverage is now required for all Phases with unit tests. 90% line coverage awards 1.25 points of extra credit. Here are the changes I made by phase:

Phase 3

  • Coverage is collected on service package
  • Videos are broken up into smaller pieces.
  • Unit test section edited.
  • Rubric edited.
  • Getting started instructions includes directions for changing the code coverage runner to JaCoCo in IntelliJ, to match the output the autograder gives the students

Phase 4

  • Coverage is collected on dataaccess.sql package
  • Unit test section edited
  • Rubric edited
  • Video for unit test updated

Phase 5

  • Coverage is collected on ServerFacade class
  • Unit test section edited
  • Rubric edtied
  • Overview video updated

Note on Video updates

I recorded the videos through the BYU Online. I do not know if I should have asked them for transcripts. Each video has closed captions. I removed transcript links for all of Phase 3, and removed the links to just the videos changed for phases 4 and 5 (the word [transcript] is still there, following the pattern with the Server Facade video).

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.

Add Code Coverage instructions

2 participants