Skip to content

Commit 23e2a76

Browse files
committed
Fix type hints.
1 parent b61dc93 commit 23e2a76

File tree

2 files changed

+10
-4
lines changed

2 files changed

+10
-4
lines changed

tests/mcstas_io_test.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
# SPDX-License-Identifier: BSD-3-Clause
22
# Copyright (c) 2025 Scipp contributors (https://github.com/scipp)
3+
import pathlib
4+
35
import pytest
46
import scipp as sc
57

@@ -8,7 +10,7 @@
810

911

1012
@pytest.fixture(params=[get_small_mcstas])
11-
def mcstas_file_path(request: pytest.FixtureRequest) -> str:
13+
def mcstas_file_path(request: pytest.FixtureRequest) -> pathlib.Path:
1214
return request.param()
1315

1416

tests/workflow_test.py

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
# SPDX-License-Identifier: BSD-3-Clause
22
# Copyright (c) 2023 Scipp contributors (https://github.com/scipp)
3+
import pathlib
4+
35
import pandas as pd
46
import pytest
57
import sciline as sl
@@ -26,12 +28,12 @@
2628

2729

2830
@pytest.fixture(params=[get_small_mcstas])
29-
def mcstas_file_path(request: pytest.FixtureRequest) -> str:
31+
def mcstas_file_path(request: pytest.FixtureRequest) -> pathlib.Path:
3032
return request.param()
3133

3234

3335
@pytest.fixture
34-
def mcstas_workflow(mcstas_file_path: str) -> sl.Pipeline:
36+
def mcstas_workflow(mcstas_file_path: pathlib.Path) -> sl.Pipeline:
3537
return sl.Pipeline(
3638
[
3739
*load_providers,
@@ -59,7 +61,9 @@ def multi_bank_mcstas_workflow(mcstas_workflow: sl.Pipeline) -> sl.Pipeline:
5961
return pl
6062

6163

62-
def test_pipeline_builder(mcstas_workflow: sl.Pipeline, mcstas_file_path: str) -> None:
64+
def test_pipeline_builder(
65+
mcstas_workflow: sl.Pipeline, mcstas_file_path: pathlib.Path
66+
) -> None:
6367
assert mcstas_workflow.get(FilePath).compute() == mcstas_file_path
6468

6569

0 commit comments

Comments
 (0)