Add test for missing labels

[jotak]

@KalmanMeth here's a test that clarifies the behaviour that IMO we should keep, wrt to missing labels and #460

[codecov]

Codecov Report

Merging #463 (9587d47) into main (f771d4b) will decrease coverage by 0.05%.
Report is 5 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #463      +/-   ##
==========================================
- Coverage   66.20%   66.15%   -0.05%     
==========================================
  Files          94       94              
  Lines        6808     6867      +59     
==========================================
+ Hits         4507     4543      +36     
- Misses       2046     2065      +19     
- Partials      255      259       +4     

Flag	Coverage Δ
unittests	66.15% <ø> (-0.05%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 5 files with indirect coverage changes

[jpinsonneau]

LGTM 👍

Is it related to a specific story ?

[jotak]

@jpinsonneau that was related to this ticket #464 where the proposed fix was to ignore counting records for metrics when there are missing labels, but that would have broken a bunch of things, hence this added test to write in stone the expected behaviour here.
Thanks for the review!
/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jotak

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [jotak]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment