Homec4science

Don't show detailed unittest result box if all tests passed

Authored by Nick Harper <nharper@fb.com> on Jan 3 2012, 22:47.

Description

Don't show detailed unittest result box if all tests passed

Summary:
When all unit tests pass, a box appears between the unit test results and lint
status (for test failures to go in). This checks if there's anything to put
in that div/ul before putting it on the page.

Test Plan:
Loaded a revision with unit tests OK and saw no box. Loaded a revision with
failing unittests, and saw the same box from before.

Reviewers: tuomaspelkonen, epriestley

Reviewed By: epriestley

CC: jungejason, aran, nh, epriestley

Differential Revision: https://secure.phabricator.com/D1295

Details

Committed
Nick Harper <nharper@fb.com>Jan 3 2012, 23:58
Pushed
aubortJan 31 2017, 17:16
Parents
rPH369079dd4522: Change default status to 'all' in DifferentialRevisionList
Branches
Unknown
Tags
Unknown

Event Timeline

Nick Harper <nharper@fb.com> committed rPH4a7790614139: Don't show detailed unittest result box if all tests passed (authored by Nick Harper <nharper@fb.com>).Jan 3 2012, 23:58