Skip to content

Commit 6df12f6

Browse files
committed
fix(test): clarify docstrings
1 parent 4a0f0da commit 6df12f6

1 file changed

Lines changed: 6 additions & 2 deletions

File tree

tests/test_well_inventory.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -565,7 +565,7 @@ def test_measuring_point_height_ft_used_for_thing_and_observation(tmp_path):
565565
def test_mp_height_used_for_thing_and_observation_when_measuring_point_height_ft_blank(
566566
tmp_path,
567567
):
568-
"""When depth to water is provided but measuring_point_height_ft is blank, the mp_height value should be used for the thing's and observation's measuring_point_height."""
568+
"""When depth to water is provided and measuring_point_height_ft is blank the mp_height value should be used for the thing's (MeasuringPointHistory) and observation's measuring_point_height."""
569569
row = _minimal_valid_well_inventory_row()
570570
row.update(
571571
{
@@ -599,7 +599,7 @@ def test_mp_height_used_for_thing_and_observation_when_measuring_point_height_ft
599599

600600

601601
def test_null_observation_allows_blank_mp_height(tmp_path):
602-
"""When depth to water is not provided, a blank measuring_point_height_ft should be allowed and result in a null measuring_point_height on the thing and observation and no associated measuring point height for the well."""
602+
"""When depth to water is not provided (ie null), blank measuring_point_height_ft and mp_height fields should be allowed and result in a null measuring_point_height for the observation and no associated measuring point height (MeasuringPointHistory) for the well."""
603603
row = _minimal_valid_well_inventory_row()
604604
row.update(
605605
{
@@ -632,7 +632,11 @@ def test_null_observation_allows_blank_mp_height(tmp_path):
632632

633633

634634
def test_conflicting_mp_heights_raises_error(tmp_path):
635+
"""
636+
When both measuring_point_height_ft and mp_height are provided, an inequality (conflict) should raise an error.
637+
"""
635638
row = _minimal_valid_well_inventory_row()
639+
636640
row.update(
637641
{
638642
"measuring_point_height_ft": 3.5,

0 commit comments

Comments
 (0)