Skip to content

Convert jaxws tests to java#12509

Merged
laurit merged 3 commits into
open-telemetry:mainfrom
laurit:jaxws-java
Oct 24, 2024
Merged

Convert jaxws tests to java#12509
laurit merged 3 commits into
open-telemetry:mainfrom
laurit:jaxws-java

Conversation

@laurit

@laurit laurit commented Oct 23, 2024

Copy link
Copy Markdown
Contributor

Part of #7195

@laurit laurit requested a review from a team as a code owner October 23, 2024 14:00
Comment on lines -20 to -34
<xs:element name="hello2Request">
<xs:complexType>
<xs:sequence>
<xs:element name="name" type="xs:string" form="unqualified"/>
</xs:sequence>
</xs:complexType>
</xs:element>

<xs:element name="hello2Response">
<xs:complexType>
<xs:sequence>
<xs:element name="message" type="xs:string" form="unqualified"/>
</xs:sequence>
</xs:complexType>
</xs:element>

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

In jax-ws 2 tests hello request also tests the jaxws-jws-api-1.1 instrumentation while hello2 test is constructed so that the jws api instrumentation does not apply to it. In jax-ws 3 we don't have an instrumentation similar to the jws api instrumentation so we need only 1 test case.

laurit and others added 2 commits October 24, 2024 12:57
Co-authored-by: Jay DeLuca <jaydeluca4@gmail.com>
@laurit laurit merged commit 3cf37dd into open-telemetry:main Oct 24, 2024
@laurit laurit deleted the jaxws-java branch October 24, 2024 11:57
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