disable send crash report, refactor (#717)

Author: marknguyen1302

cortex-js/src/usecases/telemetry/telemetry.usecases.ts

@@ -28,8 +28,7 @@ export class TelemetryUsecases {
     source: TelemetrySource,
   ): Promise<void> {
     try {
-      const isCollectingTelemetryEnabled = process.env.CORTEX_CRASH_REPORT;
-      if (isCollectingTelemetryEnabled !== '1') {
+      if (this.isCrashReportEnabled() === false) {
         return;
       }
       const crashReport: CrashReportAttributes = this.buildCrashReport(error);
@@ -40,6 +39,9 @@ export class TelemetryUsecases {
   }
 
   async sendCrashReport(): Promise<void> {
+    if (!this.isCrashReportEnabled()) {
+      return;
+    }
     const crashReport = await this.telemetryRepository.getLastCrashReport();
     if (crashReport && !crashReport.metadata.sentAt) {
       const promises = [
@@ -79,4 +81,8 @@ export class TelemetryUsecases {
       },
     };
   }
+
+  private isCrashReportEnabled(): boolean {
+    return process.env.CORTEX_CRASH_REPORT === '1';
+  }
 }