Add a failing test case for new Phortune account initialization
ClosedPublic
Actions

Authored by epriestley on Apr 14 2017, 4:53 PM.

Details

Reviewers

chad

Maniphest Tasks

Commits

Summary

Ref T12451. Ref T12484. I think D17657 fixed this, but caused the bug in D17690. The fix for that causes this bug again.

Put a unit test on it. This test currently fails; I'll correct the bug in the next change.

Test Plan

Ran arc unit, saw a failure.

Repository

rP Phabricator

Branch

phortune3

Lint

Lint Passed

Unit

Test Failures

Build Status

	Time	Test
	15 ms	PhabricatorPhortuneTestCase::Unknown Unit Message ("") EXCEPTION (PhabricatorApplicationTransactionValidationException): Validation errors: - You can not remove yourself as an account manager. #0 /Users/epriestley/dev/core/lib/phabricator/src/applications/phortune/storage/PhortuneAccount.php(51): PhabricatorApplicationTransactionEditor->applyTransactions(Object(PhortuneAccount), Array)
	17 ms	PhabricatorPhortuneTestCase::Unknown Unit Message ("") EXCEPTION (PhabricatorApplicationTransactionValidationException): Validation errors: - You can not remove yourself as an account manager. #0 /core/data/drydock/workingcopy-85/repo/phabricator/src/applications/phortune/storage/PhortuneAccount.php(51): PhabricatorApplicationTransactionEditor->applyTransactions(Object(PhortuneAccount), Array)
	1 ms	AlmanacNamesTestCase::Unknown Unit Message ("") 30 assertions passed.
	0 ms	AlmanacServiceTypeTestCase::Unknown Unit Message ("") 1 assertion passed.
	0 ms	AphrontHTTPSinkTestCase::Unknown Unit Message ("") 2 assertions passed.
		View Full Test Results (2 Failed · 343 Passed)