flake8
passed.
JSHint
passed.
Update TestCase for new fixtures loading.
Review Request #14808 — Created Feb. 10, 2026 and submitted
Fixtures are now (primarily) loaded at class setup time rather than
per-test-case. This change updates Review Board's TestCase class to
handle that, and simplifies our handling of thetest_scmtools
fixture, which is not a real fixture file anymore but ends up triggering
loading of SCMTools from the registry.
Ran unit tests.
-
-
This mentions the obsolete test_scmtools fixture, but I'm not sure it's obsolete. The fixture file itself certainly is, but we populate the fixture database when this is flagged, and iirc that still matters to many tests. Unless I'm mistaken, I think I wrote tests that needed that fixture loaded recently.
- Description:
-
Fixtures are now (primarily) loaded at class setup time rather than
per-test-case. This change updates Review Board's TestCase class to ~ handle that, and simplifies our handling of the obsolete test_scmtools ~ fixture. ~ handle that, and simplifies our handling of the test_scmtools~ fixture, which is not a real fixture file anymore but ends up triggering + loading of SCMTools from the registry. - Commits:
-
Summary ID wouswnzunlvymollknxvnnvkwxlzuzro wouswnzunlvymollknxvnnvkwxlzuzro - Diff:
-
Revision 2 (+76 -74)
Checks run (2 succeeded)
flake8
passed.
JSHint
passed.