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

Fix accidentally broken unit test#676

Merged
d-perl merged 1 commit into
mainfrom
fix_unit_test
May 22, 2023
Merged

Fix accidentally broken unit test#676
d-perl merged 1 commit into
mainfrom
fix_unit_test

Conversation

@DominicOram

Copy link
Copy Markdown
Collaborator

To test:

  1. Confirm tests pass

@DominicOram
DominicOram requested a review from d-perl May 22, 2023 13:22
@DominicOram
DominicOram marked this pull request as ready for review May 22, 2023 13:22
@DominicOram
DominicOram requested a review from noemifrisina May 22, 2023 13:22
@codecov

codecov Bot commented May 22, 2023

Copy link
Copy Markdown

Codecov Report

Merging #676 (477dd56) into main (779cb1f) will increase coverage by 0.64%.
The diff coverage is 95.23%.

@@            Coverage Diff             @@
##             main     #676      +/-   ##
==========================================
+ Coverage   89.11%   89.75%   +0.64%     
==========================================
  Files          34       34              
  Lines        1488     1494       +6     
==========================================
+ Hits         1326     1341      +15     
+ Misses        162      153       -9     
Impacted Files Coverage Δ
src/artemis/parameters/beamline_parameters.py 97.87% <90.00%> (+12.50%) ⬆️
...rc/artemis/experiment_plans/fast_grid_scan_plan.py 82.87% <100.00%> (+0.44%) ⬆️
src/artemis/experiment_plans/full_grid_scan.py 70.31% <100.00%> (+0.95%) ⬆️
...rtemis/experiment_plans/oav_grid_detection_plan.py 100.00% <100.00%> (+3.44%) ⬆️
src/artemis/parameters/constants.py 100.00% <100.00%> (ø)

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

@d-perl
d-perl merged commit bc18cce into main May 22, 2023
@d-perl
d-perl deleted the fix_unit_test branch May 22, 2023 14:00
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.

2 participants