chore: make assertions clearer and skip parallel tests

test/integration/change-streams/change_stream.test.ts

@@ -780,7 +780,7 @@ describe('Change Streams', function () {
  });
  });
 
- describe.only('when close is called while changes are pending', function () {
+ describe('when close is called while changes are pending', function () {
  let client;
  let db;
  let collection: Collection<{ insertCount: number }>;
@@ -832,17 +832,23 @@ describe('Change Streams', function () {
  await changeStream.close();
  const results = await Promise.allSettled(changes);
 
- for (const i of changes.keys()) {
- expect(results)
- .to.have.nested.property(`[${i}].reason`)
- .that.is.instanceOf(MongoAPIError);
- const message = /ChangeStream is closed/i;
- expect(results).nested.property(`[${i}].reason`).to.match(message);
- }
+ const statuses = results.map(({ status, reason, value }) => {
+ const res =
+ status === 'rejected'
+ ? reason.message
+ : value.operationType === 'insert'
+ ? `insert count = ${value.fullDocument.insertCount}`
+ : null;
+ return `${status}:${res}`;
+ });
+
+ expect(statuses).to.deep.equal(
+ Array.from({ length: 20 }, () => 'rejected:ChangeStream is closed')
+ );
  }
  );
 
- it(
+ it.skip(
  'rejects promises already returned by next after awaiting the first one',
  { requires: { topology: 'replicaset' } },
  async function () {
@@ -854,15 +860,26 @@ describe('Change Streams', function () {
 
  const results = await allChanges;
 
- const statuses = results.map(({ status }) => status);
+ const statuses = results.map(({ status, reason, value }) => {
+ const res =
+ status === 'rejected'
+ ? reason.message
+ : value.operationType === 'insert'
+ ? `insert count = ${value.fullDocument.insertCount}`
+ : null;
+ return `${status}:${res}`;
+ });
+
+ console.log(statuses);
+
  expect(statuses).to.deep.equal([
- 'fulfilled',
- ...Array.from({ length: 19 }, () => 'rejected')
+ 'fulfilled:insert count = 1',
+ ...Array.from({ length: 19 }, () => 'rejected:ChangeStream is closed')
  ]);
  }
- );
+ ).skipReason = 'TODO(NODE-5221): Parallel change streams and close are nondeterministic';
 
- it(
+ it.skip(
  'rejects promises already returned by next after awaiting half of them',
  { requires: { topology: 'replicaset' } },
  async function () {
@@ -875,13 +892,24 @@ describe('Change Streams', function () {
 
  const results = await allChanges;
 
- const statuses = results.map(({ status }) => status);
+ const statuses = results.map(({ status, reason, value }) => {
+ const res =
+ status === 'rejected'
+ ? reason.message
+ : value.operationType === 'insert'
+ ? `insert count = ${value.fullDocument.insertCount}`
+ : null;
+ return `${status}:${res}`;
+ });
+
+ console.log(statuses);
+
  expect(statuses).to.deep.equal([
- ...Array.from({ length: 10 }, () => 'fulfilled'),
- ...Array.from({ length: 10 }, () => 'rejected')
+ ...Array.from({ length: 1 }, () => 'fulfilled:insert count = 0'),
+ ...Array.from({ length: 19 }, () => 'fulfilled:insert count = 1')
  ]);
  }
- );
+ ).skipReason = 'TODO(NODE-5221): Parallel change streams and close are nondeterministic';
  });
 
  describe('iterator api', function () {