Skip to content

Commit 76dcd5f

Browse files
authored
MINOR: Reduce duplicate parameter combinations in heavy Streams integration tests (#22125)
Cuts redundant parameterized cases from three slow Streams integration tests without losing meaningful coverage: - IQv2StoreIntegrationTest — skips withHeaders=true rows for kind=PAPI and for global stores. DSL_STORE_FORMAT_CONFIG (the setting withHeaders toggles) only affects DSL stores built without an explicit supplier, so those rows are exact duplicates of the withHeaders=false runs. Reviewers: TengYao Chi <frankvicky@apache.org>, Bill Bejeck <bill@confluent.io>
1 parent 0dad6a7 commit 76dcd5f

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

streams/integration-tests/src/test/java/org/apache/kafka/streams/integration/IQv2StoreIntegrationTest.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -364,6 +364,11 @@ public static Stream<Arguments> data() {
364364
for (final String kind : Arrays.asList("DSL", "PAPI")) {
365365
for (final String groupProtocol : Arrays.asList("classic", "streams")) {
366366
for (final boolean withHeaders : Arrays.asList(true, false)) {
367+
// DSL_STORE_FORMAT_CONFIG only affects DSL stores built without an
368+
// explicit supplier; for PAPI and global stores it is a no-op, so
369+
// skip the redundant withHeaders=true duplicates.
370+
if (withHeaders && (!"DSL".equals(kind) || toTest.global()))
371+
continue;
367372
values.add(Arguments.of(cacheEnabled, logEnabled, toTest.name(), kind, groupProtocol, withHeaders));
368373
}
369374
}

0 commit comments

Comments
 (0)