JsoupSiteParser.extractSellerUrl(): add unit tests.

Fix #843

Author: php-coder

src/main/java/ru/mystamps/web/util/extractor/JsoupSiteParser.java

@@ -256,7 +256,6 @@ protected String extractSellerName(Element body) {
 		return sellerName;
 	}
 
-	// @todo #695 JsoupSiteParser.extractSellerUrl(): add unit tests
 	protected String extractSellerUrl(Element body) {
 		if (sellerLocator == null) {
 			return null;

src/test/java/ru/mystamps/web/util/extractor/JsoupSiteParserTest.java

@@ -825,6 +825,44 @@ public void extractSellerNameShouldReturnTextOfSellerLocator() {
 		assertThat(msg, name, equalTo(expectedValue));
 	}
 
+	//
+	// Tests for extractSellerUrl()
+	//
+	
+	@Test
+	public void extractSellerUrlShouldReturnNullWhenSellerLocatorIsNotSet() {
+		parser.setSellerLocator(null);
+		Element doc = createEmptyDocument();
+		
+		String url = parser.extractSellerUrl(doc);
+		
+		assertThat(url, is(nullValue()));
+	}
+	
+	@Test
+	public void extractSellerUrlShouldReturnNullWhenElementNotFound() {
+		parser.setSellerLocator(Random.jsoupLocator());
+		Element doc = createEmptyDocument();
+		
+		String url = parser.extractSellerUrl(doc);
+		
+		assertThat(url, is(nullValue()));
+	}
+	
+	@Test
+	public void extractSellerUrlShouldReturnValueOfHrefAttribute() {
+		parser.setSellerLocator("a");
+		
+		String expectedUrl = Random.url();
+		String html = String.format("<a href='%s'>test</a>", expectedUrl);
+		Element doc = createDocumentFromText(html);
+		
+		String url = parser.extractSellerUrl(doc);
+		
+		String msg = String.format("couldn't extract seller url from '%s'", doc);
+		assertThat(msg, url, equalTo(expectedUrl));
+	}
+	
 	//
 	// Tests for extractPrice()
 	//