Skip to content

Remove SchemaRegistry TestContainers from integration tests#177

Merged
jwijgerd merged 2 commits into
mainfrom
copilot/remove-schema-registry-testcontainer
Nov 16, 2025
Merged

Remove SchemaRegistry TestContainers from integration tests#177
jwijgerd merged 2 commits into
mainfrom
copilot/remove-schema-registry-testcontainer

Conversation

Copilot AI commented Nov 16, 2025

Copy link
Copy Markdown
Contributor

The SchemaRegistry TestContainer is no longer needed for integration tests as the framework now handles schema registration internally via Kafka topics.

Changes

Removed SchemaRegistry container from three crypto-trading test files:

  • CryptoTradingQueryApiTest.java
  • CryptoTradingCommandApiTest.java
  • CryptoTradingApplicationTest.java

For each file:

  • Removed GenericContainer field declaration for schemaRegistry with Confluent Schema Registry configuration
  • Removed akces.schemaregistry.url property from test initializers
  • Removed GenericContainer import

Tests continue to use KafkaContainer and PostgreSQLContainer where needed.

Original prompt

Remove all schemaRegistry TestContainer from all tests in all modules that have integration tests. Ensure all tests compile, you don't need to run them since the schemaregistry container is not used anymore


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: jwijgerd <914840+jwijgerd@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove schemaRegistry TestContainer from all integration tests Remove SchemaRegistry TestContainers from integration tests Nov 16, 2025
Copilot AI requested a review from jwijgerd November 16, 2025 19:46
@jwijgerd
jwijgerd marked this pull request as ready for review November 16, 2025 19:49
@jwijgerd
jwijgerd merged commit 797aea2 into main Nov 16, 2025
5 of 7 checks passed
@jwijgerd
jwijgerd deleted the copilot/remove-schema-registry-testcontainer branch November 16, 2025 19:57
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