git-diff on tests to trigger corresponding covered source files?

[icarus94]

git-diff on tests to trigger corresponding covered source files?

[maks-rafalko]

Hello,

unfortunately, there is no way to achieve this with the current implementation. Look at this code:

infection/src/Logger/GitHub/GitDiffFileProvider.php

Lines 69 to 77 in 3e0387b

	'git',
	'diff',
	$referenceCommit,
	'--diff-filter',
	$gitDiffFilter,
	'--name-only',
	'--',
	],
	$sourceDirectories

On line 77, we pass an array of source directories from infection.json5, in which we find changes with git diff. If we want to also check a change in a test folder, then we need to pass it as well.

However, we can't just add tests folder to infection.json5 under source.directories, as it will break Infection - it will start mutating tests...

I see no other way than creating some new option to pass tests there.

---

UPD: even if we pass tests on the line 77 above, it won't work, because Infection mutates only files from source.directories config, so tests files will be skipped.

In your case, there should be some logic that understand that tests/Unit/Service/FooTest.php corresponds to the source file src/Service/Foo.php, so the mentioned method should return src/Service/Foo.php, not tests/Unit/Service/FooTest.php.

Thus, it looks like a new feature with the logic of determining a source file by looking at the test file. This is definitely not supported.

You can however achieve this by yourself (outside of Infection).

git diff .... # get all files from the diff

# check if tests files are changed

# product a list of source files from the list of tests files

# pass them to `--filter` option to Infection

infection --filter=src/Service/Foo.php

bash examples can be found here https://infection.github.io/guide/how-to.html#By-using-filter-option-for-the-old-Infection-versions