Skip to content

test(http-client-java): add e2e test#11263

Merged
XiaofeiCao merged 6 commits into
mainfrom
copilot/http-client-java-add-test-for-new-end-to-end-cases
Jul 16, 2026
Merged

test(http-client-java): add e2e test#11263
XiaofeiCao merged 6 commits into
mainfrom
copilot/http-client-java-add-test-for-new-end-to-end-cases

Conversation

Copilot AI commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

The Java route end-to-end tests still left one unsupported query expansion explode case enabled after the earlier test split. This branch finishes that cleanup by grouping the remaining known-broken route query expansion cases under disabled coverage instead of letting them fail active runs.

  • What changed

    • Moved the remaining testQueryExpansionExplode route case into the disabled set in routes/RouteTests.
    • Kept the previously isolated record/map-valued query expansion cases in the same disabled section so unsupported route query expansion coverage is grouped consistently.
  • Why this matters

    • Active route tests now cover only supported query expansion behavior.
    • Known unsupported explode/object and record/map serialization cases remain visible in source without breaking the main test pass.
  • Code shape

    @Test
    @Disabled("Blocked by current query object expansion serialization behavior for explode cases")
    public void testQueryExpansionExplode() {
        var client = new RoutesClientBuilder().buildQueryParametersQueryExpansionExplodeClient();
        client.primitive("a");
        client.array(List.of("a", "b"));
        client.model(new ExpandParameters("status", "active"));
    }

Copilot AI and others added 2 commits July 15, 2026 09:37
…d DiscriminatedUnionTests

Co-authored-by: weidongxu-microsoft <53292327+weidongxu-microsoft@users.noreply.github.com>
Co-authored-by: weidongxu-microsoft <53292327+weidongxu-microsoft@users.noreply.github.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
1 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

…ars, routes, duration, and discriminated-union

Co-authored-by: weidongxu-microsoft <53292327+weidongxu-microsoft@users.noreply.github.com>
Copilot AI changed the title test(http-client-java): add e2e tests for SpecialChars, encode/boolean, and type/union/discriminated test(http-client-java): fix test coverage gaps for routes, duration, and union Jul 15, 2026
@github-actions

Copy link
Copy Markdown
Contributor

All changed packages have been documented.

  • @typespec/http-client-java
Show changes

@typespec/http-client-java - internal ✏️

Add SpecialChars e2e test, encode/boolean test (disabled), and type/union/discriminated e2e tests

Co-authored-by: weidongxu-microsoft <53292327+weidongxu-microsoft@users.noreply.github.com>
Copilot AI changed the title test(http-client-java): fix test coverage gaps for routes, duration, and union Stabilize Java RouteTests by isolating unsupported query-record expansion cases Jul 15, 2026
Co-authored-by: weidongxu-microsoft <53292327+weidongxu-microsoft@users.noreply.github.com>
Copilot AI changed the title Stabilize Java RouteTests by isolating unsupported query-record expansion cases test(http-client-java): isolate unsupported route query expansion cases Jul 15, 2026
@weidongxu-microsoft weidongxu-microsoft marked this pull request as ready for review July 15, 2026 15:09
@azure-sdk-automation

Copy link
Copy Markdown

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

@XiaofeiCao XiaofeiCao added this pull request to the merge queue Jul 15, 2026
Merged via the queue into main with commit 42927da Jul 16, 2026
35 checks passed
@XiaofeiCao XiaofeiCao deleted the copilot/http-client-java-add-test-for-new-end-to-end-cases branch July 16, 2026 00:29
@weidongxu-microsoft weidongxu-microsoft changed the title test(http-client-java): isolate unsupported route query expansion cases @weidongxu-microsoft test(http-client-java): add e2e test Jul 16, 2026
@weidongxu-microsoft weidongxu-microsoft changed the title @weidongxu-microsoft test(http-client-java): add e2e test test(http-client-java): add e2e test Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

emitter:client:java Issue for the Java client emitter: @typespec/http-client-java

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants