Skip to content

Commit 63ec7e1

Browse files
committed
Enable VNet RACK test stat check
1 parent 74d2499 commit 63ec7e1

File tree

1 file changed

+12
-13
lines changed

1 file changed

+12
-13
lines changed

vnet_test.go

Lines changed: 12 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -941,17 +941,16 @@ func TestRACK_RTTSwitch_Reordering_NoDrop(t *testing.T) { //nolint:gocyclo,cyclo
941941
}
942942

943943
// check FR stats reported.
944-
// we can uncomment this to check the FR stats.
945-
// I tested it and it works fine on the pch07/rack-sctp branch.
946-
// cs := <-clientStatsCh
947-
// ss := <-serverStatsCh
948-
//
949-
// if assert.True(t, cs.ok, "client assoc/stats unavailable") {
950-
// assert.LessOrEqual(t, cs.fr, uint64(2),
951-
// "client fast retransmits should be low")
952-
// }
953-
// if assert.True(t, ss.ok, "server assoc/stats unavailable") {
954-
// assert.LessOrEqual(t, ss.fr, uint64(2),
955-
// "server fast retransmits should be low")
956-
// }
944+
cs := <-clientStatsCh
945+
ss := <-serverStatsCh
946+
947+
if assert.True(t, cs.ok, "client assoc/stats unavailable") {
948+
assert.LessOrEqual(t, cs.fr, uint64(2),
949+
"client fast retransmits should be low")
950+
}
951+
952+
if assert.True(t, ss.ok, "server assoc/stats unavailable") {
953+
assert.LessOrEqual(t, ss.fr, uint64(2),
954+
"server fast retransmits should be low")
955+
}
957956
}

0 commit comments

Comments
 (0)