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

	F13816832: D8435.diff
	Thu, Sep 19, 6:50 PM

	Unknown Object (File)
	Mon, Aug 26, 3:13 AM

	Unknown Object (File)
	Thu, Aug 22, 8:20 PM

	Unknown Object (File)
	Thu, Aug 22, 8:20 PM

	Unknown Object (File)
	Thu, Aug 22, 8:20 PM

	Unknown Object (File)
	Thu, Aug 22, 7:18 PM

	Unknown Object (File)
	Thu, Aug 22, 12:05 AM

	Unknown Object (File)
	Aug 15 2024, 8:10 PM

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