Skip to content

test(procore): add pdf jwt dependency smoke#28

Merged
mcxl merged 1 commit into
mainfrom
codex/procore-pdf-jwt-smoke
Jun 17, 2026
Merged

test(procore): add pdf jwt dependency smoke#28
mcxl merged 1 commit into
mainfrom
codex/procore-pdf-jwt-smoke

Conversation

@mcxl

@mcxl mcxl commented Jun 17, 2026

Copy link
Copy Markdown
Owner
  • P2 fix(pims): whitelist + commit RPD_SSA_template.docx so route can find it #5: adds dependency smoke coverage for the pypdf and python-jose/cryptography upgrade risk.
  • Generates tiny valid PDFs in memory, so no fixture files are added.
  • Exercises document PDF extraction on a real text PDF when pypdf is installed.
  • Exercises textless/scanned-style PDF fallback without a live OCR/network call.
  • Exercises the Procore PDF green path without vision audit.
  • Exercises real ES256 JWT encode/decode through core.auth.get_current_user using a generated JWKS.

Local tests:

  • python -m pytest tests/test_dependency_smoke.py -v -> 1 passed, 3 skipped locally because this Python lacks pypdf
  • flake8 tests/test_dependency_smoke.py -> clean
  • direct bundled-runtime pypdf fixture validation -> text PDF extracted 3430 chars, textless PDF extracted empty text
  • pre-push full suite -> passed

CI should run the PDF tests after installing requirements.txt with pypdf==6.13.0.

No production code changed.

@mcxl mcxl merged commit 8a16d4f into main Jun 17, 2026
1 check passed
@mcxl mcxl deleted the codex/procore-pdf-jwt-smoke branch June 17, 2026 22:29
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