Skip to content
This repository was archived by the owner on Sep 2, 2024. It is now read-only.

Use autospec in patches, also remove useless test#813

Merged
DominicOram merged 1 commit into
mainfrom
812_add_autospec
Jul 18, 2023
Merged

Use autospec in patches, also remove useless test#813
DominicOram merged 1 commit into
mainfrom
812_add_autospec

Conversation

@DominicOram

Copy link
Copy Markdown
Collaborator

Fixes #812

To test:

  1. Confirm tests all still pass and removed test is out of date

@codecov

codecov Bot commented Jul 18, 2023

Copy link
Copy Markdown

Codecov Report

Merging #813 (6cd2c89) into main (cf1e9fc) will increase coverage by 1.12%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #813      +/-   ##
==========================================
+ Coverage   93.33%   94.46%   +1.12%     
==========================================
  Files          37       37              
  Lines        1875     1878       +3     
==========================================
+ Hits         1750     1774      +24     
+ Misses        125      104      -21     
Impacted Files Coverage Δ
...emis/experiment_plans/optimise_attenuation_plan.py 94.35% <100.00%> (+17.49%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@DominicOram
DominicOram merged commit f2d5e3a into main Jul 18, 2023
@DominicOram
DominicOram deleted the 812_add_autospec branch July 18, 2023 15:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add some autospecs in the patches

2 participants