Skip to content

Add unit tests to check for metric publishing

Adam Coldrick requested to merge sotk/metrics-improvements/tests into master

Before raising this MR, consider whether the following are required, and complete if so:

  • Unit tests
  • Metrics
  • Documentation update(s)

If not required, please explain in brief why not.

Description

This MR adds unit tests to ensure that we publish the metrics we expect, when we expect to. The actual publishing is mocked, so that mechanism isn't tested here. We also mock all the data, so could do with further testing around the actual calculation of metrics.

Issues addressed

Part of #207

Merge request reports