Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Coverage metric in report #233

Closed
therealryan opened this issue Feb 2, 2023 · 0 comments · Fixed by #391
Closed

Coverage metric in report #233

therealryan opened this issue Feb 2, 2023 · 0 comments · Fixed by #391
Labels
enhancement New feature or request javascript Pull requests that update Javascript code

Comments

@therealryan
Copy link
Collaborator

We add masking operations to messages to deal with message content that we don't want to assert on in testing (it might be unpredictable or just boring to model). These operations necessarily reduce test strength, as we're no longer asserting on the entire message.
Currently the only way to see how much of the message is actually being asserted on is to go to the execution report and manually compare the full and asserted messages. This means it's possible to add over-enthusiastic masking operations that hide message content that we really should be asserting on, and it'd take someone doing a bunch of clicking around to spot the weakened coverage.

It'd be good to highlight test strength in the execution report to reduce review effort.

For each asserted message in the report detail page, we should compare the asserted (masks applied) and full (no masking) messages and compute a test strength metric, ranging from 100% (the asserted and full messages are exactly the same) to 0% (the asserted and full messages are completely different) and display that alongside the assertion result.

@therealryan therealryan added enhancement New feature or request javascript Pull requests that update Javascript code labels Feb 2, 2023
@therealryan therealryan changed the title Coverage metric Coverage metric in report Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request javascript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant