Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
SONARJS-332 Upgrade SonarQube dependency from 3.7 to 4.5.1
- Loading branch information
1 parent
5c4729c
commit 992d3a5
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,33 @@ | ||
| /* | ||
| * SonarQube JavaScript Plugin | ||
| * Copyright (C) 2011 SonarSource and Eriks Nukis | ||
| * [email protected] | ||
| * | ||
| * This program is free software; you can redistribute it and/or | ||
| * modify it under the terms of the GNU Lesser General Public | ||
| * License as published by the Free Software Foundation; either | ||
| * version 3 of the License, or (at your option) any later version. | ||
| * | ||
| * This program is distributed in the hope that it will be useful, | ||
| * but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
| * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | ||
| * Lesser General Public License for more details. | ||
| * | ||
| * You should have received a copy of the GNU Lesser General Public | ||
| * License along with this program; if not, write to the Free Software | ||
| * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02 | ||
| */ | ||
| package org.sonar.javascript; | ||
|
|
||
| import java.io.File; | ||
| import java.net.URISyntaxException; | ||
|
|
||
| public class TestUtils { | ||
|
|
||
| private TestUtils() { | ||
| } | ||
|
|
||
| public static File getResource(String filename) throws URISyntaxException { | ||
| return new File(TestUtils.class.getResource(filename).toURI()); | ||
| } | ||
| } |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -19,9 +19,18 @@ | |
| */ | ||
| package org.sonar.plugins.javascript; | ||
|
|
||
| import java.util.Collection; | ||
| import java.util.List; | ||
| import java.util.Locale; | ||
| import java.util.Set; | ||
|
|
||
| import org.slf4j.Logger; | ||
| import org.slf4j.LoggerFactory; | ||
| import org.sonar.api.batch.Sensor; | ||
| import org.sonar.api.batch.SensorContext; | ||
| import org.sonar.api.batch.fs.FilePredicate; | ||
| import org.sonar.api.batch.fs.FileSystem; | ||
| import org.sonar.api.batch.fs.InputFile; | ||
| import org.sonar.api.checks.AnnotationCheckFactory; | ||
This comment has been minimized.
Sorry, something went wrong. |
||
| import org.sonar.api.checks.NoSonarFilter; | ||
This comment has been minimized.
Sorry, something went wrong. 
pynicolas
Contributor
|
||
| import org.sonar.api.component.ResourcePerspectives; | ||
|
|
@@ -36,8 +45,6 @@ | |
| import org.sonar.api.resources.Project; | ||
| import org.sonar.api.rule.RuleKey; | ||
| import org.sonar.api.rules.ActiveRule; | ||
| import org.sonar.javascript.EcmaScriptConfiguration; | ||
| import org.sonar.javascript.JavaScriptAstScanner; | ||
| import org.sonar.javascript.JavaScriptFileScanner; | ||
|
|
@@ -58,43 +65,43 @@ | |
| import org.sonar.squidbridge.indexer.QueryByType; | ||
| import org.sonar.sslr.parser.LexerlessGrammar; | ||
|
|
||
| import com.google.common.collect.Lists; | ||
|
|
||
| public class JavaScriptSquidSensor implements Sensor { | ||
|
|
||
| private static final Logger LOG = LoggerFactory.getLogger(JavaScriptSquidSensor.class); | ||
| private static final Number[] FUNCTIONS_DISTRIB_BOTTOM_LIMITS = {1, 2, 4, 6, 8, 10, 12, 20, 30}; | ||
| private static final Number[] FILES_DISTRIB_BOTTOM_LIMITS = {0, 5, 10, 20, 30, 60, 90}; | ||
|
|
||
| private final AnnotationCheckFactory annotationCheckFactory; | ||
| private final FileLinesContextFactory fileLinesContextFactory; | ||
| private final ResourcePerspectives resourcePerspectives; | ||
| private final FileSystem fileSystem; | ||
| private final NoSonarFilter noSonarFilter; | ||
| private final FilePredicate mainFilePredicate; | ||
|
|
||
| private SensorContext context; | ||
| private AstScanner<LexerlessGrammar> scanner; | ||
|
|
||
| public JavaScriptSquidSensor(RulesProfile profile, FileLinesContextFactory fileLinesContextFactory, | ||
| ResourcePerspectives resourcePerspectives, FileSystem fileSystem, NoSonarFilter noSonarFilter) { | ||
| this.annotationCheckFactory = AnnotationCheckFactory.create(profile, CheckList.REPOSITORY_KEY, CheckList.getChecks()); | ||
| this.fileLinesContextFactory = fileLinesContextFactory; | ||
| this.resourcePerspectives = resourcePerspectives; | ||
| this.fileSystem = fileSystem; | ||
| this.noSonarFilter = noSonarFilter; | ||
| this.mainFilePredicate = fileSystem.predicates().and( | ||
| fileSystem.predicates().hasType(InputFile.Type.MAIN), | ||
| fileSystem.predicates().hasLanguage(JavaScript.KEY)); | ||
| } | ||
|
|
||
| @Override | ||
| public boolean shouldExecuteOnProject(Project project) { | ||
| return fileSystem.hasFiles(mainFilePredicate); | ||
| } | ||
|
|
||
| @Override | ||
| public void analyse(Project project, SensorContext context) { | ||
| this.context = context; | ||
|
|
||
| List<CodeVisitor> astNodeVisitors = Lists.newArrayList(); | ||
|
|
@@ -110,31 +117,37 @@ public void analyse(Project project, SensorContext context) { | |
| } | ||
|
|
||
| astNodeVisitors.add(new VisitorsBridge(treeVisitors)); | ||
| astNodeVisitors.add(new FileLinesVisitor(fileLinesContextFactory, fileSystem)); | ||
|
|
||
| scanner = JavaScriptAstScanner.create(createConfiguration(), astNodeVisitors.toArray(new SquidAstVisitor[astNodeVisitors.size()])); | ||
| scanner.scanFiles(Lists.newArrayList(fileSystem.files(mainFilePredicate))); | ||
|
|
||
| Collection<SourceCode> squidSourceFiles = scanner.getIndex().search(new QueryByType(SourceFile.class)); | ||
| save(squidSourceFiles); | ||
| } | ||
|
|
||
| private EcmaScriptConfiguration createConfiguration() { | ||
| return new EcmaScriptConfiguration(fileSystem.encoding()); | ||
| } | ||
|
|
||
| private void save(Collection<SourceCode> squidSourceFiles) { | ||
| for (SourceCode squidSourceFile : squidSourceFiles) { | ||
| SourceFile squidFile = (SourceFile) squidSourceFile; | ||
|
|
||
| InputFile inputFile = fileSystem.inputFile(fileSystem.predicates().hasAbsolutePath(squidFile.getKey())); | ||
|
|
||
| if (inputFile != null) { | ||
| File sonarFile = File.create(inputFile.relativePath()); | ||
| noSonarFilter.addResource(sonarFile, squidFile.getNoSonarTagLines()); | ||
| saveClassComplexity(sonarFile, squidFile); | ||
| saveFilesComplexityDistribution(sonarFile, squidFile); | ||
| saveFunctionsComplexityAndDistribution(sonarFile, squidFile); | ||
| saveMeasures(sonarFile, squidFile); | ||
| saveIssues(sonarFile, squidFile); | ||
|
|
||
| } else { | ||
| LOG.warn("Cannot save analysis information for file {}. Unable to retrieve the associated sonar resource.", squidFile.getKey()); | ||
| } | ||
| } | ||
| } | ||
|
|
||
|
|
||
Oops, something went wrong.
Deprecated class. Can be replaced by org.sonar.api.batch.rule.CheckFactory (not trivial).