Skip to content
Prev Previous commit
Next Next commit
Update test_disabled_metric_ref_model to have more disabled metrics
There are metrics which depend on the metric `number_of_people`. If
`number_of_people` is disabled without the metrics that depend on it
being disabled, then a different (expected) exception would be raised
than the one this test is testing for. Thus we've disabled those
downstream metrics.
  • Loading branch information
QMalcolm committed Jun 28, 2023
commit 3c9f33643a208dc6e356c9a4d8d9f9e8c2f4a0cc
8 changes: 8 additions & 0 deletions tests/functional/metrics/test_metric_configs.py
Original file line number Diff line number Diff line change
Expand Up @@ -122,13 +122,21 @@ def test_disabled_metric_ref_model(self, project):
assert "metric.test.number_of_people" in manifest.metrics
assert "metric.test.collective_tenure" in manifest.metrics
assert "model.test.people_metrics" in manifest.nodes
assert "metric.test.average_tenure" in manifest.metrics
assert "metric.test.average_tenure_minus_people" in manifest.metrics

new_enabled_config = {
"metrics": {
"test": {
"number_of_people": {
"enabled": False,
},
"average_tenure_minus_people": {
"enabled": False,
},
"average_tenure": {
"enabled": False,
},
}
}
}
Expand Down