Skip to content

Commit 21245d2

Browse files
roed314claude
andcommitted
Register test_search_parsing.py in CI test census (review fix)
The T07 PR added lmfdb/utils/test_search_parsing.py but left the CI test-file census asserting 43 files, so every matrix job failed at the "checking that we didn't miss any test files" step before running any tests or linting. - python-package.yml: bump census count 43 -> 44. - matrix_includes.json: assign the new module to the shard that already runs lmfdb/tests/test_utils.py (both proddb and devmirror copies) so its tests actually execute; add "utils" to that shard's display folders. Coordination: open PR #24 (ai/t28-ui-knowls) adds another new test file needing the same fix. Each PR bumps the census by one for its own file, so whichever of #4/#24 merges second must bump again (44 -> 45): a trivial, intentional same-line conflict on the census line. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 805aacc commit 21245d2

2 files changed

Lines changed: 5 additions & 5 deletions

File tree

.github/workflows/matrix_includes.json

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -60,13 +60,13 @@
6060
"server": "devmirror"
6161
},
6262
{
63-
"files": "lmfdb/users/test_users.py lmfdb/lattice/test_lattice.py lmfdb/maass_forms/test_maass.py lmfdb/higher_genus_w_automorphisms/test_hgcwa.py lmfdb/belyi/test_belyi.py lmfdb/hypergm/test_hgm.py lmfdb/tests/test_utils.py",
64-
"folders": "belyi higher_genus_w_automorphisms hypergm lattice maass_forms tests users",
63+
"files": "lmfdb/users/test_users.py lmfdb/lattice/test_lattice.py lmfdb/maass_forms/test_maass.py lmfdb/higher_genus_w_automorphisms/test_hgcwa.py lmfdb/belyi/test_belyi.py lmfdb/hypergm/test_hgm.py lmfdb/tests/test_utils.py lmfdb/utils/test_search_parsing.py",
64+
"folders": "belyi higher_genus_w_automorphisms hypergm lattice maass_forms tests users utils",
6565
"server": "proddb"
6666
},
6767
{
68-
"files": "lmfdb/users/test_users.py lmfdb/lattice/test_lattice.py lmfdb/maass_forms/test_maass.py lmfdb/higher_genus_w_automorphisms/test_hgcwa.py lmfdb/belyi/test_belyi.py lmfdb/hypergm/test_hgm.py lmfdb/tests/test_utils.py",
69-
"folders": "belyi higher_genus_w_automorphisms hypergm lattice maass_forms tests users",
68+
"files": "lmfdb/users/test_users.py lmfdb/lattice/test_lattice.py lmfdb/maass_forms/test_maass.py lmfdb/higher_genus_w_automorphisms/test_hgcwa.py lmfdb/belyi/test_belyi.py lmfdb/hypergm/test_hgm.py lmfdb/tests/test_utils.py lmfdb/utils/test_search_parsing.py",
69+
"folders": "belyi higher_genus_w_automorphisms hypergm lattice maass_forms tests users utils",
7070
"server": "devmirror"
7171
},
7272
{

.github/workflows/python-package.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ jobs:
9090
- name: checking that we didn't miss any test files
9191
shell: bash -l {0}
9292
# If this fails you need to update the file list above and file count
93-
run: test $(find lmfdb -name 'test_*.py' -or -name '*_test.py' | wc -l) -eq 43
93+
run: test $(find lmfdb -name 'test_*.py' -or -name '*_test.py' | wc -l) -eq 44
9494

9595
- name: Config LMFDB to run tests against proddb
9696
if: matrix.files != 'lint' && matrix.server == 'proddb'

0 commit comments

Comments
 (0)