HomePhabricator

Call `$linter->setEngine` in linter tests

Authored by joshuaspence on May 14 2019, 11:00 PM.

Description

Call $linter->setEngine in linter tests

Summary: We aren't calling $linter->setEngine($engine), even though we do have an $engine. This causes unit tests for any linters which require an engine to fail.

Test Plan: Ran the unit tests for a third-party linter.

Reviewers: epriestley, Blessed Reviewers

Reviewed By: epriestley, Blessed Reviewers

Subscribers: Korvin

Differential Revision: https://secure.phabricator.com/D20515

Details

Committed
joshuaspenceMay 14 2019, 11:01 PM
Pushed
joshuaspenceMay 14 2019, 11:02 PM
Reviewer
Blessed Reviewers
Parents
rARC9ccbbee33675: Fix a potential double-prompt in "arc land" when landing with ongoing builds
Branches
Unknown
Tags
Unknown
Build Status
Buildable 22796
Build 31272: Run Core Tests