From 1db2ec93f3588ec9dcb01e99ecd9ad8893580078 Mon Sep 17 00:00:00 2001
From: Shivam Mathur <shivam_jpr@hotmail.com>
Date: Thu, 27 Jan 2022 08:44:53 +0530
Subject: [PATCH] Fix tools.getLatestVersion

---
 __tests__/tools.test.ts |  7 +++++--
 dist/index.js           | 15 +++++++++------
 src/tools.ts            | 23 +++++++++++++----------
 3 files changed, 27 insertions(+), 18 deletions(-)

diff --git a/__tests__/tools.test.ts b/__tests__/tools.test.ts
index 6ab0c6850..d60029479 100644
--- a/__tests__/tools.test.ts
+++ b/__tests__/tools.test.ts
@@ -42,12 +42,14 @@ jest
   .spyOn(utils, 'fetch')
   .mockImplementation(
     async (url: string, token?: string): Promise<Record<string, string>> => {
-      if (url.includes('atom') && !url.includes('no-release')) {
+      if (url.includes('atom') && !url.includes('no-')) {
         return {
           data: '"releases/tag/1.2.3", "releases/tag/3.2.1", "releases/tag/2.3.1"'
         };
+      } else if (url.includes('no-data')) {
+        return {};
       } else if (url.includes('no-release')) {
-        return {data: ''};
+        return {data: 'no-release'};
       } else if (!token || token === 'valid_token') {
         return {data: `[{"ref": "refs/tags/1.2.3", "url": "${url}"}]`};
       } else if (token === 'beta_token') {
@@ -77,6 +79,7 @@ describe('Tools tests', () => {
   it.each`
     tool                 | fetch_latest | version
     ${'tool'}            | ${'true'}    | ${'3.2.1'}
+    ${'tool-no-data'}    | ${'true'}    | ${'latest'}
     ${'tool-no-release'} | ${'true'}    | ${'latest'}
     ${'tool'}            | ${'false'}   | ${'latest'}
   `(
diff --git a/dist/index.js b/dist/index.js
index be1868ce9..c61687e08 100644
--- a/dist/index.js
+++ b/dist/index.js
@@ -548,12 +548,15 @@ async function getLatestVersion(data) {
         return 'latest';
     }
     const resp = await utils.fetch(`${data['github']}/${data['repository']}/releases.atom`);
-    const releases = [
-        ...resp['data'].matchAll(/releases\/tag\/([a-zA-Z]*)?(\d+.\d+.\d+)"/g)
-    ].map(match => match[2]);
-    return (releases
-        .sort((a, b) => a.localeCompare(b, undefined, { numeric: true }))
-        .pop() || 'latest');
+    if (resp['data']) {
+        const releases = [
+            ...resp['data'].matchAll(/releases\/tag\/([a-zA-Z]*)?(\d+.\d+.\d+)"/g)
+        ].map(match => match[2]);
+        return (releases
+            .sort((a, b) => a.localeCompare(b, undefined, { numeric: true }))
+            .pop() || 'latest');
+    }
+    return 'latest';
 }
 exports.getLatestVersion = getLatestVersion;
 async function getVersion(version, data) {
diff --git a/src/tools.ts b/src/tools.ts
index fd8de8825..8b040f5f7 100644
--- a/src/tools.ts
+++ b/src/tools.ts
@@ -43,17 +43,20 @@ export async function getLatestVersion(data: RS): Promise<string> {
   const resp: Record<string, string> = await utils.fetch(
     `${data['github']}/${data['repository']}/releases.atom`
   );
-  const releases: string[] = [
-    ...resp['data'].matchAll(/releases\/tag\/([a-zA-Z]*)?(\d+.\d+.\d+)"/g)
-  ].map(match => match[2]);
+  if (resp['data']) {
+    const releases: string[] = [
+      ...resp['data'].matchAll(/releases\/tag\/([a-zA-Z]*)?(\d+.\d+.\d+)"/g)
+    ].map(match => match[2]);
 
-  return (
-    releases
-      .sort((a: string, b: string) =>
-        a.localeCompare(b, undefined, {numeric: true})
-      )
-      .pop() || 'latest'
-  );
+    return (
+      releases
+        .sort((a: string, b: string) =>
+          a.localeCompare(b, undefined, {numeric: true})
+        )
+        .pop() || 'latest'
+    );
+  }
+  return 'latest';
 }
 
 /**