Incorrect behavior of RemoveUnusedPrivateMethodRector, RemoveAlwaysElseRector

[Bl00D4NGEL]

Bug Report

Subject	Details
Rector version	last dev-master
Installed as	composer dependency

Minimal PHP Code Causing Issue

See https://getrector.org/demo/1ec0a727-cd6b-633e-9617-33d1d4395dd3

<?php

declare(strict_types=1);

namespace App;

final class Client
{
    public function doRequest(): int
    {
        if (401 === 200) {
            return -1;
        } else {
            return $this->notUnused();
        }
    }

    private function notUnused(): int
    {
        // This is some code that is very important
    }
}

Responsible rules

• RemoveUnusedPrivateMethodRector

• RemoveAlwaysElseRector

Expected Behavior

<?php

declare(strict_types=1);

namespace App;

final class Client
{
    public function doRequest(): int
    {
        if (401 === 200) {
            return -1;
        }

        return $this->notUnused();
    }

    private function notUnused(): int
    {
        // This is some code that is very important
    }
}
?>

Test fixture will be supplied in a following MR in a bit :)