Use assertThat instead of Java assert

philwebb · rwinch · commit ef8f11361912 · 2020-08-24T17:33:09.000-05:00
Fix `DefaultSavedRequestMixinTests` so that `assertThat` is used rather than Java's `assert` keyword. Issue gh-8945
diff --git a/web/src/test/java/org/springframework/security/web/jackson2/DefaultSavedRequestMixinTests.java b/web/src/test/java/org/springframework/security/web/jackson2/DefaultSavedRequestMixinTests.java
@@ -84,7 +84,7 @@ public void matchRequestBuildWithConstructorAndBuilder() {
 		MockHttpServletRequest mockRequest = new MockHttpServletRequest();
 		mockRequest.setCookies(new Cookie("SESSION", "123456789"));
 		mockRequest.addHeader("x-auth-token", "12");
-		assert request.doesRequestMatch(mockRequest, new PortResolverImpl());
+		assertThat(request.doesRequestMatch(mockRequest, new PortResolverImpl())).isTrue();
 	}
 
 	@Test

Original file line number	Diff line number	Diff line change
`@@ -84,7 +84,7 @@ public void matchRequestBuildWithConstructorAndBuilder() {`
`84`	`84`	`MockHttpServletRequest mockRequest = new MockHttpServletRequest();`
`85`	`85`	`mockRequest.setCookies(new Cookie("SESSION", "123456789"));`
`86`	`86`	`mockRequest.addHeader("x-auth-token", "12");`
`87`		`- assert request.doesRequestMatch(mockRequest, new PortResolverImpl());`
	`87`	`+ assertThat(request.doesRequestMatch(mockRequest, new PortResolverImpl())).isTrue();`
`88`	`88`	`}`
`89`	`89`
`90`	`90`	`@Test`