Skip to content

Commit 0cf1ad7

Browse files
committed
fix(ci): fix bad unit test + add ruff dependencie + update checkout action v4 to v5
1 parent 80370b1 commit 0cf1ad7

4 files changed

Lines changed: 5 additions & 4 deletions

File tree

.github/workflows/lint.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ jobs:
66
ruff:
77
runs-on: ubuntu-latest
88
steps:
9-
- uses: actions/checkout@v4
9+
- uses: actions/checkout@v5
1010
with:
1111
submodules: true
1212

.github/workflows/tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ jobs:
66
test:
77
runs-on: ubuntu-latest
88
steps:
9-
- uses: actions/checkout@v4
9+
- uses: actions/checkout@v5
1010
with:
1111
submodules: true
1212

pyproject.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ dev = [
2727
"pdoc>=16.0.0",
2828
"pytest>=9.0.2",
2929
"pytest-cov>=7.0.0",
30+
"ruff",
3031
]
3132

3233
[tool.pytest.ini_options]

tests/unit/test_viewer/test_viewer.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -263,8 +263,8 @@ def test_bezier_conversion_success(self, MockBezierCurve):
263263
# Assertions
264264
viewer.model.get_end_points_coords.assert_called_once_with(42)
265265
MockBezierCurve._default_control_points.assert_called_once_with(coords_a, coords_b, degree)
266-
MockBezierCurve.assert_called_once_with("42", MockBezierCurve._default_control_points.return_value, degree)
267-
viewer.model.set_curve.assert_called_once_with("42", mock_curve_instance)
266+
MockBezierCurve.assert_called_once_with(42, MockBezierCurve._default_control_points.return_value, degree)
267+
viewer.model.set_curve.assert_called_once_with(42, mock_curve_instance)
268268

269269
def test_bezier_conversion_no_selection(self):
270270
"""Verifies behavior if no curve is selected/hovered."""

0 commit comments

Comments
 (0)