Skip to content

fix(tests): fix paperless-ngx container-structure-test config#335

Open
Copilot wants to merge 1 commit into
masterfrom
copilot/try-to-fix-tests
Open

fix(tests): fix paperless-ngx container-structure-test config#335
Copilot wants to merge 1 commit into
masterfrom
copilot/try-to-fix-tests

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jun 1, 2026

The tests/paperless-ngx.yaml referenced a non-existent binary (/app/bin/paperless-ng) and had a TBD placeholder checksum, causing all four tests to fail.

Changes

  • Removed commandTests — the sha256sum test had an unfilled TBD expected output; it would always fail
  • Replaced paperless-ng binary exists (path: /app/bin/paperless-ng) with manage.py exists (path: /app/manage.py) — the actual Django management entry point copied from src/manage.py during the build; no bin/ subdir exists in paperless-ngx src/, and pyproject.toml sets package = false so no pip entry-point scripts are installed either
  • Kept the two nltk data existence tests unchanged — they remain valid

@bdossantos bdossantos marked this pull request as ready for review June 1, 2026 09:36
Copilot AI review requested due to automatic review settings June 1, 2026 09:36
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes the paperless-ngx container-structure-test configuration so CI stops failing due to checks against a non-existent binary path and an unfilled checksum placeholder.

Changes:

  • Replace the file existence check from /app/bin/paperless-ng to /app/manage.py.
  • Remove the commandTests section that contained a TBD sha256 expected output (guaranteed failure).
  • Keep the existing NLTK data existence checks as-is.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

3 participants