Skip to content

fix: move Otel testing library out production runtime#446

Merged
abelonogov-ld merged 5 commits intomainfrom
andrey/open-telemetry-testing
Mar 28, 2026
Merged

fix: move Otel testing library out production runtime#446
abelonogov-ld merged 5 commits intomainfrom
andrey/open-telemetry-testing

Conversation

@abelonogov-ld
Copy link
Copy Markdown
Contributor

@abelonogov-ld abelonogov-ld commented Mar 28, 2026

Summary

How did you test this change?

Are there any deployment considerations?


Note

Medium Risk
Medium risk because it changes the SDK’s public testing/inspection surface (replaces ObservabilityClient.getTelemetryInspector() with an ObservabilityOptions.telemetryInspector injection) and refactors core initialization to use ObservabilityService, which could affect downstream integrations if any relied on the old classes or runtime testing dependency.

Overview
Moves OpenTelemetry in-memory “telemetry inspection” support out of production runtime by making opentelemetry-sdk-testing a testImplementation/testFixtures dependency and introducing testing.InMemoryTelemetryInspector (test fixtures) for assertions.

Refactors telemetry inspection wiring: TelemetryInspector becomes an interface, a new ObservabilityOptions.telemetryInspector is added, and ObservabilityService composes the provided exporters into the debug composite exporters. The old ObservabilityClient/getTelemetryInspector() path is removed, and E2E tests are updated to inject and read the in-memory inspector via TestApplication.

Written by Cursor Bugbot for commit 2941a91. This will update automatically on new commits. Configure here.

@abelonogov-ld abelonogov-ld requested a review from a team as a code owner March 28, 2026 00:01
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

@abelonogov-ld abelonogov-ld changed the title fix: move Otel testing library into testing fix: move Otel testing library out production runtime Mar 28, 2026
@abelonogov-ld abelonogov-ld enabled auto-merge (squash) March 28, 2026 00:20
@abelonogov-ld abelonogov-ld merged commit 8a8ef34 into main Mar 28, 2026
24 checks passed
@abelonogov-ld abelonogov-ld deleted the andrey/open-telemetry-testing branch March 28, 2026 00:30
Vadman97 pushed a commit that referenced this pull request Mar 28, 2026
🤖 I have created a release *beep* *boop*
---


<details><summary>launchdarkly-observability-android: 0.34.1</summary>

##
[0.34.1](launchdarkly-observability-android-0.34.0...launchdarkly-observability-android-0.34.1)
(2026-03-28)


### Bug Fixes

* move Otel testing library out production runtime
([#446](#446))
([8a8ef34](8a8ef34))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Low risk: this is a release-version bump plus changelog/manifest
updates, with no functional code changes in this PR diff.
> 
> **Overview**
> Bumps `sdk/@launchdarkly/observability-android` from `0.34.0` to
`0.34.1` in the release manifest and Gradle properties.
> 
> Updates the Android package changelog for `0.34.1`, noting the bug fix
to move an OpenTelemetry testing library out of the production runtime.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
bbd27cf. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
abelonogov-ld added a commit that referenced this pull request Mar 30, 2026
* main:
  chore: release main (#447)
  fix: move Otel testing library out production runtime (#446)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants