Skip to content

Commit

Permalink
removed unused fixture
Browse files Browse the repository at this point in the history
  • Loading branch information
kozlov721 committed Dec 4, 2024
1 parent 72edf2f commit a4efff8
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 16 deletions.
18 changes: 3 additions & 15 deletions tests/test_data/conftest.py
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
import pytest
from pytest import Metafunc, Parser

from luxonis_ml.data import BucketStorage


def pytest_addoption(parser):
def pytest_addoption(parser: Parser):
parser.addoption(
"--only-local",
action="store_true",
Expand All @@ -12,19 +12,7 @@ def pytest_addoption(parser):
)


@pytest.fixture
def only_local(request):
return request.config.getoption("--only-local")


@pytest.fixture
def storage_options(only_local: bool):
if only_local:
return [BucketStorage.LOCAL]
return [BucketStorage.LOCAL, BucketStorage.GCS, BucketStorage.S3]


def pytest_generate_tests(metafunc):
def pytest_generate_tests(metafunc: Metafunc):
if "bucket_storage" in metafunc.fixturenames:
only_local = metafunc.config.getoption("--only-local")
storage_options = (
Expand Down
4 changes: 3 additions & 1 deletion tests/test_utils/test_filesystem.py
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,9 @@ def get_os_python_specific_url(


@pytest.fixture
def fs(request, python_version: str, platform_name: str):
def fs(
request: pytest.FixtureRequest, python_version: str, platform_name: str
):
url_path = get_os_python_specific_url(
request.param, platform_name, python_version
)
Expand Down

0 comments on commit a4efff8

Please sign in to comment.