Skip to content

add new storage test panel#3487

Open
cody-littley wants to merge 3 commits into
mainfrom
cjl/storage-test-panel
Open

add new storage test panel#3487
cody-littley wants to merge 3 commits into
mainfrom
cjl/storage-test-panel

Conversation

@cody-littley
Copy link
Copy Markdown
Contributor

@cody-littley cody-littley commented May 21, 2026

Describe your changes and provide context

In order to speed up test runtime, split storage unit tests into a stand alone panel.

Screenshot 2026-05-21 at 12 49 40 PM

I downloaded the logs from the new panels and from the old panels, and used an LLM to verify that the expected tests are running in each panel.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 21, 2026

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedMay 21, 2026, 4:55 PM

@github-actions
Copy link
Copy Markdown

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedMay 21, 2026, 4:49 PM

@cody-littley cody-littley requested review from blindchaser and masih May 21, 2026 16:54
@cody-littley cody-littley marked this pull request as ready for review May 21, 2026 17:47
@cursor
Copy link
Copy Markdown

cursor Bot commented May 21, 2026

PR Summary

Medium Risk
Moderate CI risk: changes restructure test/coverage execution and package selection, which could inadvertently skip or double-run tests/coverage if package filtering or Codecov flags are misconfigured.

Overview
Splits the sei-db/state_db test suite into a new dedicated workflow state_db-tests.yml, running race tests and separate Codecov uploads/flags for that subtree.

Updates go-test.yml to exclude github.com/sei-protocol/sei-chain/sei-db/state_db from all test and coverage package lists (including PR fast-path reverse-dep selection), and adjusts the full-coverage job to build a single -coverpkg list from filtered packages to avoid clobbering coverage.out.

Reviewed by Cursor Bugbot for commit 712c704. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown
Collaborator

@masih masih left a comment

Choose a reason for hiding this comment

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

I understand this the pre-step in more tests to come since today it won't make the slowest mandatory CI job any faster. Which means total wait time is unchanged.

No blockers regardless and glad to see experimentation here 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants