Skip to content

Tests for #410 #482

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

Merged
merged 2 commits into from
May 25, 2022
Merged

Tests for #410 #482

merged 2 commits into from
May 25, 2022

Conversation

ozahorulia
Copy link
Contributor

Tests for #411 PR (fixes #410 and #481).

@codecov-commenter
Copy link

Codecov Report

Merging #482 (16950c1) into master (dc6e07d) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #482   +/-   ##
=========================================
  Coverage     96.33%   96.33%           
  Complexity     1638     1638           
=========================================
  Files           146      146           
  Lines          4115     4115           
=========================================
  Hits           3964     3964           
  Misses          151      151           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dc6e07d...16950c1. Read the comment docs.

@ozahorulia
Copy link
Contributor Author

ozahorulia commented May 25, 2022

@oojacoboo Hahaha. Okay, so after rabasing I can see that problem seems to be fixed in master already. So this PR probably only adds tests. But it would still be perfect if we could merge it and tag it, so I can require it in my project with a stable tag, rather than on master. Thanks.

@oojacoboo
Copy link
Collaborator

@ozahorulia great. We're waiting on #466 to get wrapped up before tagging a new release. It's basically done.

@oojacoboo oojacoboo mentioned this pull request May 25, 2022
@oojacoboo oojacoboo merged commit 4fcbb74 into thecodingmachine:master May 25, 2022
andrew-demb pushed a commit to andrew-demb/graphqlite that referenced this pull request Jun 14, 2022
* feat: add test for thecodingmachine#410

* Tests for thecodingmachine#411 (fixes thecodingmachine#410, thecodingmachine#481)

Co-authored-by: Marcin Hubert <marcin.hubert@zety.com>
Co-authored-by: ozahorulia <ozahorulia@unisender.com>
andrew-demb pushed a commit to andrew-demb/graphqlite that referenced this pull request Jun 14, 2022
* feat: add test for thecodingmachine#410

* Tests for thecodingmachine#411 (fixes thecodingmachine#410, thecodingmachine#481)

Co-authored-by: Marcin Hubert <marcin.hubert@zety.com>
Co-authored-by: ozahorulia <ozahorulia@unisender.com>
andrew-demb pushed a commit to andrew-demb/graphqlite that referenced this pull request Jun 14, 2022
* feat: add test for thecodingmachine#410

* Tests for thecodingmachine#411 (fixes thecodingmachine#410, thecodingmachine#481)

Co-authored-by: Marcin Hubert <marcin.hubert@zety.com>
Co-authored-by: ozahorulia <ozahorulia@unisender.com>
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.

Parameter 'locked' is missing for class 'FooInputType' constructor. It should be mapped as required field.
3 participants