corvus-dotnet / corvus-dotnet/Corvus.Monitoring

Determine what, if any, additional tests can be defined

Open
#27 0 comments 0 reactions 0 assignees View on GitHub
enhancement IMM
Dominant language
C#
Stars
5
Forks
0
PR merge metrics
No merged PRs in 30d

Description

Currently, the `Executable Specifications` entry in `imm.yaml` scores just 1 point. There are two additional ways we can increase the score:

* Specs which cover common failure cases
* Specs which explore edge cases

We can do either (for one point each) or both (for two). Arguably the tests already cover edge cases as far as possible, but someone other than @idg10 should review them to see what opportunities I have missed to cover more.

We don't currently have much by way of testing for failure, not least because we haven't really defined what we want to have happen. Failure will generally mean that the underlying instrumentation provider is throwing exceptions. Probably all we can do in this case is allow them to pass back out unimpeded. This is likely to be the default behaviour already, but we could add tests to verify that.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing the `Executable Specifications` entry in `imm.yaml` and the existing tests that contribute to its score. Look for missed common failure cases and edge cases, and determine whether instrumentation-provider exceptions should pass through unchanged. Done means adding justified coverage where opportunities exist and verifying the resulting score.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
testing-qa
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.