[jmeter] branch master updated: Fix test failure following ph-css upgrade, on parsing error, see Heavily improved fault tolerance in parsing when browser compliant mode is enabled

[pmouawad]

This is an automated email from the ASF dual-hosted git repository.

pmouawad pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/jmeter.git


The following commit(s) were added to refs/heads/master by this push:
     new 926f895  Fix test failure following ph-css upgrade, on parsing error, 
see Heavily improved fault tolerance in parsing when browser compliant mode is 
enabled
926f895 is described below

commit 926f895bfdc53f60b9d5b3e77ef231ff15d3b1e6
Author: pmouawad <p.moua...@ubik-ingenierie.com>
AuthorDate: Sun Sep 29 23:54:37 2019 +0200

    Fix test failure following ph-css upgrade, on parsing error, see Heavily 
improved fault tolerance in parsing when browser compliant mode is enabled
---
 .../org/apache/jmeter/protocol/http/parser/TestCssParser.java     | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git 
a/src/protocol/http/src/test/java/org/apache/jmeter/protocol/http/parser/TestCssParser.java
 
b/src/protocol/http/src/test/java/org/apache/jmeter/protocol/http/parser/TestCssParser.java
index 403154f..17190d4 100644
--- 
a/src/protocol/http/src/test/java/org/apache/jmeter/protocol/http/parser/TestCssParser.java
+++ 
b/src/protocol/http/src/test/java/org/apache/jmeter/protocol/http/parser/TestCssParser.java
@@ -50,14 +50,14 @@ public class TestCssParser extends JMeterTestCase {
 
     @Test
     public void testGetEmbeddedResourceURLsnOneUrl() throws Exception {
-        List<?> result;
-        result = extractUrls("@import url(http://example.com/abc.css);");
+        List<?> result = extractUrls("@import 
url(http://example.com/abc.css);");
         assertThat(result.isEmpty(), CoreMatchers.is(false));
     }
 
-    @Test(expected=LinkExtractorParseException.class)
+    @Test
     public void testExtractUrlsFromBrokenData() throws Exception {
-        extractUrls(CSS_IN_ERROR);
+        List<?> result = extractUrls(CSS_IN_ERROR);
+        assertThat(result.isEmpty(), CoreMatchers.is(true));
     }
 
     @Test