upload-snapshot: fail earlier on invalid tag-git Check Runs

When handling a `push` event on `git-for-windows/git`, it is necessary
to validate that the `tag-git` Check Run has the information needed to
determine the `git-artifacts` runs.

If that information is not there (e.g. because `tag-git` was
accidentally updated and now contains invalid/unparseable data), we
should error out earlier.

Signed-off-by: Johannes Schindelin <[email protected]>

Author: dscho

GitForWindowsHelper/cascading-runs.js

@@ -339,6 +339,11 @@ const handlePush = async (context, req) => {
         `The 'tag-git' run at ${latest.html_url} did not succeed (conclusion = ${latest.conclusion}).`
     )
 
+    const match = latest.output.summary.match(/^Tag Git (\S+) @([0-9a-f]+)$/)
+    if (!match) throw new Error(`Unexpected summary '${latest.output.summary}' of tag-git run: ${latest.html_url}`)
+    if (!match[2] === commit) throw new Error(`Unexpected revision ${match[2]} '${latest.output.summary}' of tag-git run: ${latest.html_url}`)
+    const ver = match[1]
+
     // There is already a `tag-git` workflow run; Is there already an `upload-snapshot` run?
     const latestUploadSnapshotRun = (await listCheckRunsForCommit(
         context,
@@ -363,11 +368,6 @@ const handlePush = async (context, req) => {
         tagGitCheckRunTitle
     )
 
-    const match = latest.output.summary.match(/^Tag Git (\S+) @([0-9a-f]+)$/)
-    if (!match) throw new Error(`Unexpected summary '${latest.output.summary}' of tag-git run: ${latest.html_url}`)
-    if (!match[2] === commit) throw new Error(`Unexpected revision ${match[2]} '${latest.output.summary}' of tag-git run: ${latest.html_url}`)
-    const ver = match[1]
-
     try {
         const workFlowRunIDs = {}
         for (const architecture of ['x86_64', 'i686', 'aarch64']) {