Page MenuHomePhabricator
Differential D17691

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

Authored by epriestley on Apr 14 2017, 4:53 PM.
Tags
None
Referenced Files
F19824250: D17691.id.diff
Sat, Mar 7, 8:54 PM
F19812573: D17691.diff
Wed, Mar 4, 5:29 AM
F19791844: D17691.id42551.diff
Thu, Feb 26, 12:00 AM
F19756117: D17691.id42545.diff
Tue, Feb 17, 5:57 PM
F19705479: D17691.diff
Wed, Feb 11, 7:14 AM
F19603169: D17691.id42545.diff
Feb 3 2026, 6:44 AM
F19531623: D17691.id42545.diff
Jan 20 2026, 6:01 AM
F19527982: D17691.id42545.diff
Jan 19 2026, 6:00 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T12451: Finding your subscriptions / cards on Phacility is non-obvious
T12484: Write some tests for Phortune
Commits
rP71d933d496ca: Add a failing test case for new Phortune account initialization
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.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 14 2017, 4:53 PM
Harbormaster failed remote builds in B16476: Diff 42545!Apr 14 2017, 4:53 PM
chad accepted this revision.Apr 14 2017, 4:55 PM
This revision is now accepted and ready to land.Apr 14 2017, 4:55 PM
Closed by commit rP71d933d496ca: Add a failing test case for new Phortune account initialization (authored by epriestley, committed by epriestley). · Explain WhyApr 14 2017, 5:23 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
2 lines
applications/
phortune/
__tests__/
26 lines

Diff 42551

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/phortune/__tests__/PhabricatorPhortuneTestCase.php

Loading...
Phabricator · Built by Phacility