Page MenuHomePhabricator

Add configuration option for PHPUnit test dirs
Needs RevisionPublic

Authored by sascha-egerer on Jan 27 2015, 7:35 AM.
Tags
None
Referenced Files
F11039328: D11515.diff
Tue, Aug 16, 3:19 AM
Unknown Object (File)
Wed, Aug 10, 7:38 PM
Unknown Object (File)
Thu, Aug 4, 11:06 AM
Unknown Object (File)
Tue, Aug 2, 11:07 PM
Unknown Object (File)
Mon, Jul 25, 10:31 PM
Unknown Object (File)
Jul 14 2022, 1:47 AM
Unknown Object (File)
Jul 9 2022, 3:23 PM
Unknown Object (File)
Jun 25 2022, 8:29 AM

Details

Summary

The PHPUnit testengine expects that the unit tests are located
in a folder tests or Tests. In some setups this does not fit because tests
could also be located on different places. For example they could be stored
in a folter "tests/Unit/". Therefore there should be a configuration option
for the PHPUnit tests folders instead of setting them to a fixed value.

Test Plan

Run arc unit --coverage in a setup where the unit test for the class
src/foo.php is stored in the folder tests/Unit/foo.php. There should be no
coverage report. Now add the configuration option to the .arcconfig

{
  "phpunit_test_dirs" : [ "tests/Unit" ]
}

and run arc unit --coverage again.

Diff Detail

Repository
rARC Arcanist
Branch
configurationOptionForPhpunitTestDirs
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 4075
Build 4088: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

sascha-egerer retitled this revision from to Add configuration option for PHPUnit test dirs.
sascha-egerer updated this object.
sascha-egerer edited the test plan for this revision. (Show Details)
sascha-egerer edited edge metadata.

Removed some whitespace changes

That does also fix the problem that unit tests are not executed when doing an arc diff if they are not stored directly in the tests folder.

epriestley added a reviewer: epriestley.

This should wait for T5568 and rebuild on top of that.

Particularly, I do not want to add global configuration for a specific test engine.

This revision now requires changes to proceed.May 19 2015, 2:46 PM