newrelic / newrelic-python-agent

New Relic Python Agent
https://docs.newrelic.com/docs/agents/python-agent
Apache License 2.0
178 stars 102 forks source link

Enable GraphQL master tests #1167

Closed lrafeei closed 4 months ago

lrafeei commented 4 months ago

This PR

  1. enables the testing of graphqlmaster. That is, it pulls graphql-core's code from the main branch as well as the latest release to make sure future releases of graphql-core do not conflict with New Relic's instrumentation/crash customer code
  2. adds/tweaks instrumentation to support graphql-core v3.3. Currently, this is still an alpha version
  3. updates GraphQL tests to support both v3.2 and v3.3+
github-actions[bot] commented 4 months ago

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ PYTHON bandit 2 0 5.77s
✅ PYTHON black 3 0 0 1.86s
✅ PYTHON flake8 3 0 0.97s
✅ PYTHON isort 3 0 0 0.26s
✅ PYTHON pylint 3 0 6.45s

See detailed report in MegaLinter reports _Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff_

_MegaLinter is graciously provided by OX Security_

codecov-commenter commented 4 months ago

Codecov Report

Attention: Patch coverage is 88.88889% with 1 line in your changes missing coverage. Please review.

Project coverage is 81.66%. Comparing base (6e72cdf) to head (67d8ceb).

Files Patch % Lines
newrelic/hooks/framework_graphql.py 87.50% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1167 +/- ## ========================================== + Coverage 81.65% 81.66% +0.01% ========================================== Files 195 195 Lines 21666 21667 +1 Branches 3780 3781 +1 ========================================== + Hits 17691 17694 +3 Misses 2877 2877 + Partials 1098 1096 -2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.