Use new config data when Infection has been executed in a first time …

…(without .json or json.dist)

src/Command/BaseCommand.php

@@ -74,8 +74,6 @@ public function getContainer(): Container
 
     protected function initialize(InputInterface $input, OutputInterface $output): void
     {
-        parent::initialize($input, $output);
-
         $this->input = $input;
         $this->output = $output;
     }

src/Command/InfectionCommand.php

@@ -352,10 +352,9 @@ private function runConfigurationCommand(Locator $locator): void
             $locator->locateOneOf(InfectionConfig::POSSIBLE_CONFIG_FILE_NAMES);
         } catch (\Exception $e) {
             $configureCommand = $this->getApplication()->find('configure');
-            $config = $this->getContainer()->get('infection.config');
 
             $args = [
-                '--test-framework' => $this->input->getOption('test-framework') ?: $config->getTestFramework(),
+                '--test-framework' => $this->input->getOption('test-framework') ?: TestFrameworkTypes::PHPUNIT,
             ];
 
             $newInput = new ArrayInput($args);

src/Config/InfectionConfig.php

@@ -35,6 +35,7 @@
 
 namespace Infection\Config;
 
+use Infection\TestFramework\TestFrameworkTypes;
 use Symfony\Component\Filesystem\Filesystem;
 
 /**
@@ -152,7 +153,7 @@ public function getBootstrap(): string
 
     public function getTestFramework(): string
     {
-        return $this->config->testFramework ?? 'phpunit';
+        return $this->config->testFramework ?? TestFrameworkTypes::PHPUNIT;
     }
 
     public function getInitialTestsPhpOptions(): string

tests/Fixtures/e2e/Initial_Configuration/composer.json

@@ -0,0 +1,15 @@
+{
+    "require-dev": {
+        "phpunit/phpunit": "^6.5"
+    },
+    "autoload": {
+        "psr-4": {
+            "Initial_Configuration\\": "src/"
+        }
+    },
+    "autoload-dev": {
+        "psr-4": {
+            "Initial_Configuration\\Test\\": "tests/"
+        }
+    }
+}

tests/Fixtures/e2e/Initial_Configuration/do_configure.expect

@@ -0,0 +1,22 @@
+#!/usr/bin/env expect
+set timeout 20
+eval spawn $env(INFECTION)
+
+proc configure { input value } {
+    expect $input {
+        send $value
+    } timeout {
+        send_user "Test failed\n"
+        exit 1
+    }
+}
+
+configure "directories do you want to include" "2\r"
+configure "Any directories to exclude from" "\r"
+configure "timeout in seconds" "\r"
+configure "text log file?" "\r"
+
+expect "Please note that some mutants will inevitably be harmless"
+
+send_user "Test succeeded!\n"
+exit 0

tests/Fixtures/e2e/Initial_Configuration/expected.log

@@ -0,0 +1,8 @@
+Escaped mutants:
+================
+
+Timed Out mutants:
+==================
+
+Not Covered mutants:
+====================

tests/Fixtures/e2e/Initial_Configuration/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>
+
+    <filter>
+        <whitelist>
+            <directory>./src/</directory>
+        </whitelist>
+    </filter>
+</phpunit>

tests/Fixtures/e2e/Initial_Configuration/run_tests.bash

@@ -0,0 +1,24 @@
+#!/usr/bin/env bash
+
+if test ! -f "$(which expect)"; then
+    test -x $(which tput) && tput setaf 1 # red
+    echo "Please install expect; it is readily available from apt and brew"
+    exit 1;
+fi
+
+cd $(dirname $0)
+rm -f infection.json.dist infection.log
+
+set -e
+
+if [[ "$PHPDBG" = "1" ]]
+then
+    INFECTION="phpdbg -qrr ../../../../bin/infection"
+else
+    INFECTION="php ../../../../bin/infection"
+fi
+export INFECTION
+
+./do_configure.expect
+
+diff -u expected.log infection.log

tests/Fixtures/e2e/Initial_Configuration/skipped_src/IgnoredSourceClass.php

@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Infection\Tests\Fixtures\e2e\Initial_Configuration\skipped_src;
+
+class IgnoredSourceClass
+{
+    public function doSmth()
+    {
+        return 10;
+    }
+}

tests/Fixtures/e2e/Initial_Configuration/src/SourceClass.php

@@ -0,0 +1,11 @@
+<?php
+
+namespace Initial_Configuration;
+
+class SourceClass
+{
+    public function hello(): string
+    {
+        return 'hello';
+    }
+}

tests/Fixtures/e2e/Initial_Configuration/tests/SourceClassTest.php

@@ -0,0 +1,15 @@
+<?php
+
+namespace Initial_Configuration\Test;
+
+use Initial_Configuration\SourceClass;
+use PHPUnit\Framework\TestCase;
+
+class SourceClassTest extends TestCase
+{
+    public function test_hello()
+    {
+        $sourceClass = new SourceClass();
+        $this->assertSame('hello', $sourceClass->hello());
+    }
+}