Merge pull request #485 from creative-commoners/pulls/5.0/broken-builds

MNT Fix unit test
silverstripe · Feb 15, 2023 · 14679ee · 14679ee
2 parents ecfda3b + 31c9075
commit 14679ee
Showing 1 changed file with 2 additions and 15 deletions.
diff --git a/tests/php/Report/EnabledMembersFunctionalTest.php b/tests/php/Report/EnabledMembersFunctionalTest.php
@@ -30,33 +30,20 @@ public function testReportHasMemberNames()
 
     public function testReportHasRegisteredMethods()
     {
-        $this->markTestSkipped('Temperamental test - consider revising');
-
         $result = (string) $this->get(EnabledMembers::create()->getLink())->getBody();
-
         $this->assertStringContainsString('Math problem, Null', $result);
     }
 
     public function testFilterReportByMemberName()
     {
-        $this->get(EnabledMembers::create()->getLink());
-        $response = $this->submitForm('Form_EditForm', 'action_updatereport', [
-            'filters[Member]' => 'Michelle',
-        ]);
-        $result = (string) $response->getBody();
-
+        $result = $this->get('/admin/reports/show/SilverStripe-MFA-Report-EnabledMembers?filters%5BMember%5D=Michelle');
         $this->assertStringContainsString('[email protected]', $result);
         $this->assertStringNotContainsString('[email protected]', $result);
     }
 
     public function testFilterReportBySkippedRegistration()
     {
-        $this->get(EnabledMembers::create()->getLink());
-        $response = $this->submitForm('Form_EditForm', 'action_updatereport', [
-            'filters[Skipped]' => 'yes',
-        ]);
-        $result = (string) $response->getBody();
-
+        $result = $this->get('/admin/reports/show/SilverStripe-MFA-Report-EnabledMembers?filters%5BSkipped%5D=yes');
         $this->assertStringContainsString('[email protected]', $result);
         $this->assertStringNotContainsString('[email protected]', $result);
     }