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

GraphQL Server Testing #506

Merged
merged 9 commits into from
Mar 31, 2022
Merged

GraphQL Server Testing #506

merged 9 commits into from
Mar 31, 2022

Conversation

TimPansino
Copy link
Contributor

@TimPansino TimPansino commented Mar 28, 2022

Before contributing, please read our contributing guidelines and code of conduct.

Overview

  • Port existing test suites for GraphQL to GraphQL Server.
  • Fix framework_details in Flask with delayed callable logic in WSGI Application Wrapper
  • Parametrize graphql-server testing for Sanic and Flask

Related Github Issue

N/A

Testing

The agent includes a suite of tests which should be used to
verify your changes don't break existing functionality. These tests will run with
Github Actions when a pull request is made. More details on running the tests locally can be found
here,
For most contributions it is strongly recommended to add additional tests which
exercise your changes.

@TimPansino TimPansino force-pushed the feature-graphqlserver-testing branch from 4323a67 to a25b73f Compare March 28, 2022 22:38
@TimPansino TimPansino marked this pull request as ready for review March 28, 2022 22:39
@TimPansino TimPansino requested a review from a team March 28, 2022 22:39
@github-actions
Copy link

github-actions bot commented Mar 28, 2022

MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ PYTHON bandit 5 0 0.61s
✅ PYTHON black 5 0 0 0.61s
✅ PYTHON flake8 5 0 0.5s
✅ PYTHON isort 5 0 0 0.19s
✅ PYTHON pylint 5 0 6.05s

See errors details in artifact MegaLinter reports on CI Job page
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

@umaannamalai umaannamalai added this to the 7.8.0.174 milestone Mar 30, 2022
@umaannamalai umaannamalai merged commit 367e57b into main Mar 31, 2022
@umaannamalai umaannamalai deleted the feature-graphqlserver-testing branch March 31, 2022 20:28
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.

2 participants