Paths

Table of Contentst

Skip unit tests if `ArcanistLinter::getCacheVersion` throws an `ArcanistUsageException`
ClosedPublic
Actions

Authored by joshuaspence on Jan 9 2014, 1:56 AM.

Tags

None

Referenced Files

	F14036029: D7913.diff
	Sun, Nov 10, 8:17 AM

	F13999310: D7913.diff
	Thu, Oct 24, 2:07 PM

	F13979986: D7913.id17910.diff
	Sat, Oct 19, 7:45 AM

	F13972667: D7913.id.diff
	Thu, Oct 17, 8:25 PM

	Unknown Object (File)
	Sep 20 2024, 9:24 AM

	Unknown Object (File)
	Sep 19 2024, 12:50 AM

	Unknown Object (File)
	Aug 29 2024, 1:00 PM

	Unknown Object (File)
	Aug 29 2024, 3:23 AM

Subscribers

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T4288: Unit test is not skipped if ArcanistLinter::getCacheVersion throws an ArcanistUsageException

Commits

rARCef598794a812: Skip unit tests if `ArcanistLinter::getCacheVersion` throws an…

Summary

Fixes T4288

Test Plan

N/A

Diff Detail

Branch

external_linter

Lint

Lint Passed

Unit

Tests Passed

Event Timeline

Can you throw a comment in the catch block about why/when this happens?

Also, walk me through how this causes the tests to be skipped? That doesn't seem to follow obviously from this change...

Yeah sorry, that was a really bad attempt. Pushing a new diff

Previous attempt didn't work as expected.

joshuaspence added inline comments.Jan 9 2014, 2:19 AM

src/lint/linter/__tests__/ArcanistLinterTestCase.php
122	`ArcanistLinter::getCacheVersion` throws `ArcanistUsageException`... it doesn't get converted into a `PhutilAggregateException`.

Ah, nice, this is much cleaner.

Closed by commit rARCef598794a812 (authored by @joshuaspence, committed by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

lint/

linter/

__tests__/

ArcanistLinterTestCase.php

2 lines

Diff 17911

View Options

src/lint/linter/tests/ArcanistLinterTestCase.php

Skip unit tests if `ArcanistLinter::getCacheVersion` throws an `ArcanistUsageException`ClosedPublicActions