Skip to content

Commit

Permalink
kv: fix assertion output in TestReplicationStatusReportIntegration
Browse files Browse the repository at this point in the history
Informs #117175.

This doesn't fix the failing test, but it resolves the nonsensical
failure message and allows us to debug the test.

Release note: None
  • Loading branch information
nvanbenschoten committed Jan 16, 2024
1 parent 3ccf718 commit 4facf2c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/kv/kvserver/reports/reporter_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -375,7 +375,7 @@ func checkZoneReplication(db *gosql.DB, zoneID, total, under, over, unavailable
return fmt.Errorf("expected total: %d, got: %d", total, gotTotal)
}
if under != gotUnder {
return fmt.Errorf("expected under: %d, got: %d", total, gotUnder)
return fmt.Errorf("expected under: %d, got: %d", under, gotUnder)
}
if over != gotOver {
return fmt.Errorf("expected over: %d, got: %d", over, gotOver)
Expand Down

0 comments on commit 4facf2c

Please sign in to comment.