fix: do not show if not explicitly enabled

Author: gagik

packages/shell-api/src/helpers.spec.ts

@@ -224,7 +224,7 @@ describe('getPrintableShardStatus', function () {
     );
     expect(status['most recently active mongoses']).to.have.lengthOf(1);
     expect(status.autosplit['Currently enabled']).to.equal('yes');
-    expect(status.automerge['Currently enabled']).to.equal('yes');
+    expect((status.automerge ?? {})['Currently enabled']).to.equal('yes');
     expect(status.balancer['Currently enabled']).to.equal('yes');
     expect(
       status.balancer['Failed balancer rounds in last 5 attempts']

packages/shell-api/src/helpers.ts

@@ -330,10 +330,11 @@ export async function getPrintableShardStatus(
     (async (): Promise<void> => {
       // Is automerge currently enabled, available since >= 7.0
       const automerge = await settingsColl.findOne({ _id: 'automerge' });
-      result.automerge = {
-        'Currently enabled':
-          automerge === null || automerge.enabled ? 'yes' : 'no',
-      };
+      if (automerge) {
+        result.automerge = {
+          'Currently enabled': automerge.enabled ? 'yes' : 'no',
+        };
+      }
     })(),
     (async (): Promise<void> => {
       // Is the balancer currently enabled
@@ -721,8 +722,8 @@ export type ShardingStatusResult = {
   autosplit: {
     'Currently enabled': 'yes' | 'no';
   };
-  /** Available from 7.0.0 */
-  automerge: {
+  /** Shown if explicitly set, available and enabled by default from 7.0.0 */
+  automerge?: {
     'Currently enabled': 'yes' | 'no';
   };
   balancer: {

packages/shell-api/src/shard.spec.ts

@@ -2519,18 +2519,23 @@ describe('Shard', function () {
       });
     });
     describe('automerge', function () {
-      skipIfServerVersion(mongos, '< 7.0'); // Available from 7.0
-      it('stops correctly', async function () {
-        expect((await sh.stopAutoMerger()).acknowledged).to.equal(true);
-        expect(
-          (await sh.status()).value.automerge['Currently enabled']
-        ).to.equal('no');
+      it('not shown if sh.status() if not explicitly enabled', async function () {
+        expect((await sh.status()).value.automerge).is.undefined;
       });
-      it('enables correctly', async function () {
-        expect((await sh.startAutoMerger()).acknowledged).to.equal(true);
-        expect(
-          (await sh.status()).value.automerge['Currently enabled']
-        ).to.equal('yes');
+      describe('from 7.0', function () {
+        skipIfServerVersion(mongos, '< 7.0'); // Available from 7.0
+        it('stops correctly', async function () {
+          expect((await sh.stopAutoMerger()).acknowledged).to.equal(true);
+          expect(
+            ((await sh.status()).value.automerge ?? {})['Currently enabled']
+          ).to.equal('no');
+        });
+        it('enables correctly', async function () {
+          expect((await sh.startAutoMerger()).acknowledged).to.equal(true);
+          expect(
+            ((await sh.status()).value.automerge ?? {})['Currently enabled']
+          ).to.equal('yes');
+        });
       });
     });
     describe('autosplit', function () {