MNT Make data providers static

silverstripe · Jan 14, 2025 · 227bb5e · 227bb5e
1 parent 9b61628
commit 227bb5e
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/tests/php/Core/XssSanitiserTest.php b/tests/php/Core/XssSanitiserTest.php
@@ -11,7 +11,7 @@ class XssSanitiserTest extends SapphireTest
 {
     protected $usesDatabase = false;
 
-    public function provideSanitise(): array
+    public static function provideSanitise(): array
     {
         // Most of these scenarios are inspired by Symfony's HtmlSanitizerAllTest scenarios
         return [
@@ -341,7 +341,7 @@ public function testSanitiseElement(string $input, string $expected): void
         $this->assertSame($expected, $htmlValue->getContent());
     }
 
-    public function provideSanitiseElementsAllowed(): array
+    public static function provideSanitiseElementsAllowed(): array
     {
         return [
             'disallow these by default' => [
@@ -374,7 +374,7 @@ public function testSanitiseElementsAllowed(string $input, ?array $removeElement
         $this->assertSame($expected, $sanitiser->sanitiseString($input));
     }
 
-    public function provideSanitiseAttributesAllowed(): array
+    public static function provideSanitiseAttributesAllowed(): array
     {
         return [
             'disallow these by default' => [
@@ -425,7 +425,7 @@ public function testSanitiseAttributesAllowed(string $input, ?array $removeAttri
         $this->assertSame($expected, $sanitiser->sanitiseString($input));
     }
 
-    public function provideSanitiseNoKeepInnerHtml(): array
+    public static function provideSanitiseNoKeepInnerHtml(): array
     {
         return [
             'keeps inner html' => [

diff --git a/tests/php/Forms/FormMessageTest.php b/tests/php/Forms/FormMessageTest.php
@@ -6,13 +6,13 @@
 use SilverStripe\Core\XssSanitiser;
 use SilverStripe\Dev\SapphireTest;
 use SilverStripe\Forms\Tests\FormMessageTest\TestFormMessage;
-use SilverStripe\ORM\ValidationResult;
+use SilverStripe\Core\Validation\ValidationResult;
 
 class FormMessageTest extends SapphireTest
 {
     protected $usesDatabase = false;
 
-    public function provideGetMessage(): array
+    public static function provideGetMessage(): array
     {
         return [
             'empty HTML' => [

diff --git a/tests/php/View/Shortcodes/EmbedShortcodeProviderTest.php b/tests/php/View/Shortcodes/EmbedShortcodeProviderTest.php
@@ -222,7 +222,7 @@ public function testOnlyWhitelistedAttributesAllowed()
         );
     }
 
-    public function provideSandboxHtml(): array
+    public static function provideSandboxHtml(): array
     {
         return [
             'normal' => [