MNT Fix unit tests

silverstripe · Oct 21, 2024 · 63770ec · 63770ec
1 parent 89213c9
commit 63770ec
Show file tree

Hide file tree

Showing 2 changed files with 32 additions and 9 deletions.
diff --git a/tests/php/Dev/DeprecationTest.php b/tests/php/Dev/DeprecationTest.php
@@ -111,7 +111,7 @@ public function testNoticeNoReplacement()
             'Will be removed without equivalent functionality to replace it.',
             'Called from SilverStripe\Dev\Tests\DeprecationTest->testNoticeNoReplacement.'
         ]);
-        $this->expectDeprecation();
+        $this->expectException(DeprecationTestException::class);
         $this->expectDeprecationMessage($message);
         $this->enableDeprecationNotices(true);
         $ret = $this->myDeprecatedMethodNoReplacement();

diff --git a/tests/php/ORM/Search/SearchContextTest.php b/tests/php/ORM/Search/SearchContextTest.php
@@ -135,19 +135,42 @@ public function testUserDefinedFiltersAppearInSearchContext()
         );
     }
 
+    public static $debug = false;
+
     public function testUserDefinedFieldsAppearInSearchContext()
     {
         $company = SearchContextTest\Company::singleton();
+        self::$debug = true;
+        $searchName = $company->getGeneralSearchFieldName();
+        $expected = new FieldList(
+            new HiddenField($searchName, 'General Search'),
+            (new TextField("Name", 'Name'))
+                ->setMaxLength(255),
+            new TextareaField("Industry", 'Industry'),
+            new NumericField("AnnualProfit", 'The Almighty Annual Profit')
+        );
         $context = $company->getDefaultSearchContext();
+        $actual = $context->getFields();
+        $this->assertSame($expected->count(), $actual->count());
+        $this->assertEquals(
+            $expected->fieldByName($searchName)->Title,
+            $actual->fieldByName($searchName)->Title
+        );
+        $this->assertEquals(
+            $expected->fieldByName('Name')->Title,
+            $actual->fieldByName('Name')->Title
+        );
+        $this->assertEquals(
+            $expected->fieldByName('Name')->getMaxLength(),
+            $actual->fieldByName('Name')->getMaxLength()
+        );
+        $this->assertEquals(
+            $expected->fieldByName('Industry')->Title,
+            $actual->fieldByName('Industry')->Title
+        );
         $this->assertEquals(
-            new FieldList(
-                new HiddenField($company->getGeneralSearchFieldName(), 'General Search'),
-                (new TextField("Name", 'Name'))
-                    ->setMaxLength(255),
-                new TextareaField("Industry", 'Industry'),
-                new NumericField("AnnualProfit", 'The Almighty Annual Profit')
-            ),
-            $context->getFields()
+            $expected->fieldByName('AnnualProfit')->Title,
+            $actual->fieldByName('AnnualProfit')->Title
         );
     }