Skip to content

Commit 971ea68

Browse files
authored
Add more test cases for diamond and rectangle shapes (#292)
* Make line tests easier to debug * Update diamond test cases * Update rectangle test cases
1 parent 659bf66 commit 971ea68

File tree

3 files changed

+18
-3
lines changed

3 files changed

+18
-3
lines changed

tests/shapes/lines/bases.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,8 @@ def test_distance(self, shape, test_point, expected_distance):
9191
Test the distance() method parametrized by distance_test_cases
9292
(see conftest.py).
9393
"""
94-
assert pytest.approx(shape.distance(*test_point)) == expected_distance
94+
actual_distance = shape.distance(*test_point)
95+
assert pytest.approx(actual_distance) == expected_distance
9596

9697
def test_lines_form_polygon(self, shape):
9798
"""Test that the lines form a polygon."""

tests/shapes/lines/test_diamond.py

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,14 @@ class TestDiamond(PolygonsLineModuleTestBase):
1212
"""Test the Diamond class."""
1313

1414
shape_name = 'diamond'
15-
distance_test_cases = (((20, 50), 0.0), ((30, 60), 2.773501))
15+
distance_test_cases = (
16+
((20, 50), 0),
17+
((20, 77), 0),
18+
((11, 63.5), 0),
19+
((29, 63.5), 0),
20+
((30, 63.5), 1),
21+
((30, 60), 2.773501),
22+
)
1623
expected_line_count = 4
1724

1825
def test_slopes(self, slopes):

tests/shapes/lines/test_rectangle.py

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,14 @@ class TestRectangle(PolygonsLineModuleTestBase):
1212
"""Test the Rectangle class."""
1313

1414
shape_name = 'rectangle'
15-
distance_test_cases = (((20, 50), 0.0), ((30, 60), 2.0))
15+
distance_test_cases = (
16+
((12, 60), 0),
17+
((28, 60), 0),
18+
((20, 50), 0),
19+
((20, 74), 0),
20+
((20, 75), 1),
21+
((30, 80), 6.324555320336759),
22+
)
1623
expected_line_count = 4
1724

1825
def test_slopes(self, slopes):

0 commit comments

Comments
 (0)