skip correct tests and update runner

W-A-James · W-A-James · commit 890e8052ac86 · 2024-04-19T16:55:07.000-04:00
diff --git a/test/spec/server-discovery-and-monitoring/unified/replicaset-emit-topology-changed-before-close.json b/test/spec/server-discovery-and-monitoring/unified/replicaset-emit-topology-changed-before-close.json
@@ -57,44 +57,16 @@
               "topologyOpeningEvent": {}
             },
             {
-              "topologyDescriptionChangedEvent": {
-                "previousDescription": {
-                  "type": "Unknown"
-                },
-                "newDescription": {
-                  "type": "ReplicaSetNoPrimary"
-                }
-              }
+              "topologyDescriptionChangedEvent": {}
             },
             {
-              "topologyDescriptionChangedEvent": {
-                "previousDescription": {
-                  "type": "ReplicaSetNoPrimary"
-                },
-                "newDescription": {
-                  "type": "ReplicaSetWithPrimary"
-                }
-              }
+              "topologyDescriptionChangedEvent": {}
             },
             {
-              "topologyDescriptionChangedEvent": {
-                "previousDescription": {
-                  "type": "ReplicaSetWithPrimary"
-                },
-                "newDescription": {
-                  "type": "ReplicaSetWithPrimary"
-                }
-              }
+              "topologyDescriptionChangedEvent": {}
             },
             {
-              "topologyDescriptionChangedEvent": {
-                "previousDescription": {
-                  "type": "ReplicaSetWithPrimary"
-                },
-                "newDescription": {
-                  "type": "ReplicaSetWithPrimary"
-                }
-              }
+              "topologyDescriptionChangedEvent": {}
             },
             {
               "topologyDescriptionChangedEvent": {
diff --git a/test/spec/server-discovery-and-monitoring/unified/replicaset-emit-topology-changed-before-close.yml b/test/spec/server-discovery-and-monitoring/unified/replicaset-emit-topology-changed-before-close.yml
@@ -37,26 +37,10 @@ tests:
         ignoreExtraEvents: false
         events:
           - topologyOpeningEvent: {}
-          - topologyDescriptionChangedEvent: # unknown -> replset no primary
-              previousDescription: 
-                type: "Unknown"
-              newDescription:
-                type: "ReplicaSetNoPrimary"
-          - topologyDescriptionChangedEvent:  # server connected
-              previousDescription:
-                type: "ReplicaSetNoPrimary"
-              newDescription:
-                type: "ReplicaSetWithPrimary"
-          - topologyDescriptionChangedEvent:  # server connected
-              previousDescription:
-                type: "ReplicaSetWithPrimary"
-              newDescription:
-                type: "ReplicaSetWithPrimary"
-          - topologyDescriptionChangedEvent:  # server connected
-              previousDescription:
-                type: "ReplicaSetWithPrimary"
-              newDescription:
-                type: "ReplicaSetWithPrimary"
+          - topologyDescriptionChangedEvent: {} # unknown -> replset no primary
+          - topologyDescriptionChangedEvent: {} # server connected
+          - topologyDescriptionChangedEvent: {} # server connected
+          - topologyDescriptionChangedEvent: {} # server connected
           - topologyDescriptionChangedEvent:  # replicaset -> unknown
               previousDescription:
                 type: "ReplicaSetWithPrimary"
diff --git a/test/spec/server-discovery-and-monitoring/unified/standalone-emit-topology-changed-before-close.json b/test/spec/server-discovery-and-monitoring/unified/standalone-emit-topology-changed-before-close.json
@@ -24,9 +24,7 @@
                   "observeEvents": [
                     "topologyDescriptionChangedEvent",
                     "topologyOpeningEvent",
-                    "topologyClosedEvent",
-                    "serverOpeningEvent",
-                    "serverClosedEvent"
+                    "topologyClosedEvent"
                   ]
                 }
               }
