GetMockBuilderGetMockToCreateMockRector breaks test code

[kenjis]

1) tests/system/Email/EmailTest.php:138

    ---------- begin diff ----------
@@ @@

     public function testDestructDoesNotThrowException()
     {
-        $email = $this->getMockBuilder(Email::class)
-            ->disableOriginalConstructor()
-            ->onlyMethods(['sendCommand'])
-            ->getMock();
+        $email = $this->createMock(Email::class);
         $email->method('sendCommand')
             ->willThrowException(new ErrorException('SMTP Error.'));
    ----------- end diff -----------

Applied rules:
 * GetMockBuilderGetMockToCreateMockRector (https://github.com/lmc-eu/steward/pull/187/files#diff-c7e8c65e59b8b4ff8b54325814d4ba55L80)

But when I run the test, I got the error:

$ vendor/bin/phpunit tests/system/Email/EmailTest.php
PHPUnit 9.5.21 #StandWithUkraine

Runtime:       PHP 8.0.20
Configuration: .../CodeIgniter4/phpunit.xml

........E                                                           9 / 9 (100%)E                                                           9 / 9 (100%)

Time: 00:00.358, Memory: 14.00 MB

There was 1 error:

1) CodeIgniter\Email\EmailTest::testDestructDoesNotThrowException
PHPUnit\Framework\MockObject\MethodNameNotConfiguredException: Method name is not configured

ERRORS!
Tests: 9, Assertions: 20, Errors: 1.

[samsonasik]

it seems there is test that remove onlyMethods for it:

rector-phpunit/tests/Rector/MethodCall/GetMockBuilderGetMockToCreateMockRector/Fixture/only_methods.php.inc

Line 11 in 3f4cc81

->onlyMethods(['hey'])

and it registered at:

rector-phpunit/src/Rector/MethodCall/GetMockBuilderGetMockToCreateMockRector.php

Line 25 in 3f4cc81

'onlyMethods',

@kenjis is the error happen when ->method('sendCommand') is called?

[kenjis]

Yes, I think so.

[TomasVotruba]

Let's continue discussion in one place to keep it simple: #87 (comment)

[SantosSi]

Find what could explain the misbehavior in my comment in #87.

[samsonasik]

Resolved at #205