Change flake8 github action (#248)

dd-jy · web-flow · commit d8e975540ed5 · 2025-03-25T13:43:28.000+09:00
Signed-off-by: jiyeong.seok &lt;jiyeong.seok@lge.com&gt;
diff --git a/.github/workflows/pull-request.yml b/.github/workflows/pull-request.yml
@@ -30,6 +30,7 @@ jobs:
         sudo npm install -g license-checker
     - name: Run Tox
       run: |
+        tox -e release_flake8
         tox -e run_ubuntu
   build_windows:
     runs-on: windows-latest
diff --git a/src/fosslight_dependency/package_manager/Cargo.py b/src/fosslight_dependency/package_manager/Cargo.py
@@ -132,11 +132,11 @@ def get_matched_dependencies(match_id, resolve_node):
                 try:
                     match = re.findall(r'^.*#(\S*)@(\S*)', dep_pkg)
                     dependencies_list.append(f'{match[0][0]}({match[0][1]})')
-                except:
+                except Exception:
                     try:
                         match = re.findall(r'^(\S*)\s(\S*)\s', dep_pkg)
                         dependencies_list.append(f'{match[0][0]}({match[0][1]})')
-                    except:
+                    except Exception:
                         logger.info(f'cannot find name and version for dependencies: {match_id}')
                         pass
             break
diff --git a/tests/pytest/package_manager/test_android.py b/tests/pytest/package_manager/test_android.py
@@ -6,7 +6,7 @@
 import pytest
 import subprocess
 
-DIST_PATH = os.path.join(os.environ.get("TOX_PATH"), "dist", "cli.exe")
+DIST_PATH = os.path.join(os.environ.get("TOX_PATH", ""), "dist", "cli.exe")
 
 
 @pytest.mark.parametrize("input_path, output_path, extra_args", [
diff --git a/tests/pytest/package_manager/test_cargo.py b/tests/pytest/package_manager/test_cargo.py
@@ -6,7 +6,7 @@
 import pytest
 import subprocess
 
-DIST_PATH = os.path.join(os.environ.get("TOX_PATH"), "dist", "cli.exe")
+DIST_PATH = os.path.join(os.environ.get("TOX_PATH", ""), "dist", "cli.exe")
 
 
 @pytest.mark.parametrize("input_path, output_path", [
diff --git a/tests/pytest/package_manager/test_gradle.py b/tests/pytest/package_manager/test_gradle.py
@@ -6,7 +6,7 @@
 import pytest
 import subprocess
 
-DIST_PATH = os.path.join(os.environ.get("TOX_PATH"), "dist", "cli.exe")
+DIST_PATH = os.path.join(os.environ.get("TOX_PATH", ""), "dist", "cli.exe")
 
 
 @pytest.mark.parametrize("input_path, output_path", [
diff --git a/tests/pytest/package_manager/test_maven.py b/tests/pytest/package_manager/test_maven.py
@@ -6,7 +6,7 @@
 import pytest
 import subprocess
 
-DIST_PATH = os.path.join(os.environ.get("TOX_PATH"), "dist", "cli.exe")
+DIST_PATH = os.path.join(os.environ.get("TOX_PATH", ""), "dist", "cli.exe")
 
 
 @pytest.mark.parametrize("input_path, output_path", [
diff --git a/tests/pytest/package_manager/test_mod.py b/tests/pytest/package_manager/test_mod.py
@@ -7,7 +7,7 @@
 import subprocess
 
 
-DIST_PATH = os.path.join(os.environ.get("TOX_PATH"), "dist", "cli.exe")
+DIST_PATH = os.path.join(os.environ.get("TOX_PATH", ""), "dist", "cli.exe")
 
 
 @pytest.mark.parametrize("input_path, output_path, extra_args", [
diff --git a/tests/pytest/package_manager/test_nuget.py b/tests/pytest/package_manager/test_nuget.py
@@ -11,7 +11,7 @@
     "fosslight_dependency -p tests/test_nuget2 -o tests/result/nuget2"
 ]
 
-DIST_PATH = os.path.join(os.environ.get("TOX_PATH"), "dist", "cli.exe")
+DIST_PATH = os.path.join(os.environ.get("TOX_PATH", ""), "dist", "cli.exe")
 
 
 @pytest.mark.parametrize("input_path, output_path", [
diff --git a/tests/pytest/package_manager/test_pub.py b/tests/pytest/package_manager/test_pub.py
@@ -6,7 +6,7 @@
 import pytest
 import subprocess
 
-DIST_PATH = os.path.join(os.environ.get("TOX_PATH"), "dist", "cli.exe")
+DIST_PATH = os.path.join(os.environ.get("TOX_PATH", ""), "dist", "cli.exe")
 
 
 @pytest.mark.parametrize("input_path, output_path", [
diff --git a/tests/pytest/package_manager/test_pypi.py b/tests/pytest/package_manager/test_pypi.py
@@ -6,7 +6,7 @@
 import pytest
 import subprocess
 
-DIST_PATH = os.path.join(os.environ.get("TOX_PATH"), "dist", "cli.exe")
+DIST_PATH = os.path.join(os.environ.get("TOX_PATH", ""), "dist", "cli.exe")
 
 
 @pytest.mark.parametrize("input_path, output_path, extra_args", [
diff --git a/tox.ini b/tox.ini
@@ -23,18 +23,15 @@ markers =
 [flake8]
 max-line-length = 130
 exclude = .tox/*
-ignore = E722, W503
 
 [testenv:test_run]
 deps =
   -r{toxinidir}/requirements-dev.txt
 commands =
   # Print Help Message
   fosslight_dependency -h
-  # Print Current Version
-  fosslight_dependency -v
   # Test for PEP8
-  pytest -v --flake8
+  pytest -v --flake8 src
 
 [testenv:run_ubuntu]
 deps =
@@ -49,4 +46,9 @@ commands =
   # Test for making excutable file
   pyinstaller --onefile cli.py -n cli --additional-hooks-dir=hooks --add-binary=LICENSE:LICENSES --add-binary=LICENSES\LicenseRef-3rd_party_licenses.txt:LICENSES --collect-datas fosslight_util --hidden-import=_cffi_backend
   pytest -m "windows"
-  pytest -v --flake8
+
+[testenv:release_flake8]
+deps =
+  -r{toxinidir}/requirements-dev.txt
+commands =
+  pytest -v --flake8 src

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@`
`11`	`11`	`"fosslight_dependency -p tests/test_nuget2 -o tests/result/nuget2"`
`12`	`12`	`]`
`13`	`13`
`14`		`-DIST_PATH = os.path.join(os.environ.get("TOX_PATH"), "dist", "cli.exe")`
	`14`	`+DIST_PATH = os.path.join(os.environ.get("TOX_PATH", ""), "dist", "cli.exe")`
`15`	`15`
`16`	`16`
`17`	`17`	`@pytest.mark.parametrize("input_path, output_path", [`