Applied StyleCI changes, added php/larastan to attribution

[bookstack] / tests / Entity / PageContentTest.php

diff --git a/tests/Entity/PageContentTest.php b/tests/Entity/PageContentTest.php
index 47a3c9c13614fe2d9f351fce1270d5c6db579cf5..0b99c63c3971ab92f393e90123c04053bba922a4 100644 (file)
--- a/tests/Entity/PageContentTest.php
+++ b/tests/Entity/PageContentTest.php
@@ -74,7 +74,7 @@ class PageContentTest extends TestCase
 
         $this->asEditor();
         $pageResp = $this->get($page->getUrl());
 
         $this->asEditor();
         $pageResp = $this->get($page->getUrl());

-        $pageResp->assertSee($content);
+        $pageResp->assertSee($content, false);

     }
 
     public function test_page_includes_rendered_on_book_export()
     }
 
     public function test_page_includes_rendered_on_book_export()


@@ -106,7 +106,7 @@ class PageContentTest extends TestCase
 
         $pageView = $this->get($page->getUrl());
         $pageView->assertStatus(200);
 
         $pageView = $this->get($page->getUrl());
         $pageView->assertStatus(200);

-        $pageView->assertDontSee($script);
+        $pageView->assertDontSee($script, false);

         $pageView->assertSee('abc123abc123');
     }
 
         $pageView->assertSee('abc123abc123');
     }
 


@@ -260,8 +260,8 @@ class PageContentTest extends TestCase
 
         $pageView = $this->get($page->getUrl());
         $pageView->assertStatus(200);
 
         $pageView = $this->get($page->getUrl());
         $pageView->assertStatus(200);

-        $pageView->assertDontSee($script);
-        $pageView->assertSee('<p>Hello</p>');
+        $pageView->assertDontSee($script, false);
+        $pageView->assertSee('<p>Hello</p>', false);

     }
 
     public function test_more_complex_inline_on_attributes_escaping_scenarios()
     }
 
     public function test_more_complex_inline_on_attributes_escaping_scenarios()


@@ -301,7 +301,7 @@ class PageContentTest extends TestCase
         $page->save();
 
         $pageView = $this->get($page->getUrl());
         $page->save();
 
         $pageView = $this->get($page->getUrl());

-        $pageView->assertSee($script);
+        $pageView->assertSee($script, false);

         $pageView->assertDontSee('abc123abc123');
     }
 
         $pageView->assertDontSee('abc123abc123');
     }
 


@@ -338,8 +338,8 @@ class PageContentTest extends TestCase
         $page->save();
 
         $pageView = $this->get($page->getUrl());
         $page->save();
 
         $pageView = $this->get($page->getUrl());

-        $pageView->assertSee($script);
-        $pageView->assertDontSee('<p>Hello</p>');
+        $pageView->assertSee($script, false);
+        $pageView->assertDontSee('<p>Hello</p>', false);

     }
 
     public function test_duplicate_ids_does_not_break_page_render()
     }
 
     public function test_duplicate_ids_does_not_break_page_render()


@@ -545,7 +545,7 @@ class PageContentTest extends TestCase
 
         $pageView = $this->get($page->getUrl());
         $pageView->assertStatus(200);
 
         $pageView = $this->get($page->getUrl());
         $pageView->assertStatus(200);

-        $pageView->assertSee($content);
+        $pageView->assertSee($content, false);

     }
 
     public function test_base64_images_get_extracted_from_page_content()
     }
 
     public function test_base64_images_get_extracted_from_page_content()


@@ -596,31 +596,24 @@ class PageContentTest extends TestCase
 
     public function test_base64_images_within_html_blanked_if_not_supported_extension_for_extract()
     {
 
     public function test_base64_images_within_html_blanked_if_not_supported_extension_for_extract()
     {

-        $this->asEditor();
-        $page = Page::query()->first();
-
-        $this->put($page->getUrl(), [
-            'name' => $page->name, 'summary' => '',
-            'html' => '<p>test<img src="data:image/jiff;base64,' . $this->base64Jpeg . '"/></p>',
-        ]);
+        // Relevant to https://github.com/BookStackApp/BookStack/issues/3010 and other cases
+        $extensions = [
+            'jiff', 'pngr', 'png ', ' png', '.png', 'png.', 'p.ng', ',png',
+            'data:image/png', ',data:image/png',
+        ];

 
 

-        $page->refresh();
-        $this->assertStringContainsString('<img src=""', $page->html);
-    }
+        foreach ($extensions as $extension) {
+            $this->asEditor();
+            $page = Page::query()->first();

 
 

-    // Relevant to https://github.com/BookStackApp/BookStack/issues/3010
-    public function test_base64_images_within_html_blanked_if_extension_incorrect_but_prefix_matches_correct_extension()
-    {
-        $this->asEditor();
-        $page = Page::query()->first();
-
-        $this->put($page->getUrl(), [
-            'name' => $page->name, 'summary' => '',
-            'html' => '<p>test<img src="data:image/pngr;base64,' . $this->base64Jpeg . '"/></p>',
-        ]);
+            $this->put($page->getUrl(), [
+                'name' => $page->name, 'summary' => '',
+                'html' => '<p>test<img src="data:image/' . $extension . ';base64,' . $this->base64Jpeg . '"/></p>',
+            ]);

 
 

-        $page->refresh();
-        $this->assertStringContainsString('<img src=""', $page->html);
+            $page->refresh();
+            $this->assertStringContainsString('<img src=""', $page->html);
+        }

     }
 
     public function test_base64_images_get_extracted_from_markdown_page_content()
     }
 
     public function test_base64_images_get_extracted_from_markdown_page_content()