SERVER-32393 Use same comparison for eq and neq assertions

2 files changed, 19 insertions, 1 deletions

diff --git a/jstests/multiVersion/upgrade_cluster.js b/jstests/multiVersion/upgrade_cluster.js
index 4f861ed7860..5769bed44ec 100644
--- a/jstests/multiVersion/upgrade_cluster.js
+++ b/jstests/multiVersion/upgrade_cluster.js
@@ -99,7 +99,7 @@ TestData.skipCheckingUUIDsConsistentAcrossCluster = true;
         version = st.s.getCollection('config.version').findOne();
         assert.eq(version.minCompatibleVersion, 5);
         assert.eq(version.currentVersion, 6);
-        assert.neq(clusterID, version.clusterId);
+        assert.eq(clusterID, version.clusterId);
         assert.eq(version.excluding, undefined);
 
         st.stop();
diff --git a/jstests/noPassthrough/shell_assertions.js b/jstests/noPassthrough/shell_assertions.js
index 299dbc00d53..75a0a7ee47e 100644
--- a/jstests/noPassthrough/shell_assertions.js
+++ b/jstests/noPassthrough/shell_assertions.js
@@ -182,6 +182,15 @@
         assert.eq(true, called, 'msg function should have been called');
     });
 
+    tests.push(function eqShouldPassOnObjectsWithSameContent() {
+        const a = {'foo': true};
+        const b = {'foo': true};
+
+        assert.doesNotThrow(() => {
+            assert.eq(a, b);
+        }, [], 'eq should not throw exception on two objects with the same content');
+    });
+
     /* assert.eq.automsg tests */
 
     tests.push(function eqAutomsgShouldCreateMessage() {
@@ -207,6 +216,15 @@
         });
     });
 
+    tests.push(function neqShouldFailOnObjectsWithSameContent() {
+        const a = {'foo': true};
+        const b = {'foo': true};
+
+        assert.throws(() => {
+            assert.neq(a, b);
+        }, [], 'neq should throw exception on two objects with the same content');
+    });
+
     /* assert.hasFields tests */
 
     tests.push(function hasFieldsRequiresAnArrayOfFields() {