Merge pull request #113 from azerothcore/feat/e2e-tests-and-fix-npm

feat: update dependencies and implemented e2e tests

Author: Yehonal

.github/workflows/e2e.yml

@@ -0,0 +1,76 @@
+name: E2E
+
+on:
+  push:
+    branches: [ main, master ]
+  pull_request:
+    branches: [ '**' ]
+
+jobs:
+  test-e2e:
+    runs-on: ubuntu-latest
+    env:
+      CI: true
+      NODE_OPTIONS: --openssl-legacy-provider
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: '18'
+
+      - name: Show Node/npm versions
+        run: |
+          node -v
+          npm -v
+
+      - name: Setup Chrome
+        uses: browser-actions/setup-chrome@v1
+
+      - name: Cache npm
+        uses: actions/cache@v4
+        with:
+          path: ~/.npm
+          key: ${{ runner.os }}-npm-${{ hashFiles('**/package-lock.json') }}
+          restore-keys: |
+            ${{ runner.os }}-npm-
+
+      - name: Install dependencies
+        run: npm ci --no-audit --no-fund
+
+      - name: Prepare E2E config and data fixtures
+        run: |
+          mkdir -p src/assets/data
+          cat > src/assets/config.json << 'JSON'
+          {
+            "perPage": 50,
+            "pageSize": 8,
+            "usePreGeneratedFile": true,
+            "preGeneratedFileUrl": "assets/data/catalogue.json",
+            "globalSearch": false,
+            "tabs": {
+              "All Modules": {
+                "topic": "e2e-topic",
+                "org": "e2e-org",
+                "path": "/module"
+              }
+            }
+          }
+          JSON
+          cat > src/assets/data/catalogue.json << 'JSON'
+          {
+            "organizations": {
+              "e2e-org": {
+                "e2e-topic": []
+              }
+            }
+          }
+          JSON
+
+      - name: Run unit tests (headless)
+        run: npm run test-ci -- --browsers=ChromeHeadless
+
+      - name: Run E2E tests (Protractor, headless)
+        run: npm run e2e -- --configuration=e2e --webdriver-update=false

angular.json

@@ -65,6 +65,15 @@
                 }
               ]
             }
+            ,
+            "e2e": {
+              "fileReplacements": [
+                {
+                  "replace": "src/environments/environment.ts",
+                  "with": "src/environments/environment.e2e.ts"
+                }
+              ]
+            }
           }
         },
         "serve": {
@@ -75,6 +84,9 @@
           "configurations": {
             "production": {
               "browserTarget": "git-catalogue:build:production"
+            },
+            "e2e": {
+              "browserTarget": "git-catalogue:build:e2e"
             }
           }
         },
@@ -128,6 +140,9 @@
           "configurations": {
             "production": {
               "devServerTarget": "git-catalogue:serve:production"
+            },
+            "e2e": {
+              "devServerTarget": "git-catalogue:serve:e2e"
             }
           }
         }
@@ -138,4 +153,4 @@
   "cli": {
     "analytics": false
   }
-}
+}

e2e/protractor.conf.js

@@ -13,9 +13,14 @@ exports.config = {
     './src/**/*.e2e-spec.ts'
   ],
   capabilities: {
-    browserName: 'chrome'
+    browserName: 'chrome',
+    chromeOptions: process.env.CI ? {
+      args: ['--headless', '--no-sandbox', '--disable-dev-shm-usage', '--window-size=1280,720']
+    } : {}
   },
   directConnect: true,
+  chromeDriver: '/usr/bin/chromedriver', // Use system chromedriver
+  seleniumAddress: null, // Force direct connection
   baseUrl: 'http://localhost:4200/',
   framework: 'jasmine',
   jasmineNodeOpts: {
@@ -29,4 +34,4 @@ exports.config = {
     });
     jasmine.getEnv().addReporter(new SpecReporter({ spec: { displayStacktrace: true } }));
   }
-};
+};

e2e/src/app.e2e-spec.ts

@@ -14,9 +14,21 @@ describe('workspace-project App', () => {
 
   afterEach(async () => {
     // Assert that there are no errors emitted from the browser
+    // Filter out network-related errors that are expected in CI environments
     const logs = await browser.manage().logs().get(logging.Type.BROWSER);
-    expect(logs).not.toContain(jasmine.objectContaining({
-      level: logging.Level.SEVERE,
-    } as logging.Entry));
+    const filteredLogs = logs.filter(entry => {
+      if (entry.level !== logging.Level.SEVERE) return false;
+      
+      // Filter out network-related errors (fonts, external resources)
+      const message = entry.message || '';
+      const isNetworkError = message.includes('net::ERR_NAME_NOT_RESOLVED') ||
+                            message.includes('Failed to load resource') ||
+                            message.includes('fonts.googleapis.com') ||
+                            message.includes('fonts.gstatic.com');
+      
+      return !isNetworkError;
+    });
+    
+    expect(filteredLogs).toEqual([]);
   });
 });