Browse Source

Rename doesExist() to exists() for header assertions

pull/1655/head
Sam Brannen 8 years ago
parent
commit
b612f53e27
  1. 2
      spring-test/src/main/java/org/springframework/test/web/reactive/server/HeaderAssertions.java
  2. 4
      spring-test/src/main/java/org/springframework/test/web/servlet/result/HeaderResultMatchers.java
  3. 6
      spring-test/src/test/java/org/springframework/test/web/reactive/server/HeaderAssertionTests.java
  4. 20
      spring-test/src/test/java/org/springframework/test/web/servlet/samples/standalone/resultmatchers/HeaderAssertionTests.java

2
spring-test/src/main/java/org/springframework/test/web/reactive/server/HeaderAssertions.java

@ -77,7 +77,7 @@ public class HeaderAssertions { @@ -77,7 +77,7 @@ public class HeaderAssertions {
* Expect that the header with the given name is present.
* @since 5.0.3
*/
public WebTestClient.ResponseSpec doesExist(String name) {
public WebTestClient.ResponseSpec exists(String name) {
if (!getHeaders().containsKey(name)) {
String message = getMessage(name) + " does not exist";
this.exchangeResult.assertWithDiagnostics(() -> fail(message));

4
spring-test/src/main/java/org/springframework/test/web/servlet/result/HeaderResultMatchers.java

@ -91,10 +91,10 @@ public class HeaderResultMatchers { @@ -91,10 +91,10 @@ public class HeaderResultMatchers {
}
/**
* Assert that the named response header does exist.
* Assert that the named response header exists.
* @since 5.0.3
*/
public ResultMatcher doesExist(final String name) {
public ResultMatcher exists(final String name) {
return result -> assertTrue("Response should contain header '" + name + "'",
result.getResponse().containsHeader(name));
}

6
spring-test/src/test/java/org/springframework/test/web/reactive/server/HeaderAssertionTests.java

@ -126,16 +126,16 @@ public class HeaderAssertionTests { @@ -126,16 +126,16 @@ public class HeaderAssertionTests {
}
@Test
public void doesExist() {
public void exists() {
HttpHeaders headers = new HttpHeaders();
headers.setContentType(MediaType.APPLICATION_JSON_UTF8);
HeaderAssertions assertions = headerAssertions(headers);
// Success
assertions.doesExist("Content-Type");
assertions.exists("Content-Type");
try {
assertions.doesExist("Framework");
assertions.exists("Framework");
fail("Header should not exist");
}
catch (AssertionError error) {

20
spring-test/src/test/java/org/springframework/test/web/servlet/samples/standalone/resultmatchers/HeaderAssertionTests.java

@ -144,6 +144,16 @@ public class HeaderAssertionTests { @@ -144,6 +144,16 @@ public class HeaderAssertionTests {
}
}
@Test
public void exists() throws Exception {
this.mockMvc.perform(get("/persons/1")).andExpect(header().exists(LAST_MODIFIED));
}
@Test(expected = AssertionError.class)
public void existsFail() throws Exception {
this.mockMvc.perform(get("/persons/1")).andExpect(header().exists("X-Custom-Header"));
}
@Test // SPR-10771
public void doesNotExist() throws Exception {
this.mockMvc.perform(get("/persons/1")).andExpect(header().doesNotExist("X-Custom-Header"));
@ -154,16 +164,6 @@ public class HeaderAssertionTests { @@ -154,16 +164,6 @@ public class HeaderAssertionTests {
this.mockMvc.perform(get("/persons/1")).andExpect(header().doesNotExist(LAST_MODIFIED));
}
@Test
public void doesExist() throws Exception {
this.mockMvc.perform(get("/persons/1")).andExpect(header().doesExist(LAST_MODIFIED));
}
@Test(expected = AssertionError.class)
public void doesExistFail() throws Exception {
this.mockMvc.perform(get("/persons/1")).andExpect(header().doesExist("X-Custom-Header"));
}
@Test
public void stringWithIncorrectResponseHeaderValue() throws Exception {
assertIncorrectResponseHeader(header().string(LAST_MODIFIED, secondLater), secondLater);

Loading…
Cancel
Save