Skip to content

test(python): add a pytest lane, bugsink R2 adapter as first suite - #504

Merged
zgeoff merged 3 commits into
mainfrom
feat/bugsink-workspace-package
Jul 12, 2026
Merged

test(python): add a pytest lane, bugsink R2 adapter as first suite#504
zgeoff merged 3 commits into
mainfrom
feat/bugsink-workspace-package

Conversation

@zgeoff

@zgeoff zgeoff commented Jul 12, 2026

Copy link
Copy Markdown
Owner

Description

Establish the monorepo's Python test lane and land its first suite. Python packages sit outside the Bun test graph, so a python-tests workflow runs their pytest under uv on change; apps/bugsink (the Bugsink R2 storage adapter) is the first and only one.

  • @vers/bugsink package (turbo.json app tag) with a test:adapter script running pytest under uv.
  • pytest suite for the R2 adapter — moto-mocked S3, Bugsink's ObjectStorage base stubbed so it runs without installing Bugsink.
  • One-line Python convention in agents/project.md; bun test rule scoped to JS/TS; uv added to knip ignoreBinaries; python cache gitignored.

Testing

  • bun run format:check, deadcode, boundaries pass
  • python-tests job green; bun run test:adapter — 5 passed (moto-mocked)

zgeoff added 3 commits July 12, 2026 21:38
Add @vers/bugsink (package.json + turbo.json app tag) and a pytest suite for the
R2 storage adapter, moto-mocking S3 and stubbing Bugsink's ObjectStorage base.
The Python sits outside the Bun test graph, so a test:adapter script runs it
under uv and a path-filtered bugsink-adapter-test workflow runs it in CI.
Ignore uv in knip and add python cache to gitignore.
Rename the workflow to python-tests, add a one-line Python convention to
project.md, and scope the bun-test rule to JS/TS packages.
@zgeoff zgeoff changed the title test(bugsink): make it a workspace package with adapter tests test(python): add a pytest lane, bugsink R2 adapter as first suite Jul 12, 2026
@zgeoff
zgeoff merged commit 8edde24 into main Jul 12, 2026
4 checks passed
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