Page MenuHomePhabricator

Allow an arc test to execute without a git author configured
ClosedPublic

Authored by epriestley on Oct 4 2015, 3:22 PM.
Tags
None
Referenced Files
F18804660: D14232.id34367.diff
Sat, Oct 18, 10:26 AM
F18783329: D14232.diff
Mon, Oct 13, 5:59 AM
F18649214: D14232.diff
Sat, Sep 20, 7:38 PM
F18506568: D14232.id.diff
Sep 5 2025, 1:36 AM
F18499177: D14232.diff
Sep 4 2025, 7:30 PM
F18098393: D14232.diff
Aug 8 2025, 7:55 PM
Unknown Object (File)
Jun 16 2025, 8:33 PM
Unknown Object (File)
Jun 1 2025, 3:57 PM
Subscribers
None

Details

Summary

On sbuild, we currently get a failure on this test. Use an explicit --author so we can run the test even if user.email and user.name are not set in global Git config.

   FAIL  ArcanistBundleTestCase::testGitRepository
15	EXCEPTION (CommandException): Command failed with error #128!
16	COMMAND
17	git commit -m 'Mark koan2 +x and edit it.'
18	
19	STDOUT
20	(empty)
21	
22	STDERR
23	
24	*** Please tell me who you are.
25	
26	Run
27	
28	  git config --global user.email "you@example.com"
29	  git config --global user.name "Your Name"
30	
31	to set your account's default identity.
32	Omit --global to set the identity only in this repository.
33	
34	fatal: empty ident name (for <builder@sbuild001.phacility.net>) not allowed
Test Plan

Ran arc unit --everything. Will verify in production.

Diff Detail

Repository
rARC Arcanist
Branch
unit3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 8178
Build 9345: Run Core Tests
Build 9344: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Allow an arc test to execute without a git author configured.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Oct 4 2015, 3:30 PM
This revision was automatically updated to reflect the committed changes.

Unfortunately, this didn't work at all. I'll figure out what's up...