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

	F15392872: D8435.diff
	Sat, Mar 15, 6:08 PM

	Unknown Object (File)
	Mon, Feb 24, 11:33 AM

	Unknown Object (File)
	Sun, Feb 23, 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 8 2025, 6:25 AM

	Unknown Object (File)
	Feb 1 2025, 2:21 PM

	Unknown Object (File)
	Jan 29 2025, 6:10 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