Page MenuHomePhabricator

Fix failing unit tests for `ArcanistLesscLinter`
ClosedPublic

Authored by joshuaspence on Jan 27 2015, 11:51 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jan 1, 6:05 AM
Unknown Object (File)
Wed, Jan 1, 6:05 AM
Unknown Object (File)
Wed, Jan 1, 6:03 AM
Unknown Object (File)
Wed, Jan 1, 5:45 AM
Unknown Object (File)
Dec 8 2024, 10:00 AM
Unknown Object (File)
Nov 29 2024, 5:49 PM
Unknown Object (File)
Nov 26 2024, 2:49 AM
Unknown Object (File)
Nov 9 2024, 8:48 AM
Subscribers

Details

Summary

A bunch of unit tests are failing with the latest version of lessc (v2.3.0). I decided to delete a bunch of test cases for this linter as we have far too many at the moment.

Test Plan

arc unit

Diff Detail

Repository
rARC Arcanist
Branch
master
Lint
No Lint Coverage
Unit
Tests Passed
Build Status
Buildable 4080
Build 4093: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Fix failing unit tests for `ArcanistLesscLinter`.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.

Cool, I think this is desirable/correct. As elsewhere, we don't really benefit from covering the nuances in behavior of external linters, and primarily care that the binding works -- and we don't need 300 tests to establish that.

This revision is now accepted and ready to land.Jan 27 2015, 2:25 PM
This revision was automatically updated to reflect the committed changes.