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
F14758583: D8436.id.diff
Wed, Jan 22, 7:57 AM
Unknown Object (File)
Mon, Jan 6, 2:20 PM
Unknown Object (File)
Tue, Dec 24, 5:08 AM
Unknown Object (File)
Dec 22 2024, 2:00 AM
Unknown Object (File)
Dec 13 2024, 7:52 AM
Unknown Object (File)
Dec 7 2024, 1:34 AM
Unknown Object (File)
Dec 3 2024, 11:52 AM
Unknown Object (File)
Dec 3 2024, 11:51 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