Skip to content

Commit 40d5974

Browse files
Applied code style (zalando#1026)
1 parent 29a3f20 commit 40d5974

File tree

2 files changed

+20
-10
lines changed

2 files changed

+20
-10
lines changed

logbook-core/src/main/java/org/zalando/logbook/Conditions.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,8 +78,8 @@ public static <T extends HttpMessage> Predicate<T> header(final BiPredicate<Stri
7878
e.getValue().stream().anyMatch(v -> predicate.test(e.getKey(), v)));
7979
}
8080

81-
public static <T extends HttpMessage> Predicate<T> withoutHeader(String headerName) {
82-
return message -> !message.getHeaders().containsKey(headerName);
81+
public static <T extends HttpMessage> Predicate<T> withoutHeader(final String key) {
82+
return message -> !message.getHeaders().containsKey(key);
8383
}
8484

8585
}

logbook-core/src/test/java/org/zalando/logbook/ConditionsTest.java

Lines changed: 18 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -158,20 +158,30 @@ void headerShouldNotMatchPredicateWhenHeaderIsAbsent() {
158158
}
159159

160160
@Test
161-
void withoutHeader() {
162-
final Predicate<HttpMessage> unit = Conditions.withoutHeader("Authorization");
161+
void matchesWithoutHeader() {
162+
final Predicate<HttpMessage> unit = withoutHeader("Authorization");
163163

164164
assertThat(unit.test(request)).isTrue();
165165
}
166166

167167
@Test
168-
void withEmptyHeader() {
169-
final MockHttpRequest requestWithEmptyAuthorizationHeader = MockHttpRequest.create()
170-
.withHeaders(HttpHeaders.of("Authorization", ""))
171-
.withContentType("application/json");
168+
void doesNotMatchWithHeader() {
169+
final MockHttpRequest request = this.request
170+
.withHeaders(HttpHeaders.of("Authorization", "Bearer Unw62Gp9okJFN1AAHm8xtR"));
172171

173-
final Predicate<HttpMessage> unit = Conditions.withoutHeader("Authorization");
172+
final Predicate<HttpMessage> unit = withoutHeader("Authorization");
174173

175-
assertThat(unit.test(requestWithEmptyAuthorizationHeader)).isFalse();
174+
assertThat(unit.test(request)).isFalse();
175+
}
176+
177+
@Test
178+
void doesNotMatchWithEmptyHeaderValue() {
179+
final MockHttpRequest request = this.request
180+
.withHeaders(HttpHeaders.of("Authorization", ""));
181+
182+
final Predicate<HttpMessage> unit = withoutHeader("Authorization");
183+
184+
assertThat(unit.test(request)).isFalse();
176185
}
186+
177187
}

0 commit comments

Comments
 (0)