From 1410b857a28a37feb6268c27fd3b075b691a234f Mon Sep 17 00:00:00 2001
From: Elena Vilchik
Date: Tue, 11 Jun 2019 13:00:18 +0200
Subject: Add warning when old NodeJS property is used (#175)

---
 .../test/java/org/sonar/css/plugin/CssRuleSensorTest.java   | 13 +++++++++++++
 1 file changed, 13 insertions(+)

(limited to 'sonar-css-plugin/src/test')

diff --git a/sonar-css-plugin/src/test/java/org/sonar/css/plugin/CssRuleSensorTest.java b/sonar-css-plugin/src/test/java/org/sonar/css/plugin/CssRuleSensorTest.java
index f7f0d31..cf33d50 100644
--- a/sonar-css-plugin/src/test/java/org/sonar/css/plugin/CssRuleSensorTest.java
+++ b/sonar-css-plugin/src/test/java/org/sonar/css/plugin/CssRuleSensorTest.java
@@ -106,6 +106,19 @@ public class CssRuleSensorTest {
     verifyZeroInteractions(analysisWarnings);
   }
 
+  @Test
+  public void test_old_property_is_provided() {
+    TestLinterCommandProvider commandProvider = getCommandProvider();
+    CssRuleSensor sensor = createCssRuleSensor(commandProvider, analysisWarnings);
+    context.settings().setProperty(CssPlugin.FORMER_NODE_EXECUTABLE, "foo");
+    sensor.execute(context);
+
+    assertThat(logTester.logs(LoggerLevel.WARN)).contains("Property 'sonar.css.node' is ignored, 'sonar.nodejs.executable' should be used instead");
+    verify(analysisWarnings).addUnique(eq("Property 'sonar.css.node' is ignored, 'sonar.nodejs.executable' should be used instead"));
+
+    assertThat(context.allIssues()).hasSize(1);
+  }
+
   @Test
   public void test_invalid_node() {
     InvalidCommandProvider commandProvider = new InvalidCommandProvider();
-- 
cgit v1.2.3