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
F18904879: D8436.id.diff
Nov 8 2025, 11:36 AM
F18756491: D8436.id.diff
Oct 5 2025, 11:42 AM
F18749718: D8436.diff
Oct 4 2025, 4:52 AM
F18734661: D8436.id.diff
Sep 30 2025, 10:59 PM
F18589558: D8436.diff
Sep 12 2025, 3:40 AM
F18107057: D8436.diff
Aug 10 2025, 11:37 PM
F18093618: D8436.id20020.diff
Aug 7 2025, 7:37 PM
F18087140: D8436.diff
Aug 6 2025, 2:34 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