Merge pull request #208 from codecrafters-io/update-tolerance-logs

UdeshyaDhungana · web-flow · commit 37e48e1386b2 · 2025-08-14T12:46:59.000+05:45
add braces around tolerance
diff --git a/internal/resp_assertions/floating_point_bulk_string_assertion.go b/internal/resp_assertions/floating_point_bulk_string_assertion.go
@@ -36,7 +36,7 @@ func (a FloatingPointBulkStringAssertion) Run(value resp_value.Value) error {
 	if diff > a.Tolerance {
 		// display full precision
 		expectedValueStr := strconv.FormatFloat(a.ExpectedValue, 'f', -1, 64)
-		expectedStr := fmt.Sprintf("%s ± %g", expectedValueStr, a.Tolerance)
+		expectedStr := fmt.Sprintf("%s (± %g)", expectedValueStr, a.Tolerance)
 		return fmt.Errorf("Expected %s, got %s", expectedStr, stringValue)
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ func (a FloatingPointBulkStringAssertion) Run(value resp_value.Value) error {`
`36`	`36`	`if diff > a.Tolerance {`
`37`	`37`	`// display full precision`
`38`	`38`	`expectedValueStr := strconv.FormatFloat(a.ExpectedValue, 'f', -1, 64)`
`39`		`- expectedStr := fmt.Sprintf("%s ± %g", expectedValueStr, a.Tolerance)`
	`39`	`+ expectedStr := fmt.Sprintf("%s (± %g)", expectedValueStr, a.Tolerance)`
`40`	`40`	`return fmt.Errorf("Expected %s, got %s", expectedStr, stringValue)`
`41`	`41`	`}`
`42`	`42`