@@ -68,9 +66,6 @@
                 }
               }
             },
-            {
-              "serverOpeningEvent": {}
-            },
             {
               "topologyDescriptionChangedEvent": {
                 "previousDescription": {
@@ -81,9 +76,6 @@
                 }
               }
             },
-            {
-              "serverClosedEvent": {}
-            },
             {
               "topologyDescriptionChangedEvent": {
                 "previousDescription": {
diff --git a/test/spec/server-discovery-and-monitoring/unified/standalone-emit-topology-changed-before-close.yml b/test/spec/server-discovery-and-monitoring/unified/standalone-emit-topology-changed-before-close.yml
@@ -20,8 +20,6 @@ tests:
                   - topologyDescriptionChangedEvent
                   - topologyOpeningEvent
                   - topologyClosedEvent
-                  - serverOpeningEvent
-                  - serverClosedEvent
       # ensure the topology has been fully discovered before closing the client.
       # expected events are initial server discovery and server connect event.
       - name: waitForEvent
@@ -44,13 +42,11 @@ tests:
                 type: "Unknown"
               newDescription: 
                 type: "Unknown"
-          - serverOpeningEvent: {}
           - topologyDescriptionChangedEvent:  # unknown w disconnected server -> standalone 
               previousDescription:
                 type: "Unknown"
               newDescription:
                 type: "Single"
-          - serverClosedEvent: {}
           - topologyDescriptionChangedEvent:  # standalone -> unknown
               previousDescription:
                 type: "Single"
diff --git a/test/tools/unified-spec-runner/match.ts b/test/tools/unified-spec-runner/match.ts
@@ -38,6 +38,7 @@ import {
 } from '../../mongodb';
 import { ejson } from '../utils';
 import { type CmapEvent, type CommandEvent, type EntitiesMap, type SdamEvent } from './entities';
+import { trace } from './runner';
 import {
   type ExpectedCmapEvent,
   type ExpectedCommandEvent,
@@ -621,7 +622,7 @@ function compareEvents(
         );
       }
     } else {
-      expect.fail(`Encountered unexpected event - ${inspect(actualEvent)}`);
+      expect.fail(`Encountered unexpected event - ${inspect(actualEvent)} `);
     }
   }
 }
