-
Notifications
You must be signed in to change notification settings - Fork 10
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
test: add agent info endpoint malfunctioning test cases #891
Conversation
.NET has test cases which verify the behavior of traces endpoint, that it works even other endpoints could be having problems like config, telemetry or remote config. This PR to add similar level of testing coverage.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #891 +/- ##
==========================================
+ Coverage 71.67% 71.72% +0.05%
==========================================
Files 328 328
Lines 48413 48485 +72
==========================================
+ Hits 34699 34778 +79
+ Misses 13714 13707 -7
|
BenchmarksComparisonBenchmark execution time: 2025-02-20 14:43:30 Comparing candidate commit 2ab082a in PR branch Found 0 performance improvements and 1 performance regressions! Performance is the same for 51 metrics, 2 unstable metrics. scenario:benching deserializing traces from msgpack to their internal representation
CandidateCandidate benchmark detailsGroup 1
Group 2
Group 3
Group 4
Group 5
Group 6
Group 7
Group 8
Group 9
Group 10
Group 11
Group 12
Group 13
BaselineOmitted due to size. |
What does this PR do?
.NET has test cases which verify the behavior of traces endpoint, that it works even other endpoints could be having problems like config, telemetry or remote config.
This PR to add similar level of testing coverage.
Motivation
https://github.com/DataDog/dd-trace-dotnet/blob/4d06db4f8b87670fbfaf3e582198b3d9e74dc8bd/tracer/test/Datadog.Trace.ClrProfiler.IntegrationTests/AgentMalfunctionTests.cs#L54-L55
Additional Notes
None
How to test the change?
No implementation changes, only more test coverage.