Debug

Author: romaricpascal

bin/generate-npm-tag.sh

@@ -9,12 +9,16 @@ HIGHEST_PUBLISHED_VERSION=${2:-$(git tag --list 2>/dev/null | sort -V | tail -n1
 
 version() { echo "$@" | awk -F. '{ printf("%d%03d%03d\n", $1,$2,$3); }'; }
 
+echo $CURRENT_VERSION
+echo $HIGHEST_PUBLISHED_VERSION
+echo $(git tag --list 2>/dev/null | sort -V | tail -n1 2>/dev/null | sed 's/v//g')
+
 if [ "$CURRENT_VERSION" = "$HIGHEST_PUBLISHED_VERSION" ]; then
   echo "⚠️ Git tag $CURRENT_VERSION already exists. Check you have run \`npm version\` correctly."
   exit 1
 elif echo "$CURRENT_VERSION" | grep -q "internal"; then
   NPM_TAG="internal"
-elif echo "$CURRENT_VERSION" | grep -q -E '^\d+\.\d+\.\d+-\D+(\.\d+)?$'; then
+elif echo "$CURRENT_VERSION" | grep -q "-"; then
   NPM_TAG="next"
 elif [ $(version "$CURRENT_VERSION") -ge $(version "$HIGHEST_PUBLISHED_VERSION") ]; then
   NPM_TAG="latest"

bin/generate-npm-tag.unit.test.mjs

@@ -1,12 +1,15 @@
 import { exec } from 'node:child_process'
 
-describe('generate-npm-tag.sh', () => {
-  it('errors if the version in the GOV.UK Frontend package is currently published', async () => {
+// Our build scripts are Shell scripts so won't run on Windows
+// Unfortunately, Jest wants test files to have test suites,
+// so we need to only run `describe` if the platform is not Windows
+describeIf(process.platform !== 'win32')('generate-npm-tag.sh', () => {
+  it.only('errors if the version in the GOV.UK Frontend package is currently published', async () => {
     expect.assertions(2)
     // Unfortunately Jest's `.rejects.toThrow` only allows to test the type of error or its message
     // so we need to explicitly `try/catch` to check the exit code and message
     try {
-      await execute('bin/generate-npm-tag.sh')
+      await execute('bin/generate-npm-tag.sh 6.2.0-beta.0 6.2.0-beta.0')
     } catch (e) {
       expect(e.code).toBe(1)
       expect(e.stdout).toMatch(
@@ -22,8 +25,8 @@ describe('generate-npm-tag.sh', () => {
     ['minor', '6.1.0', 'latest'],
     ['major', '7.0.0', 'latest'],
     ['internal', '6.0.0-internal', 'internal'],
-    ['beta', '6.0.0-beta', 'next'],
-    ['rc', '6.0.0-rc', 'next'],
+    ['beta', '6.1.0-beta.0', 'next'],
+    ['rc', '6.1.0-rc.1', 'next'],
     ['latest-...', '4.0.0', 'latest-v4']
   ])('%s bump', async (label, version, expectedTag) => {
     const { stdout } = await execute(
@@ -34,6 +37,19 @@ describe('generate-npm-tag.sh', () => {
   })
 })
 
+/**
+ * Allows to skip a `describe` block if the condition is not met
+ *
+ * Jest does not allow empty test suites, so this allows to skip all tests
+ * if a condition is met
+ *
+ * @param {boolean} condition - The condition that needs to be met for tests to run
+ * @returns {Function} Jest's `describe` function if the `condition` is met, `describe.skip` otherwise
+ */
+function describeIf(condition) {
+  return condition ? describe : describe.skip
+}
+
 /**
  * Executes the given command and returns the content of `stdout` and `stderr`
  *