Page MenuHomePhabricator

Fix failing unit tests for `ArcanistLesscLinter`
ClosedPublic

Authored by joshuaspence on Jan 27 2015, 11:51 AM.
Tags
None
Referenced Files
F21799344: D11521.id27701.diff
Tue, Aug 11, 11:36 AM
F21794513: D11521.diff
Tue, Aug 11, 7:33 AM
Unknown Object (File)
Jan 30 2026, 4:58 PM
Unknown Object (File)
Jan 22 2026, 12:01 PM
Unknown Object (File)
Nov 10 2025, 8:05 PM
Unknown Object (File)
Oct 21 2025, 10:04 AM
Unknown Object (File)
Oct 6 2025, 6:14 AM
Unknown Object (File)
Sep 30 2025, 2:07 PM
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.