Update dependency ruff to ^0.7.1 (#2633)

Co-authored-by: Kyle Finley <kyle@finley.sh>

pyproject.toml

@@ -76,7 +76,7 @@ sphinxcontrib-programoutput = "^0.17"
 
 [tool.poetry.group.lint.dependencies]
 black = "^24.10.0"
-ruff = "^0.6.9"
+ruff = "^0.7.1"
 
 [tool.poetry.group.test.dependencies]
 coverage = {extras = ["toml"], version = "^7.6.4"}

tests/conftest.py

@@ -50,20 +50,20 @@ def pytest_addoption(parser: Parser) -> None:
     )
 
 
-@pytest.fixture()
+@pytest.fixture
 def cli_runner(request: SubRequest) -> CliRunner:
     """Initialize instance of `click.testing.CliRunner`."""
     return cli_runner_factory(request)
 
 
-@pytest.fixture()
+@pytest.fixture
 def cli_runner_isolated(cli_runner: CliRunner) -> Generator[CliRunner, None, None]:
     """Initialize instance of `click.testing.CliRunner` with `isolate_filesystem()` called."""
     with cli_runner.isolated_filesystem():
         yield cli_runner
 
 
-@pytest.fixture()
+@pytest.fixture
 def cd_tmp_path(tmp_path: Path) -> Iterator[Path]:
     """Change directory to a temporary path.
 
@@ -79,7 +79,7 @@ def cd_tmp_path(tmp_path: Path) -> Iterator[Path]:
         os.chdir(prev_dir)
 
 
-@pytest.fixture()
+@pytest.fixture
 def root_dir() -> Path:
     """Return a path object to the root directory."""
     return Path(__file__).parent.parent

tests/functional/terraform/conftest.py

@@ -20,7 +20,7 @@ def fixture_dir() -> Path:
     return Path(__file__).parent / "fixtures"
 
 
-@pytest.fixture()
+@pytest.fixture
 def local_backend(fixture_dir: Path, request: SubRequest) -> Generator[Path, None, None]:
     """Copy local_backend.tf into the test directory."""
     file_name = "local_backend.tf"
@@ -31,7 +31,7 @@ def local_backend(fixture_dir: Path, request: SubRequest) -> Generator[Path, Non
     new_file.unlink()
 
 
-@pytest.fixture()
+@pytest.fixture
 def no_backend(fixture_dir: Path, request: SubRequest) -> Generator[Path, None, None]:
     """Copy no_backend.tf into the test directory."""
     file_name = "no_backend.tf"
@@ -42,7 +42,7 @@ def no_backend(fixture_dir: Path, request: SubRequest) -> Generator[Path, None,
     new_file.unlink()
 
 
-@pytest.fixture()
+@pytest.fixture
 def s3_backend(fixture_dir: Path, request: SubRequest) -> Generator[Path, None, None]:
     """Copy s3_backend.tf into the test directory."""
     file_name = "s3_backend.tf"

tests/functional/terraform/test_backend_local_2_s3/test_runner.py

@@ -46,15 +46,15 @@ def tf_version(request: SubRequest) -> Iterator[str]:
     file_path.unlink(missing_ok=True)
 
 
-@pytest.fixture()
+@pytest.fixture
 def deploy_local_backend_result(
     cli_runner: CliRunner, local_backend: Path  # noqa: ARG001
 ) -> Result:
     """Execute `runway deploy` with `runway destroy` as a cleanup step."""
     return cli_runner.invoke(cli, ["deploy", "--tag", "local"], env={"CI": "1"})
 
 
-@pytest.fixture()
+@pytest.fixture
 def deploy_s3_backend_result(
     cli_runner: CliRunner, s3_backend: Path  # noqa: ARG001
 ) -> Iterator[Result]:

tests/functional/terraform/test_backend_no_2_local/test_runner.py

@@ -37,7 +37,7 @@ def tf_version(request: SubRequest) -> Generator[str, None, None]:
     file_path.unlink(missing_ok=True)
 
 
-@pytest.fixture()
+@pytest.fixture
 def deploy_local_backend_result(
     cli_runner: CliRunner,
     local_backend: Path,  # noqa: ARG001
@@ -53,7 +53,7 @@ def deploy_local_backend_result(
     (CURRENT_DIR / ".terraform.lock.hcl").unlink(missing_ok=True)
 
 
-@pytest.fixture()
+@pytest.fixture
 def deploy_no_backend_result(
     cli_runner: CliRunner,
     no_backend: Path,  # noqa: ARG001

tests/functional/terraform/test_base/test_runner.py

@@ -42,7 +42,7 @@ def tf_version(request: SubRequest) -> Generator[str, None, None]:
     file_path.unlink(missing_ok=True)
 
 
-@pytest.fixture()
+@pytest.fixture
 def deploy_result(
     cli_runner: CliRunner, no_backend: Path  # noqa: ARG001
 ) -> Generator[Result, None, None]:

tests/integration/conftest.py

@@ -25,13 +25,13 @@ def pytest_ignore_collect(path: Any, config: Config) -> bool:  # noqa: ARG001
     return not (config.option.integration or config.option.integration_only)
 
 
-@pytest.fixture()
+@pytest.fixture
 def configs() -> Path:
     """Path to Runway config fixtures."""
     return TEST_ROOT.parent / "fixtures" / "configs"
 
 
-@pytest.fixture()
+@pytest.fixture
 def cp_config(configs: Path) -> Callable[[str, Path], Path]:
     """Copy a config file."""
 

tests/unit/cfngin/actions/conftest.py

@@ -16,7 +16,7 @@
     from pytest_mock import MockerFixture
 
 
-@pytest.fixture()
+@pytest.fixture
 def provider_get_stack(mocker: MockerFixture) -> MagicMock:
     """Patches ``runway.cfngin.providers.aws.default.Provider.get_stack``."""
     return_value: StackTypeDef = {

tests/unit/cfngin/conftest.py

@@ -13,13 +13,13 @@ def cfngin_fixtures() -> Path:
     return Path(__file__).parent / "fixtures"
 
 
-@pytest.fixture()
+@pytest.fixture
 def empty_dag() -> DAG:
     """Create an empty DAG."""
     return DAG()
 
 
-@pytest.fixture()
+@pytest.fixture
 def basic_dag() -> DAG:
     """Create a basic DAG."""
     dag = DAG()

tests/unit/cfngin/hooks/awslambda/test__python_hooks.py

@@ -19,7 +19,7 @@
 MODULE = "runway.cfngin.hooks.awslambda._python_hooks"
 
 
-@pytest.fixture()
+@pytest.fixture
 def args(tmp_path: Path) -> PythonHookArgs:
     """Fixture for creating default function args."""
     return PythonHookArgs(

tests/unit/cfngin/hooks/awslambda/test_deployment_package.py

@@ -47,7 +47,7 @@
 ProjectTypeAlias = Project[AwsLambdaHookArgs]
 
 
-@pytest.fixture()
+@pytest.fixture
 def project(cfngin_context: CfnginContext, tmp_path: Path) -> ProjectTypeAlias:
     """Mock project object."""
     args = AwsLambdaHookArgs(

tests/unit/cfngin/hooks/docker/image/test__build.py

@@ -34,7 +34,7 @@
 MODULE = "runway.cfngin.hooks.docker.image._build"
 
 
-@pytest.fixture()
+@pytest.fixture
 def tmp_dockerfile(cd_tmp_path: Path) -> Path:
     """Create temporary Dockerfile."""
     dockerfile = cd_tmp_path / "Dockerfile"

tests/unit/cfngin/hooks/docker/test_data_models.py

@@ -29,7 +29,7 @@
 }
 
 
-@pytest.fixture()
+@pytest.fixture
 def mock_image() -> MagicMock:
     """Return a mock docker.models.images.Image."""
     return MagicMock(spec=Image, **MOCK_IMAGE_PROPS)

tests/unit/cfngin/hooks/ssm/conftest.py

@@ -13,15 +13,15 @@
     from ....factories import MockCfnginContext
 
 
-@pytest.fixture()
+@pytest.fixture
 def ssm_client(
     cfngin_context: MockCfnginContext, ssm_stubber: Stubber  # noqa: ARG001
 ) -> SSMClient:
     """Create SSM client."""
     return cfngin_context.get_session().client("ssm")
 
 
-@pytest.fixture()
+@pytest.fixture
 def ssm_stubber(cfngin_context: MockCfnginContext) -> Stubber:
     """Create SSM stubber."""
     return cfngin_context.add_stubber("ssm")

tests/unit/cfngin/hooks/test_keypair.py

@@ -47,7 +47,7 @@ def ssh_key(cfngin_fixtures: Path) -> SSHKey:
     )
 
 
-@pytest.fixture()
+@pytest.fixture
 def context() -> CfnginContext:
     """Mock context."""
     return mock_context(namespace="fake")

tests/unit/cfngin/lookups/handlers/test_awslambda.py

@@ -22,7 +22,7 @@
 QUERY = "test::foo=bar"
 
 
-@pytest.fixture()
+@pytest.fixture
 def hook_data() -> AwsLambdaHookDeployResponse:
     """Fixture for hook response data."""
     return AwsLambdaHookDeployResponse(

tests/unit/cfngin/test_cfngin.py

@@ -31,7 +31,7 @@ def copy_basic_fixtures(cfngin_fixtures: Path, tmp_path: Path) -> None:
     copy_fixture(src=cfngin_fixtures / "configs" / "basic.yml", dest=tmp_path / "basic.yml")
 
 
-@pytest.fixture()
+@pytest.fixture
 def patch_safehaven(mocker: MockerFixture) -> Mock:
     """Patch SafeHaven."""
     mock_haven = mocker.patch("runway.cfngin.cfngin.SafeHaven")

tests/unit/conftest.py

@@ -89,19 +89,19 @@ def fx_config() -> YamlLoader:
     )
 
 
-@pytest.fixture()
+@pytest.fixture
 def fx_deployments() -> YamlLoaderDeployment:
     """Return YAML loader for deployment fixtures."""
     return YamlLoaderDeployment(TEST_ROOT / "fixtures" / "deployments")
 
 
-@pytest.fixture()
+@pytest.fixture
 def mock_docker_client() -> DockerClient:
     """Create a docker client with mock API backend."""
     return make_fake_client()
 
 
-@pytest.fixture()
+@pytest.fixture
 def tempfile_temporary_directory(mocker: MockerFixture, tmp_path: Path) -> MagicMock:
     """Mock tempfile.TemporaryDirectory."""
     return mocker.patch(
@@ -127,43 +127,43 @@ def yaml_fixtures(request: pytest.FixtureRequest, fixture_dir: Path) -> dict[str
     return result
 
 
-@pytest.fixture()
+@pytest.fixture
 def deploy_environment(tmp_path: Path) -> DeployEnvironment:
     """Create a deploy environment that can be used for testing."""
     return DeployEnvironment(explicit_name="test", root_dir=tmp_path)
 
 
-@pytest.fixture()
+@pytest.fixture
 def cfngin_context(runway_context: MockRunwayContext) -> MockCfnginContext:
     """Create a mock CFNgin context object."""
     return MockCfnginContext(deploy_environment=runway_context.env, parameters={})
 
 
-@pytest.fixture()
+@pytest.fixture
 def mock_sleep(mocker: MockerFixture) -> Mock:
     """Patch built-in ``time.sleep``."""
     return mocker.patch("time.sleep", return_value=None)
 
 
-@pytest.fixture()
+@pytest.fixture
 def platform_darwin(mocker: MockerFixture) -> None:
     """Patch platform.system to always return "Darwin"."""
     mocker.patch("platform.system", return_value="Darwin")
 
 
-@pytest.fixture()
+@pytest.fixture
 def platform_linux(mocker: MockerFixture) -> None:
     """Patch platform.system to always return "Linux"."""
     mocker.patch("platform.system", return_value="Linux")
 
 
-@pytest.fixture()
+@pytest.fixture
 def platform_windows(mocker: MockerFixture) -> None:
     """Patch platform.system to always return "Windows"."""
     mocker.patch("platform.system", return_value="Windows")
 
 
-@pytest.fixture()
+@pytest.fixture
 def patch_runway_config(
     request: pytest.FixtureRequest, monkeypatch: pytest.MonkeyPatch, runway_config: MockRunwayConfig
 ) -> MockRunwayConfig:
@@ -174,13 +174,13 @@ def patch_runway_config(
     return runway_config
 
 
-@pytest.fixture()
+@pytest.fixture
 def runway_config() -> MockRunwayConfig:
     """Create a mock runway config object."""
     return MockRunwayConfig()
 
 
-@pytest.fixture()
+@pytest.fixture
 def runway_context(request: pytest.FixtureRequest, tmp_path: Path) -> MockRunwayContext:
     """Create a mock Runway context object."""
     env_vars = {

tests/unit/context/test_base.py

@@ -29,15 +29,15 @@
 }
 
 
-@pytest.fixture()
+@pytest.fixture
 def mock_boto3_session(mocker: MockerFixture) -> MagicMock:
     """Mock boto3.Session."""
     mock_session = MagicMock(autospec=boto3.Session)
     mocker.patch(f"{MODULE}.boto3", Session=mock_session)
     return mock_session
 
 
-@pytest.fixture()
+@pytest.fixture
 def mock_sso_botocore_session(mocker: MockerFixture) -> MagicMock:
     """Mock runway.aws_sso_botocore.session.Session."""
     return mocker.patch(f"{MODULE}.Session")