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
F15515581: D8436.id.diff
Fri, Apr 18, 11:33 AM
F15511717: D8436.diff
Thu, Apr 17, 4:56 AM
F15425645: D8436.diff
Sun, Mar 23, 5:15 AM
F15368631: D8436.diff
Mar 12 2025, 3:18 AM
Unknown Object (File)
Feb 23 2025, 10:17 PM
Unknown Object (File)
Feb 14 2025, 10:51 PM
Unknown Object (File)
Feb 14 2025, 10:51 PM
Unknown Object (File)
Feb 14 2025, 10:51 PM
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