Skip to content

test(integration): cover protocol negotiation defaults#911

Merged
jbachorik merged 1 commit into
developfrom
agent/issue-891-integration-coverage
Jul 19, 2026
Merged

test(integration): cover protocol negotiation defaults#911
jbachorik merged 1 commit into
developfrom
agent/issue-891-integration-coverage

Conversation

@jbachorik

@jbachorik jbachorik commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

Summary

Fixes #891’s release-critical integration coverage gaps.

  • Adds end-to-end dynamic-attach coverage for the shipped V2 auto-negotiating client configuration.
  • Adds V1-client compatibility coverage against the V2-capable agent.
  • Replaces @OnProbe's silent modern-JDK pass with a report-visible JAXB capability assumption.

Why

The normal integration harness previously forced V2/no-negotiation, leaving default negotiation and V1 compatibility outside ordinary PR coverage. @OnProbe also returned green when JAXB was unavailable.

Validation

  • :btrace-dist:build
  • -Pintegration :integration-tests:test --tests tests.BTraceFunctionalTests
  • :integration-tests:spotlessCheck
  • git diff --check

This change is Reviewable

@jbachorik
jbachorik force-pushed the agent/issue-891-integration-coverage branch from 6da07ed to 26f85fd Compare July 19, 2026 12:03
@jbachorik
jbachorik marked this pull request as ready for review July 19, 2026 12:05
@jbachorik
jbachorik merged commit 0fd3de8 into develop Jul 19, 2026
16 of 17 checks passed
@jbachorik
jbachorik deleted the agent/issue-891-integration-coverage branch July 19, 2026 16:08
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.

[3.0] Integration coverage gaps: default auto-negotiate/V1 untested on PRs; @OnProbe silently skipped

1 participant