From 492c3a12894d5967bb84957eb05ecd6a366cba19 Mon Sep 17 00:00:00 2001 From: "deepsource-autofix[bot]" <62050782+deepsource-autofix[bot]@users.noreply.github.com> Date: Tue, 16 Jan 2024 05:06:42 +0000 Subject: [PATCH 1/2] refactor: remove unused imports An object has been imported but is not used anywhere in the file. It should either be used or the import should be removed. --- codeinsight_sdk/handler.py | 3 --- tests/test_experimental.py | 1 - tests/test_handlers.py | 2 -- 3 files changed, 6 deletions(-) diff --git a/codeinsight_sdk/handler.py b/codeinsight_sdk/handler.py index 8a0b5b3..34174a1 100644 --- a/codeinsight_sdk/handler.py +++ b/codeinsight_sdk/handler.py @@ -1,9 +1,6 @@ import abc from typing import List -from .models import Project, ProjectInventory, ProjectInventoryItem, Report -from .exceptions import CodeInsightError - class Handler(abc.ABC): def __init__(self, client): self.client = client diff --git a/tests/test_experimental.py b/tests/test_experimental.py index fe0e0ea..8d94584 100644 --- a/tests/test_experimental.py +++ b/tests/test_experimental.py @@ -5,7 +5,6 @@ from codeinsight_sdk import CodeInsightClient -from codeinsight_sdk.exceptions import CodeInsightError logger = logging.getLogger(__name__) diff --git a/tests/test_handlers.py b/tests/test_handlers.py index 6bc3bdd..98e72fc 100644 --- a/tests/test_handlers.py +++ b/tests/test_handlers.py @@ -1,8 +1,6 @@ import pytest from codeinsight_sdk import CodeInsightClient -from codeinsight_sdk.handlers import ProjectHandler, ReportHandler -from codeinsight_sdk.models import Project, Report class TestHandlers(object): From 265110c577364292e23c50a4122a7fbce5adc7de Mon Sep 17 00:00:00 2001 From: "deepsource-autofix[bot]" <62050782+deepsource-autofix[bot]@users.noreply.github.com> Date: Tue, 16 Jan 2024 05:06:53 +0000 Subject: [PATCH 2/2] style: format code with Black This commit fixes the style issues introduced in 492c3a1 according to the output from Black. Details: https://github.com/zkarpinski/codeinsight-sdk-python/pull/18 --- codeinsight_sdk/handler.py | 2 +- tests/test_experimental.py | 27 +++++++++++++++++++-------- tests/test_handlers.py | 2 +- 3 files changed, 21 insertions(+), 10 deletions(-) diff --git a/codeinsight_sdk/handler.py b/codeinsight_sdk/handler.py index 34174a1..302ebff 100644 --- a/codeinsight_sdk/handler.py +++ b/codeinsight_sdk/handler.py @@ -1,6 +1,7 @@ import abc from typing import List + class Handler(abc.ABC): def __init__(self, client): self.client = client @@ -9,4 +10,3 @@ def __init__(self, client): @abc.abstractmethod def get(self): pass - diff --git a/tests/test_experimental.py b/tests/test_experimental.py index 8d94584..551beff 100644 --- a/tests/test_experimental.py +++ b/tests/test_experimental.py @@ -12,14 +12,15 @@ TEST_URL = "https://api.revenera.com" TEST_API_TOKEN = "your_api_token" + class TestExperimental: @pytest.fixture def client(self): return CodeInsightClient(TEST_URL, TEST_API_TOKEN, experimental=True) - + def test_experimental_enabled(self, client): assert client.experimental_enabled == True - + def test_get_project_vulnerabilities(self, client): project_id = 1 total_pages = 4 @@ -85,11 +86,21 @@ def test_get_project_vulnerabilities(self, client): "vulnerabilityCvssV3Severity":"CRITICAL"} ] } """ with requests_mock.Mocker() as m: - m.get(f"{TEST_URL}/codeinsight/api/projects/{project_id}/inventorySummary", - text=fake_response_json, headers=response_header) - m.get(f"{TEST_URL}/codeinsight/api/inventories/12346/vulnerabilities", - text=mock_resp_vuln, headers=response_header) - vulnerable_items = client.experimental.get_project_vulnerabilities(project_id) + m.get( + f"{TEST_URL}/codeinsight/api/projects/{project_id}/inventorySummary", + text=fake_response_json, + headers=response_header, + ) + m.get( + f"{TEST_URL}/codeinsight/api/inventories/12346/vulnerabilities", + text=mock_resp_vuln, + headers=response_header, + ) + vulnerable_items = client.experimental.get_project_vulnerabilities( + project_id + ) assert len(vulnerable_items) > 0 assert vulnerable_items[0].vulnerabilities is not None - assert vulnerable_items[0].vulnerabilities[1].vulnerabilityName == "CVE-987-65432" + assert ( + vulnerable_items[0].vulnerabilities[1].vulnerabilityName == "CVE-987-65432" + ) diff --git a/tests/test_handlers.py b/tests/test_handlers.py index 98e72fc..b37081a 100644 --- a/tests/test_handlers.py +++ b/tests/test_handlers.py @@ -6,4 +6,4 @@ class TestHandlers(object): @pytest.fixture def client(self): - return CodeInsightClient("","") + return CodeInsightClient("", "")