refactor(JsoupSiteParserTest): replace ExpectedException by assertThat() for handling exceptions.

Author: php-coder

src/test/java/ru/mystamps/web/feature/series/importing/extractor/JsoupSiteParserTest.java

@@ -22,9 +22,7 @@
 import org.jsoup.nodes.Document;
 import org.jsoup.nodes.Element;
 import org.junit.Before;
-import org.junit.Rule;
 import org.junit.Test;
-import org.junit.rules.ExpectedException;
 import ru.mystamps.web.tests.Random;
 import java.util.Locale;
 
@@ -34,9 +32,6 @@
 
 public class JsoupSiteParserTest implements WithAssertions {
 
-	@Rule
-	public ExpectedException thrown = ExpectedException.none();
-	
 	private JsoupSiteParser parser;
 
 	@Before
@@ -50,10 +45,15 @@ public void init() {
 
 	@Test
 	public void parseShouldRequireNonBlankPageContent() {
-		thrown.expect(IllegalArgumentException.class);
-		thrown.expectMessage("Page content must be non-blank");
-		
-		parser.parse(nullOrBlank());
+		// given
+		// when
+		Throwable thrown = catchThrowable(() -> {
+			parser.parse(nullOrBlank());
+		});
+		// then
+		assertThat(thrown)
+			.isInstanceOf(IllegalArgumentException.class)
+			.hasMessage("Page content must be non-blank");
 	}
 
 	@Test