Simplify bitrise run step

MetaMask · Oct 26, 2023 · 4feb455 · 4feb455
1 parent e75b0d9
commit 4feb455
Showing 1 changed file with 48 additions and 69 deletions.
diff --git a/.github/scripts/check-bitrise-e2e-smoke/run-bitrise-e2e-smoke.ts b/.github/scripts/check-bitrise-e2e-smoke/run-bitrise-e2e-smoke.ts
@@ -10,7 +10,6 @@ main().catch((error: Error): void => {
 
 async function main(): Promise<void> {
   const githubToken = process.env.GITHUB_TOKEN;
-  const e2eLabel = process.env.E2E_LABEL;
   const e2ePipeline = process.env.E2E_PIPELINE;
   const pullRequestLink = `https://github.com/MetaMask/metamask-mobile/pull/${context.issue.number}`;
 
@@ -19,90 +18,70 @@ async function main(): Promise<void> {
     process.exit(1);
   }
 
-  if (!e2eLabel) {
-    core.setFailed('E2E_LABEL not found');
-    process.exit(1);
-  }
-
   if (!e2ePipeline) {
     core.setFailed('E2E_PIPELINE not found');
     process.exit(1);
   }
 
   const octokit: InstanceType<typeof GitHub> = getOctokit(githubToken);
 
-  const { pull_request, label } = context.payload;
+  const data = {
+    hook_info: {
+      type: 'bitrise',
+      build_trigger_token: process.env.BITRISE_BUILD_TRIGGER_TOKEN,
+    },
+    build_params: {
+      branch: process.env.GITHUB_HEAD_REF,
+      pipeline_id: e2ePipeline,
+      commit_message: `Triggered by (run-bitrise-e2e) workflow in ${pullRequestLink}`,
+    },
+    triggered_by: 'run-bitrise-e2e',
+  };
+
+  const bitriseProjectUrl = `https://app.bitrise.io/app/${process.env.BITRISE_APP_ID}`;
+
+  // Start Bitrise build.
+  const bitriseBuildResponse = await axios.post(
+    `${bitriseProjectUrl}/build/start.json`,
+    data,
+    {
+      headers: {
+        'Content-Type': 'application/json',
+      },
+    },
+  );
 
-  if (!label || !pull_request) {
-    core.setFailed(
-      'label or pull_request property from context.payload not found',
-    );
+  if (!bitriseBuildResponse.data.build_slug) {
+    core.setFailed(`Bitrise build slug not found`);
     process.exit(1);
   }
 
-  if (label.name === e2eLabel) {
-    const data = {
-      hook_info: {
-        type: 'bitrise',
-        build_trigger_token: process.env.BITRISE_BUILD_TRIGGER_TOKEN,
-      },
-      build_params: {
-        branch: process.env.GITHUB_HEAD_REF,
-        pipeline_id: e2ePipeline,
-        commit_message: `Triggered by (run-bitrise-e2e) workflow in ${pullRequestLink}`,
-      },
-      triggered_by: 'run-bitrise-e2e',
-    };
+  const buildLink = `${bitriseProjectUrl}/pipelines/${bitriseBuildResponse.data.build_slug}`;
+  const message = `E2E test started on Bitrise: ${buildLink}`;
 
-    const bitriseProjectUrl = `https://app.bitrise.io/app/${process.env.BITRISE_APP_ID}`;
-
-    // Start Bitrise build.
-    const bitriseBuildResponse = await axios.post(
-      `${bitriseProjectUrl}/build/start.json`,
-      data,
-      {
-        headers: {
-          'Content-Type': 'application/json',
-        },
-      },
+  if (bitriseBuildResponse.status === 201) {
+    console.log(message);
+  } else {
+    core.setFailed(
+      `Bitrise build request returned with status code ${bitriseBuildResponse.status}`,
     );
+    process.exit(1);
+  }
 
-    if (!bitriseBuildResponse.data.build_slug) {
-      core.setFailed(`Bitrise build slug not found`);
-      process.exit(1);
-    }
-
-    const buildLink = `${bitriseProjectUrl}/pipelines/${bitriseBuildResponse.data.build_slug}`;
-    const message = `E2E test started on Bitrise: ${buildLink}`;
-
-    if (bitriseBuildResponse.status === 201) {
-      console.log(message);
-    } else {
-      core.setFailed(
-        `Bitrise build request returned with status code ${bitriseBuildResponse.status}`,
-      );
-      process.exit(1);
-    }
-
-    // Post build link in PR comments.
-    const postCommentResponse = await octokit.rest.issues.createComment({
-      owner: context.repo.owner,
-      repo: context.repo.repo,
-      issue_number: context.issue.number,
-      body: message,
-    });
+  // Post build link in PR comments.
+  const postCommentResponse = await octokit.rest.issues.createComment({
+    owner: context.repo.owner,
+    repo: context.repo.repo,
+    issue_number: context.issue.number,
+    body: message,
+  });
 
-    if (postCommentResponse.status === 201) {
-      console.log(`Posting comment in pull request ${pullRequestLink}`);
-    } else {
-      core.setFailed(
-        `Post comment request returned with status code ${postCommentResponse.status}`,
-      );
-      process.exit(1);
-    }
+  if (postCommentResponse.status === 201) {
+    console.log(`Posting comment in pull request ${pullRequestLink}`);
   } else {
-    console.log(
-      `Skipping E2E build on PR #${pull_request.number} since workflow was not triggered by (${e2eLabel}) label`,
+    core.setFailed(
+      `Post comment request returned with status code ${postCommentResponse.status}`,
     );
+    process.exit(1);
   }
 }