Skip to content

Conversation

EasterTheBunny
Copy link
Contributor

@EasterTheBunny EasterTheBunny commented Oct 13, 2025


Below is a summarization created by an LLM (gpt-4-0125-preview). Be mindful of hallucinations and verify accuracy.

Why

The changes implement configuration adjustments and operational improvements in the Docker Compose setup for the billing platform service. Specifically, these adjustments aim at enhancing integration testing capabilities by introducing new environment variables and modifying existing commands to better support automated testing environments.

What

  • framework/components/dockercompose/billing_platform_service/docker-compose.yml
    • Added N_WORKFLOW_ORACLES: 4 and ORGANIZATION_ID: "integration-test-aggregation-org-happy-path-odd-quorum" to the billing-platform-service and populate_test_data service environments. This introduces more explicit control over the number of workflow oracles and specifies an organization ID for integration tests.
    • Updated the populate_test_data service's command from using a complex set of flags to a simpler --linking-mode=basic. This simplifies the data population process for testing purposes.
    • Added BILLING_CLIENT_URL: "billing-platform-service:2222", DISABLE_AUTH: true, DB_HOST: postgres, DB_PORT: 5432, DB_NAME: billing_platform, DB_USERNAME: postgres, and DB_PASSWORD: postgres to the populate_test_data service environment. These changes standardize database connection settings and disable authentication for easier testing setup.
    • Removed the detailed command in populate_test_data service in favor of a more streamlined approach, improving clarity and ease of use for testing scenarios.

@EasterTheBunny EasterTheBunny force-pushed the etb/update-testing-framework-billing branch from 01c3fb7 to 0f2ba9a Compare October 16, 2025 17:21
@EasterTheBunny EasterTheBunny force-pushed the etb/update-testing-framework-billing branch from 0f2ba9a to 12d26c7 Compare October 16, 2025 17:22
@EasterTheBunny EasterTheBunny marked this pull request as ready for review October 16, 2025 17:22
@EasterTheBunny EasterTheBunny requested a review from a team as a code owner October 16, 2025 17:22
@EasterTheBunny EasterTheBunny enabled auto-merge (squash) October 16, 2025 17:28
@EasterTheBunny EasterTheBunny merged commit a9e72b8 into main Oct 16, 2025
68 checks passed
@EasterTheBunny EasterTheBunny deleted the etb/update-testing-framework-billing branch October 16, 2025 18:14
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