From 060cc5bfb5cfd8347d9f9798d3a99b1e52bc3ace Mon Sep 17 00:00:00 2001
From: Zaid-maker <pzhafeez@gmail.com>
Date: Thu, 21 Nov 2024 16:04:28 +0500
Subject: [PATCH] Revert "improve workflow"

This reverts commit a3358e298359bf7824e313cae819e70366e44fcd.
---
 .github/workflows/auto-test.yml | 91 +++++++++++----------------------
 1 file changed, 30 insertions(+), 61 deletions(-)

diff --git a/.github/workflows/auto-test.yml b/.github/workflows/auto-test.yml
index 403b48866..bf76d9eb6 100644
--- a/.github/workflows/auto-test.yml
+++ b/.github/workflows/auto-test.yml
@@ -1,3 +1,6 @@
+# This workflow will do a clean install of node dependencies, build the source code and run tests across different versions of node
+# For more information see: https://help.github.com/actions/language-and-framework-guides/using-nodejs-with-github-actions
+
 name: Auto Test
 
 on:
@@ -11,19 +14,7 @@ on:
       - '*.md'
 
 jobs:
-  check-linters:
-    runs-on: ubuntu-latest
-    steps:
-    - uses: actions/checkout@v4
-    - name: Use Node.js 20
-      uses: actions/setup-node@v4
-      with:
-        node-version: 20
-        cache: 'npm'
-    - run: npm install
-    - run: npm run lint:prod
-
-  test:
+  auto-test:
     needs: [ check-linters ]
     runs-on: ${{ matrix.os }}
     timeout-minutes: 15
@@ -32,41 +23,23 @@ jobs:
       matrix:
         os: [macos-latest, ubuntu-latest, windows-latest, ARM64]
         node: [ 18, 20 ]
+        # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
 
     steps:
+    - run: git config --global core.autocrlf false  # Mainly for Windows
     - uses: actions/checkout@v4
 
     - name: Use Node.js ${{ matrix.node }}
       uses: actions/setup-node@v4
       with:
         node-version: ${{ matrix.node }}
-        cache: 'npm'
-    
-    - name: Install dependencies
-      run: npm install
-
-    - name: Build project
-      run: npm run build
-
-    - name: Run backend tests
-      run: npm run test-backend
+    - run: npm install
+    - run: npm run build
+    - run: npm run test-backend
       env:
         HEADLESS_TEST: 1
         JUST_FOR_TEST: ${{ secrets.JUST_FOR_TEST }}
 
-    - name: Run component tests
-      run: npm run test-component
-
-    - name: Upload test results
-      if: always()
-      uses: actions/upload-artifact@v3
-      with:
-        name: test-results-${{ matrix.os }}-node-${{ matrix.node }}
-        path: |
-          ./test-results
-          ./coverage
-        retention-days: 30
-
   # As a lot of dev dependencies are not supported on ARMv7, we have to test it separately and just test if `npm ci --production` works
   armv7-simple-test:
     needs: [ ]
@@ -77,48 +50,44 @@ jobs:
       matrix:
         os: [ ARMv7 ]
         node: [ 18, 20 ]
+        # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
 
     steps:
+      - run: git config --global core.autocrlf false  # Mainly for Windows
       - uses: actions/checkout@v4
 
       - name: Use Node.js ${{ matrix.node }}
         uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node }}
-          cache: 'npm'
       - run: npm ci --production
 
-  e2e-test:
-    needs: [ check-linters ]
-    runs-on: ARM64
-    timeout-minutes: 30
+  check-linters:
+    runs-on: ubuntu-latest
+
     steps:
+    - run: git config --global core.autocrlf false  # Mainly for Windows
     - uses: actions/checkout@v4
 
     - name: Use Node.js 20
       uses: actions/setup-node@v4
       with:
         node-version: 20
-        cache: 'npm'
-    
-    - name: Install dependencies
-      run: npm install
-
-    - name: Install Playwright browsers
-      run: npx playwright install --with-deps chromium firefox
-
-    - name: Build project
-      run: npm run build
+    - run: npm install
+    - run: npm run lint:prod
 
-    - name: Run E2E tests
-      run: npm run test-e2e
-      env:
-        HEADLESS_TEST: 1
+  e2e-test:
+    needs: [ ]
+    runs-on: ARM64
+    steps:
+    - run: git config --global core.autocrlf false  # Mainly for Windows
+    - uses: actions/checkout@v4
 
-    - name: Upload Playwright report
-      if: always()
-      uses: actions/upload-artifact@v3
+    - name: Use Node.js 20
+      uses: actions/setup-node@v4
       with:
-        name: playwright-report
-        path: ./private/playwright-report
-        retention-days: 30
+        node-version: 20
+    - run: npm install
+    - run: npx playwright install
+    - run: npm run build
+    - run: npm run test-e2e