vh/forgefirm
1
0
Fork 0
mirror of https://github.com/openglow-org/forgefirm.git synced 2026-09-27 08:41:13 -07:00
Code Issues Packages Projects Releases Wiki Activity

forgetest CI: run the python steps with -B so no bytecode enters the recipe fetch

Browse Source 
B-16: the forgetest recipe fetches the package directory whole, and the
file fetcher's checksum is taken before do_unpack drops __pycache__, so a
host `__pycache__` written by running the tests moves the recipe's task
hash with no source change. The three python steps run with -B now;
`src-sync` to the build VM excludes the caches as well. A developer who
runs the tests locally without -B still creates them; there is no exclude
on the file fetcher, so this is the floor, not a cure (BRINGUP item 11).
This commit is contained in:
ScottW514
2026-09-02 19:09:07 -04:00
parent 05cf7341a4
commit 77c11c52e4
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.github/workflows/forgetest-ci.yml
+5 -3
View File
@@ -54,13 +54,15 @@ jobs:
- name: Unit tests - name: Unit tests
working-directory: forgefirm/forgetest working-directory: forgefirm/forgetest
run: python -m unittest discover -s tests -v # -B: no bytecode caches under the package directory (the recipe's fetch
# checksum would otherwise move without a source change)
run: python -B -m unittest discover -s tests -v
- name: No undefined names (the live drills never execute on the host) - name: No undefined names (the live drills never execute on the host)
working-directory: forgefirm/forgetest working-directory: forgefirm/forgetest
run: | run: |
pip install pyflakes pip install pyflakes
if python -m pyflakes forgetest tests | grep -E "undefined name"; then exit 1; fi if python -B -m pyflakes forgetest tests | grep -E "undefined name"; then exit 1; fi
- name: Shared UI files match forgectrl at its pinned revision - name: Shared UI files match forgectrl at its pinned revision
working-directory: forgefirm working-directory: forgefirm
@@ -68,7 +70,7 @@ jobs:
- name: Coverage lint (enforced) - name: Coverage lint (enforced)
working-directory: forgefirm/forgetest working-directory: forgefirm/forgetest
run: python -m forgetest.coverage --manifest ../tree-manifest.json --enforce | tee "$GITHUB_STEP_SUMMARY" run: python -B -m forgetest.coverage --manifest ../tree-manifest.json --enforce | tee "$GITHUB_STEP_SUMMARY"
- name: Gate self-check on the tree manifest - name: Gate self-check on the tree manifest
working-directory: forgefirm working-directory: forgefirm