Skip to content

AMP-187 Trigger api test as downstream of building docker image(push to main)#24

Merged
samirgarg merged 2 commits intomainfrom
chore/amp-187-api_tests
Jan 19, 2026
Merged

AMP-187 Trigger api test as downstream of building docker image(push to main)#24
samirgarg merged 2 commits intomainfrom
chore/amp-187-api_tests

Conversation

@samirgarg
Copy link
Contributor

@samirgarg samirgarg commented Jan 16, 2026

Summary of Changes

.github/workflows/ci-build-publish.yml

  • Renamed job: api-testAPI-Test-PR

  • Added new job API-Test-Main:

    • Runs on trigger_docker (push to main/master)
    • Depends on Build-Docker and Artefact-Version
    • Tests the published Docker image from GitHub Container Registry
    • Uses DOCKER_IMAGE environment variable to pull and run the image

apiTest/docker-compose.yml

  • Added image: ${DOCKER_IMAGE} field to the app service
  • Now supports both modes:
    • When DOCKER_IMAGE is set: Uses the published image
    • When DOCKER_IMAGE is not set: Builds from source using Dockerfile
  • Added comments explaining the conditional behavior

Gradle wrapper

  • Added gradle wrapper for all gradle tasks

Result

Single docker-compose file works for both PR (build from source) and push (use published image) scenarios, controlled by the DOCKER_IMAGE environment variable.

@samirgarg samirgarg requested a review from a team as a code owner January 16, 2026 17:54
@samirgarg samirgarg marked this pull request as draft January 16, 2026 17:58
@samirgarg samirgarg marked this pull request as ready for review January 17, 2026 08:00
@samirgarg samirgarg changed the title AMP-187 Trigger api test as downstream of building docker image AMP-187 Trigger api test as downstream of building docker image(push to main) Jan 19, 2026
@samirgarg samirgarg merged commit 0e2db20 into main Jan 19, 2026
12 checks passed
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.

2 participants