Skip to content

Add tests for dart-dio anyof - #24615

Merged
wing328 merged 2 commits into
masterfrom
dart-anyof-test
Aug 5, 2026
Merged

Add tests for dart-dio anyof#24615
wing328 merged 2 commits into
masterfrom
dart-anyof-test

Conversation

@wing328

@wing328 wing328 commented Aug 5, 2026

Copy link
Copy Markdown
Member

a follow up pr to #24605

PR checklist

  • Read the contribution guidelines.
  • Run the following to build the project and update samples:
    ./mvnw clean package || exit
    ./bin/generate-samples.sh ./bin/configs/*.yaml || exit
    ./bin/utils/export_docs_generators.sh || exit
    
    (For Windows users, please run the script in WSL)
    Commit all changed files.
    This is important, as CI jobs will verify all generator outputs of your HEAD commit as it would merge with master.
    These must match the expectations made by your contribution.
    You may regenerate an individual generator by passing the relevant config(s) as an argument to the script, for example ./bin/generate-samples.sh bin/configs/java*.
    IMPORTANT: Do NOT purge/delete any folders/files (e.g. tests) when regenerating the samples as manually written tests may be removed.
  • If your PR is targeting a particular programming language, @mention the technical committee members, so they are more likely to review the pull request.

Summary by cubic

Adds a new dart-dio sample and CI coverage for AnyOf (primitive) to verify codegen, serialization, and build stability for the Dart client. Updates samples and workflow to catch regressions in AnyOf handling.

  • New Features
    • Added bin/configs/dart-dio-anyof.yaml to generate from built_value_anyof_primitive.yaml.
    • Added sample at samples/openapi3/client/petstore/dart-dio/anyof with generated FieldValue (AnyOf<String|int>), DefaultApi, serializers/docs, and placeholder tests; pubspec.yaml includes dio, one_of, one_of_serializer, and built_value.
    • Updated .github/workflows/samples-dart-build-test.yaml to build/test the new sample.

Written for commit 78f1ab1. Summary will update on new commits.

Review in cubic

@wing328
wing328 marked this pull request as ready for review August 5, 2026 08:10
@wing328
wing328 merged commit 554e1a5 into master Aug 5, 2026
20 of 33 checks passed
@wing328 wing328 mentioned this pull request Aug 5, 2026
3 tasks
@wing328
wing328 deleted the dart-anyof-test branch August 5, 2026 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant