Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

token clean-up test_instruments #226

Merged
merged 5 commits into from
Nov 17, 2023
Merged

token clean-up test_instruments #226

merged 5 commits into from
Nov 17, 2023

Conversation

burnout87
Copy link
Collaborator

No description provided.

Copy link

codecov bot commented Nov 9, 2023

Codecov Report

Merging #226 (ddd651f) into master (b211cf3) will increase coverage by 0.14%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #226      +/-   ##
==========================================
+ Coverage   57.55%   57.70%   +0.14%     
==========================================
  Files          22       22              
  Lines        4686     4686              
==========================================
+ Hits         2697     2704       +7     
+ Misses       1989     1982       -7     
Files Coverage Δ
tests/test_live.py 52.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

@burnout87
Copy link
Collaborator Author

interestingly, I had already developed a fixture specific for cleaning up the tokens from the environment

@burnout87 burnout87 changed the title token clean-up test_cli token clean-up test_instruments Nov 13, 2023
@burnout87 burnout87 merged commit 7a4a1c4 into master Nov 17, 2023
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants