Skip to content

Replace Cargo Gradle tasks with bootRun in UAA test harness#700

Open
duanemay wants to merge 1 commit into
mainfrom
boot
Open

Replace Cargo Gradle tasks with bootRun in UAA test harness#700
duanemay wants to merge 1 commit into
mainfrom
boot

Conversation

@duanemay

Copy link
Copy Markdown
Member

Summary

  • UAA no longer uses the Gradle Cargo plugin, so npm run start-uaa / npm run stop-uaa were broken: cargoStartLocal/cargoStopLocal no longer exist.
  • test/startUAA.sh: replace ./gradlew cargoStartLocal with ./gradlew run &. UAA's run task kills any stale UAA process then runs bootRun. It's backgrounded because, unlike the old Cargo deploy task, bootRun blocks in the foreground until killed — without & the script would hang before ever reaching the readiness poll / test-client creation.
  • package.json: replace stop-uaa's ./gradlew cargoStopLocal with UAA's own ./scripts/kill_uaa.sh, which looks up the UaaBootApplication process via jps (falling back to killing whatever's on port 8080).

Test plan

  • npm run start-uaa — UAA boots, OAuth token fetched, singular-test-client created
  • npm run start-singular-app — static bundle served
  • npm run unit-test — 25 specs, 0 failures
  • npm run integration-test (Nightwatch) — 56/56 assertions passed (Access Tests + Identity Tests)
  • npm run stop-uaa / stop-singular-app — both processes cleaned up correctly

🤖 Generated with Claude Code

UAA no longer uses the Gradle Cargo plugin for local test runs. Use the
gradlew "run" task (kills any stale UAA process, then bootRun) to start
it and UAA's own scripts/kill_uaa.sh to stop it. startUAA.sh now
backgrounds the boot process since bootRun blocks in the foreground,
unlike the old Cargo deploy task.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 10, 2026 22:12

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the UAA test harness to work with current UAA Gradle tasks by replacing the removed Cargo plugin tasks with UAA’s own run/kill mechanisms, so npm run start-uaa / npm run stop-uaa function again.

Changes:

  • Switch test/startUAA.sh from cargoStartLocal to a backgrounded ./gradlew run.
  • Update stop-uaa npm script to use UAA’s ./scripts/kill_uaa.sh instead of cargoStopLocal.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
test/startUAA.sh Starts UAA via Gradle run in the background so the script can continue to readiness polling and client setup.
package.json Updates npm scripts to stop UAA via UAA’s kill script now that Cargo tasks are gone.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread test/startUAA.sh

cd test/tmp/uaa
./gradlew cargoStartLocal 2>&1 > /dev/null
./gradlew run 2>&1 > /dev/null &
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants