green-code-initiative#92 refactor(rule/python): moves Python rules in…

…to `ecocode-rules-specifications` module

ecocode-rules-specifications/pom.xml

@@ -80,6 +80,13 @@
 									</fileset>
 									<flattenmapper/>
 								</copy>
+								<copy todir="${project.build.directory}/python">
+									<fileset dir="${project.build.directory}/rules">
+										<include name="*/python/EC*.html"/>
+										<include name="*/EC*.json"/>
+									</fileset>
+									<flattenmapper/>
+								</copy>
 							</target>
 						</configuration>
 						<goals>
@@ -116,6 +123,18 @@
 							</descriptors>
 						</configuration>
 					</execution>
+					<execution>
+						<id>assembly-python</id>
+						<phase>prepare-package</phase>
+						<goals>
+							<goal>single</goal>
+						</goals>
+						<configuration>
+							<descriptors>
+								<descriptor>${project.basedir}/src/main/assembly/python.xml</descriptor>
+							</descriptors>
+						</configuration>
+					</execution>
 				</executions>
 				<configuration>
 					<appendAssemblyId>true</appendAssemblyId>

ecocode-rules-specifications/src/main/assembly/python.xml

@@ -0,0 +1,22 @@
+<assembly xmlns="http://maven.apache.org/ASSEMBLY/2.1.1"
+          xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+          xsi:schemaLocation="http://maven.apache.org/ASSEMBLY/2.1.1 https://maven.apache.org/xsd/assembly-2.1.1.xsd">
+	<id>python</id>
+	<formats>
+		<format>jar</format>
+	</formats>
+	<includeBaseDirectory>false</includeBaseDirectory>
+	<fileSets>
+		<fileSet>
+			<directory>${project.build.outputDirectory}</directory>
+			<includes>
+				<include>io/ecocode/rules/python/*.class</include>
+			</includes>
+			<outputDirectory/>
+		</fileSet>
+		<fileSet>
+			<directory>${project.build.directory}/python</directory>
+			<outputDirectory>io/ecocode/rules/python/specifications</outputDirectory>
+		</fileSet>
+	</fileSets>
+</assembly>

ecocode-rules-specifications/src/main/java/io/ecocode/rules/python/PythonRulesSpecificationsRepository.java

@@ -0,0 +1,8 @@
+package io.ecocode.rules.python;
+
+public class PythonRulesSpecificationsRepository {
+  public static final String REPOSITORY_KEY = "ecocode-python";
+  public static final String NAME = "ecoCode";
+  public static final String LANGUAGE = "py";
+  public static final String RESOURCE_BASE_PATH = PythonRulesSpecificationsRepository.class.getPackageName().replace('.', '/') + "/specifications";
+}

ecocode-rules-specifications/src/main/rules/EC10/EC10.json

@@ -0,0 +1,14 @@
+{
+  "title": "Avoid using unoptimized vector images",
+  "type": "CODE_SMELL",
+  "status": "ready",
+  "remediation": {
+    "func": "Constant\/Issue",
+    "constantCost": "60min"
+  },
+  "tags": [
+    "eco-design",
+    "ecocode"
+  ],
+  "defaultSeverity": "Minor"
+}

ecocode-rules-specifications/src/main/rules/EC203/EC203.json

@@ -0,0 +1,16 @@
+{
+  "title": "Detect unoptimized image format",
+  "type": "CODE_SMELL",
+  "status": "ready",
+  "remediation": {
+    "func": "Constant\/Issue",
+    "constantCost": "60min"
+  },
+  "tags": [
+    "performance",
+    "user-experience",
+    "eco-design",
+    "ecocode"
+  ],
+  "defaultSeverity": "Minor"
+}

ecocode-rules-specifications/src/main/rules/EC404/EC404.json

@@ -0,0 +1,15 @@
+{
+  "title": "Use generator comprehension instead of list comprehension in for loop declaration",
+  "type": "CODE_SMELL",
+  "status": "ready",
+  "remediation": {
+    "func": "Constant\/Issue",
+    "constantCost": "15min"
+  },
+  "tags": [
+    "performance",
+    "eco-design",
+    "ecocode"
+  ],
+  "defaultSeverity": "Minor"
+}

ecocode-rules-specifications/src/main/rules/EC7/EC7.json

@@ -0,0 +1,15 @@
+{
+  "title": "Avoid creating getter and setter methods in classes",
+  "type": "CODE_SMELL",
+  "status": "ready",
+  "remediation": {
+    "func": "Constant\/Issue",
+    "constantCost": "5min"
+  },
+  "tags": [
+    "eco-design",
+    "performance",
+    "ecocode"
+  ],
+  "defaultSeverity": "Minor"
+}

python-plugin/pom.xml

@@ -16,20 +16,36 @@
     <url>https://github.com/green-code-initiative/ecoCode/tree/main/python-plugin</url>
 
     <dependencies>
+        <dependency>
+            <groupId>${project.groupId}</groupId>
+            <artifactId>ecocode-rules-specifications</artifactId>
+            <version>${project.version}</version>
+            <classifier>python</classifier>
+        </dependency>
 
         <dependency>
             <groupId>org.sonarsource.python</groupId>
             <artifactId>sonar-python-plugin</artifactId>
             <type>sonar-plugin</type>
+            <scope>provided</scope>
         </dependency>
 
         <dependency>
             <groupId>org.sonarsource.sonarqube</groupId>
             <artifactId>sonar-plugin-api</artifactId>
+            <scope>provided</scope>
+        </dependency>
+
+        <dependency>
+            <groupId>org.sonarsource.analyzer-commons</groupId>
+            <artifactId>sonar-analyzer-commons</artifactId>
+            <scope>provided</scope>
         </dependency>
+
         <dependency>
             <groupId>org.sonarsource.sonarqube</groupId>
             <artifactId>sonar-plugin-api-impl</artifactId>
+            <scope>test</scope>
         </dependency>
 
         <!-- for security on regex patterns : https://github.com/google/re2j -->
@@ -58,44 +74,32 @@
                     <pluginClass>fr.greencodeinitiative.python.PythonPlugin</pluginClass>
                     <sonarLintSupported>true</sonarLintSupported>
                     <sonarQubeMinVersion>${sonarqube.version}</sonarQubeMinVersion>
+                    <basePlugin>python</basePlugin>
                     <jreMinVersion>${java.version}</jreMinVersion>
                 </configuration>
             </plugin>
-<!--            <plugin>-->
-<!--                <groupId>org.apache.maven.plugins</groupId>-->
-<!--                <artifactId>maven-shade-plugin</artifactId>-->
-<!--                <executions>-->
-<!--                    <execution>-->
-<!--                        <phase>package</phase>-->
-<!--                        <goals>-->
-<!--                            <goal>shade</goal>-->
-<!--                        </goals>-->
-<!--                        <configuration>-->
-<!--                            <filters>-->
-<!--                                <filter>-->
-<!--                                    <artifact>commons-*:*</artifact>-->
-<!--                                    <excludes>-->
-<!--                                        <exclude>META-INF/**</exclude>-->
-<!--                                    </excludes>-->
-<!--                                </filter>-->
-<!--                                <filter>-->
-<!--                                    <artifact>org.*:*</artifact>-->
-<!--                                    <excludes>-->
-<!--                                        <exclude>META-INF/**</exclude>-->
-<!--                                        <exclude>org/sonar/api/batch/sensor/**</exclude>-->
-<!--                                    </excludes>-->
-<!--                                </filter>-->
-<!--                                <filter>-->
-<!--                                    <artifact>com.*:*</artifact>-->
-<!--                                    <excludes>-->
-<!--                                        <exclude>META-INF/**</exclude>-->
-<!--                                    </excludes>-->
-<!--                                </filter>-->
-<!--                            </filters>-->
-<!--                        </configuration>-->
-<!--                    </execution>-->
-<!--                </executions>-->
-<!--            </plugin>-->
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-shade-plugin</artifactId>
+                <executions>
+                    <execution>
+                        <phase>package</phase>
+                        <goals>
+                            <goal>shade</goal>
+                        </goals>
+                        <configuration>
+                            <filters>
+                                <filter>
+                                    <artifact>*:ecocode-rules-specifications</artifact>
+                                    <excludes>
+                                        <exclude>META-INF/**</exclude>
+                                    </excludes>
+                                </filter>
+                            </filters>
+                        </configuration>
+                    </execution>
+                </executions>
+            </plugin>
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-dependency-plugin</artifactId>

python-plugin/src/main/java/fr/greencodeinitiative/python/PythonRuleRepository.java

@@ -19,61 +19,62 @@
  */
 package fr.greencodeinitiative.python;
 
