Skip to content

chore(datastore): Change the DatastoreTester interface to take a testing.TB#2925

Merged
jakedt merged 1 commit intomainfrom
datastore-tester-interface
Feb 27, 2026
Merged

chore(datastore): Change the DatastoreTester interface to take a testing.TB#2925
jakedt merged 1 commit intomainfrom
datastore-tester-interface

Conversation

@jakedt
Copy link
Member

@jakedt jakedt commented Feb 26, 2026

Description

This change allows us to (optionally) cleanup datastore instances on a per-test basis, rather than waiting for the whole test suite to finish.

@jakedt jakedt requested a review from a team as a code owner February 26, 2026 23:09
@github-actions github-actions bot added area/datastore Affects the storage system area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools) labels Feb 26, 2026
@github-actions
Copy link

github-actions bot commented Feb 26, 2026

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

Copy link
Member

@josephschorr josephschorr left a comment

Choose a reason for hiding this comment

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

You'll need to update the CHANGLOG as well

@jakedt
Copy link
Member Author

jakedt commented Feb 27, 2026

I have read the CLA Document and I hereby sign the CLA

authzedbot added a commit to authzed/cla that referenced this pull request Feb 27, 2026
@jakedt jakedt force-pushed the datastore-tester-interface branch from 4e3ba89 to 325da5e Compare February 27, 2026 00:24
@codecov
Copy link

codecov bot commented Feb 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.53%. Comparing base (68bc889) to head (7a19dda).
⚠️ Report is 1 commits behind head on main.

❌ Your project status has failed because the head coverage (74.53%) is below the target coverage (75.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2925      +/-   ##
==========================================
+ Coverage   73.57%   74.53%   +0.96%     
==========================================
  Files         489      489              
  Lines       60373    60373              
==========================================
+ Hits        44414    44990     +576     
+ Misses      12798    12232     -566     
+ Partials     3161     3151      -10     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jakedt jakedt force-pushed the datastore-tester-interface branch from 325da5e to 7a19dda Compare February 27, 2026 18:29
Copy link
Member

@josephschorr josephschorr left a comment

Choose a reason for hiding this comment

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

LGTM

@jakedt jakedt merged commit acd9e4e into main Feb 27, 2026
44 of 45 checks passed
@jakedt jakedt deleted the datastore-tester-interface branch February 27, 2026 19:43
@github-actions github-actions bot locked and limited conversation to collaborators Feb 27, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area/datastore Affects the storage system area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants