Skip to content

[v2] [8/X] Unit Test Overhaul - Part 2 #644

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

Draft
wants to merge 1 commit into
base: QueryWatcher-tasklocal-contextid
Choose a base branch
from

Conversation

AnthonyMDev
Copy link
Contributor

A few tests are still failing when all tests are run together. I think this is due to async tasks holding up other tasks and causing timeouts. All tests pass when run individually.

@svc-apollo-docs
Copy link
Collaborator

svc-apollo-docs commented May 15, 2025

⚠️ Docs preview not attached to branch

The preview was not built because the PR's base branch QueryWatcher-tasklocal-contextid is not in the list of sources.

An Apollo team member can comment one of the following commands to dictate which branch to attach the preview to:

  • !docs set-base-branch main

Build ID: 561218969640bf570adc16f8

@AnthonyMDev AnthonyMDev changed the title [v2] Fix remaining tests [v2] Unit Test Overhaul - Part 2 May 15, 2025
@AnthonyMDev AnthonyMDev changed the title [v2] Unit Test Overhaul - Part 2 [v2] [8/X] Unit Test Overhaul - Part 2 May 16, 2025
@AnthonyMDev AnthonyMDev force-pushed the QueryWatcher-tasklocal-contextid branch 2 times, most recently from 26560c8 to cd8c068 Compare May 16, 2025 00:41
@AnthonyMDev AnthonyMDev force-pushed the unit-tests-2 branch 2 times, most recently from b0d32f0 to 6405384 Compare May 16, 2025 00:47
@AnthonyMDev AnthonyMDev force-pushed the QueryWatcher-tasklocal-contextid branch from cd8c068 to 3a81cf5 Compare May 16, 2025 00:49
@AnthonyMDev AnthonyMDev force-pushed the QueryWatcher-tasklocal-contextid branch 2 times, most recently from 1ec7c50 to 54a017d Compare May 16, 2025 00:54
@AnthonyMDev AnthonyMDev force-pushed the unit-tests-2 branch 3 times, most recently from d4d7076 to 7524805 Compare May 16, 2025 00:59
@AnthonyMDev AnthonyMDev force-pushed the QueryWatcher-tasklocal-contextid branch from 54a017d to 531c677 Compare May 16, 2025 01:01
@calvincestari
Copy link
Member

Going to slowly make our way through these during the week - there is a lot to review. 😓

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.

3 participants