various fixes

Author: jochen-testingbot

src/auth.ts

@@ -5,13 +5,17 @@ import Credentials from './models/credentials';
 
 export default class Auth {
   public static async getCredentials(): Promise<Credentials | null> {
-    const savedCredentials = (
-      await fs.promises.readFile(path.join(os.homedir(), '.testingbot'))
-    ).toString();
-    if (savedCredentials.length > 0) {
-      const [userName, accessKey] = savedCredentials.split(':');
-      return new Credentials(userName, accessKey);
+    try {
+      const savedCredentials = (
+        await fs.promises.readFile(path.join(os.homedir(), '.testingbot'))
+      ).toString();
+      if (savedCredentials.length > 0) {
+        const [userName, accessKey] = savedCredentials.split(':');
+        return new Credentials(userName, accessKey);
+      }
+      return null;
+    } catch {
+      return null;
     }
-    return null;
   }
 }

src/cli.ts

@@ -41,8 +41,10 @@ program
       }
       const espresso = new Espresso(credentials, options);
       await espresso.run();
-    } catch (err: any) {
-      logger.error(`Espresso error: ${err.message}`);
+    } catch (err) {
+      logger.error(
+        `Espresso error: ${err instanceof Error ? err.message : err}`,
+      );
     }
   })
   .showHelpAfterError(true);
@@ -68,10 +70,12 @@ program
       if (credentials === null) {
         throw new Error('Please specify credentials');
       }
-      const maestto = new Maestro(credentials, options);
-      await maestto.run();
-    } catch (err: any) {
-      logger.error(`Maestro error: ${err.message}`);
+      const maestro = new Maestro(credentials, options);
+      await maestro.run();
+    } catch (err) {
+      logger.error(
+        `Maestro error: ${err instanceof Error ? err.message : err}`,
+      );
     }
   })
   .showHelpAfterError(true);
@@ -91,8 +95,10 @@ program
       }
       const xcuitest = new XCUITest(credentials, options);
       await xcuitest.run();
-    } catch (err: any) {
-      logger.error(`XCUITest error: ${err.message}`);
+    } catch (err) {
+      logger.error(
+        `XCUITest error: ${err instanceof Error ? err.message : err}`,
+      );
     }
   });
 

src/providers/espresso.ts

@@ -68,8 +68,8 @@ export default class Espresso {
 
       logger.info('Running Espresso Tests');
       await this.runTests();
-    } catch (error: any) {
-      logger.error(error.message);
+    } catch (error) {
+      logger.error(error instanceof Error ? error.message : error);
     }
   }
 
@@ -112,7 +112,7 @@ export default class Espresso {
       formData,
       {
         headers: {
-          'Content-Type': 'application/zip,',
+          'Content-Type': 'application/zip',
           'Content-Disposition': `attachment; filename=${fileName}`,
           'User-Agent': utils.getUserAgent(),
         },

src/providers/maestro.ts

@@ -68,8 +68,8 @@ export default class Maestro {
 
       logger.info('Running Maestro Tests');
       await this.runTests();
-    } catch (error: any) {
-      logger.error(error.message);
+    } catch (error) {
+      logger.error(error instanceof Error ? error.message : error);
     }
   }
 
@@ -112,7 +112,7 @@ export default class Maestro {
       formData,
       {
         headers: {
-          'Content-Type': 'application/zip,',
+          'Content-Type': 'application/zip',
           'Content-Disposition': `attachment; filename=${fileName}`,
           'User-Agent': utils.getUserAgent(),
         },

src/providers/xcuitest.ts

@@ -6,6 +6,7 @@ import path from 'node:path';
 import FormData from 'form-data';
 import TestingBotError from '../models/testingbot_error';
 import XCUITestOptions from '../models/xcuitest_options';
+import utils from '../utils';
 
 export default class XCUITest {
   private readonly URL = 'https://api.testingbot.com/v1/app-automate/xcuitest';
@@ -64,7 +65,7 @@ export default class XCUITest {
       logger.info('Running XCUITests');
       await this.runTests();
     } catch (error) {
-      logger.error(error);
+      logger.error(error instanceof Error ? error.message : error);
     }
   }
 
@@ -78,6 +79,7 @@ export default class XCUITest {
       headers: {
         'Content-Type': 'application/octet-stream',
         'Content-Disposition': `attachment; filename=${fileName}`,
+        'User-Agent': utils.getUserAgent(),
       },
       auth: {
         username: this.credentials.userName,
@@ -106,8 +108,9 @@ export default class XCUITest {
       formData,
       {
         headers: {
-          'Content-Type': 'application/zip,',
+          'Content-Type': 'application/zip',
           'Content-Disposition': `attachment; filename=${fileName}`,
+          'User-Agent': utils.getUserAgent(),
         },
         auth: {
           username: this.credentials.userName,
@@ -138,6 +141,7 @@ export default class XCUITest {
         {
           headers: {
             'Content-Type': 'application/json',
+            'User-Agent': utils.getUserAgent(),
           },
           auth: {
             username: this.credentials.userName,