Skip to content

test: isolate S3 integration prefixes per test#24

Merged
DiTo97 merged 1 commit intomainfrom
test/integration-isolation-and-smoke-cleanup
Apr 15, 2026
Merged

test: isolate S3 integration prefixes per test#24
DiTo97 merged 1 commit intomainfrom
test/integration-isolation-and-smoke-cleanup

Conversation

@DiTo97
Copy link
Copy Markdown
Owner

@DiTo97 DiTo97 commented Apr 15, 2026

Summary

  • give each S3 integration test a unique MinIO prefix instead of sharing the fixed test-run namespace
  • reduce the chance of order-dependent failures caused by leftover objects from earlier tests
  • keep this PR focused on the high-value integration isolation fix while Docker is unavailable locally

Test plan

  • uv run pytest -m unit -q
  • uv run ruff check .
  • uv run pytest -m integration -q (Docker not running in this environment)

Stack

🤖 Generated with Claude Code

Give each S3 integration test a unique object prefix so MinIO-backed tests no longer share a keyspace and leak state across test ordering.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@DiTo97 DiTo97 deleted the branch main April 15, 2026 08:33
@DiTo97 DiTo97 closed this Apr 15, 2026
@DiTo97 DiTo97 deleted the test/integration-isolation-and-smoke-cleanup branch April 15, 2026 08:33
@DiTo97 DiTo97 restored the test/integration-isolation-and-smoke-cleanup branch April 15, 2026 08:35
@DiTo97 DiTo97 reopened this Apr 15, 2026
@DiTo97 DiTo97 deleted the branch main April 15, 2026 08:38
@DiTo97 DiTo97 closed this Apr 15, 2026
@DiTo97 DiTo97 deleted the test/integration-isolation-and-smoke-cleanup branch April 15, 2026 08:39
@DiTo97 DiTo97 restored the test/integration-isolation-and-smoke-cleanup branch April 15, 2026 08:40
@DiTo97 DiTo97 reopened this Apr 15, 2026
@DiTo97 DiTo97 changed the base branch from test/read-edit-grep-edge-cases to main April 15, 2026 08:41
@DiTo97 DiTo97 merged commit 369bd66 into main Apr 15, 2026
@DiTo97 DiTo97 deleted the test/integration-isolation-and-smoke-cleanup branch April 15, 2026 08:41
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.

1 participant