Skip to content

[chore]: Improve test coverage of remotesampling extension #7432

@yurishkuro

Description

@yurishkuro

Requirement

We are aiming for 95% code coverage in the project

Problem

https://app.codecov.io/gh/jaegertracing/jaeger/blob/main/cmd%2Fjaeger%2Finternal%2Fextension%2Fremotesampling%2Fextension.go

Lots of uncovered lines that could use extra tests. Also the Shutdown function does not need to wrap all errors, just append them to errors array, which will reduce the amount of uncovered (and hard to test) code.

Proposal

No response

Open questions

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    area/samplinggood first issueGood for beginnershelp wantedFeatures that maintainers are willing to accept but do not have cycles to implement

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions