fix: multiple appium session creation (#231)

* fix: multiple appium session creation

* chore: added changeset

* fix: ios app restart

.changeset/tame-meals-hunt.md

@@ -0,0 +1,5 @@
+---
+"appwright": patch
+---
+
+fix: multiple appium session creation

src/fixture/index.ts

@@ -9,6 +9,7 @@ import {
 import { Device } from "../device";
 import { createDeviceProvider } from "../providers";
 import { WorkerInfoStore } from "./workerInfo";
+import { stopAppiumServer } from "../providers/appium";
 
 type TestLevelFixtures = {
   /**
@@ -36,10 +37,12 @@ export const test = base.extend<TestLevelFixtures, WorkerLevelFixtures>({
   },
   device: async ({ deviceProvider }, use, testInfo) => {
     const device = await deviceProvider.getDevice();
+    const deviceProviderName = (
+      testInfo.project as FullProject<AppwrightConfig>
+    ).use.device?.provider;
     testInfo.annotations.push({
       type: "providerName",
-      description: (testInfo.project as FullProject<AppwrightConfig>).use.device
-        ?.provider,
+      description: deviceProviderName,
     });
     testInfo.annotations.push({
       type: "sessionId",
@@ -48,6 +51,12 @@ export const test = base.extend<TestLevelFixtures, WorkerLevelFixtures>({
     await deviceProvider.syncTestDetails?.({ name: testInfo.title });
     await use(device);
     await device.close();
+    if (
+      deviceProviderName === "emulator" ||
+      deviceProviderName === "local-device"
+    ) {
+      await stopAppiumServer();
+    }
     await deviceProvider.syncTestDetails?.({
       name: testInfo.title,
       status: testInfo.status,

src/providers/appium.ts

@@ -45,9 +45,12 @@ export async function startAppiumServer(
     const appiumProcess = spawn("npx", ["appium"], {
       stdio: "pipe",
     });
-
+    appiumProcess.stderr.on("data", async (data: Buffer) => {
+      console.log(data.toString());
+    });
     appiumProcess.stdout.on("data", async (data: Buffer) => {
       const output = data.toString();
+      console.log(output);
 
       if (output.includes("Error: listen EADDRINUSE")) {
         // TODO: Kill the appium server if it is already running
@@ -86,6 +89,19 @@ export async function startAppiumServer(
   });
 }
 
+export function stopAppiumServer() {
+  return new Promise((resolve, reject) => {
+    exec(`pkill -f appium`, (error, stdout) => {
+      if (error) {
+        logger.error(`Error stopping Appium server: ${error.message}`);
+        reject(error);
+      }
+      logger.log("Appium server stopped successfully.");
+      resolve(stdout);
+    });
+  });
+}
+
 export function isEmulatorInstalled(platform: Platform): Promise<boolean> {
   return new Promise((resolve) => {
     if (platform == Platform.ANDROID) {

src/providers/emulator/index.ts

@@ -117,6 +117,7 @@ Follow the steps mentioned in ${androidSimulatorConfigDocLink} to run test on An
         "appium:fullReset": true,
         "appium:deviceOrientation": this.project.use.device?.orientation,
         "appium:settings[snapshotMaxDepth]": 62,
+        "appium:wdaLaunchTimeout": 300_000,
       },
     };
   }