Differential D19465

[ARCUNIT] Set the ConfigurationManager of ConfigurationDrivenUnitTestEngines
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Jan 23, 1:13 PM

	Unknown Object (File)
	Tue, Jan 21, 12:36 PM

	Unknown Object (File)
	Tue, Jan 21, 9:19 AM

	Unknown Object (File)
	Wed, Jan 15, 7:53 AM

	Unknown Object (File)
	Sat, Jan 4, 12:58 PM

	Unknown Object (File)
	Dec 28 2024, 5:18 PM

	Unknown Object (File)
	Dec 27 2024, 8:35 PM

	Unknown Object (File)
	Dec 24 2024, 3:05 AM

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 46561

src/unit/engine/ArcanistConfigurationDrivenUnitTestEngine.php

Loading...