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

Overhaul GraphQL test suite #1750

Merged
merged 2 commits into from Mar 25, 2022
Merged

Overhaul GraphQL test suite #1750

merged 2 commits into from Mar 25, 2022

Conversation

zshipko
Copy link
Contributor

@zshipko zshipko commented Feb 2, 2022

@codecov-commenter
Copy link

codecov-commenter commented Feb 2, 2022

Codecov Report

Merging #1750 (f7c5eff) into main (e54478b) will increase coverage by 0.35%.
The diff coverage is 100.00%.

❗ Current head f7c5eff differs from pull request most recent head 4a972ad. Consider uploading reports for the commit 4a972ad to get more accurate results

@@            Coverage Diff             @@
##             main    #1750      +/-   ##
==========================================
+ Coverage   64.63%   64.98%   +0.35%     
==========================================
  Files         116      116              
  Lines       14159    14164       +5     
==========================================
+ Hits         9152     9205      +53     
+ Misses       5007     4959      -48     
Impacted Files Coverage Δ
src/irmin-graphql/server.ml 59.95% <100.00%> (+10.42%) ⬆️
src/irmin-test/store.ml 94.93% <0.00%> (-0.06%) ⬇️
src/irmin/tree.ml 81.42% <0.00%> (+0.14%) ⬆️
src/irmin/store.ml 64.96% <0.00%> (+0.29%) ⬆️
src/irmin/commit.ml 63.98% <0.00%> (+0.32%) ⬆️
src/irmin/import.ml 73.91% <0.00%> (+4.34%) ⬆️

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@zshipko zshipko force-pushed the graphql-test-suite branch 2 times, most recently from 4dfcac6 to df7167a Compare February 3, 2022 00:09
@zshipko zshipko added no-changelog-needed No changelog is needed here type/tests About tests labels Feb 3, 2022
@zshipko zshipko marked this pull request as ready for review February 16, 2022 04:03
@zshipko zshipko force-pushed the graphql-test-suite branch 3 times, most recently from 5e71e8d to dbaaf6c Compare March 11, 2022 00:27
@zshipko zshipko force-pushed the graphql-test-suite branch 3 times, most recently from 5e04c99 to e1f9958 Compare March 24, 2022 20:02
@zshipko zshipko merged commit 77445d2 into mirage:main Mar 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog-needed No changelog is needed here type/tests About tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants