search

inspectIT / inspectit-ocelot

inspectIT Ocelot - Java agent for collecting application performance, tracing and behavior data
http://www.inspectit.rocks/
Apache License 2.0
204 stars 69 forks source link

Adding integration tests for DynamicallyActivatableServiceObserver and update normal tests #1545

Closed danipaniii closed 1 year ago

danipaniii commented 1 year ago

In a meeting we discussed about implementing integration tests for the DynamicallyActivatableServiceObserver and with this PR those should be implemented.

Reference: https://github.com/inspectIT/inspectit-ocelot/pull/1527

This change is Reviewable

codecov[bot] commented 1 year ago

Codecov Report

Merging #1545 (96b82a4) into master (6b9c5d1) will increase coverage by 0.04%. The diff coverage is n/a.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1545 +/- ## ============================================ + Coverage 77.46% 77.50% +0.04% Complexity 2319 2319 ============================================ Files 240 240 Lines 7728 7728 Branches 923 923 ============================================ + Hits 5986 5989 +3 + Misses 1354 1351 -3 Partials 388 388 ``` | [Impacted Files](https://codecov.io/gh/inspectIT/inspectit-ocelot/pull/1545?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT) | Coverage Δ | | |---|---|---| | [...entation/transformer/AbstractClassTransformer.java](https://codecov.io/gh/inspectIT/inspectit-ocelot/pull/1545/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb3JlL3NyYy9tYWluL2phdmEvcm9ja3MvaW5zcGVjdGl0L29jZWxvdC9jb3JlL2luc3RydW1lbnRhdGlvbi90cmFuc2Zvcm1lci9BYnN0cmFjdENsYXNzVHJhbnNmb3JtZXIuamF2YQ==) | `83.81% <0.00%> (-0.95%)` | :arrow_down: | | [...it/ocelot/core/exporter/JaegerExporterService.java](https://codecov.io/gh/inspectIT/inspectit-ocelot/pull/1545/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb3JlL3NyYy9tYWluL2phdmEvcm9ja3MvaW5zcGVjdGl0L29jZWxvdC9jb3JlL2V4cG9ydGVyL0phZWdlckV4cG9ydGVyU2VydmljZS5qYXZh) | `72.88% <0.00%> (+6.78%)` | :arrow_up: |