Skip to content

feat(ci): Add integration tests with testcontainers#12

Merged
ATMackay merged 5 commits into
mainfrom
feat/integration-tests
Oct 20, 2025
Merged

feat(ci): Add integration tests with testcontainers#12
ATMackay merged 5 commits into
mainfrom
feat/integration-tests

Conversation

@ATMackay

@ATMackay ATMackay commented Oct 8, 2025

Copy link
Copy Markdown
Owner

@ATMackay ATMackay requested a review from Copilot October 8, 2025 00:54

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

Add integration tests powered by Testcontainers to spin up Postgres and the checkout app in CI and locally, and wire them into the build.

  • Introduce a reusable Testcontainers stack (network, Postgres, app) with logging support.
  • Add an integration test to add inventory items and verify pricing.
  • Update Makefile, Dockerfile, and GitHub Actions workflow to build and run integration tests.

Reviewed Changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
integration/utils.go New helpers to create a Docker network, run Postgres and the app containers, and build a test client.
integration/add_inventory_test.go New integration test using the Testcontainers stack to add items and verify prices.
integration/concurrent_write_test.go Commented-out scaffold for a future concurrent write test (no functional changes).
go.mod Adds testcontainers and updates dependencies to support integration tests.
Makefile Adds a dedicated integration test target with tag and timeout.
Dockerfile Passes build metadata as args into the make build during image build.
.github/workflows/go.yml Adds an integration-test job running the new tests on ubuntu-latest.

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread integration/utils.go Outdated
Comment thread integration/add_inventory_test.go
Comment thread integration/add_inventory_test.go Outdated
Comment thread integration/utils.go Outdated
Comment thread integration/utils.go Outdated
@ATMackay ATMackay merged commit a4df263 into main Oct 20, 2025
3 checks passed
@ATMackay ATMackay deleted the feat/integration-tests branch October 20, 2025 09:40
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