Replace deprecated ->setMethods by ->onlyMethods

See: sebastianbergmann/phpunit#3687 Signed-off-by: William Desportes <williamdes@wdes.fr>
williamdes · Mar 4, 2021 · 76803f9 · 76803f9
1 parent 09736fc
commit 76803f9
Show file tree

Hide file tree

Showing 18 changed files with 55 additions and 55 deletions.
diff --git a/test/classes/AbstractNetworkTestCase.php b/test/classes/AbstractNetworkTestCase.php
@@ -42,7 +42,7 @@ public function mockResponse(...$param): MockObject
     {
         $mockResponse = $this->getMockBuilder(Response::class)
             ->disableOriginalConstructor()
-            ->setMethods([
+            ->onlyMethods([
                 'header',
                 'headersSent',
                 'disable',

diff --git a/test/classes/Config/FormDisplayTest.php b/test/classes/Config/FormDisplayTest.php
@@ -107,7 +107,7 @@ public function testProcess(): void
 
         $this->object = $this->getMockBuilder(FormDisplay::class)
             ->disableOriginalConstructor()
-            ->setMethods(['save'])
+            ->onlyMethods(['save'])
             ->getMock();
 
         $attrForms = new ReflectionProperty(FormDisplay::class, 'forms');

diff --git a/test/classes/Config/FormTest.php b/test/classes/Config/FormTest.php
@@ -268,7 +268,7 @@ public function testLoadForm(): void
     {
         $this->object = $this->getMockBuilder(Form::class)
             ->disableOriginalConstructor()
-            ->setMethods(['readFormPaths', 'readTypes'])
+            ->onlyMethods(['readFormPaths', 'readTypes'])
             ->getMock();
 
         $this->object->expects($this->exactly(1))
@@ -293,7 +293,7 @@ public function testCleanGroupPaths(): void
     {
         $this->object = $this->getMockBuilder(Form::class)
             ->disableOriginalConstructor()
-            ->setMethods(['readFormPaths', 'readTypes'])
+            ->onlyMethods(['readFormPaths', 'readTypes'])
             ->getMock();
 
         $this->object->expects($this->exactly(1))->method('readFormPaths')->with([

diff --git a/test/classes/Config/ServerConfigChecksTest.php b/test/classes/Config/ServerConfigChecksTest.php
@@ -62,7 +62,7 @@ public function testManyErrors(): void
         $_SESSION[$this->sessionID]['ZipDump'] = true;
 
         $configChecker = $this->getMockBuilder(ServerConfigChecks::class)
-            ->setMethods(['functionExists'])
+            ->onlyMethods(['functionExists'])
             ->setConstructorArgs([$GLOBALS['ConfigFile']])
             ->getMock();
 

diff --git a/test/classes/ErrorReportTest.php b/test/classes/ErrorReportTest.php
@@ -117,7 +117,7 @@ public function testSend(): void
         $return = 'return';
 
         $httpRequest = $this->getMockBuilder(HttpRequest::class)
-            ->setMethods(['create'])
+            ->onlyMethods(['create'])
             ->getMock();
         $httpRequest->expects($this->once())
             ->method('create')

diff --git a/test/classes/InsertEditTest.php b/test/classes/InsertEditTest.php
@@ -271,7 +271,7 @@ public function testShowEmptyResultMessageOrSetUniqueCondition(): void
 
         $responseMock = $this->getMockBuilder(Response::class)
             ->disableOriginalConstructor()
-            ->setMethods(['addHtml'])
+            ->onlyMethods(['addHtml'])
             ->getMock();
 
         $restoreInstance = Response::getInstance();
@@ -2773,15 +2773,15 @@ public function testIsInsertRow(): void
 
         $scriptsMock = $this->getMockBuilder(Scripts::class)
             ->disableOriginalConstructor()
-            ->setMethods(['addFile'])
+            ->onlyMethods(['addFile'])
             ->getMock();
 
         $scriptsMock->expects($this->exactly(2))
             ->method('addFile');
 
         $headerMock = $this->getMockBuilder(Header::class)
             ->disableOriginalConstructor()
-            ->setMethods(['getScripts'])
+            ->onlyMethods(['getScripts'])
             ->getMock();
 
         $headerMock->expects($this->once())
@@ -2790,7 +2790,7 @@ public function testIsInsertRow(): void
 
         $responseMock = $this->getMockBuilder(Response::class)
             ->disableOriginalConstructor()
-            ->setMethods(['getHeader'])
+            ->onlyMethods(['getHeader'])
             ->getMock();
 
         $responseMock->expects($this->once())
@@ -3886,7 +3886,7 @@ public function testDetermineInsertOrEdit(): void
 
         $responseMock = $this->getMockBuilder(Response::class)
             ->disableOriginalConstructor()
-            ->setMethods(['addHtml'])
+            ->onlyMethods(['addHtml'])
             ->getMock();
 
         $restoreInstance = Response::getInstance();

diff --git a/test/classes/Plugins/Auth/AuthenticationCookieTest.php b/test/classes/Plugins/Auth/AuthenticationCookieTest.php
@@ -99,7 +99,7 @@ private function getAuthErrorMockResponse(): void
         // mock footer
         $mockFooter = $this->getMockBuilder(Footer::class)
             ->disableOriginalConstructor()
-            ->setMethods(['setMinimal'])
+            ->onlyMethods(['setMinimal'])
             ->getMock();
 
         $mockFooter->expects($this->once())
@@ -110,7 +110,7 @@ private function getAuthErrorMockResponse(): void
 
         $mockHeader = $this->getMockBuilder(Header::class)
             ->disableOriginalConstructor()
-            ->setMethods(
+            ->onlyMethods(
                 [
                     'setBodyId',
                     'setTitle',
@@ -157,7 +157,7 @@ private function getAuthErrorMockResponse(): void
 
         $mockErrorHandler = $this->getMockBuilder(ErrorHandler::class)
             ->disableOriginalConstructor()
-            ->setMethods(['hasDisplayErrors'])
+            ->onlyMethods(['hasDisplayErrors'])
             ->getMock();
 
         $mockErrorHandler->expects($this->once())
@@ -604,7 +604,7 @@ public function testAuthCheckDecryptUser(): void
         // mock for blowfish function
         $this->object = $this->getMockBuilder(AuthenticationCookie::class)
             ->disableOriginalConstructor()
-            ->setMethods(['cookieDecrypt'])
+            ->onlyMethods(['cookieDecrypt'])
             ->getMock();
 
         $this->object->expects($this->once())
@@ -643,7 +643,7 @@ public function testAuthCheckDecryptPassword(): void
         // mock for blowfish function
         $this->object = $this->getMockBuilder(AuthenticationCookie::class)
             ->disableOriginalConstructor()
-            ->setMethods(['cookieDecrypt'])
+            ->onlyMethods(['cookieDecrypt'])
             ->getMock();
 
         $this->object->expects($this->at(1))
@@ -686,7 +686,7 @@ public function testAuthCheckAuthFails(): void
         // mock for blowfish function
         $this->object = $this->getMockBuilder(AuthenticationCookie::class)
             ->disableOriginalConstructor()
-            ->setMethods(['showFailure', 'cookieDecrypt'])
+            ->onlyMethods(['showFailure', 'cookieDecrypt'])
             ->getMock();
 
         $this->object->expects($this->once())
@@ -780,7 +780,7 @@ public function testAuthFailsNoPass(): void
     {
         $this->object = $this->getMockBuilder(AuthenticationCookie::class)
             ->disableOriginalConstructor()
-            ->setMethods(['showLoginForm'])
+            ->onlyMethods(['showLoginForm'])
             ->getMock();
 
         $GLOBALS['server'] = 2;
@@ -861,7 +861,7 @@ public function testAuthFailsDeny(): void
     {
         $this->object = $this->getMockBuilder(AuthenticationCookie::class)
             ->disableOriginalConstructor()
-            ->setMethods(['showLoginForm'])
+            ->onlyMethods(['showLoginForm'])
             ->getMock();
 
         $GLOBALS['server'] = 2;
@@ -883,7 +883,7 @@ public function testAuthFailsActivity(): void
     {
         $this->object = $this->getMockBuilder(AuthenticationCookie::class)
             ->disableOriginalConstructor()
-            ->setMethods(['showLoginForm'])
+            ->onlyMethods(['showLoginForm'])
             ->getMock();
 
         $GLOBALS['server'] = 2;
@@ -909,7 +909,7 @@ public function testAuthFailsDBI(): void
     {
         $this->object = $this->getMockBuilder(AuthenticationCookie::class)
             ->disableOriginalConstructor()
-            ->setMethods(['showLoginForm'])
+            ->onlyMethods(['showLoginForm'])
             ->getMock();
 
         $GLOBALS['server'] = 2;
@@ -942,7 +942,7 @@ public function testAuthFailsErrno(): void
     {
         $this->object = $this->getMockBuilder(AuthenticationCookie::class)
             ->disableOriginalConstructor()
-            ->setMethods(['showLoginForm'])
+            ->onlyMethods(['showLoginForm'])
             ->getMock();
 
         $dbi = $this->getMockBuilder(DatabaseInterface::class)

diff --git a/test/classes/Plugins/Auth/AuthenticationHttpTest.php b/test/classes/Plugins/Auth/AuthenticationHttpTest.php
@@ -59,7 +59,7 @@ public function doMockResponse($set_minimal, $body_id, $set_title, ...$headers):
         // mock footer
         $mockFooter = $this->getMockBuilder(Footer::class)
             ->disableOriginalConstructor()
-            ->setMethods(['setMinimal'])
+            ->onlyMethods(['setMinimal'])
             ->getMock();
 
         $mockFooter->expects($this->exactly($set_minimal))
@@ -70,7 +70,7 @@ public function doMockResponse($set_minimal, $body_id, $set_title, ...$headers):
 
         $mockHeader = $this->getMockBuilder(Header::class)
             ->disableOriginalConstructor()
-            ->setMethods(
+            ->onlyMethods(
                 [
                     'setBodyId',
                     'setTitle',
@@ -415,7 +415,7 @@ public function testAuthFails(): void
 
         $this->object = $this->getMockBuilder(AuthenticationHttp::class)
             ->disableOriginalConstructor()
-            ->setMethods(['authForm'])
+            ->onlyMethods(['authForm'])
             ->getMock();
 
         $this->object->expects($this->exactly(2))

diff --git a/test/classes/Plugins/Auth/AuthenticationSignonTest.php b/test/classes/Plugins/Auth/AuthenticationSignonTest.php
@@ -238,7 +238,7 @@ public function testAuthFailsForbidden(): void
 
         $this->object = $this->getMockBuilder(AuthenticationSignon::class)
             ->disableOriginalConstructor()
-            ->setMethods(['showLoginForm'])
+            ->onlyMethods(['showLoginForm'])
             ->getMock();
 
         $this->object->expects($this->exactly(1))
@@ -260,7 +260,7 @@ public function testAuthFailsDeny(): void
 
         $this->object = $this->getMockBuilder(AuthenticationSignon::class)
             ->disableOriginalConstructor()
-            ->setMethods(['showLoginForm'])
+            ->onlyMethods(['showLoginForm'])
             ->getMock();
 
         $this->object->expects($this->exactly(1))
@@ -281,7 +281,7 @@ public function testAuthFailsTimeout(): void
 
         $this->object = $this->getMockBuilder(AuthenticationSignon::class)
             ->disableOriginalConstructor()
-            ->setMethods(['showLoginForm'])
+            ->onlyMethods(['showLoginForm'])
             ->getMock();
 
         $this->object->expects($this->exactly(1))
@@ -306,7 +306,7 @@ public function testAuthFailsMySQLError(): void
 
         $this->object = $this->getMockBuilder(AuthenticationSignon::class)
             ->disableOriginalConstructor()
-            ->setMethods(['showLoginForm'])
+            ->onlyMethods(['showLoginForm'])
             ->getMock();
 
         $this->object->expects($this->exactly(1))
@@ -338,7 +338,7 @@ public function testAuthFailsConnect(): void
 
         $this->object = $this->getMockBuilder(AuthenticationSignon::class)
             ->disableOriginalConstructor()
-            ->setMethods(['showLoginForm'])
+            ->onlyMethods(['showLoginForm'])
             ->getMock();
 
         $this->object->expects($this->exactly(1))
@@ -366,7 +366,7 @@ public function testSetCookieParamsDefaults(): void
     {
         $this->object = $this->getMockBuilder(AuthenticationSignon::class)
         ->disableOriginalConstructor()
-        ->setMethods(['setCookieParams'])
+        ->onlyMethods(['setCookieParams'])
         ->getMock();
 
         $this->object->setCookieParams([]);

diff --git a/test/classes/Plugins/Export/ExportHtmlwordTest.php b/test/classes/Plugins/Export/ExportHtmlwordTest.php
@@ -372,7 +372,7 @@ public function testExportData(): void
     public function testGetTableDefStandIn(): void
     {
         $this->object = $this->getMockBuilder(ExportHtmlword::class)
-            ->setMethods(['formatOneColumnDefinition'])
+            ->onlyMethods(['formatOneColumnDefinition'])
             ->getMock();
 
         // case 1
@@ -423,7 +423,7 @@ public function testGetTableDefStandIn(): void
     public function testGetTableDef(): void
     {
         $this->object = $this->getMockBuilder(ExportHtmlword::class)
-            ->setMethods(['formatOneColumnDefinition'])
+            ->onlyMethods(['formatOneColumnDefinition'])
             ->getMock();
 
         $keys = [
@@ -730,7 +730,7 @@ public function testExportStructure(): void
             ->will($this->returnValue(1));
 
         $this->object = $this->getMockBuilder(ExportHtmlword::class)
-            ->setMethods(['getTableDef', 'getTriggers', 'getTableDefStandIn'])
+            ->onlyMethods(['getTableDef', 'getTriggers', 'getTableDefStandIn'])
             ->getMock();
 
         $this->object->expects($this->at(0))

diff --git a/test/classes/Plugins/Export/ExportOdtTest.php b/test/classes/Plugins/Export/ExportOdtTest.php
@@ -602,7 +602,7 @@ public function testGetTableDefStandIn(): void
 
         $this->object = $this->getMockBuilder(ExportOdt::class)
             ->disableOriginalConstructor()
-            ->setMethods(['formatOneColumnDefinition'])
+            ->onlyMethods(['formatOneColumnDefinition'])
             ->getMock();
 
         $this->object->expects($this->at(0))
@@ -635,7 +635,7 @@ public function testGetTableDefStandIn(): void
     public function testGetTableDef(): void
     {
         $this->object = $this->getMockBuilder(ExportOdt::class)
-            ->setMethods(['formatOneColumnDefinition'])
+            ->onlyMethods(['formatOneColumnDefinition'])
             ->getMock();
 
         // case 1
@@ -884,7 +884,7 @@ public function testExportStructure(): void
             ->will($this->returnValue(1));
 
         $this->object = $this->getMockBuilder(ExportOdt::class)
-            ->setMethods(['getTableDef', 'getTriggers', 'getTableDefStandIn'])
+            ->onlyMethods(['getTableDef', 'getTriggers', 'getTableDefStandIn'])
             ->getMock();
 
         $this->object->expects($this->at(0))

diff --git a/test/classes/Plugins/Export/ExportSqlTest.php b/test/classes/Plugins/Export/ExportSqlTest.php
@@ -1279,7 +1279,7 @@ public function testExportStructure(): void
             );
 
         $this->object = $this->getMockBuilder(ExportSql::class)
-            ->setMethods(['getTableDef', 'getTriggers', 'getTableDefStandIn'])
+            ->onlyMethods(['getTableDef', 'getTriggers', 'getTableDefStandIn'])
             ->getMock();
 
         $this->object->expects($this->at(0))

diff --git a/test/classes/Plugins/Export/ExportTexytextTest.php b/test/classes/Plugins/Export/ExportTexytextTest.php
@@ -302,7 +302,7 @@ public function testGetTableDefStandIn(): void
 
         $this->object = $this->getMockBuilder(ExportTexytext::class)
             ->disableOriginalConstructor()
-            ->setMethods(['formatOneColumnDefinition'])
+            ->onlyMethods(['formatOneColumnDefinition'])
             ->getMock();
 
         $this->object->expects($this->at(0))
@@ -326,7 +326,7 @@ public function testGetTableDefStandIn(): void
     public function testGetTableDef(): void
     {
         $this->object = $this->getMockBuilder(ExportTexytext::class)
-            ->setMethods(['formatOneColumnDefinition'])
+            ->onlyMethods(['formatOneColumnDefinition'])
             ->getMock();
 
         // case 1
@@ -469,7 +469,7 @@ public function testExportStructure(): void
             ->will($this->returnValue(1));
 
         $this->object = $this->getMockBuilder(ExportTexytext::class)
-            ->setMethods(['getTableDef', 'getTriggers', 'getTableDefStandIn'])
+            ->onlyMethods(['getTableDef', 'getTriggers', 'getTableDefStandIn'])
             ->getMock();
 
         $this->object->expects($this->at(0))

diff --git a/test/classes/RelationCleanupTest.php b/test/classes/RelationCleanupTest.php
@@ -77,7 +77,7 @@ protected function setUp(): void
 
         $this->relation = $this->getMockBuilder(Relation::class)
             ->disableOriginalConstructor()
-            ->setMethods(['queryAsControlUser'])
+            ->onlyMethods(['queryAsControlUser'])
             ->getMock();
         $this->relationCleanup = new RelationCleanup($GLOBALS['dbi'], $this->relation);
     }