Refactor CompletedOrder_TotalIsCorrect to express intent with IsFullyPaid()
This commit is contained in:
@@ -15,4 +15,7 @@ public class Order
|
|||||||
=> Lines[lineIndex].ApplyDiscount();
|
=> Lines[lineIndex].ApplyDiscount();
|
||||||
|
|
||||||
public void Complete() => IsCompleted = true;
|
public void Complete() => IsCompleted = true;
|
||||||
|
|
||||||
|
public bool IsFullyPaid(decimal expectedTotal)
|
||||||
|
=> IsCompleted && Total == expectedTotal;
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -12,6 +12,6 @@ public class OrderCompletionTests
|
|||||||
order.AddLine(100m, 1);
|
order.AddLine(100m, 1);
|
||||||
order.Complete();
|
order.Complete();
|
||||||
|
|
||||||
Assert.That(order.Total, Is.EqualTo(100m));
|
Assert.That(order.IsFullyPaid(100m), Is.True);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user