I decided to open this issue as a place to document any future discussion about this topic. The PISA guide on general conventions has been stating for a long time that the plan is to integrate pytest, I presume to auto-detect, run, and report on the results of unit tests.
https://github.com/icecube/pisa/pull/580 looks as if it was meant as a custom implementation of some pytest-like functionality, avoiding it as external dependency, but I'm not sure whether this interpretation is correct. The merged branch was even called "pytest".
I decided to open this issue as a place to document any future discussion about this topic. The PISA guide on general conventions has been stating for a long time that the plan is to integrate pytest, I presume to auto-detect, run, and report on the results of unit tests.
Installing pytest has been part of several workflows since https://github.com/icecube/pisa/pull/701, but apparently only because of the existence of two lines in two unit tests (
binning.test_MultiDimBinning()
andpisa_tests.test_kde_stage
) that are relying onpytest.raises()
as a context manager.https://github.com/icecube/pisa/pull/580 looks as if it was meant as a custom implementation of some pytest-like functionality, avoiding it as external dependency, but I'm not sure whether this interpretation is correct. The merged branch was even called "pytest".