Changed using custom test for pylint to opencv sample

opencv · asenyaev · Jun 1, 2022 · Mar 22, 2022 · Mar 23, 2022 · Mar 23, 2022
commit b01f2cfe5d425533c3206846f282029f433826ee
diff --git a/.github/workflows/build_wheels_linux.yml b/.github/workflows/build_wheels_linux.yml
@@ -102,7 +102,7 @@ jobs:
    - name: Checkout
      uses: actions/checkout@v2
      with:
-        submodules: false
+        submodules: true
        fetch-depth: 0

    - name: Setup Environment variables

diff --git a/.github/workflows/build_wheels_macos.yml b/.github/workflows/build_wheels_macos.yml
@@ -137,6 +137,7 @@ jobs:
      CONFIG_PATH: travis_config.sh
      PLAT: x86_64
      OPENCV_TEST_DATA_PATH: ${{ github.workspace }}/opencv_extra/testdata
+      PYLINT_TEST_FILE: ${{ github.workspace }}/opencv/samples/python/squares.py

    steps:
    - name: Checkout
@@ -174,7 +175,8 @@ jobs:
    - name: Pylint test
      run: |
        python -m pip install pylint==2.12.2
-        python -m pylint ${{ github.workspace }}/tests/pylint.py --extension-pkg-whitelist=cv2 --disable=missing-docstring
+        cd ${{ github.workspace }}/tests
+        python -m pylint $PYLINT_TEST_FILE

  test_release_opencv_python:
    if: github.event_name == 'release' && github.event.release.prerelease

diff --git a/.github/workflows/build_wheels_macos_m1.yml b/.github/workflows/build_wheels_macos_m1.yml
@@ -81,6 +81,7 @@ jobs:

    env:
      OPENCV_TEST_DATA_PATH: ${{ github.workspace }}/opencv_extra/testdata
+      PYLINT_TEST_FILE: ${{ github.workspace }}/opencv/samples/python/squares.py

    steps:
    - name: Checkout
@@ -109,7 +110,8 @@ jobs:
    - name: Pylint test
      run: |
        arch -arm64 python${{ matrix.python-version }} -m pip install pylint==2.12.2
-        arch -arm64 python${{ matrix.python-version }} -m pylint ${{ github.workspace }}/tests/pylint.py --extension-pkg-whitelist=cv2 --disable=missing-docstring
+        cd ${{ github.workspace }}/tests
+        arch -arm64 python${{ matrix.python-version }} -m pylint $PYLINT_TEST_FILE

  test_release_opencv_python:
    if: github.event_name == 'release' && github.event.release.prerelease

diff --git a/.github/workflows/build_wheels_windows.yml b/.github/workflows/build_wheels_windows.yml
@@ -96,6 +96,7 @@ jobs:
    env:
      ACTIONS_ALLOW_UNSECURE_COMMANDS: true
      OPENCV_TEST_DATA_PATH: ${{ github.workspace }}\opencv_extra\testdata
+      PYLINT_TEST_FILE: ${{ github.workspace }}\opencv\samples\python\squares.py

    steps:
    - name: Checkout
@@ -133,7 +134,8 @@ jobs:
    - name: Pylint test
      run: |
        python -m pip install pylint==2.12.2
-        python -m pylint ${{ github.workspace }}\tests\pylint.py --extension-pkg-whitelist=cv2 --disable=missing-docstring
+        cd ${{ github.workspace }}\tests
+        python -m pylint $PYLINT_TEST_FILE
      shell: cmd

  test_release_opencv_python:

diff --git a/tests/pylint.py b/tests/pylint.py
diff --git a/tests/pylintrc b/tests/pylintrc
@@ -0,0 +1,12 @@
+# Source: opencv/platforms/scripts/pylintrc
+
+[MESSAGES CONTROL]
+
+# Disable all to choose the Tests one by one
+disable=all
+
+# Tests
+enable=bad-indentation,       # Used when an unexpected number of indentation’s tabulations or spaces has been found.
+       mixed-indentation,     # Used when there are some mixed tabs and spaces in a module.
+       unnecessary-semicolon, # Used when a statement is ended by a semi-colon (”;”), which isn’t necessary.
+       unused-variable        # Used when a variable is defined but not used. (Use _var to ignore var).
diff --git a/travis_config.sh b/travis_config.sh
@@ -169,7 +169,8 @@ function pylint_test {
    echo "Starting Pylint tests..."

    $PYTHON -m pip install pylint==2.12.2
-    $PYTHON -m pylint /io/tests/pylint.py --extension-pkg-whitelist=cv2 --disable=missing-docstring
+    cd /io/tests
+    $PYTHON -m pylint /io/opencv/samples/python/squares.py
 }

 export PS4='+(${BASH_SOURCE}:${LINENO}): ${FUNCNAME[0]:+${FUNCNAME[0]}(): }'