Delegate OpenTelemetryTab.targetInfo to the wrapped tab

[nathanfallet]

Summary

OpenTelemetryTab.targetInfo was declared as override var targetInfo = tab.targetInfo — a stored property initialized once at construction. So a traced tab kept reporting stale target info after navigation (the wrapped tab's targetInfo updates via events / updateTarget() were never reflected), and writes to the wrapper never reached the underlying tab.

This affected every targetId/url/type read on a traced tab — activate, getWindow, saveScreenshot filename, getAllUrls, and screenshotB64's wait()→updateTarget() (which updated the wrong object).

Fix

Delegate get/set onto the wrapped tab, matching the existing lastMouseX/lastMouseY pattern:

override var targetInfo: Target.TargetInfo?
    get() = tab.targetInfo
    set(value) { tab.targetInfo = value }

Verification (red → green)

Added OpenTelemetryTabTargetInfoTest (mockk Tab with a mutable backing targetInfo, no browser):

• getter test: mutate the underlying value after wrapping → wrapper must reflect it (not a snapshot).
• setter test: write through the wrapper → must propagate to the wrapped tab.

Both fail on the snapshot implementation and pass after the fix. Full :opentelemetry:jvmTest passes.

Test plan

• ./gradlew :opentelemetry:jvmTest --tests "*.OpenTelemetryTabTargetInfoTest" — red on unfixed code, green after fix
• ./gradlew :opentelemetry:jvmTest — all pass

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!