refactoring code

Author: h0x0er

kbanalysis/src/issues-util.ts

@@ -21,10 +21,11 @@ export async function handleKBIssue(
 
   if (resp.status == 200) {
     let old_body = resp.data.body;
-    let action_name = get_action(issue.title);
+    let action_name = getAction(issue.title);
     if (old_body.indexOf(action_name) >= 0) {
       core.info(`[!] Action ${action_name} is already being tracked`);
-      return "Issue already being tracked";
+      let ret = await closeIssue(octokit, owner, repo, issue)
+      return ret;
     }
 
     let new_body = old_body + comment;
@@ -38,19 +39,8 @@ export async function handleKBIssue(
       core.info(`[X] Unable to add: ${issue.number} in the tracking comment`);
     } else {
       core.info(`[!] Added ${issue.title} in tracking comment.`);
-      let resp3 = await octokit.rest.issues.update({
-        owner: owner,
-        repo: repo,
-        issue_number: issue.number,
-        state: "closed",
-      });
-      if (resp3.status === 200) {
-        core.info(`[!] Closed Issue ${issue.number}`);
-        return "success";
-      } else {
-        core.info(`[X] Unable to close issue ${issue.number}`);
-        return "error: unable to close issue";
-      }
+      let ret = await closeIssue(octokit, owner, repo, issue)
+      return ret;
     }
   }
   core.info(`[X] Unable to handle: ${issue.title} `);
@@ -100,7 +90,23 @@ async function prepareComment(
   });
 }
 
-function get_action(x) {
+function getAction(x) {
   x = x.split(" ");
   return x[6];
-}
+}
+
+async function closeIssue(octokit, owner, repo, issue) {
+  let resp3 = await octokit.rest.issues.update({
+    owner: owner,
+    repo: repo,
+    issue_number: issue.number,
+    state: "closed",
+  });
+  if (resp3.status === 200) {
+    core.info(`[!] Closed Issue ${issue.number}`);
+    return "success";
+  } else {
+    core.info(`[X] Unable to close issue ${issue.number}`);
+    return "error: unable to close issue";
+  }
+}