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.

Sign up for GitHub

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

Jump to bottom

feat: make router-tests less flaky #1484

Draft
wants to merge 41 commits into
base: main
Choose a base branch
from
Draft

feat: make router-tests less flaky #1484

wants to merge 41 commits into from

Conversation

alepane21
Copy link
Contributor

Motivation and Context

Checklist

  • I have discussed my proposed changes in an issue and have received approval to proceed.
  • I have followed the coding standards of the project.
  • Tests or benchmarks have been added or updated.
  • Documentation has been updated on https://github.com/wundergraph/cosmo-docs.
  • I have read the Contributors Guide.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 3, 2025 •
edited
Loading

Router image scan passed

✅ No security vulnerabilities found in image:

ghcr.io/wundergraph/cosmo/router:sha-2317f64cd0574ecf370a0f8413fb8062d998e8d7

@github-actions github-actions bot added the router label Jan 4, 2025
alepane21 and others added 24 commits January 7, 2025 12:26
@alepane21
fix: less strict condition on test
d03a4fc
@alepane21
Merge branch 'refs/heads/main' into ale/eng-6219-make-router-tests-le…
1727feb 
…ss-flaky
@alepane21
fix: less strict condition on test
e91500e
@alepane21
fix: stricter condition on test
f3ce9c9
@alepane21
fix: stricter condition on test
51ec7b8
@alepane21
chore: change a require to an assert to see next assertion value
c1f79de
@alepane21
chore: try a different order of the checks for subscribe sync
70f34c8
@alepane21
Merge branch 'main' into ale/eng-6219-make-router-tests-less-flaky
42ca681
@alepane21
Merge branch 'main' into ale/eng-6219-make-router-tests-less-flaky
82f7408
@alepane21
Merge branch 'main' into ale/eng-6219-make-router-tests-less-flaky
eeda567
@alepane21
Merge branch 'main' into ale/eng-6219-make-router-tests-less-flaky
a0d380a
@alepane21
fix: use fixed graphql-go-tools
7af651e
@alepane21
fix: use specified metric reader
14f2622
@alepane21
chore: limit parallelization
b17e17c
@alepane21
chore: changed flaky test to give more details when an error occurs
4158be9
@alepane21
fix: always call the env Shutdown
e3f5d59
@alepane21
fix: static check
440f2e1
@alepane21
Merge branch 'main' into ale/eng-6219-make-router-tests-less-flaky
d794103
@alepane21
fix: set higher timeout default on events checks
478b768
@alepane21
fix: add WaitForTriggerCount
bab0852
@alepane21
Merge branch 'refs/heads/main' into ale/eng-6219-make-router-tests-le…
4b3499e 
…ss-flaky

# Conflicts:
#	router-tests/go.mod
#	router-tests/go.sum
@alepane21
Merge branch 'main' into ale/eng-6219-make-router-tests-less-flaky
3101301
@alepane21
feat: now the nats provider can log errors
f2c6d40
@alepane21
feat: try less time sensitive approach with multipart tests
6f05f79
alepane21 and others added 13 commits January 21, 2025 18:32
@alepane21
chore: wait before publishing second message
233207a
@alepane21
fix: deflake ws connections
907043c
@alepane21
fix: deflake subscribe sse with filter
3accd1c
@alepane21
fix: make retry waiting time grow each iteration
985522a
@alepane21
fix: make deflake functions return the error
51c47b2
@alepane21
fix: deflake "subscribe sync sse works without query param"
8e8353e
@alepane21
Merge branch 'refs/heads/main' into ale/eng-6219-make-router-tests-le…
a220066 
…ss-flaky

# Conflicts:
#	router-tests/testenv/testenv.go
@alepane21
fix: add Shutdown to cleanup methods also on error
c726296
@alepane21
fix: unflake two hot reload test
a7c4426
@alepane21
chore: try to execute only flaky test with retry
6d3b17c
@alepane21
chore: more readable regexp
afdae6a
@alepane21
Merge branch 'main' into ale/eng-6219-make-router-tests-less-flaky
6af87ed
@alepane21
chore: make flaky test less parallel, to see if the behave better
6e7714b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
router
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@alepane21