Page MenuHomePhabricator

Improve `ArcanistLinterStandardTestCase`
Needs ReviewPublic

Authored by wjiang on Apr 24 2017, 1:04 AM.
Tags
None
Referenced Files
F12842414: D17783.id42759.diff
Thu, Mar 28, 10:00 PM
Unknown Object (File)
Sat, Mar 16, 4:36 PM
Unknown Object (File)
Feb 3 2024, 3:41 PM
Unknown Object (File)
Jan 25 2024, 11:38 AM
Unknown Object (File)
Dec 23 2023, 1:03 PM
Unknown Object (File)
Dec 21 2023, 5:55 PM
Unknown Object (File)
Dec 21 2023, 5:55 PM
Unknown Object (File)
Dec 21 2023, 5:55 PM

Details

Summary

Ref T12631. Let unit test actually test ArcanistLinterStandard.

Test Plan

Run unit test

Diff Detail

Repository
rARC Arcanist
Branch
arcpatch-D17783
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16794
Build 22413: arc lint + arc unit

Event Timeline

joshuaspence added a subscriber: joshuaspence.

LGTM. @epriestley, it would be greatly appreciated if this could be accepted upstream. We have a large number of XHPAST linter rules (I think ~200) and we want to write integration tests to ensure that they play nicely together.

src/lint/linter/standards/__tests__/ArcanistLinterStandardTestCase.php
4

You should extend ArcanistLinterTestCase instead

5–8

You might want to leave this.

8

Maybe call this getBaseLinter?

10

I think this should be called getStandard

15–18

Ideally, we should have some sort of $linter->applyStandard($standard) method... because this code is currently duplicated from ArcanistLinter

wjiang marked 5 inline comments as done.
  • Add applyStandard