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

test: backport test from graphql-js #6305

Merged
merged 1 commit into from
Jul 1, 2024
Merged

Conversation

n1ru4l
Copy link
Collaborator

@n1ru4l n1ru4l commented Jul 1, 2024

Backports the test case from graphql/graphql-js@)4a82557ae6d3b3c6cd72bcd528254296ecf7e9e8

The actual fix is already here.

Copy link

changeset-bot bot commented Jul 1, 2024

⚠️ No Changeset found

Latest commit: 32fc189

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Contributor

github-actions bot commented Jul 1, 2024

✅ Benchmark Results

     ✓ no_errors
     ✓ expected_result

     checks.........................: 100.00% ✓ 338       ✗ 0  
     data_received..................: 39 MB   3.9 MB/s
     data_sent......................: 145 kB  14 kB/s
     http_req_blocked...............: avg=3.62µs   min=1.86µs   med=2.49µs   max=152.59µs p(90)=3.55µs   p(95)=3.89µs  
     http_req_connecting............: avg=596ns    min=0s       med=0s       max=100.77µs p(90)=0s       p(95)=0s      
     http_req_duration..............: avg=55ms     min=47.75ms  med=50.9ms   max=156.84ms p(90)=62.56ms  p(95)=85.95ms 
       { expected_response:true }...: avg=55ms     min=47.75ms  med=50.9ms   max=156.84ms p(90)=62.56ms  p(95)=85.95ms 
     http_req_failed................: 0.00%   ✓ 0         ✗ 169
     http_req_receiving.............: avg=132.22µs min=103.23µs med=129.18µs max=313.26µs p(90)=151.04µs p(95)=160.43µs
     http_req_sending...............: avg=24.88µs  min=18.99µs  med=23.7µs   max=58.41µs  p(90)=30.51µs  p(95)=34.7µs  
     http_req_tls_handshaking.......: avg=0s       min=0s       med=0s       max=0s       p(90)=0s       p(95)=0s      
     http_req_waiting...............: avg=54.85ms  min=47.58ms  med=50.74ms  max=156.46ms p(90)=62.42ms  p(95)=85.77ms 
     http_reqs......................: 169     16.836273/s
     iteration_duration.............: avg=59.37ms  min=51.3ms   med=54.98ms  max=161.02ms p(90)=68.01ms  p(95)=89.95ms 
     iterations.....................: 169     16.836273/s
     vus............................: 1       min=1       max=1
     vus_max........................: 1       min=1       max=1

Copy link
Contributor

github-actions bot commented Jul 1, 2024

💻 Website Preview

The latest changes are available as preview in: https://974d8865.graphql-tools.pages.dev

@n1ru4l n1ru4l requested review from saihaj and ardatan July 1, 2024 09:31
@ardatan ardatan merged commit 374961f into master Jul 1, 2024
30 checks passed
@ardatan ardatan deleted the test-executor-backoport branch July 1, 2024 10:53
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.

None yet

2 participants