Skip to content

test(deploy): licence gate for the publish set - #73

Open
qte77 wants to merge 2 commits into
PhysicalAIBenchmarks:mainfrom
qte77:test/licence-gate
Open

qte77 wants to merge 2 commits into
PhysicalAIBenchmarks:mainfrom
qte77:test/licence-gate

Conversation

@qte77

@qte77 qte77 commented Nov 14, 2026

Copy link
Copy Markdown
Collaborator

Adds tests/test_licence_gate.py: nothing PROVENANCE.csv records as licence-less (upstream_licence = none) may enter the publish set. tests/test_deploy.py already asserts the allowlist excludes sources/*/code by path; this adds the licence-driven check that #13 and #26 ask for, plus a SHA-256 match so a renamed copy is caught too. It also fails if those rows disappear, so the gate can't silently check nothing. It deletes nothing from the dataset (that is PR #56).

Checked: passes on main (today's publish set is clean); fails when the allowlist stops hiding sources/*/code and when a licence-less script is copied under docs/ (only the hash catches that one); full pytest -q -rs with harbor 0.23.0: 221 passed, 6 skipped. Limits: it only knows what PROVENANCE.csv records (last regenerated 2026-10-04; two rows still use the old harbor/ paths, still matched by hash).

Generated with Claude noreply@anthropic.com

qte77 and others added 2 commits November 5, 2026 00:41
Nothing from an upstream with no declared licence can enter the dataset,
by path or by content, using the licences and hashes in PROVENANCE.csv.

Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>

This branch has not been deployed

No deployments
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