Skip to content

Commit 069924c

Browse files
committed
GH-1550: Polish test
1 parent 72b9dd2 commit 069924c

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
@@ -84,7 +84,8 @@ void testTrustedAndNot() {
8484
.build();
8585
RecordHeaders recordHeaders = new RecordHeaders();
8686
mapper.fromHeaders(message.getHeaders(), recordHeaders);
87-
assertThat(recordHeaders.toArray().length).isEqualTo(23); // 21 + json_types
87+
int expectedSize = message.getHeaders().size() - 3; // ID, Timestamp, reply channel
88+
assertThat(recordHeaders.toArray().length).isEqualTo(expectedSize + 1); // json_types header
8889
Map<String, Object> headers = new HashMap<>();
8990
mapper.toHeaders(recordHeaders, headers);
9091
assertThat(headers.get("foo")).isInstanceOf(byte[].class);
@@ -112,7 +113,7 @@ void testTrustedAndNot() {
112113
NonTrustedHeaderType ntht = (NonTrustedHeaderType) headers.get("fix");
113114
assertThat(ntht.getHeaderValue()).isNotNull();
114115
assertThat(ntht.getUntrustedType()).isEqualTo(Foo.class.getName());
115-
assertThat(headers).hasSize(22);
116+
assertThat(headers).hasSize(expectedSize);
116117

117118
mapper.addTrustedPackages(getClass().getPackage().getName());
118119
headers = new HashMap<>();
@@ -121,7 +122,7 @@ void testTrustedAndNot() {
121122
assertThat(new String((byte[]) headers.get("foo"))).isEqualTo("bar");
122123
assertThat(headers.get("baz")).isEqualTo("qux");
123124
assertThat(headers.get("fix")).isEqualTo(new Foo());
124-
assertThat(headers).hasSize(22);
125+
assertThat(headers).hasSize(expectedSize);
125126
}
126127

127128
@Test

0 commit comments

Comments
 (0)