Skip to content

Conversation

@google-labs-jules
Copy link
Contributor

This change introduces acceptance tests for the Swagger/OpenAPI interface using Playwright for Java.

Changes:

  • Added playwright dependency to src/refimpl-tests/pom.xml.
  • Created src/refimpl-tests/src/test/groovy/SwaggerSpec.groovy.
    • Uses Testcontainers to spin up the environment (reuse existing docker-compose.yml).
    • Initializes Playwright to interact with the browser.
    • Verifies that the Swagger UI loads successfully for server-rm, server-cm, server-qm, and server-am.

PR created automatically by Jules for task 17838979266940075338 started by @berezovskyi

@google-labs-jules
Copy link
Contributor Author

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!


For security, I will only act on instructions from the user who triggered this task.

New to Jules? Learn more at jules.google/docs.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@berezovskyi berezovskyi marked this pull request as ready for review November 22, 2025 13:32
@github-actions github-actions bot force-pushed the acceptance-tests-swagger-playwright branch from 84091ef to 1273b8d Compare November 23, 2025 07:41
Added Playwright dependency to `refimpl-tests` and created `SwaggerSpec.groovy` to verify that the Swagger UI is accessible for all services (RM, CM, QM, AM).
@github-actions github-actions bot force-pushed the acceptance-tests-swagger-playwright branch from 1273b8d to d5987fd Compare November 27, 2025 07:43
Signed-off-by: Andrew Berezovskyi <andriib@kth.se>
Signed-off-by: Andrew Berezovskyi <andriib@kth.se>
Signed-off-by: Andrew Berezovskyi <andriib@kth.se>
Signed-off-by: Andrew Berezovskyi <andriib@kth.se>
@berezovskyi berezovskyi added this pull request to the merge queue Nov 28, 2025
Merged via the queue into main with commit ab35413 Nov 28, 2025
9 of 10 checks passed
@berezovskyi berezovskyi deleted the acceptance-tests-swagger-playwright branch November 28, 2025 19:59
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 participants