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

Add coverage improvement to report #21

Merged
merged 6 commits into from
Jun 18, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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.
Loading