Paths

Table of Contentst

Differential D15178

Add some test coverage for board moves
ClosedPublic
Actions

Authored by epriestley on Feb 3 2016, 9:07 PM.

Details

Reviewers

chad

Maniphest Tasks

T10010: Implement Subprojects

Commits

Restricted Diffusion Commit
rP00165424d0d4: Add some test coverage for board moves

Summary

Ref T10010. This isn't totally comprehensive, and a lot of behaviors aren't testable (e.g., all the Javascript stuff) but at least covers the basic create/move/reorder operations.

Test Plan

arc unit

Diff Detail

Repository

rP Phabricator

Branch

col6

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 10477
Build 12817: Run Core Tests
Build 12816: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 36637.Feb 3 2016, 9:07 PM

epriestley retitled this revision from to Add some test coverage for board moves.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: chad.

epriestley added a task: T10010: Implement Subprojects.

chad accepted this revision.Feb 3 2016, 9:11 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Feb 3 2016, 9:11 PM

Closed by commit rP00165424d0d4: Add some test coverage for board moves (authored by epriestley, committed by epriestley). · Explain WhyFeb 3 2016, 11:08 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

project/

__tests__/

PhabricatorProjectCoreTestCase.php

159 lines

engine/

PhabricatorBoardLayoutEngine.php

4 lines

Diff 36637

View Options

src/applications/project/tests/PhabricatorProjectCoreTestCase.php

View Options

src/applications/project/engine/PhabricatorBoardLayoutEngine.php

Add some test coverage for board movesClosedPublicActions