test: refactor client's reconnect tests to cleanup properly (#4909)

Author: snitin315

test/e2e/client-reconnect.test.js

@@ -45,9 +45,14 @@ describe("client.reconnect option", () => {
         waitUntil: "networkidle0",
       });
 
-      expect(response.status()).toMatchSnapshot("response status");
+      try {
+        expect(response.status()).toMatchSnapshot("response status");
+      } catch (error) {
+        throw error;
+      } finally {
+        await server.stop();
+      }
 
-      await server.stop();
       // Can't wait to check for unlimited times so wait only for 5-6 retries
       // eslint-disable-next-line no-restricted-properties
       await page.waitForTimeout(1000 * Math.pow(2, 5) + Math.random() * 100);
@@ -101,9 +106,14 @@ describe("client.reconnect option", () => {
         waitUntil: "networkidle0",
       });
 
-      expect(response.status()).toMatchSnapshot("response status");
+      try {
+        expect(response.status()).toMatchSnapshot("response status");
+      } catch (error) {
+        throw error;
+      } finally {
+        await server.stop();
+      }
 
-      await server.stop();
       // Can't wait to check for unlimited times so wait only for 5-6 retries
       // eslint-disable-next-line no-restricted-properties
       await page.waitForTimeout(1000 * Math.pow(2, 2) + Math.random() * 100);
@@ -154,9 +164,14 @@ describe("client.reconnect option", () => {
         waitUntil: "networkidle0",
       });
 
-      expect(response.status()).toMatchSnapshot("response status");
+      try {
+        expect(response.status()).toMatchSnapshot("response status");
+      } catch (error) {
+        throw error;
+      } finally {
+        await server.stop();
+      }
 
-      await server.stop();
       // eslint-disable-next-line no-restricted-properties
       await page.waitForTimeout(1000 * Math.pow(2, 3) + Math.random() * 100);