Skip to content

📦 Ship the complete test tree and its resources in the sdist - #616

Merged
MiWeiss merged 1 commit into
mainfrom
fix/sdist-ships-tests-and-resources
Sep 3, 2026
Merged

📦 Ship the complete test tree and its resources in the sdist#616
MiWeiss merged 1 commit into
mainfrom
fix/sdist-ships-tests-and-resources

Conversation

@MiWeiss

@MiWeiss MiWeiss commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

The sdist only shipped the four top-level test*.py files that setuptools picks up by default. tests/resources/ and the splitter_tests / middleware_tests subpackages were missing, so running the suite from an unpacked sdist failed with FileNotFoundError: tests/resources/gbk_test.bib.

Fix: add a MANIFEST.in that grafts tests (with a global-exclude for __pycache__ / *.pyc so stale caches do not leak in).

Verified:

  • pytest -q in an unpacked sdist: 2730 passed, 12 skipped — identical to a checkout.
  • Wheel unchanged, still contains no tests ([tool.setuptools.packages.find] keeps it to bibtexparser*).
  • No caches or build artefacts added to the sdist.

Fixes #486

🤖 Generated with Claude Code

The sdist only contained the four top-level `test*.py` files that
setuptools picks up by default: `tests/resources/` and the
`splitter_tests` and `middleware_tests` subpackages were missing, so
running the test suite from an unpacked sdist failed with
`FileNotFoundError: tests/resources/gbk_test.bib`.

Add a `MANIFEST.in` grafting `tests`. The wheel is unaffected, as
`[tool.setuptools.packages.find]` restricts it to `bibtexparser*`.

Fixes #486

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@MiWeiss
MiWeiss merged commit ef2fa54 into main Sep 3, 2026
16 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.

Packaging: Include all files necessary for testing

1 participant