Fixes.

silverstripe-terraformers · Dec 21, 2023 · 4c6c0cb · 4c6c0cb
1 parent cc21e76
commit 4c6c0cb
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 11 deletions.
diff --git a/tests/php/Form/GridField/RichFilterHeaderTest.php b/tests/php/Form/GridField/RichFilterHeaderTest.php
@@ -76,7 +76,7 @@ public function testRenderFilteredHeaderStandard(): void
         $component = $config->getComponentByType(RichFilterHeader::class);
         $htmlFragment = $component->getHTMLFragments($gridField);
 
-        $this->assertContains(
+        $this->assertStringContainsString(
             '<input type="text" name="filter[testfield][City]"'
             . ' class="text grid-field__sort-field no-change-track form-group--no-label"'
             . ' id="Form_mockform_filter_testfield_City" placeholder="Filter by City" />',
@@ -99,15 +99,15 @@ public function testRenderFilterHeaderWithCustomFields(): void
 
         $htmlFragment = $component->getHTMLFragments($gridField);
 
-        $this->assertContains(
+        $this->assertStringContainsString(
             '<select name="filter[testfield][Name]" '
             . 'class="dropdown grid-field__sort-field no-change-track form-group--no-label"'
             . ' id="Form_mockform_filter_testfield_Name" placeholder="Filter by Name">',
             $htmlFragment['header'],
             'We expect a rendered Name filter'
         );
 
-        $this->assertContains(
+        $this->assertStringContainsString(
             '<select name="filter[testfield][City]" '
             . 'class="dropdown grid-field__sort-field no-change-track form-group--no-label"'
             . ' id="Form_mockform_filter_testfield_City" placeholder="Filter by City">',
@@ -141,15 +141,15 @@ public function testRenderFilterHeaderWithFullConfig(): void
 
         $htmlFragment = $component->getHTMLFragments($gridField);
 
-        $this->assertContains(
+        $this->assertStringContainsString(
             '<select name="filter[testfield][Name]" '
             . 'class="dropdown grid-field__sort-field no-change-track form-group--no-label"'
             . ' id="Form_mockform_filter_testfield_Name" placeholder="Filter by Name">',
             $htmlFragment['header'],
             'We expect a rendered Name filter'
         );
 
-        $this->assertContains(
+        $this->assertStringContainsString(
             '<select name="filter[testfield][City]" '
             . 'class="dropdown grid-field__sort-field no-change-track form-group--no-label"'
             . ' id="Form_mockform_filter_testfield_City" placeholder="Filter by City">',

diff --git a/tests/php/Form/GridField/RichFilterHeaderTest/Cheerleader.php b/tests/php/Form/GridField/RichFilterHeaderTest/Cheerleader.php
@@ -4,10 +4,12 @@
 
 use SilverStripe\Dev\TestOnly;
 use SilverStripe\ORM\DataObject;
+use SilverStripe\ORM\HasManyList;
 
 /**
- * Class Cheerleader
- * @package Terraformers\RichFilterHeader\Tests\Form\GridField\RichFilterHeaderTest
+ * @property string $Name
+ * @method Team Team()
+ * @method HasManyList|CheerleaderHat[] Hats()
  */
 class Cheerleader extends DataObject implements TestOnly
 {

diff --git a/tests/php/Form/GridField/RichFilterHeaderTest/CheerleaderHat.php b/tests/php/Form/GridField/RichFilterHeaderTest/CheerleaderHat.php
@@ -4,10 +4,11 @@
 
 use SilverStripe\Dev\TestOnly;
 use SilverStripe\ORM\DataObject;
+use SilverStripe\ORM\ManyManyList;
 
 /**
- * Class CheerleaderHat
- * @package Terraformers\RichFilterHeader\Tests\Form\GridField\RichFilterHeaderTest
+ * @property string $Colour
+ * @method ManyManyList|Cheerleader[] Cheerleaders()
  */
 class CheerleaderHat extends DataObject implements TestOnly
 {

diff --git a/tests/php/Form/GridField/RichFilterHeaderTest/Team.php b/tests/php/Form/GridField/RichFilterHeaderTest/Team.php
@@ -6,8 +6,9 @@
 use SilverStripe\ORM\DataObject;
 
 /**
- * Class Team
- * @package Terraformers\RichFilterHeader\Tests\Form\GridField\RichFilterHeaderTest
+ * @property string $Name
+ * @property string $City
+ * @method Cheerleader Cheerleader()
  */
 class Team extends DataObject implements TestOnly
 {