Make sure dg/bypass-finals works with Infection (#1728)

* Add tests to make sure `bypass-finals` lib does not lead to escaped mutants

* Test dg/bypass-finals@7c08b98

* Tests different approaches

* Use the latest `master` of `db/bypass-finals`

* Update `db/bypass-finals` to the latest commit to check if the issue is fixed

* Update `conflict` section with new version of `dg/bypass-finals`

* Run `composer update --lock`

* Update `expected-output.txt` to fix e2e test

* Fix incorrect versions

* Run `composer update --lock`

* For some unknown reason, MSI has decreased with no changes in `src`

.github/workflows/mt.yaml

@@ -12,7 +12,7 @@ on:
 
 env:
   MIN_MSI: 71.35
-  MIN_COVERED_MSI: 86.54
+  MIN_COVERED_MSI: 86.50
 
 jobs:
   tests:

composer.json

@@ -68,7 +68,7 @@
     },
     "conflict": {
         "phpunit/php-code-coverage": ">9 <9.1.4",
-        "dg/bypass-finals": "*"
+        "dg/bypass-finals": "<1.4.1"
     },
     "require-dev": {
         "ext-simplexml": "*",

tests/e2e/Stream_Wrapper_Execution/README.md

@@ -0,0 +1,10 @@
+# Ensures Infection works with `dg/bypass-finals`
+
+* https://github.com/dg/bypass-finals/issues/9
+* https://github.com/infection/infection/issues/1275
+
+## Summary
+
+`dg/bypass-finals` before version 1.4.1 overridden Infection's Stream Wrapper and Infection did not create any Mutants.
+
+This tests ensures that starting from 1.4.1 `dg/bypass-finals` works good and do not "disables" Infection's Stream Wrapper.

tests/e2e/Stream_Wrapper_Execution/composer.json

@@ -0,0 +1,16 @@
+{
+    "require-dev": {
+        "phpunit/phpunit": "^9.5.5",
+        "dg/bypass-finals": "^1.4.1"
+    },
+    "autoload": {
+        "psr-4": {
+            "Stream_Wrapper_Execution\\": "src/"
+        }
+    },
+    "autoload-dev": {
+        "psr-4": {
+            "Stream_Wrapper_Execution\\Test\\": "tests/"
+        }
+    }
+}

tests/e2e/Stream_Wrapper_Execution/expected-output.txt

@@ -0,0 +1,10 @@
+Total: 5
+
+Killed: 5
+Errored: 0
+Syntax Errors: 0
+Escaped: 0
+Timed Out: 0
+Skipped: 0
+Ignored: 0
+Not Covered: 0

tests/e2e/Stream_Wrapper_Execution/infection.json

@@ -0,0 +1,15 @@
+{
+    "timeout": 5,
+    "source": {
+        "directories": [
+            "src"
+        ]
+    },
+    "logs": {
+        "summary": "infection.log"
+    },
+    "tmpDir": ".",
+    "mutators": {
+        "@default": true
+    }
+}

tests/e2e/Stream_Wrapper_Execution/phpunit.xml

@@ -0,0 +1,18 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="./vendor/phpunit/phpunit/phpunit.xsd"
+         bootstrap="./vendor/autoload.php"
+         colors="true"
+>
+    <testsuites>
+        <testsuite name="Test Suite">
+            <directory>./tests/</directory>
+        </testsuite>
+    </testsuites>
+
+    <coverage>
+        <include>
+            <directory>./src/</directory>
+        </include>
+    </coverage>
+</phpunit>

tests/e2e/Stream_Wrapper_Execution/src/FinalClass.php

@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Stream_Wrapper_Execution;
+
+final class FinalClass
+{
+    public function get(): int
+    {
+        return 1;
+    }
+}

tests/e2e/Stream_Wrapper_Execution/src/SourceClass.php

@@ -0,0 +1,18 @@
+<?php
+
+namespace Stream_Wrapper_Execution;
+
+class SourceClass
+{
+    private $finalClass;
+
+    public function __construct(FinalClass $finalClass)
+    {
+        $this->finalClass = $finalClass;
+    }
+
+    public function getOne(): int
+    {
+        return $this->finalClass->get();
+    }
+}

tests/e2e/Stream_Wrapper_Execution/tests/SourceClassTest.php

@@ -0,0 +1,37 @@
+<?php
+
+namespace Stream_Wrapper_Execution\Test;
+
+use Stream_Wrapper_Execution\FinalClass;
+use Stream_Wrapper_Execution\SourceClass;
+use PHPUnit\Framework\TestCase;
+use DG\BypassFinals;
+
+class SourceClassTest extends TestCase
+{
+    public static function setUpBeforeClass(): void
+    {
+        parent::setUpBeforeClass();
+
+        BypassFinals::enable();
+    }
+
+
+    public function test_returns_one()
+    {
+        $sourceClass = new SourceClass(new FinalClass());
+
+        $this->assertSame(1, $sourceClass->getOne());
+    }
+
+    public function test_final_class_can_be_mocked()
+    {
+        $mock = $this->createMock(FinalClass::class);
+
+        $mock->expects(self::once())->method('get')->willReturn(1);
+
+        $sourceClass = new SourceClass($mock);
+
+        self::assertSame(1, $sourceClass->getOne());
+    }
+}