@@ -724,7 +725,7 @@ export function expectErrorCheck(
   expected: ExpectedError,
   entities: EntitiesMap
 ): void {
-  const expectMessage = `\n\nOriginal Error Stack:\n${error.stack}\n\n`;
+  const expectMessage = `\n\nOriginal Error Stack: \n${error.stack} \n\n`;
 
   if (!isMongoCryptError(error)) {
     expect(error, expectMessage).to.be.instanceOf(MongoError);
@@ -755,7 +756,7 @@ export function expectErrorCheck(
     for (const errorLabel of expected.errorLabelsContain) {
       expect(
         mongoError.hasErrorLabel(errorLabel),
-        `Error was supposed to have label ${errorLabel}, has [${mongoError.errorLabels}] -- ${expectMessage}`
+        `Error was supposed to have label ${errorLabel}, has[${mongoError.errorLabels}]--${expectMessage} `
       ).to.be.true;
     }
   }
@@ -765,7 +766,7 @@ export function expectErrorCheck(
     for (const errorLabel of expected.errorLabelsOmit) {
       expect(
         mongoError.hasErrorLabel(errorLabel),
-        `Error was not supposed to have label ${errorLabel}, has [${mongoError.errorLabels}] -- ${expectMessage}`
+        `Error was not supposed to have label ${errorLabel}, has[${mongoError.errorLabels}]--${expectMessage} `
       ).to.be.false;
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -24,9 +24,7 @@`
`24`	`24`	`"observeEvents": [`
`25`	`25`	`"topologyDescriptionChangedEvent",`
`26`	`26`	`"topologyOpeningEvent",`
`27`		`- "topologyClosedEvent",`
`28`		`- "serverOpeningEvent",`
`29`		`- "serverClosedEvent"`
	`27`	`+ "topologyClosedEvent"`
`30`	`28`	`]`
`31`	`29`	`}`
`32`	`30`	`}`
`@@ -68,9 +66,6 @@`
`68`	`66`	`}`
`69`	`67`	`}`
`70`	`68`	`},`
`71`		`- {`
`72`		`- "serverOpeningEvent": {}`
`73`		`- },`
`74`	`69`	`{`
`75`	`70`	`"topologyDescriptionChangedEvent": {`
`76`	`71`	`"previousDescription": {`
`@@ -81,9 +76,6 @@`
`81`	`76`	`}`
`82`	`77`	`}`
`83`	`78`	`},`
`84`		`- {`
`85`		`- "serverClosedEvent": {}`
`86`		`- },`
`87`	`79`	`{`
`88`	`80`	`"topologyDescriptionChangedEvent": {`
`89`	`81`	`"previousDescription": {`
Original file line number	Diff line number	Diff line change
`@@ -38,6 +38,7 @@ import {`
`38`	`38`	`} from '../../mongodb';`
`39`	`39`	`import { ejson } from '../utils';`
`40`	`40`	`import { type CmapEvent, type CommandEvent, type EntitiesMap, type SdamEvent } from './entities';`
	`41`	`+import { trace } from './runner';`
`41`	`42`	`import {`
`42`	`43`	`type ExpectedCmapEvent,`
`43`	`44`	`type ExpectedCommandEvent,`
`@@ -621,7 +622,7 @@ function compareEvents(`
`621`	`622`	`);`
`622`	`623`	`}`
`623`	`624`	`} else {`
`624`		- expect.fail(`Encountered unexpected event - ${inspect(actualEvent)}`);
	`625`	+ expect.fail(`Encountered unexpected event - ${inspect(actualEvent)} `);
`625`	`626`	`}`
`626`	`627`	`}`
`627`	`628`	`}`
`@@ -724,7 +725,7 @@ export function expectErrorCheck(`
`724`	`725`	`expected: ExpectedError,`
`725`	`726`	`entities: EntitiesMap`
`726`	`727`	`): void {`
`727`		- const expectMessage = `\n\nOriginal Error Stack:\n${error.stack}\n\n`;
	`728`	+ const expectMessage = `\n\nOriginal Error Stack: \n${error.stack} \n\n`;
`728`	`729`
`729`	`730`	`if (!isMongoCryptError(error)) {`
`730`	`731`	`expect(error, expectMessage).to.be.instanceOf(MongoError);`
`@@ -755,7 +756,7 @@ export function expectErrorCheck(`
`755`	`756`	`for (const errorLabel of expected.errorLabelsContain) {`
`756`	`757`	`expect(`
`757`	`758`	`mongoError.hasErrorLabel(errorLabel),`
`758`		- `Error was supposed to have label ${errorLabel}, has [${mongoError.errorLabels}] -- ${expectMessage}`
	`759`	+ `Error was supposed to have label ${errorLabel}, has[${mongoError.errorLabels}]--${expectMessage} `
`759`	`760`	`).to.be.true;`
`760`	`761`	`}`
`761`	`762`	`}`
`@@ -765,7 +766,7 @@ export function expectErrorCheck(`
`765`	`766`	`for (const errorLabel of expected.errorLabelsOmit) {`
`766`	`767`	`expect(`
`767`	`768`	`mongoError.hasErrorLabel(errorLabel),`
`768`		- `Error was not supposed to have label ${errorLabel}, has [${mongoError.errorLabels}] -- ${expectMessage}`
	`769`	+ `Error was not supposed to have label ${errorLabel}, has[${mongoError.errorLabels}]--${expectMessage} `
`769`	`770`	`).to.be.false;`
`770`	`771`	`}`
`771`	`772`	`}`