Differential D19465

[ARCUNIT] Set the ConfigurationManager of ConfigurationDrivenUnitTestEngines
ClosedPublic
Actions

Authored by tmcjilton on Jun 5 2018, 8:30 PM.

Tags

None

Referenced Files

	F14076125: D19465.diff
	Thu, Nov 21, 2:52 PM

	Unknown Object (File)
	Tue, Nov 19, 4:27 PM

	Unknown Object (File)
	Sun, Nov 3, 10:15 AM

	Unknown Object (File)
	Oct 15 2024, 10:05 PM

	Unknown Object (File)
	Oct 15 2024, 10:00 PM

	Unknown Object (File)
	Oct 15 2024, 9:59 PM

	Unknown Object (File)
	Oct 15 2024, 9:58 PM

	Unknown Object (File)
	Oct 15 2024, 4:26 PM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Commits

rARCb199ca808660: [ARCUNIT] Set the ConfigurationManager of ConfigurationDrivenUnitTestEngines

Required Signatures

L28 Phacility Individual Contributor License Agreement

Summary

The Configuration Manager is supported by ArcanistUnitTestEngine but not support by the ArcanistConfigurationDrivenUnitTestEngine.

Added the configuration manager as one of the initially set properties of an ArcUnitTestEngine created by the ArcanistConfigurationDrivenTestEngine

Test Plan

Ran arc unit against a project without the change, verified the Configuration was none. Added this change and ran again and verified it was set

Diff Detail

Repository

Branch

master

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 20368
Build 27660: arc lint + arc unit

Event Timeline

tmcjilton created this revision.Jun 5 2018, 8:30 PM

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJun 5 2018, 8:30 PM

Herald added a required legal document: L28 Phacility Individual Contributor License Agreement. · View Herald Transcript

tmcjilton requested review of this revision.Jun 5 2018, 8:30 PM

Herald added a subscriber: Korvin. · View Herald TranscriptJun 5 2018, 8:30 PM

Harbormaster completed remote builds in B20368: Diff 46560.Jun 5 2018, 8:30 PM

Thanks! I added a CLA exemption for you and added you to Community (granting you sweeping janitorial powers) and Blessed Committers (so you should be able to land this yourself). See that project's detail page for instructions, and yell if you run into issues.

This revision is now accepted and ready to land.Jun 5 2018, 8:46 PM

(For future archaeology, see also PHI694.)

Closed by commit rARCb199ca808660: [ARCUNIT] Set the ConfigurationManager of ConfigurationDrivenUnitTestEngines (authored by tmcjilton). · Explain WhyJun 5 2018, 8:58 PM

This revision was automatically updated to reflect the committed changes.

image.png (448×484 px, 356 KB)

Revision Contents
Changeset List

Path

Size

src/

unit/

engine/

ArcanistConfigurationDrivenUnitTestEngine.php

1 line

Diff 46560

src/unit/engine/ArcanistConfigurationDrivenUnitTestEngine.php

Loading...