Skip to content

Comments

CBG-4902: Fix Flaky Test TestBlipOneShotChangesSubscription#7815

Merged
torcolvin merged 1 commit intomainfrom
CBG-4902
Oct 10, 2025
Merged

CBG-4902: Fix Flaky Test TestBlipOneShotChangesSubscription#7815
torcolvin merged 1 commit intomainfrom
CBG-4902

Conversation

@RIT3shSapata
Copy link
Contributor

CBG-4902

Describe your PR here...

  • The NumPullReplActiveOneShot's decrement is deferred, so updated the require condition to wait for the metric to be eventually updated to the desired value

Pre-review checklist

  • Removed debug logging (fmt.Print, log.Print, ...)
  • Logging sensitive data? Make sure it's tagged (e.g. base.UD(docID), base.MD(dbName))
  • Updated relevant information in the API specifications (such as endpoint descriptions, schemas, ...) in docs/api

Dependencies (if applicable)

  • Link upstream PRs
  • Update Go module dependencies when merged

Integration Tests

Copilot AI review requested due to automatic review settings October 10, 2025 14:56
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes a flaky test TestBlipOneShotChangesSubscription by addressing a timing issue with metric updates. The test was failing because it was checking for an immediate value of a metric that gets decremented asynchronously.

  • Updated the test assertion to wait for the NumPullReplActiveOneShot metric to eventually reach the expected value

@torcolvin torcolvin enabled auto-merge (squash) October 10, 2025 15:00
@torcolvin torcolvin merged commit bcb36b4 into main Oct 10, 2025
41 of 42 checks passed
@torcolvin torcolvin deleted the CBG-4902 branch October 10, 2025 15:04
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.

2 participants