fix(ci): push all tags when doing a mongosh release

This is a drive-by fix which makes sure that all tags get pushed when we actually release mongosh.

Author: gagik

packages/build/src/npm-packages/push-tags.spec.ts

@@ -54,7 +54,10 @@ describe('pushing tags', function () {
 
       expect(() =>
         pushTags(
-          { useAuxiliaryPackagesOnly: false },
+          {
+            isDryRun: false,
+            useAuxiliaryPackagesOnly: false,
+          },
           listNpmPackages,
           existsVersionTag,
           spawnSync
@@ -65,6 +68,7 @@ describe('pushing tags', function () {
     it('takes mongosh version and pushes tags when releasing', function () {
       pushTags(
         {
+          isDryRun: false,
           useAuxiliaryPackagesOnly: false,
         },
         listNpmPackages,
@@ -95,6 +99,7 @@ describe('pushing tags', function () {
     it('pushes only package tags when using auxiliary packages', function () {
       pushTags(
         {
+          isDryRun: false,
           useAuxiliaryPackagesOnly: true,
         },
         listNpmPackages,
@@ -146,6 +151,7 @@ describe('pushing tags', function () {
 
       pushTags(
         {
+          isDryRun: false,
           useAuxiliaryPackagesOnly: true,
         },
         listNpmPackages,
@@ -194,6 +200,7 @@ describe('pushing tags', function () {
       pushTags(
         {
           useAuxiliaryPackagesOnly: false,
+          isDryRun: false,
         },
         listNpmPackages,
         existsVersionTag,
@@ -209,5 +216,29 @@ describe('pushing tags', function () {
       ]);
       expect(spawnSync).calledWith('git', ['push', '--follow-tags']);
     });
+
+    it('skips tag push if it is a dry run', function () {
+      existsVersionTag.withArgs(`v${mongoshVersion}`).returns(true);
+
+      pushTags(
+        {
+          useAuxiliaryPackagesOnly: false,
+          isDryRun: true,
+        },
+        listNpmPackages,
+        existsVersionTag,
+        spawnSync
+      );
+
+      expect(spawnSync).not.calledWith('git', [
+        'tag',
+        '-a',
+        `v${mongoshVersion}`,
+        '-m',
+        `v${mongoshVersion}`,
+      ]);
+
+      expect(spawnSync).not.calledWith('git', ['push', '--follow-tags']);
+    });
   });
 });

packages/build/src/publish-mongosh.ts

@@ -106,7 +106,7 @@ export async function publishMongosh(
   );
 
   pushTags({
-    useAuxiliaryPackagesOnly: true,
+    useAuxiliaryPackagesOnly: false,
     isDryRun: config.isDryRun || false,
   });