diff --git a/tests/php/ContentReviewBaseTest.php b/tests/php/ContentReviewBaseTest.php deleted file mode 100644 index 0ac0015..0000000 --- a/tests/php/ContentReviewBaseTest.php +++ /dev/null @@ -1,18 +0,0 @@ -getMockBuilder(ContentReviewCMSExtension::class) - ->setMethods(['getReviewContentForm']) + ->onlyMethods(['getReviewContentForm']) ->getMock(); $mock->expects($this->once())->method('getReviewContentForm')->with(123)->willReturn(true); @@ -45,7 +45,7 @@ public function testGetReviewContentFormThrowsExceptionWhenObjectCannotBeReviewe $this->logOut(); $mock = $this->getMockBuilder(ContentReviewCMSExtension::class) - ->setMethods(['findRecord']) + ->onlyMethods(['findRecord']) ->getMock(); $mock->setOwner(new Controller); @@ -62,7 +62,7 @@ public function testGetReviewContentFormThrowsExceptionWhenObjectCannotBeReviewe public function testSaveReviewCallsHandler() { $mock = $this->getMockBuilder(ContentReviewCMSExtension::class) - ->setMethods(['findRecord', 'getReviewContentHandler']) + ->onlyMethods(['findRecord', 'getReviewContentHandler']) ->getMock(); $mock->setOwner(new Controller); @@ -71,7 +71,7 @@ public function testSaveReviewCallsHandler() $mock->expects($this->once())->method('findRecord')->willReturn($mockPage); $mockHandler = $this->getMockBuilder(ReviewContentHandler::class) - ->setMethods(['submitReview']) + ->onlyMethods(['submitReview']) ->getMock(); $mockHandler->expects($this->once()) diff --git a/tests/php/Forms/ReviewContentHandlerTest.php b/tests/php/Forms/ReviewContentHandlerTest.php index a544562..76e4904 100644 --- a/tests/php/Forms/ReviewContentHandlerTest.php +++ b/tests/php/Forms/ReviewContentHandlerTest.php @@ -63,7 +63,7 @@ public function testAddReviewNoteCalledWhenSubmittingReview() $mock = $this->getMockBuilder(ReviewContentHandler::class) ->setConstructorArgs([$controller]) - ->setMethods(['canSubmitReview']) + ->onlyMethods(['canSubmitReview']) ->getMock(); $mock->expects($this->exactly(3))->method('canSubmitReview')->willReturn(true); diff --git a/tests/php/SiteTreeContentReviewTest.php b/tests/php/SiteTreeContentReviewTest.php index 3b1eb20..57ab2ee 100644 --- a/tests/php/SiteTreeContentReviewTest.php +++ b/tests/php/SiteTreeContentReviewTest.php @@ -18,8 +18,9 @@ use SilverStripe\SiteConfig\SiteConfig; use SilverStripe\Versioned\Versioned; use SilverStripe\ORM\ArrayList; +use SilverStripe\Dev\FunctionalTest; -class SiteTreeContentReviewTest extends ContentReviewBaseTest +class SiteTreeContentReviewTest extends FunctionalTest { protected $usesTransactions = false; @@ -374,7 +375,7 @@ public function testPermissionCheckByOnDataObject() // Mock Page class with canReviewContent method to return true on first call and false on second call $mock = $this->getMockBuilder(Page::class) - ->setMethods(['canReviewContent', 'NextReviewDate', 'OwnerUsers']) + ->onlyMethods(['canReviewContent', 'NextReviewDate', 'OwnerUsers']) ->getMock(); $mock->expects($this->exactly(2))->method('canReviewContent')->willReturnOnConsecutiveCalls(false, true); $mock->method('NextReviewDate')->willReturn('2020-02-20 12:00:00');