From 2bbe2243f0a393dd3c9884dca5560f583a843f88 Mon Sep 17 00:00:00 2001 From: Elena Vilchik Date: Fri, 27 Dec 2019 15:34:39 +0100 Subject: Analyze CSS in Vue Single File Components (#232) --- sonar-css-plugin/pom.xml | 4 ++-- .../main/java/org/sonar/css/plugin/CssRuleSensor.java | 16 ++++++++++++---- .../java/org/sonar/css/plugin/CssRuleSensorTest.java | 11 +++++++---- .../src/test/resources/mock-start-server/startServer.js | 1 + 4 files changed, 22 insertions(+), 10 deletions(-) (limited to 'sonar-css-plugin') diff --git a/sonar-css-plugin/pom.xml b/sonar-css-plugin/pom.xml index c79869e..b8576c9 100644 --- a/sonar-css-plugin/pom.xml +++ b/sonar-css-plugin/pom.xml @@ -135,8 +135,8 @@ - 6800000 - 9000000 + 8000000 + 9500000 ${project.build.directory}/${project.build.finalName}.jar diff --git a/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssRuleSensor.java b/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssRuleSensor.java index db961df..326c94f 100644 --- a/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssRuleSensor.java +++ b/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssRuleSensor.java @@ -88,7 +88,7 @@ public class CssRuleSensor implements Sensor { List inputFiles = getInputFiles(context); if (inputFiles.isEmpty()) { - LOG.info("No CSS, PHP or HTML files are found in the project. CSS analysis is skipped."); + LOG.info("No CSS, PHP, HTML or VueJS files are found in the project. CSS analysis is skipped."); return; } @@ -142,8 +142,8 @@ public class CssRuleSensor implements Sensor { LOG.info(e.toString()); } catch (Exception e) { - // we can end up here in the following cases: file analysis request sending, or response parsing, server is not answering - // or some unpredicted state + // we can end up here in the following cases: fail to send file analysis request, fail to parse the response, server is not answering + // or some other unpredicted state String msg = "Failure during CSS analysis, " + cssAnalyzerBridgeServer.getCommandInfo(); logErrorOrWarn(context, msg, e); throwFailFast(context, e); @@ -245,10 +245,18 @@ public class CssRuleSensor implements Sensor { private static List getInputFiles(SensorContext context) { FileSystem fileSystem = context.fileSystem(); + FilePredicate mainFilePredicate = fileSystem.predicates().and( fileSystem.predicates().hasType(InputFile.Type.MAIN), fileSystem.predicates().hasLanguages(CssLanguage.KEY, "php", "web")); - return StreamSupport.stream(fileSystem.inputFiles(mainFilePredicate).spliterator(), false) + + FilePredicate vueFilePredicate = fileSystem.predicates().and( + fileSystem.predicates().hasType(InputFile.Type.MAIN), + fileSystem.predicates().hasExtension("vue"), + // by default 'vue' extension is defined for JS language, but 'vue' files can contain TS code and thus language can be changed + fileSystem.predicates().hasLanguages("js", "ts")); + + return StreamSupport.stream(fileSystem.inputFiles(fileSystem.predicates().or(mainFilePredicate, vueFilePredicate)).spliterator(), false) .collect(Collectors.toList()); } 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 429722d..4a2fbd4 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 @@ -129,14 +129,15 @@ public class CssRuleSensorTest { DefaultInputFile fileCss = addInputFile("file.css"); DefaultInputFile fileHtml = addInputFile("file.web"); DefaultInputFile filePhp = addInputFile("file.php"); + DefaultInputFile fileVue = addInputFile("file.vue"); addInputFile("file.js"); sensor.execute(context); - assertThat(context.allIssues()).hasSize(3); + assertThat(context.allIssues()).hasSize(4); assertThat(context.allIssues()) .extracting("primaryLocation.component") - .containsOnly(fileCss, fileHtml, filePhp); + .containsOnly(fileCss, fileHtml, filePhp, fileVue); } @Test @@ -144,7 +145,7 @@ public class CssRuleSensorTest { sensor.execute(context); assertThat(context.allIssues()).hasSize(0); assertThat(logTester.logs(LoggerLevel.ERROR)).isEmpty(); - assertThat(logTester.logs(LoggerLevel.INFO)).contains("No CSS, PHP or HTML files are found in the project. CSS analysis is skipped."); + assertThat(logTester.logs(LoggerLevel.INFO)).contains("No CSS, PHP, HTML or VueJS files are found in the project. CSS analysis is skipped."); } @Test @@ -322,8 +323,10 @@ public class CssRuleSensorTest { } private DefaultInputFile addInputFile(String relativePath) { + String extension = relativePath.split("\\.")[1]; + String language = extension.equals("vue") ? "js" : extension; DefaultInputFile inputFile = new TestInputFileBuilder("moduleKey", relativePath) - .setLanguage(relativePath.split("\\.")[1]) + .setLanguage(language) .setCharset(StandardCharsets.UTF_8) .setContents("some css content\n on 2 lines") .build(); diff --git a/sonar-css-plugin/src/test/resources/mock-start-server/startServer.js b/sonar-css-plugin/src/test/resources/mock-start-server/startServer.js index 87298a9..b224e48 100644 --- a/sonar-css-plugin/src/test/resources/mock-start-server/startServer.js +++ b/sonar-css-plugin/src/test/resources/mock-start-server/startServer.js @@ -24,6 +24,7 @@ const requestHandler = (request, response) => { case "file.css": case "file.web": case "file.php": + case "file.vue": case "file.js": // to test that we will not save this issue even if it's provided by response response.end(JSON.stringify([ {line: 2, rule: "block-no-empty", text: "Unexpected empty block"} -- cgit v1.2.3