-import java.io.ByteArrayOutputStream;
-import java.io.IOException;
-import java.io.InputStream;
-import java.net.URL;
-import java.nio.charset.StandardCharsets;
-import java.util.Arrays;
-import java.util.HashMap;
-import java.util.List;
-import java.util.Map;
-
-
-import fr.greencodeinitiative.python.checks.*;
-import org.apache.commons.lang.StringUtils;
-import org.sonar.api.rules.RuleType;
+import fr.greencodeinitiative.python.checks.AvoidDoubleQuoteCheck;
+import fr.greencodeinitiative.python.checks.AvoidFullSQLRequest;
+import fr.greencodeinitiative.python.checks.AvoidGettersAndSetters;
+import fr.greencodeinitiative.python.checks.AvoidGlobalVariableInFunctionCheck;
+import fr.greencodeinitiative.python.checks.AvoidListComprehensionInIterations;
+import fr.greencodeinitiative.python.checks.AvoidSQLRequestInLoop;
+import fr.greencodeinitiative.python.checks.AvoidTryCatchFinallyCheck;
+import fr.greencodeinitiative.python.checks.AvoidUnoptimizedVectorImagesCheck;
+import fr.greencodeinitiative.python.checks.DetectUnoptimizedImageFormat;
+import fr.greencodeinitiative.python.checks.NoFunctionCallWhenDeclaringForLoop;
+import org.sonar.api.SonarEdition;
+import org.sonar.api.SonarProduct;
+import org.sonar.api.SonarQubeSide;
+import org.sonar.api.SonarRuntime;
 import org.sonar.api.server.rule.RulesDefinition;
