Fix flakey test

[eloff]

Skips over any msgN messages until finding the disconnect message. This makes the test robust if the consumer and producer happen to interleave in a way that one or more of the messages go through.

Hopefully this is enough now.

[Copilot]

PR Overview

This PR fixes a flaky test by modifying the test flow to skip intermediary "msg" messages until a disconnect message is received.

• Introduces a loop to ignore non-disconnect messages.
• Refines message parsing by extracting the message text for comparison.

Reviewed Changes

File	Description
rust/foxglove/src/websocket/tests.rs	Updated test logic to handle interleaved messages robustly.

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

rust/foxglove/src/websocket/tests.rs:1445

• The comment contains a spelling error: 'disconencted' should be corrected to 'disconnected'.

// Client should have been disconencted

[Copilot]

Consider adding a timeout or a maximum iteration safeguard in this loop to prevent a potential infinite loop if the expected disconnect message is never received.

Suggested change

	loop {
	// Client should have been disconencted
	let mut iteration_count = 0;
	let max_iterations = 100; // Set a reasonable limit for the number of iterations
	loop {
	iteration_count += 1;
	if iteration_count > max_iterations {
	panic!("Exceeded maximum iterations while waiting for disconnect message");
	}
	// Client should have been disconnected