Page MenuHomePhabricator
Differential D8436

Add assertions to no-assertions tests in phabricator/
ClosedPublic
Actions

Authored by epriestley on Mar 7 2014, 2:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Feb 14, 10:51 PM
Unknown Object (File)
Fri, Feb 14, 10:51 PM
Unknown Object (File)
Fri, Feb 14, 10:51 PM
Unknown Object (File)
Sun, Feb 2, 6:01 PM
Unknown Object (File)
Sun, Feb 2, 3:26 PM
Unknown Object (File)
Fri, Jan 31, 4:07 AM
Unknown Object (File)
Thu, Jan 30, 6:42 PM
Unknown Object (File)
Tue, Jan 28, 6:24 AM
View All Files
Subscribers
aran

Details

Reviewers
leebyron
btrahan
Maniphest Tasks
T4570: ArcanistLinterTestCase::executeTestsInDirectory() should fail when it finds 0 tests.
Commits
Restricted Diffusion Commit
rP0488ce73c4b2: Add assertions to no-assertions tests in phabricator/
Summary

Ref T4570. Add trivial assertions to tests which fail-by-exploding so we can fail tests with no assertions.

Test Plan

Ran arc unit --everything with Arcanist patched to fail with no assertions.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan accepted this revision.Mar 7 2014, 5:48 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 7 2014, 5:48 PM
epriestley closed this revision.Mar 7 2014, 6:02 PM

Closed by commit rP0488ce73c4b2 (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
people/
editor/
__tests__/
2 lines
infrastructure/
__tests__/
1 line
storage/
__tests__/
4 lines
lisk/
__tests__/
1 line

Diff 20029

View Options

src/applications/people/editor/__tests__/PhabricatorUserEditorTestCase.php

Loading...
View Options

src/infrastructure/__tests__/PhabricatorInfrastructureTestCase.php

Loading...
View Options

src/infrastructure/storage/__tests__/AphrontIsolatedDatabaseConnectionTestCase.php

Loading...
View Options

src/infrastructure/storage/lisk/__tests__/LiskIsolationTestCase.php

Loading...
Phabricator · Built by Phacility