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

Improve unit tests #14

Merged
merged 31 commits into from
Dec 1, 2022
Merged

Improve unit tests #14

merged 31 commits into from
Dec 1, 2022

Conversation

p-gw
Copy link
Owner

@p-gw p-gw commented Nov 28, 2022

closes #10

TODO

  • split up modifying tests between lts and latest
  • Reconstruct 813998.lss for lts tests
  • Fix Docker volumes for lts release

@p-gw p-gw linked an issue Nov 28, 2022 that may be closed by this pull request
@p-gw p-gw added this to the 0.1.0 milestone Nov 28, 2022
@codecov-commenter
Copy link

codecov-commenter commented Nov 29, 2022

Codecov Report

Merging #14 (d305558) into main (a9e5c2e) will increase coverage by 46.46%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##             main      #14       +/-   ##
===========================================
+ Coverage   30.06%   76.52%   +46.46%     
===========================================
  Files          54       54               
  Lines         306      311        +5     
===========================================
+ Hits           92      238      +146     
+ Misses        214       73      -141     
Impacted Files Coverage Δ
src/methods/release_session_key.jl 100.00% <ø> (+100.00%) ⬆️
src/CitrusAPI.jl 81.25% <100.00%> (+31.25%) ⬆️
src/methods/activate_tokens.jl 100.00% <100.00%> (+100.00%) ⬆️
src/methods/export_responses_by_token.jl 100.00% <100.00%> (+100.00%) ⬆️
src/methods/list_participants.jl 100.00% <100.00%> (+100.00%) ⬆️
src/utils.jl 80.00% <0.00%> (+6.66%) ⬆️
src/query.jl 100.00% <0.00%> (+9.09%) ⬆️
src/methods/set_survey_properties.jl 57.14% <0.00%> (+57.14%) ⬆️
... and 29 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@p-gw p-gw marked this pull request as ready for review December 1, 2022 14:06
@p-gw p-gw merged commit e081a41 into main Dec 1, 2022
@p-gw p-gw deleted the 10-improve-unit-tests branch December 1, 2022 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve unit tests
2 participants