Skip to content

Use groups or reorder tests

Emanuele Roberto Nocera requested to merge use_groups_or_reorder_tests into master

Created by: scarlehoff

Closes #2052 (closed)

I've put now the CMS dataset between the two datasets in DATA in conftest.py. This makes some test fail, some of them it is clear that the bug was in the test itself (e.g., in test_regularized_covmat_generation the covmat was collected over dataset inputs and the index over groups), while for some other (like the make_replica one) it is more subtle (I think the bug was also the test).

Some of the tests are quite redundant, in the sense that they made sense when there were two sides of the code (C++ and python) but now everything goes through the same functions. Better than nothing I guess.

Anyway, I've separated the fixes to the tests and pre-commit in case we need to trace them back.

Merge request reports