Page MenuHomePhabricator

Add support for xUnit.NET 2 and improve build / test performance
AbandonedPublic

Authored by hach-que on Feb 5 2015, 6:06 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 27, 9:43 PM
Unknown Object (File)
Thu, Mar 21, 1:01 PM
Unknown Object (File)
Fri, Mar 15, 9:37 PM
Unknown Object (File)
Feb 14 2024, 11:42 AM
Unknown Object (File)
Feb 2 2024, 4:43 PM
Unknown Object (File)
Jan 9 2024, 11:35 AM
Unknown Object (File)
Jan 9 2024, 3:36 AM
Unknown Object (File)
Dec 28 2023, 1:17 AM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Summary

This adds support for xUnit.NET 2 and improves the performance of building and testing. Building is improved by getting MSBuild to build everything in parallel, rather than invoking MSBuild for each project, and testing is improved because xUnit.NET 2 supports testing multiple assemblies in parallel. This means that the Arcanist unit test runner only needs to exec two processes for the whole operation, instead of two processes per assembly.

This also depends on D11688 landing in libphutil first.

Test Plan

Ran it against a code base using xUnit.NET 2.

Diff Detail

Repository
rARC Arcanist
Branch
xunit2
Lint
Lint Skipped
Unit
Test Failures
Build Status
Buildable 4346
Build 4359: [Placeholder Plan] Wait for 30 Seconds

Unit TestsFailed

TimeTest
198 msArcanistLibraryTestCase::testLibraryMap
29 msPhpunitTestEngineTestCase::testSearchLocations
41 msArcanistLibraryTestCase::testEverythingImplemented
50 msArcanistLibraryTestCase::testMethodVisibility
0 msPhutilUnitTestEngineTestCase::testFailSkip
View Full Test Results (2 Failed · 6 Passed)

Event Timeline

hach-que retitled this revision from to Add support for xUnit.NET 2 and improve build / test performance.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: epriestley.
hach-que edited edge metadata.
  • Remove "Discovered" test notifications
epriestley edited edge metadata.

New test engines should wait for T5568.

This revision now requires changes to proceed.Aug 8 2015, 6:40 PM