Skip to content

Conversation

@juliannguyen4
Copy link
Collaborator

@juliannguyen4 juliannguyen4 commented Jan 5, 2026

This test case passes a low total_timeout to client.put() which still passes in most environments that have a low e2e latency. But if the e2e latency is high enough, this client.put() would fail and the test will be unable to get the record metadata using client.exists(). We increase the total_timeout so that client.put() still succeeds with a relatively low total timeout.

TODO

  • Run macos x86 tests multiple times to check that test case no longer fails intermittently

…rd. This doesn't seem like a negative test, though
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.37%. Comparing base (6739ae7) to head (cbb859c).
⚠️ Report is 3 commits behind head on dev.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #917   +/-   ##
=======================================
  Coverage   83.37%   83.37%           
=======================================
  Files          99       99           
  Lines       14389    14389           
=======================================
  Hits        11997    11997           
  Misses       2392     2392           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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