jihun4452 opened a new pull request, #7584: URL: https://github.com/apache/incubator-seata/pull/7584
- [x] I have read the [CONTRIBUTING.md](https://github.com/apache/incubator-seata/blob/2.x/CONTRIBUTING.md) guidelines. - [ ] I have registered the PR [changes](https://github.com/apache/incubator-seata/tree/2.x/changes). (N/A for this PR: test-only, no user-facing change) ### Ⅰ. Describe what this PR did Deflake `ConsulConfigurationTest.testInitSeataConfig` by adding a short await/retry loop (up to ~3s with 100ms backoff). This absorbs occasional CI timing/propagation delays when reading the mocked Consul KV so the assertion remains stable. **No production code changes — test-only.** ### Ⅱ. Does this pull request fix one issue? Fixes #7583. ### Ⅲ. Why don't you add test cases (unit test/integration test)? This PR modifies the failing test itself. The flakiness comes from timing, not functionality, so adding new tests would not add coverage. ### Ⅳ. Describe how to verify it Run the module tests repeatedly and confirm they pass: ```bash # run just the consul config module (build deps included) ./mvnw -pl config/seata-config-consul -am test # or target the single test method ./mvnw -q -pl config/seata-config-consul -am \ -Dtest=org.apache.seata.config.consul.ConsulConfigurationTest#testInitSeataConfig \ -Dsurefire.printSummary=true -Dsurefire.useFile=false \ -DfailIfNoTests=false \ test ``` ### Ⅴ. Special notes for reviews - Adds a bounded wait (≤3s) only to absorb CI timing; the original assertion remains. - No production code touched. - If preferred, we can swap to Awaitility, but avoided adding deps. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@seata.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@seata.apache.org For additional commands, e-mail: notifications-h...@seata.apache.org
