Skip to content

Commit c1f5efd

Browse files
Copilotpetesramek
andauthored
fix confusing block-length test comments
Agent-Logs-Url: https://github.com/petesramek/polyline-algorithm-csharp/sessions/b16b1bac-62fc-44de-9c08-88c566cf476e Co-authored-by: petesramek <2333452+petesramek@users.noreply.github.com>
1 parent c375141 commit c1f5efd

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

tests/PolylineAlgorithm.Tests/PolylineEncodingTests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -609,7 +609,7 @@ public void ValidateFormat_With_Single_Terminator_Does_Not_Throw() {
609609
[TestMethod]
610610
public void ValidateFormat_With_Block_Too_Long_Throws_InvalidPolylineException() {
611611
// Act & Assert
612-
Assert.ThrowsExactly<InvalidPolylineException>(() => PolylineEncoding.ValidateFormat("_____________?")); // 14-char block (max is 13)
612+
Assert.ThrowsExactly<InvalidPolylineException>(() => PolylineEncoding.ValidateFormat("_____________?")); // 13 continuation chars + terminator = block length 14, exceeds max of 13
613613
}
614614

615615
#endregion
@@ -668,7 +668,7 @@ public void ValidateBlockLength_With_Valid_Polyline_Does_Not_Throw(string polyli
668668
/// Tests that ValidateBlockLength throws <see cref="InvalidPolylineException"/> for invalid block structures.
669669
/// </summary>
670670
[TestMethod]
671-
[DataRow("_____________?")] // 13-char block (exceeds max of 13 = blockLen 14)
671+
[DataRow("_____________?")] // 13 continuation chars + terminator = block length 14, exceeds max of 13
672672
[DataRow("________")] // all continuation chars, no terminator
673673
[DataRow("")] // empty polyline has no terminator
674674
[DataRow("?_____________?")] // valid block then too-long block

0 commit comments

Comments
 (0)