Skip to content

Commit 045b1d0

Browse files
committed
GH-1550: Polish test
1 parent af0ad5b commit 045b1d0

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

spring-kafka/src/test/java/org/springframework/kafka/support/DefaultKafkaHeaderMapperTests.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,8 @@ void testTrustedAndNot() {
8282
.build();
8383
RecordHeaders recordHeaders = new RecordHeaders();
8484
mapper.fromHeaders(message.getHeaders(), recordHeaders);
85-
assertThat(recordHeaders.toArray().length).isEqualTo(23); // 21 + json_types
85+
int expectedSize = message.getHeaders().size() - 3; // ID, Timestamp, reply channel
86+
assertThat(recordHeaders.toArray().length).isEqualTo(expectedSize + 1); // json_types header
8687
Map<String, Object> headers = new HashMap<>();
8788
mapper.toHeaders(recordHeaders, headers);
8889
assertThat(headers.get("foo")).isInstanceOf(byte[].class);
@@ -110,7 +111,7 @@ void testTrustedAndNot() {
110111
NonTrustedHeaderType ntht = (NonTrustedHeaderType) headers.get("fix");
111112
assertThat(ntht.getHeaderValue()).isNotNull();
112113
assertThat(ntht.getUntrustedType()).isEqualTo(Foo.class.getName());
113-
assertThat(headers).hasSize(22);
114+
assertThat(headers).hasSize(expectedSize);
114115

115116
mapper.addTrustedPackages(getClass().getPackage().getName());
116117
headers = new HashMap<>();
@@ -119,7 +120,7 @@ void testTrustedAndNot() {
119120
assertThat(new String((byte[]) headers.get("foo"))).isEqualTo("bar");
120121
assertThat(headers.get("baz")).isEqualTo("qux");
121122
assertThat(headers.get("fix")).isEqualTo(new Foo());
122-
assertThat(headers).hasSize(22);
123+
assertThat(headers).hasSize(expectedSize);
123124
}
124125

125126
@Test

0 commit comments

Comments
 (0)