Skip to content

test: expand legacy piece storage coverage - #308

Open
Kubuxu wants to merge 2 commits into
mainfrom
tests/expand-legacy
Open

test: expand legacy piece storage coverage#308
Kubuxu wants to merge 2 commits into
mainfrom
tests/expand-legacy

Conversation

@Kubuxu

@Kubuxu Kubuxu commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add an environment-controlled test hook that creates datasets using legacy piece storage
  • rerun the full suite in legacy mode from make test and make test-fast without compiling duplicate inherited test contracts
  • remove noisy piece-construction logging

Verification

  • make test
    • compact pass: 232 passed
    • legacy-mode pass: 232 passed
    • 0 failures
  • forge fmt --check
  • Solidity diagnostics: no issues

Closes #301

Kubuxu added 2 commits August 25, 2026 15:28
Signed-off-by: Jakub Sztandera <oss@kubuxu.com>
Signed-off-by: Jakub Sztandera <oss@kubuxu.com>
@Kubuxu
Kubuxu requested a review from wjmelements as a code owner August 25, 2026 13:31
@FilOzzy FilOzzy added this to FOC Aug 25, 2026
@github-project-automation github-project-automation Bot moved this to 📌 Triage in FOC Aug 25, 2026
@FilOzzy FilOzzy moved this from 📌 Triage to 🔎 Awaiting review in FOC Aug 25, 2026
@github-project-automation github-project-automation Bot moved this from 🔎 Awaiting review to ✔️ Approved by reviewer in FOC Aug 25, 2026
@Kubuxu

Kubuxu commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator Author

Waiting for release before merging.

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

Labels

None yet

Projects

Status: ✔️ Approved by reviewer
Status: No status

Development

Successfully merging this pull request may close these issues.

testing: improve testing of legacy storage format

3 participants