-import org.sonar.api.server.rule.RulesDefinitionAnnotationLoader;
+import org.sonar.api.utils.Version;
 import org.sonar.plugins.python.api.PythonCustomRuleRepository;
+import org.sonarsource.analyzer.commons.RuleMetadataLoader;
 
-public class PythonRuleRepository implements RulesDefinition, PythonCustomRuleRepository {
-
-  public static final String LANGUAGE = "py";
-  public static final String NAME = "ecoCode";
-  public static final String RESOURCE_BASE_PATH = "/fr/greencodeinitiative/l10n/python/rules/python/";
-  public static final String REPOSITORY_KEY = "ecocode-python";
+import java.util.Arrays;
+import java.util.List;
 
-  @Override
-  public void define(Context context) {
-    NewRepository repository = context.createRepository(repositoryKey(), LANGUAGE).setName(NAME);
+import static io.ecocode.rules.python.PythonRulesSpecificationsRepository.LANGUAGE;
+import static io.ecocode.rules.python.PythonRulesSpecificationsRepository.NAME;
+import static io.ecocode.rules.python.PythonRulesSpecificationsRepository.REPOSITORY_KEY;
+import static io.ecocode.rules.python.PythonRulesSpecificationsRepository.RESOURCE_BASE_PATH;
 
-    new RulesDefinitionAnnotationLoader().load(repository, checkClasses().toArray(new Class[] {}));
+public class PythonRuleRepository implements RulesDefinition, PythonCustomRuleRepository {
+  private static final Version SONARQUBE_RUNTIME_VERSION = Version.create(9, 8);
+  private static final SonarRuntime SONARQUBE_RUNTIME = new SonarRuntime() {
+    @Override
+    public Version getApiVersion() {
+      return SONARQUBE_RUNTIME_VERSION;
+    }
 
-    // technical debt
-    Map<String, String> remediationCosts = new HashMap<>();
-    remediationCosts.put(AvoidSQLRequestInLoop.RULE_KEY, "10min");
-    remediationCosts.put(AvoidFullSQLRequest.RULE_KEY, "20min");
-    repository.rules().forEach(rule -> {
-      rule.setType(RuleType.CODE_SMELL);
-      String debt = remediationCosts.get(rule.key());
+    @Override
+    public SonarProduct getProduct() {
+      return SonarProduct.SONARQUBE;
+    }
 
-      // TODO DDC : create support to use org.apache.commons.lang.StringUtils
-//      if (StringUtils.isBlank(debt)) {
-      if (debt == null || debt.trim().equals("")) {
-        // default debt to 5min for issue correction
-        rule.setDebtRemediationFunction(
-                rule.debtRemediationFunctions().constantPerIssue("5min"));
-      } else {
-        rule.setDebtRemediationFunction(
-                rule.debtRemediationFunctions().constantPerIssue(debt));
-      }
-    });
+    @Override
+    public SonarQubeSide getSonarQubeSide() {
+      return SonarQubeSide.SCANNER;
+    }
 
-    // HTML description
-    repository.rules().forEach(rule ->
-            rule.setHtmlDescription(loadResource(RESOURCE_BASE_PATH + rule.key() + ".html")));
+    @Override
+    public SonarEdition getEdition() {
+      return SonarEdition.COMMUNITY;
+    }
+  };
 
+  @Override
+  public void define(Context context) {
+    NewRepository repository = context.createRepository(REPOSITORY_KEY, LANGUAGE).setName(NAME);
+    RuleMetadataLoader ruleMetadataLoader = new RuleMetadataLoader(RESOURCE_BASE_PATH, SONARQUBE_RUNTIME);
+    ruleMetadataLoader.addRulesByAnnotatedClass(repository, (List<Class<?>>) (List) checkClasses());
     repository.done();
   }
 
@@ -97,21 +98,4 @@ public List<Class> checkClasses() {
             DetectUnoptimizedImageFormat.class
     );
   }
-
-  private String loadResource(String path) {
-    URL resource = getClass().getResource(path);
-    if (resource == null) {
-      throw new IllegalStateException("Resource not found: " + path);
-    }
-    ByteArrayOutputStream result = new ByteArrayOutputStream();
-    try (InputStream in = resource.openStream()) {
-      byte[] buffer = new byte[1024];
-      for (int len = in.read(buffer); len != -1; len = in.read(buffer)) {
-        result.write(buffer, 0, len);
-      }
-      return new String(result.toByteArray(), StandardCharsets.UTF_8);
-    } catch (IOException e) {
-      throw new IllegalStateException("Failed to read resource: " + path, e);
-    }
-  }
 }

python-plugin/src/test/java/fr/greencodeinitiative/python/PythonRuleRepositoryTest.java

@@ -21,6 +21,7 @@
 
 import static org.assertj.core.api.Assertions.assertThat;
 
+import io.ecocode.rules.python.PythonRulesSpecificationsRepository;
 import org.assertj.core.api.SoftAssertions;
 import org.junit.Before;
 import org.junit.Test;
@@ -37,12 +38,12 @@ public void init() {
     pythonRuleRepository = new PythonRuleRepository();
     context = new RulesDefinition.Context();
     pythonRuleRepository.define(context);
-    repository = context.repository(PythonRuleRepository.REPOSITORY_KEY);
+    repository = context.repository(PythonRulesSpecificationsRepository.REPOSITORY_KEY);
   }
 
   @Test
   public void test() {
-    assertThat(pythonRuleRepository.repositoryKey()).isEqualTo(PythonRuleRepository.REPOSITORY_KEY);
+    assertThat(pythonRuleRepository.repositoryKey()).isEqualTo(PythonRulesSpecificationsRepository.REPOSITORY_KEY);
     assertThat(context.repositories()).hasSize(1).extracting("key").containsExactly(pythonRuleRepository.repositoryKey());
     assertThat(context.repositories().get(0).rules()).hasSize(10);
     assertThat(pythonRuleRepository.checkClasses()).hasSize(10);