Reduce pylint python versions, add sonar-scanner download and install (#2564)

* Reduce pylint python versions, add sonar-scanner download and install

* Change installation way of sonar-scanner

* Fix installation way of sonar-scanner

* Change test-ci to not run sonar and create new sonar task

Author: HelioGuilherme66

.github/workflows/build.yml

@@ -37,8 +37,18 @@ jobs:
         run: pip install .
       - name: Run RIDE unit-tests
         run: invoke test-ci
-      - name: SonarCloud Scan
-        uses: SonarSource/sonarcloud-github-action@master
+      - name: Analyze with SonarCloud
+        uses: sonarsource/sonarcloud-github-action@master
+        with:
+          projectBaseDir: ${{ github.workspace }}
+          args: >
+            -Dsonar.organization=helioguilherm66
+            -Dsonar.projectKey=HelioGuilherme66_RIDE
+            -Dsonar.python.coverage.reportPaths=.coverage-reports/coverage.xml
+            -Dsonar.sources=src/
+            -Dsonar.test.exclusions=utest/**
+            -Dsonar.tests=utest/
+            -Dsonar.verbose=true
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}  # Needed to get PR information, if any
           SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}

.github/workflows/pylint.yml

@@ -7,7 +7,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: ["3.8", "3.9", "3.10"]
+        python-version: ["3.10"]
     steps:
     - uses: actions/checkout@v2
     - name: Set up Python ${{ matrix.python-version }}
@@ -20,4 +20,4 @@ jobs:
         pip install pylint
     - name: Analysing the code with pylint
       run: |
-        pylint src  # $(git ls-files '*.py')
+        pylint --max-line-length 120 --ignore lib/robot src  # $(git ls-files '*.py')

.pylintrc

@@ -388,7 +388,7 @@ indent-after-paren=4
 indent-string='    '
 
 # Maximum number of characters on a single line.
-max-line-length=100
+max-line-length=120
 
 # Maximum number of lines in a module.
 max-module-lines=1000

tasks.py

@@ -358,8 +358,8 @@ def tags_test(ctx):
         pass
 
 @task
-def test_ci(ctx, test_filter=''):
-    """Run unit tests and coverage"""
+def sonar(ctx, test_filter=''):
+    """Run unit tests and coverage and send to SonarCloud"""
 
     _set_development_path()
 
@@ -378,6 +378,25 @@ def test_ci(ctx, test_filter=''):
     finally:
         pass
 
+@task
+def test_ci(ctx, test_filter=''):
+    """Run unit tests and coverage"""
+    
+    _set_development_path()
+
+    try:
+        import subprocess
+        c = subprocess.Popen(["coverage", "run" , "-m", "pytest", "--cov-config=.coveragerc", "-k test_", "-v", TEST_DIR])
+        c.communicate('')
+        r = subprocess.Popen(["coverage", "report"])
+        r.communicate('')
+        x = subprocess.Popen(["coverage", "xml"])
+        x.communicate('')
+        h = subprocess.Popen(["coverage", "html"])
+        h.communicate('')
+    finally:
+        pass
+
 
 # Helper functions