Resolve issues in fvt_client_test

Matt Brittan · Matt Brittan · commit 601453b38642 · 2025-09-16T12:29:25.000+12:00
An error in reporting result and Test_cleanUpMids_2 intermittently failed due to race.
diff --git a/fvt_client_test.go b/fvt_client_test.go
@@ -985,7 +985,7 @@ func Test_CallbackOverrun(t *testing.T) {
 
 	err := <-handlerError
 	if err != nil {
-		t.Fatalf(err.Error())
+		t.Fatal(err.Error())
 	}
 
 	time.Sleep(time.Microsecond) // Allow a little time in case the handler returning after connection dropped causes an issue (panic)
@@ -1206,7 +1206,7 @@ func Test_cleanUpMids_2(t *testing.T) {
 		t.Fatalf("Should be a token in the messageIDs, none found")
 	}
 	// fmt.Println("Disconnecting", len(cl.messageIds.index))
-	c.Disconnect(0)
+	c.Disconnect(500) // Wait half a second to allow operation to complete
 
 	fmt.Println("Wait on Token")
 	// We should be able to wait on this token without any issue

Original file line number	Diff line number	Diff line change
`@@ -985,7 +985,7 @@ func Test_CallbackOverrun(t *testing.T) {`
`985`	`985`
`986`	`986`	`err := <-handlerError`
`987`	`987`	`if err != nil {`
`988`		`- t.Fatalf(err.Error())`
	`988`	`+ t.Fatal(err.Error())`
`989`	`989`	`}`
`990`	`990`
`991`	`991`	`time.Sleep(time.Microsecond) // Allow a little time in case the handler returning after connection dropped causes an issue (panic)`
`@@ -1206,7 +1206,7 @@ func Test_cleanUpMids_2(t *testing.T) {`
`1206`	`1206`	`t.Fatalf("Should be a token in the messageIDs, none found")`
`1207`	`1207`	`}`
`1208`	`1208`	`// fmt.Println("Disconnecting", len(cl.messageIds.index))`
`1209`		`- c.Disconnect(0)`
	`1209`	`+ c.Disconnect(500) // Wait half a second to allow operation to complete`
`1210`	`1210`
`1211`	`1211`	`fmt.Println("Wait on Token")`
`1212`	`1212`	`// We should be able to wait on this token without any issue`