Make even more weblinks tests (#2)

frontkom · Sep 30, 2024 · 8b950e8 · 8b950e8
1 parent 5318a0d
commit 8b950e8
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 14 deletions.
diff --git a/src/Transformer.php b/src/Transformer.php
@@ -17,17 +17,8 @@ public function processWeblinks(string $data, bool $nofollow = true) : string
         // </linkName></webLink>.
         // And then the output should become
         // <a rel="nofollow" href="https://www.example.com">gjeldende forskrift og tilhørende retningslinjer.</a>
-        if ($nofollow) {
-            return preg_replace(
-                '/<webLink><href>(.*?)<\/href>(<linkName>(.*?)<\/linkName>)?<\/webLink>/',
-                '<a rel="nofollow" href="$1">$3</a>',
-                $data,
-            );
-        }
-        return preg_replace(
-            '/<webLink><href>(.*?)<\/href>(<linkName>(.*?)<\/linkName>)?<\/webLink>/',
-            '<a href="$1">$3</a>',
-            $data,
-        );
+        $regex = '<webLink><href>(.*?)<\/href>(<linkName>([\S\s]*?)<\/linkName>)?<\/webLink>';
+        $replace = $nofollow ? '<a rel="nofollow" href="$1">$3</a>' : '<a href="$1">$3</a>';
+        return preg_replace('/' . $regex . '/', $replace, $data);
     }
 }
diff --git a/tests/WebLinksTest.php b/tests/WebLinksTest.php
@@ -14,7 +14,7 @@ class WebLinksTest extends TestCase
      *
      * @dataProvider weblinksProvider
      */
-    public function testWeblinks($input, $expected, $nofollow)
+    public function testWeblinks($input, $expected, $nofollow = true)
     {
         $transformer = new Transformer();
         $result = $transformer->processWeblinks($input, $nofollow);
@@ -29,14 +29,22 @@ public static function weblinksProvider()
                 '<webLink><href>https://www.example.com</href><linkName>gjeldende forskrift og tilhørende retningslinjer.</linkName></webLink>',
                 // @phpcs:ignore
                 '<a rel="nofollow" href="https://www.example.com">gjeldende forskrift og tilhørende retningslinjer.</a>',
-                true,
             ],
             [
                 // @phpcs:ignore
                 '<webLink><href>https://www.example.com</href><linkName>gjeldende forskrift og tilhørende retningslinjer.</linkName></webLink>',
                 '<a href="https://www.example.com">gjeldende forskrift og tilhørende retningslinjer.</a>',
                 false,
             ],
+            [
+                // Now one where we have a line break in the middle of a tag.
+                // @phpcs:ignore
+                '<webLink><href>https://www.example.com</href><linkName>gjeldende forskrift og tilhørende 
+retningslinjer.</linkName></webLink>',
+                // @phpcs:ignore
+                '<a rel="nofollow" href="https://www.example.com">gjeldende forskrift og tilhørende 
+retningslinjer.</a>',
+            ],
         ];
     }
 }