Skip to content

Commit

Permalink
[pre-commit.ci] auto fixes from pre-commit.com hooks
Browse files Browse the repository at this point in the history
for more information, see https://pre-commit.ci
  • Loading branch information
pre-commit-ci[bot] committed Sep 9, 2024
1 parent 0db310a commit 4ac1c0a
Show file tree
Hide file tree
Showing 7 changed files with 20 additions and 18 deletions.
3 changes: 2 additions & 1 deletion tests/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
import pytest
from click.testing import CliRunner
from pytask import storage

from pytask_stata.config import STATA_COMMANDS

needs_stata = pytest.mark.skipif(
Expand Down Expand Up @@ -78,6 +79,6 @@ def invoke(self, *args, **kwargs):
return super().invoke(*args, **kwargs)


@pytest.fixture()
@pytest.fixture
def runner():
return CustomCliRunner()
5 changes: 3 additions & 2 deletions tests/test_collect.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,12 @@

import pytest
from pytask import Mark

from pytask_stata.collect import _parse_stata_mark
from pytask_stata.collect import stata


@pytest.mark.unit()
@pytest.mark.unit
@pytest.mark.parametrize(
("args", "kwargs", "expectation", "expected"),
[
Expand All @@ -32,7 +33,7 @@ def test_stata(args, kwargs, expectation, expected):
assert options == expected


@pytest.mark.unit()
@pytest.mark.unit
@pytest.mark.parametrize(
("mark", "expectation", "expected"),
[
Expand Down
2 changes: 1 addition & 1 deletion tests/test_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
from pytask import build


@pytest.mark.end_to_end()
@pytest.mark.end_to_end
def test_marker_is_configured(tmp_path):
session = build(paths=tmp_path)

Expand Down
18 changes: 9 additions & 9 deletions tests/test_execute.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,13 +12,13 @@
from pytask import Task
from pytask import build
from pytask import cli

from pytask_stata.config import STATA_COMMANDS
from pytask_stata.execute import pytask_execute_task_setup

from tests.conftest import needs_stata


@pytest.mark.unit()
@pytest.mark.unit
@pytest.mark.parametrize(
("stata", "expectation"),
[(executable, does_not_raise()) for executable in STATA_COMMANDS]
Expand All @@ -42,7 +42,7 @@ def test_pytask_execute_task_setup_raise_error(stata, platform, expectation):


@needs_stata
@pytest.mark.end_to_end()
@pytest.mark.end_to_end
def test_run_do_file(runner, tmp_path):
task_source = """
import pytask
Expand Down Expand Up @@ -72,7 +72,7 @@ def task_run_do_file():


@needs_stata
@pytest.mark.end_to_end()
@pytest.mark.end_to_end
def test_run_do_file_w_task_decorator(runner, tmp_path):
task_source = """
import pytask
Expand Down Expand Up @@ -102,7 +102,7 @@ def run_do_file():
assert tmp_path.joinpath("script.log").exists()


@pytest.mark.end_to_end()
@pytest.mark.end_to_end
def test_raise_error_if_stata_is_not_found(tmp_path, monkeypatch):
task_source = """
from pytask import mark, task
Expand All @@ -128,7 +128,7 @@ def task_run_do_file():


@needs_stata
@pytest.mark.end_to_end()
@pytest.mark.end_to_end
def test_run_do_file_w_wrong_cmd_option(runner, tmp_path):
"""Apparently, Stata simply discards wrong cmd options."""
task_source = """
Expand All @@ -154,7 +154,7 @@ def task_run_do_file():


@needs_stata
@pytest.mark.end_to_end()
@pytest.mark.end_to_end
def test_run_do_file_by_passing_path(runner, tmp_path):
"""Replicates example under "Command Line Arguments" in Readme."""
task_source = """
Expand All @@ -180,7 +180,7 @@ def task_run_do_file():


@needs_stata
@pytest.mark.end_to_end()
@pytest.mark.end_to_end
def test_run_do_file_fails_with_multiple_marks(runner, tmp_path):
task_source = """
import pytask
Expand All @@ -201,7 +201,7 @@ def task_run_do_file():


@needs_stata
@pytest.mark.end_to_end()
@pytest.mark.end_to_end
def test_with_task_without_path(runner, tmp_path):
task_source = """
import pytask
Expand Down
2 changes: 1 addition & 1 deletion tests/test_normal_execution_w_plugin.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
from pytask import cli


@pytest.mark.end_to_end()
@pytest.mark.end_to_end
@pytest.mark.parametrize(
"dependencies",
[(), ("in.txt",), ("in_1.txt", "in_2.txt")],
Expand Down
4 changes: 2 additions & 2 deletions tests/test_parallel.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@


@needs_stata
@pytest.mark.end_to_end()
@pytest.mark.end_to_end
def test_parallel_parametrization_over_source_files_w_loop(runner, tmp_path):
source = """
import pytask
Expand Down Expand Up @@ -56,7 +56,7 @@ def task_execute_do_file():


@needs_stata
@pytest.mark.end_to_end()
@pytest.mark.end_to_end
def test_parallel_parametrization_over_source_file_w_loop(runner, tmp_path):
source = """
import pytask
Expand Down
4 changes: 2 additions & 2 deletions tests/test_parametrize.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@


@needs_stata
@pytest.mark.end_to_end()
@pytest.mark.end_to_end
def test_parametrized_execution_of_do_file_w_loop(runner, tmp_path):
source = """
import pytask
Expand Down Expand Up @@ -41,7 +41,7 @@ def task_execute_do_file():


@needs_stata
@pytest.mark.end_to_end()
@pytest.mark.end_to_end
def test_parametrize_command_line_options_w_loop(runner, tmp_path):
task_source = """
import pytask
Expand Down

0 comments on commit 4ac1c0a

Please sign in to comment.