Skip to content

feat: [Gax] Add script to download and generate the showcase client and download the showcase server#9347

Merged
bshaffer merged 14 commits into
mainfrom
generate-conformance-tests
Jul 24, 2026
Merged

feat: [Gax] Add script to download and generate the showcase client and download the showcase server#9347
bshaffer merged 14 commits into
mainfrom
generate-conformance-tests

Conversation

@Hectorhammett

@Hectorhammett Hectorhammett commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Created a script for downloading and generating the showcase client for the conformance tests. This client was generated a long time ago and we need a consistent way to generate and update the showcase client and server, specially with PQC around the corner.

Closes: #9229

Depends on: googleapis/gapic-generator-php#838

BEGIN_COMMIT_OVERRIDE
feat(dev): add showcase:generate command and update showcase client (#9347)
END_COMMIT_OVERRIDE

@Hectorhammett
Hectorhammett marked this pull request as ready for review July 16, 2026 22:03
@Hectorhammett
Hectorhammett requested a review from a team as a code owner July 16, 2026 22:03
Comment thread Gax/tests/Conformance/src/V1beta1/Client/TestingClient.php
Comment thread Gax/tests/Conformance/bin/gapic-showcase Outdated
Comment thread Gax/generate-showcase.sh Outdated
@Hectorhammett
Hectorhammett force-pushed the generate-conformance-tests branch from 67b386e to f5c0273 Compare July 20, 2026 21:08
Comment thread Gax/tests/Conformance/README.md Outdated
Comment thread Gax/tests/Conformance/README.md
Comment thread dev/src/Command/ShowcaseGenerateCommand.php
Comment thread Gax/tests/Conformance/src/V1beta1/Test_Blueprint.php Outdated
Comment thread dev/src/Command/GaxGenerateShowcaseCommand.php Outdated
Comment thread dev/src/Command/GaxGenerateShowcaseCommand.php Outdated
Comment thread dev/src/Command/ShowcaseGenerateCommand.php
Comment thread dev/src/Command/GaxGenerateShowcaseCommand.php Outdated
Comment thread dev/src/Command/ShowcaseGenerateCommand.php Outdated
Comment thread dev/src/Command/ShowcaseGenerateCommand.php
Comment thread dev/src/Command/ShowcaseGenerateCommand.php Outdated
Comment thread dev/src/Command/ShowcaseGenerateCommand.php

@cy-yun cy-yun left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

Thanks for adding this!

@bshaffer bshaffer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great! Most of what I have is nits - I think we have a lot of opportunities to simplify this.

Comment thread dev/src/Command/ShowcaseGenerateCommand.php Outdated
Comment thread dev/src/Command/ShowcaseGenerateCommand.php Outdated
Comment thread dev/src/Command/ShowcaseGenerateCommand.php Outdated
Comment thread dev/src/Command/ShowcaseGenerateCommand.php Outdated
Comment thread dev/src/Command/ShowcaseGenerateCommand.php Outdated
Comment thread dev/src/Command/ShowcaseGenerateCommand.php Outdated
Comment thread dev/src/Command/ShowcaseGenerateCommand.php Outdated
Comment thread dev/src/Command/ShowcaseGenerateCommand.php
Comment thread dev/src/Command/ShowcaseGenerateCommand.php Outdated
Comment thread dev/src/Command/ShowcaseGenerateCommand.php

@bshaffer bshaffer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is great! I tested it locally and it ran first time! Woohoo.

However when I ran it, all the message files were updated. I am running libprotoc 35.0, so maybe that's why? Can we do this with the most recent version of protoc so that we get all the latest versions of these files?

Also, the update to symfony/yaml seems to have broken our DocFX tests. We could update them, but I'd rather understand why it's necessary, as this is downgrading our installed version from v6 to v5

Comment thread dev/src/Command/ShowcaseGenerateCommand.php Outdated
Comment thread dev/composer.json Outdated
@Hectorhammett
Hectorhammett force-pushed the generate-conformance-tests branch from ce9a0ad to 6686e3b Compare July 24, 2026 19:00
@bshaffer bshaffer added the next release PRs to be included in the next release label Jul 24, 2026
@bshaffer
bshaffer merged commit 2aaffe7 into main Jul 24, 2026
38 of 40 checks passed
@bshaffer
bshaffer deleted the generate-conformance-tests branch July 24, 2026 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

next release PRs to be included in the next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improvements to Showcase conformance tests

3 participants