At the moment it seems that we only test the summary method for the pymc_experiments.DifferenceInDifferences class, and that is in the test_pymc_experiments.py file...
We should make sure that we are testing the summary method for all of the pymc_experiment classes. Though I believe the InstrumentalVariable class doesn't have a summery method.
This could be done by adding more to the current test_pymc_experiments.py file, but it could be worth thinking about simply adding to existing tests in test_integration_pymc_examples.py. My thinking is that this would involve less setup / computation time as we wouldn't be doing any additional model fitting.
At the moment it seems that we only test the
summarymethod for thepymc_experiments.DifferenceInDifferencesclass, and that is in thetest_pymc_experiments.pyfile...https://github.com/pymc-labs/CausalPy/blob/3bb81a702a19e725b1b180b8ddbc1aa3e453204c/causalpy/tests/test_pymc_experiments.py#L10-L23
We should make sure that we are testing the summary method for all of the
pymc_experimentclasses. Though I believe theInstrumentalVariableclass doesn't have asummerymethod.This could be done by adding more to the current
test_pymc_experiments.pyfile, but it could be worth thinking about simply adding to existing tests intest_integration_pymc_examples.py. My thinking is that this would involve less setup / computation time as we wouldn't be doing any additional model fitting.