Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Tidy up tests #91

Merged
merged 5 commits into from
Sep 8, 2023
Merged

Tidy up tests #91

merged 5 commits into from
Sep 8, 2023

Conversation

gilesknap
Copy link
Member

New tests have no dependency on ibek-defs.
Are all based on two minimal support modules YAML and two ioc YAML.
Are shorter and contain much less hard coded strings for comparison.

@codecov-commenter
Copy link

Codecov Report

Merging #91 (c9359c5) into main (dd1906b) will decrease coverage by 1.02%.
Report is 5 commits behind head on main.
The diff coverage is 77.77%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##             main      #91      +/-   ##
==========================================
- Coverage   97.24%   96.22%   -1.02%     
==========================================
  Files           9        9              
  Lines         363      371       +8     
==========================================
+ Hits          353      357       +4     
- Misses         10       14       +4     
Files Changed Coverage Δ
src/ibek/ioc.py 96.15% <75.00%> (+0.15%) ⬆️
src/ibek/support.py 97.50% <80.00%> (-1.17%) ⬇️

... and 1 file with indirect coverage changes

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

@gilesknap gilesknap merged commit 27bc903 into main Sep 8, 2023
This was referenced Sep 8, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants