Skip to content

Commit ecf0a74

Browse files
authored
Minor test fix
Avoid timing issues in a insights unit test.
1 parent 5357496 commit ecf0a74

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

test/unit/insights-client-tests.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -218,7 +218,7 @@ describe('InsightsClient', function () {
218218
insights.shutdown();
219219

220220
// The startup and status events are expected
221-
assert.ok(messages.length > 4, `Message length should be greater than 4, was ${messages.length}`);
221+
assert.ok(messages.length > 2, `Message length should be greater than 2, was ${messages.length}`);
222222

223223
const startupMessage = JSON.parse(messages[0]);
224224
assert.strictEqual(startupMessage.metadata.name, startupEventName);
@@ -227,6 +227,7 @@ describe('InsightsClient', function () {
227227

228228
const firstStatusMessage = JSON.parse(messages[1]);
229229

230+
// Following status messages should be identical
230231
messages.slice(1).forEach(m => {
231232
const statusMessage = JSON.parse(m);
232233
assert.strictEqual(statusMessage.metadata.clientId, startupMessage.metadata.clientId);

0 commit comments

Comments
 (0)