Skip to content
This repository has been archived by the owner on Jun 28, 2024. It is now read-only.

Commit

Permalink
Add coverage improvement to report (#21)
Browse files Browse the repository at this point in the history
Co-authored-by: Maurice Ibrahim <[email protected]>
Co-authored-by: tthijm <[email protected]>
  • Loading branch information
3 people authored Jun 18, 2024
1 parent 0374ebe commit a9bede1
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 34 deletions.
40 changes: 6 additions & 34 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -24,20 +24,6 @@ Running `coverage html` and opening the document in the browser gives the follow

### Your own coverage tool

<The following is supposed to be repeated for each group member>

<Group member name>

<Function 1 name>

<Show a patch (diff) or a link to a commit made in your forked repository that shows the instrumented code to gather coverage measurements>

<Provide a screenshot of the coverage results output by the instrumentation>

<Function 2 name>

<Provide the same kind of information provided for Function 1>

#### Thijmen

flatten_literal_params
Expand Down Expand Up @@ -100,24 +86,6 @@ from_str

### Individual tests

<The following is supposed to be repeated for each group member>

<Group member name>

<Test 1>

<Show a patch (diff) or a link to a commit made in your forked repository that shows the new/enhanced test>

<Provide a screenshot of the old coverage results (the same as you already showed above)>

<Provide a screenshot of the new coverage results>

<State the coverage improvement with a number and elaborate on why the coverage is improved>

<Test 2>

<Provide the same kind of information provided for Test 1>

#### Thijmen

test_flatten_literal_params
Expand Down Expand Up @@ -215,9 +183,13 @@ The coverage improved from 85% to 100%. Since the tests resulted in each of the

### Overall

<Provide a screenshot of the old coverage results by running an existing tool (the same as you already showed above)>
The total coverage before adding our own tests

![](./assets/coverage_run_before.png)

The total coverage after adding our own tests

<Provide a screenshot of the new coverage results by running the existing tool using all test modifications made by the group>
![](./assets/coverage_run_after.png)

## Statement of individual contributions

Expand Down
Binary file added assets/coverage_run_after.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added assets/coverage_run_before.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

0 comments on commit a9bede1

Please sign in to comment.