Paths

Table of Contentst

Add assertions to every test in libphutil/
ClosedPublic
Actions

Authored by epriestley on Mar 7 2014, 2:16 PM.

Tags

None

Referenced Files

	F19023430: D8435.id20019.diff
	Nov 24 2025, 12:24 AM

	F18906833: D8435.id.diff
	Nov 8 2025, 8:36 PM

	F18886945: D8435.diff
	Nov 7 2025, 5:22 AM

	F18873438: D8435.id.diff
	Nov 5 2025, 9:49 AM

	F18749877: D8435.diff
	Oct 4 2025, 5:26 AM

	F18671168: D8435.diff
	Sep 25 2025, 2:54 AM

	F18600799: D8435.diff
	Sep 13 2025, 10:37 AM

	F18589846: D8435.diff
	Sep 12 2025, 4:26 AM

View All Files

Subscribers

aran

Details

Reviewers

leebyron
btrahan

Maniphest Tasks

T4570: ArcanistLinterTestCase::executeTestsInDirectory() should fail when it finds 0 tests.

Commits

rPHU01e849c9bafc: Add assertions to every test in libphutil/

Summary

Ref T4570. Add an assertion to all tests which pass by not exploding, so we can fail tests which make no assertions.

Test Plan

arc unit --everything with the test engine set to fail on 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

Closed by commit rPHU01e849c9bafc (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

channel/

__tests__/

PhutilPHPObjectProtocolChannelTestCase.php

2 lines

infrastructure/

__tests__/

PhutilInfrastructureTestCase.php

1 line

parser/

__tests__/

PhutilParserGeneratorTestCase.php

2 lines

PhutilTypeSpecTestCase.php

2 lines

utils/

__tests__/

PhutilUTF8TestCase.php

2 lines

Diff 20030

View Options

src/channel/tests/PhutilPHPObjectProtocolChannelTestCase.php

View Options

src/infrastructure/tests/PhutilInfrastructureTestCase.php

View Options

src/parser/tests/PhutilParserGeneratorTestCase.php

View Options

src/parser/tests/PhutilTypeSpecTestCase.php

View Options

src/utils/tests/PhutilUTF8TestCase.php

Add assertions to every test in libphutil/ClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 20030

src/channel/__tests__/PhutilPHPObjectProtocolChannelTestCase.php

src/infrastructure/__tests__/PhutilInfrastructureTestCase.php

src/parser/__tests__/PhutilParserGeneratorTestCase.php

src/parser/__tests__/PhutilTypeSpecTestCase.php

src/utils/__tests__/PhutilUTF8TestCase.php

Add assertions to every test in libphutil/
ClosedPublic
Actions

Revision Contents
Changeset List

src/channel/tests/PhutilPHPObjectProtocolChannelTestCase.php

src/infrastructure/tests/PhutilInfrastructureTestCase.php

src/parser/tests/PhutilParserGeneratorTestCase.php

src/parser/tests/PhutilTypeSpecTestCase.php

src/utils/tests/PhutilUTF8TestCase.php