[DeadCode] Skip fluent return $this on RemoveEmptyMethodCallRector

[samsonasik]

Given the following code:

class SkipFluentReturnThis
{
    /**
     * @return $this
     */
    public function prepare()
    {
        return $this;
    }

    public function run()
    {
    }

   public function call(string $class)
   {
        /** @var SkipFluentReturnThis $obj */
        $obj = new $class();
        $obj->prepare()->run();
   }
}

class Extended extends SkipFluentReturnThis
{
    public function run()
    {
        echo 'hello';
    }
}

(new SkipFluentReturnThis())->call(Extended::class);

It currently got error:

1) Rector\Tests\DeadCode\Rector\MethodCall\RemoveEmptyMethodCallRector\RemoveEmptyMethodCallRectorTest::test with data set #10 (Symplify\SmartFileSystem\SmartFileInfo Object (...))

Rector\Core\Exception\ShouldNotHappenException: Chain method calls cannot be removed this way. It would remove the whole tree of calls. Remove them manually by creating new parent node with no following method.

It should be skipped. Ref https://3v4l.org/P5pK2

[samsonasik]

Fixed 🎉

[samsonasik]

All checks have passed 🎉 @TomasVotruba it is ready for review.