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
Unknown Object (File)
Sun, Dec 15, 6:15 AM
Unknown Object (File)
Thu, Nov 28, 9:25 PM
Unknown Object (File)
Nov 1 2024, 10:05 PM
Unknown Object (File)
Oct 25 2024, 3:06 AM
Unknown Object (File)
Oct 1 2024, 3:50 AM
Unknown Object (File)
Sep 17 2024, 5:25 AM
Unknown Object (File)
Sep 9 2024, 6:05 PM
Unknown Object (File)
Sep 5 2024, 11:30 PM

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 4073
Build 4086: [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