From ca9cde8972a9de858ffacadaa5827d8c6d3e4852 Mon Sep 17 00:00:00 2001
From: Fabio Fernandes <fabiofernandes@fabios-mbp.home>
Date: Mon, 24 Jun 2024 15:39:42 +0100
Subject: [PATCH 01/10] Delete a todo e2e test

---
 client/src/components/ListTodos.jsx                   | 1 +
 server/src/tests/end-to-end/pern-todo-app.end.spec.js | 8 ++++++--
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/client/src/components/ListTodos.jsx b/client/src/components/ListTodos.jsx
index dede996..b2ba4da 100644
--- a/client/src/components/ListTodos.jsx
+++ b/client/src/components/ListTodos.jsx
@@ -34,6 +34,7 @@ const ListTodos = ({ todos, deleteTodo, onUpdateTodo }) => {
               <td>
                 <button
                   className="btn btn-danger"
+                  id="deleteBtn"
                   onClick={() => deleteTodo(todo.id)}
                 >
                   Delete
diff --git a/server/src/tests/end-to-end/pern-todo-app.end.spec.js b/server/src/tests/end-to-end/pern-todo-app.end.spec.js
index d01adc3..bb062e9 100644
--- a/server/src/tests/end-to-end/pern-todo-app.end.spec.js
+++ b/server/src/tests/end-to-end/pern-todo-app.end.spec.js
@@ -7,7 +7,6 @@ test.beforeEach(async ({ page }) => {
 test.describe("PERN Todo App", () => {
   test("should add a todo", async ({ page }) => {
     await page.fill("#todoInput", "Test Todo");
-    await page.pause();
     await page.click("#addTodoBtn");
     await expect(page.getByTestId("todo-description")).toHaveText("Test Todo", {
       timeout: 5000,
@@ -15,7 +14,6 @@ test.describe("PERN Todo App", () => {
   });
 
   test("should edit a todo", async ({ page }) => {
-    await page.pause();
     await page.click("#editBtn");
     await page.fill(".modal-body input", "Updated Test Todo");
     await page.click("#editDoneBtn");
@@ -23,4 +21,10 @@ test.describe("PERN Todo App", () => {
       "Updated Test Todo"
     );
   });
+
+  test("should delete a todo", async ({ page }) => {
+    await page.pause();
+    await page.click("#deleteBtn");
+    await expect(page.getByTestId("todo-description")).toHaveCount(0);
+  });
 });

From 3e3ddbc1edbfbc051207bbe3cbc5d5e4351634cc Mon Sep 17 00:00:00 2001
From: Fabio Fernandes <fabiofernandes@fabios-mbp.home>
Date: Mon, 24 Jun 2024 16:35:03 +0100
Subject: [PATCH 02/10] Automated e2e tests to run in GH Actions

---
 .github/workflows/clientBuildAndDeploy.yml | 28 +++++++++++++++-------
 1 file changed, 20 insertions(+), 8 deletions(-)

diff --git a/.github/workflows/clientBuildAndDeploy.yml b/.github/workflows/clientBuildAndDeploy.yml
index 7cbc53a..4bbb5a8 100644
--- a/.github/workflows/clientBuildAndDeploy.yml
+++ b/.github/workflows/clientBuildAndDeploy.yml
@@ -17,19 +17,31 @@ jobs:
     steps:
       - name: Code checkout
         uses: actions/checkout@v3
+
+        # Sets up a specific version of Node.js for use in the workflow
       - name: Install NodeJS
         uses: actions/setup-node@v3
         with:
           node-version: 20.x
+        # Installs your project dependencies using npm ci
       - name: Install dependencies
         run: npm ci
+
+        # Builds your application using your package.json's build script
       - name: Build app
         run: npm run build
-      - name: Deploy
-        run: | 
-          cd build 
-          aws s3 sync . s3://perntodo
-        env:
-          AWS_ACCESS_KEY_ID: ${{ secrets.AWS_ACCESS_KEY_ID }}
-          AWS_SECRET_ACCESS_KEY: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
-          AWS_DEFAULT_REGION: us-east-1
\ No newline at end of file
+
+        # Run E2E tests
+      - name: Run E2E tests
+        run: npx playwright test
+        working-directory: ../server
+
+        # Deploys your application to AWS S3
+      # - name: Deploy
+      #   run: |
+      #     cd build
+      #     aws s3 sync . s3://perntodo
+      #   env:
+      #     AWS_ACCESS_KEY_ID: ${{ secrets.AWS_ACCESS_KEY_ID }}
+      #     AWS_SECRET_ACCESS_KEY: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
+      #     AWS_DEFAULT_REGION: us-east-1

From 83a14cda51da96fafe8f887afa3cf83955571290 Mon Sep 17 00:00:00 2001
From: Fabio Fernandes <fabiofernandes@fabios-mbp.home>
Date: Mon, 24 Jun 2024 17:50:39 +0100
Subject: [PATCH 03/10] Add GitHub Actions workflow for e2e tests

---
 .github/workflows/clientBuildAndDeploy.yml | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/clientBuildAndDeploy.yml b/.github/workflows/clientBuildAndDeploy.yml
index 4bbb5a8..1623983 100644
--- a/.github/workflows/clientBuildAndDeploy.yml
+++ b/.github/workflows/clientBuildAndDeploy.yml
@@ -1,11 +1,17 @@
-name: Client build and deploy
+# name: Client build and deploy
 
+# on:
+#   push:
+#     branches:
+#       - main
+#     paths:
+#       - "client/**"
+
+name: Run e2e tests
 on:
-  push:
+  pull_request:
     branches:
       - main
-    paths:
-      - "client/**"
 
 jobs:
   deploy:

From a65f858ca780bdfc8ac86eda314bc7f7da8f50e5 Mon Sep 17 00:00:00 2001
From: Fabio Fernandes <fabiofernandes@fabios-mbp.home>
Date: Tue, 25 Jun 2024 10:22:43 +0100
Subject: [PATCH 04/10] Fix build and deploy errors in GitHub Actions E2E tests

---
 client/src/components/EditTodo.jsx  |  4 ----
 client/src/components/ListTodos.jsx | 11 +----------
 2 files changed, 1 insertion(+), 14 deletions(-)

diff --git a/client/src/components/EditTodo.jsx b/client/src/components/EditTodo.jsx
index 099af71..d68584d 100644
--- a/client/src/components/EditTodo.jsx
+++ b/client/src/components/EditTodo.jsx
@@ -1,8 +1,4 @@
 import React, { Fragment, useState } from "react";
-import axios from "axios";
-
-const baseUrl =
-  process.env.REACT_APP_ECS_SERVICE_URL || `http://localhost:5000`;
 
 const EditTodo = ({ todo, onUpdateTodo }) => {
   const [description, setDescription] = useState(todo.description);
diff --git a/client/src/components/ListTodos.jsx b/client/src/components/ListTodos.jsx
index b2ba4da..514272e 100644
--- a/client/src/components/ListTodos.jsx
+++ b/client/src/components/ListTodos.jsx
@@ -1,9 +1,5 @@
-import React, { Fragment, useEffect, useState } from "react";
+import React, { Fragment } from "react";
 import EditTodo from "./EditTodo";
-import axios from "axios";
-
-const baseUrl =
-  process.env.REACT_APP_ECS_SERVICE_URL || `http://localhost:5000`;
 
 const ListTodos = ({ todos, deleteTodo, onUpdateTodo }) => {
   console.log(todos);
@@ -20,11 +16,6 @@ const ListTodos = ({ todos, deleteTodo, onUpdateTodo }) => {
           </tr>
         </thead>
         <tbody>
-          {/*<tr>
-            <td>John</td>
-            <td>Doe</td>
-            <td>john@example.com</td>
-          </tr> */}
           {todos.map((todo) => (
             <tr key={`${todo.id}`}>
               <td data-testid="todo-description">{todo.description}</td>

From 06d57de28c4b06b357a91384ec9cd12b4adaac79 Mon Sep 17 00:00:00 2001
From: Fabio Fernandes <fabiofernandes@fabios-mbp.home>
Date: Tue, 25 Jun 2024 10:29:21 +0100
Subject: [PATCH 05/10] Fix working directory for E2E tests in GitHub Actions
 workflow

---
 .github/workflows/clientBuildAndDeploy.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/clientBuildAndDeploy.yml b/.github/workflows/clientBuildAndDeploy.yml
index 1623983..0490aa5 100644
--- a/.github/workflows/clientBuildAndDeploy.yml
+++ b/.github/workflows/clientBuildAndDeploy.yml
@@ -40,7 +40,7 @@ jobs:
         # Run E2E tests
       - name: Run E2E tests
         run: npx playwright test
-        working-directory: ../server
+        working-directory: ./server
 
         # Deploys your application to AWS S3
       # - name: Deploy

From c0e94c0a9255eb0d6a8c06688cd9ba42e571816b Mon Sep 17 00:00:00 2001
From: Fabio Miguel <fabiomigf@gmail.com>
Date: Thu, 27 Jun 2024 15:32:36 +0100
Subject: [PATCH 06/10] Fix working directory for E2E tests in GitHub Actions
 workflow

---
 .github/workflows/clientBuildAndDeploy.yml | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/clientBuildAndDeploy.yml b/.github/workflows/clientBuildAndDeploy.yml
index 0490aa5..9580a28 100644
--- a/.github/workflows/clientBuildAndDeploy.yml
+++ b/.github/workflows/clientBuildAndDeploy.yml
@@ -29,10 +29,14 @@ jobs:
         uses: actions/setup-node@v3
         with:
           node-version: 20.x
-        # Installs your project dependencies using npm ci
-      - name: Install dependencies
+        # Installs your project dependencies on client directory using npm ci
+      - name: Install client dependencies
         run: npm ci
 
+        - name: Install server dependencies
+        run: npm ci
+        working-directory: ../server
+
         # Builds your application using your package.json's build script
       - name: Build app
         run: npm run build
@@ -40,7 +44,7 @@ jobs:
         # Run E2E tests
       - name: Run E2E tests
         run: npx playwright test
-        working-directory: ./server
+        working-directory: ../server
 
         # Deploys your application to AWS S3
       # - name: Deploy

From 69aff221fe4c7da1b1f7ad7936ae5a208f8fc8ff Mon Sep 17 00:00:00 2001
From: Fabio Miguel <fabiomigf@gmail.com>
Date: Thu, 27 Jun 2024 15:37:14 +0100
Subject: [PATCH 07/10] Fix working directory for E2E tests in GitHub Actions
 workflow

---
 .github/workflows/clientBuildAndDeploy.yml | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/clientBuildAndDeploy.yml b/.github/workflows/clientBuildAndDeploy.yml
index 9580a28..4ac76e0 100644
--- a/.github/workflows/clientBuildAndDeploy.yml
+++ b/.github/workflows/clientBuildAndDeploy.yml
@@ -24,24 +24,21 @@ jobs:
       - name: Code checkout
         uses: actions/checkout@v3
 
-        # Sets up a specific version of Node.js for use in the workflow
       - name: Install NodeJS
         uses: actions/setup-node@v3
         with:
           node-version: 20.x
-        # Installs your project dependencies on client directory using npm ci
+
       - name: Install client dependencies
         run: npm ci
 
-        - name: Install server dependencies
+      - name: Install server dependencies
         run: npm ci
         working-directory: ../server
 
-        # Builds your application using your package.json's build script
       - name: Build app
         run: npm run build
 
-        # Run E2E tests
       - name: Run E2E tests
         run: npx playwright test
         working-directory: ../server

From 9e109696f8361cebb8e7e0d7a8adca53fd437655 Mon Sep 17 00:00:00 2001
From: Fabio Miguel <fabiomigf@gmail.com>
Date: Thu, 27 Jun 2024 15:39:21 +0100
Subject: [PATCH 08/10] Fix working directory for E2E tests in GitHub Actions
 workflow

---
 .github/workflows/clientBuildAndDeploy.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/clientBuildAndDeploy.yml b/.github/workflows/clientBuildAndDeploy.yml
index 4ac76e0..a7c8879 100644
--- a/.github/workflows/clientBuildAndDeploy.yml
+++ b/.github/workflows/clientBuildAndDeploy.yml
@@ -34,14 +34,14 @@ jobs:
 
       - name: Install server dependencies
         run: npm ci
-        working-directory: ../server
+        working-directory: server
 
       - name: Build app
         run: npm run build
 
       - name: Run E2E tests
         run: npx playwright test
-        working-directory: ../server
+        working-directory: server
 
         # Deploys your application to AWS S3
       # - name: Deploy

From 1a9cd2981afd62f26992273201182334140c22f2 Mon Sep 17 00:00:00 2001
From: Fabio Miguel <fabiomigf@gmail.com>
Date: Thu, 27 Jun 2024 16:40:24 +0100
Subject: [PATCH 09/10] Add Playwright browser installation step for E2E tests

---
 .github/workflows/clientBuildAndDeploy.yml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/.github/workflows/clientBuildAndDeploy.yml b/.github/workflows/clientBuildAndDeploy.yml
index a7c8879..5b91938 100644
--- a/.github/workflows/clientBuildAndDeploy.yml
+++ b/.github/workflows/clientBuildAndDeploy.yml
@@ -39,6 +39,10 @@ jobs:
       - name: Build app
         run: npm run build
 
+      - name: Install Playwright browsers
+        run: npx playwright install
+        working-directory: server
+
       - name: Run E2E tests
         run: npx playwright test
         working-directory: server

From d6771d67731d87cfec077da4160e88be60adc18b Mon Sep 17 00:00:00 2001
From: Fabio Miguel <fabiomigf@gmail.com>
Date: Thu, 27 Jun 2024 16:46:08 +0100
Subject: [PATCH 10/10] Set playwright.config.js headless: true

---
 server/playwright.config.js | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/server/playwright.config.js b/server/playwright.config.js
index 2f74a10..e8d4f57 100644
--- a/server/playwright.config.js
+++ b/server/playwright.config.js
@@ -25,8 +25,7 @@ module.exports = defineConfig({
   globalTeardown: "./playwrightTeardown.js",
   use: {
     trace: "on-first-retry",
-    /* Playwright will run the browsers in a non-headless mode, i.e., with a graphical user interface (GUI). */
-    headless: false,
+    headless: true,
   },
 
   /* Configure projects for major browsers */