Maniphest T12484

Write some tests for Phortune
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	chad
	Mar 30 2017, 11:01 PM

Tags

Referenced Files

None

Subscribers

Description

Given its importance for the monetary health of the company, maybe I should learn to write unit tests.

Revisions and Commits

rP Phabricator
	D17692	rP505b1d8379a5 Fix member edit transaction validation so it works for both implicit and…
	D17691	rP71d933d496ca Add a failing test case for new Phortune account initialization

Event Timeline

chad created this task.Mar 30 2017, 11:01 PM

Herald added a subscriber: revi. · View Herald TranscriptMar 30 2017, 11:01 PM

epriestley added a revision: D17691: Add a failing test case for new Phortune account initialization.Apr 14 2017, 4:53 PM

epriestley added a revision: D17692: Fix member edit transaction validation so it works for both implicit and explicit account creation.Apr 14 2017, 4:58 PM

epriestley added a commit: rP71d933d496ca: Add a failing test case for new Phortune account initialization.Apr 14 2017, 5:23 PM

epriestley added a commit: rP505b1d8379a5: Fix member edit transaction validation so it works for both implicit and….

This hasn't had any stability issues for a while.