Python: ecoCode plugin with SonarQube, no code-smell detection

[matthieumarrast]

I installed the plugin in my local SonarQube installation (v9.9).
I activated the "eco-conception" rules in my Quality Profile:

This Quality Profile is well "applied" to my project.

But the rules are no detecting code smells on my Python code :

Example in screenshot above for rules:

• S74: Don't use the query SELECT * FROM
• S72 : Avoid SQL request in loop
  I also tried, S34 (Avoid using try-catch-finally statement) without "success".

[matthieumarrast]

Way to reproduce:

# 1. run server
$ docker run -d --name sonarqube -e SONAR_ES_BOOTSTRAP_CHECKS_DISABLE=true -p 9000:9000 sonarqube:latest
# 2. go to http://localhost:9000, update admin password and create a project
# 3. add plugin
$ wget https://github.com/green-code-initiative/ecoCode/releases/download/0.2.2/ecocode-python-plugin-0.2.2.jar
$ docker cp ecocode-python-plugin-0.2.2.jar sonarqube:/opt/sonarqube/extensions/plugins
$ docker restart sonarqube
# 4. create a new Quality Profile with "eco-conception" rules activated, and apply on project
# 5. run scanner
$ sudo docker run --network=host --rm -e SONAR_HOST_URL="http://localhost:9000" -e SONAR_SCANNER_OPTS="-Dsonar.projectKey=<project-key>" -e SONAR_LOGIN="<sonar-token>" -v "<project_repository>:/usr/src" sonarsource/sonar-scanner-cli

[dedece35]

Hi @matthieumarrast,
thank you for your issue.

The best thing to do to check this issue is to create a test project like mentionned here #65 (in progress for java language).
I will work on it soon unless if you want to create it ;p

[dedece35]

Hi @matthieumarrast,
I'm working on this issue with the creation of a python test-project like mentionned above.

[dedece35]

Hi @matthieumarrast,
you're right, there was a problem with python plugin.
my PR #77 and the new test project for python (https://github.com/green-code-initiative/ecoCode-python-test-project) was modified to check now the good processing.

[matthieumarrast]

ok as soon as the plugin will be released with your PR #77 I'm gonna test it

[dedece35]

Hi @matthieumarrast,
PR #77 merged on main branch.
the correction is available on main branch (SNAPSHOT version) or in the next release version.

[dedece35]

Hi @matthieumarrast,
all is ok now.
please, test it with out new real test project above.

[matthieumarrast]

Hi @dedece35

Thank you it's OK now.