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

Fix race condition in web vital unit test #904

Merged
merged 1 commit into from May 23, 2023
Merged

Conversation

ankur22
Copy link
Collaborator

@ankur22 ankur22 commented May 23, 2023

Description of changes

This returns when all the conditions are met from within the goroutine so that the expected map is not written to once the context is cancelled.

Checklist

Tasks

Edit tasklist title
Beta Give feedback Tasklist Tasks, more options

Delete tasklist

Delete tasklist block?
Are you sure? All relationships in this tasklist will be removed.
  1. Written tests for the changes
    Options

This returns when all the conditions are met from within the goroutine
so that the expected map is not written to once the context is
cancelled.
@ankur22 ankur22 requested review from inancgumus and ka3de May 23, 2023 15:11
@ankur22 ankur22 self-assigned this May 23, 2023
@ankur22 ankur22 added bug Something isn't working tests labels May 23, 2023
@ankur22 ankur22 added this to the v0.10.0 milestone May 23, 2023
Copy link
Collaborator

@ka3de ka3de left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@ankur22 ankur22 merged commit da822f3 into main May 23, 2023
13 checks passed
@ankur22 ankur22 deleted the fix/web-vital-test-race branch May 23, 2023 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants