updated libraries

Author: xinredhat

.eslintrc.js

@@ -4,13 +4,23 @@
   "useTabs": false,
   "semi": true,
   "singleQuote": true,
+  "quoteProps": "as-needed",
   "trailingComma": "es5",
   "bracketSpacing": true,
+  "bracketSameLine": false,
   "arrowParens": "avoid",
   "endOfLine": "lf",
+  "embeddedLanguageFormatting": "auto",
   "plugins": [
     "@trivago/prettier-plugin-sort-imports"
   ],
+  "importOrder": [
+    "^@playwright/(.*)$",
+    "^@types/(.*)$",
+    "^@/(.*)$",
+    "^[./]"
+  ],
   "importOrderSeparation": true,
-  "importOrderSortSpecifiers": true
+  "importOrderSortSpecifiers": true,
+  "importOrderCaseInsensitive": true
 }

.prettierrc

@@ -0,0 +1,43 @@
+import js from '@eslint/js';
+import typescript from '@typescript-eslint/eslint-plugin';
+import typescriptParser from '@typescript-eslint/parser';
+import prettier from 'eslint-config-prettier';
+import globals from 'globals';
+
+export default [
+  js.configs.recommended,
+  {
+    files: ['**/*.ts'],
+    languageOptions: {
+      parser: typescriptParser,
+      parserOptions: {
+        ecmaVersion: 2020,
+        sourceType: 'module',
+        project: './tsconfig.json',
+      },
+      globals: {
+        ...globals.node,
+        ...globals.es2020,
+      },
+    },
+    plugins: {
+      '@typescript-eslint': typescript,
+    },
+    rules: {
+      '@typescript-eslint/explicit-function-return-type': 'warn',
+      '@typescript-eslint/no-explicit-any': 'error',
+      '@typescript-eslint/no-unused-vars': ['error', { argsIgnorePattern: '^_' }],
+      'prefer-const': 'error',
+      'no-console': ['warn', { allow: ['warn', 'error'] }],
+      'no-var': 'error',
+    },
+  },
+  {
+    files: ['tests/**/*', '**/*.test.ts'],
+    rules: {
+      '@typescript-eslint/no-explicit-any': 'off',
+      'no-console': 'off',
+    },
+  },
+  prettier,
+]; 

eslint.config.js

@@ -1,10 +1,14 @@
 // Import default logger as fallback
-import { closeAllLoggers, defaultLogger } from './src/log/logger';
-import { resetTestItemsFile } from './src/utils/testItemExporter';
 import { FullConfig } from '@playwright/test';
 import { Logger } from 'pino';
 
-async function globalSetup(config: FullConfig) {
+import { closeAllLoggers, defaultLogger } from './src/log/logger';
+import { resetTestItemsFile } from './src/utils/testItemExporter';
+
+/**
+ * Global setup function for Playwright tests
+ */
+async function globalSetup(config: FullConfig): Promise<void> {
   const log: Logger = defaultLogger;
 
   log.info('Starting test suite setup (Global Setup)');
@@ -14,20 +18,31 @@ async function globalSetup(config: FullConfig) {
     const isUITest = process.env.UI_TEST === 'true';
 
     if (!isUITest) {
+      log.info('Resetting test items file...');
       resetTestItemsFile();
+    } else {
+      log.info('Skipping test items reset for UI tests');
     }
 
-    log.info('Global setup completed successfully.');
+    log.info('Global setup completed successfully');
   } catch (error) {
-    log.error({ err: error }, 'Global setup failed!'); // Use err serializer
-    // Re-throw the error to fail the setup process
+    log.error({ err: error }, 'Global setup failed!');
     throw error;
   }
 }
 
-async function globalTeardown(config: FullConfig) {
+/**
+ * Global teardown function for Playwright tests
+ */
+async function globalTeardown(config: FullConfig): Promise<void> {
   console.log('Ensuring all logs are properly flushed to disk...');
-  await closeAllLoggers();
+  
+  try {
+    await closeAllLoggers();
+    console.log('All loggers closed successfully');
+  } catch (error) {
+    console.error('Error closing loggers:', error);
+  }
 }
 
 export { globalTeardown };

global-setup.ts

@@ -1,61 +1,68 @@
 {
     "name": "tssc-test",
     "version": "1.0.0",
+    "type": "module",
     "description": "TSSC Test is a testing framework for TSSC API using Playwright and TypeScript.",
     "scripts": {
-        "lint": "eslint --ext .ts src tests",
-        "lint:fix": "eslint --ext .ts src tests --fix",
-        "format": "prettier --write \"src/**/*.ts\" \"tests/**/*.ts\"",
-        "check-types": "tsc --noEmit",
-        "validate": "npm run lint && npm run format && npm run check-types",
+        "lint": "eslint \"src/**/*.ts\" \"tests/**/*.ts\"",
+        "lint:fix": "eslint \"src/**/*.ts\" \"tests/**/*.ts\" --fix",
+        "format": "prettier --write \"src/**/*.ts\" \"tests/**/*.ts\" \"*.json\" \"*.js\" \"*.md\"",
+        "format:check": "prettier --check \"src/**/*.ts\" \"tests/**/*.ts\" \"*.json\" \"*.js\" \"*.md\"",
+        "type-check": "tsc --noEmit",
+        "validate": "npm run type-check && npm run lint && npm run format:check",
         "test": "playwright test",
         "test:report": "playwright show-report",
         "test:tssc": "playwright test tests/tssc/full_workflow.test.ts",
-        "test:ui": "UI_TEST=true playwright test tests/tssc/ui.test.ts",
-        "ui": "UI_TEST=true playwright test --ui tests/tssc/ui.test.ts"
+        "test:ui": "UI_TEST=true playwright test tests/ui/ui.test.ts",
+        "test:ui-interactive": "UI_TEST=true playwright test --ui tests/ui/ui.test.ts",
+        "pretest": "npm run validate"
     },
     "keywords": [
         "typescript",
         "playwright",
         "testing",
-        "api"
+        "api",
+        "e2e"
     ],
     "author": "TSSC Team",
-    "license": "Apache License 2.0",
+    "license": "Apache-2.0",
     "dependencies": {
-        "@gitbeaker/rest": "^42.5.0",
+        "@gitbeaker/rest": "^43.0.0",
         "@janus-idp/shared-react": "^2.18.0",
         "@types/async-retry": "^1.4.9",
         "async-retry": "^1.3.3",
         "p-retry": "^6.2.1",
-        "sodium-native": "^5.0.4",
-        "typescript": "^5.8.3"
+        "sodium-native": "^5.0.6"
     },
     "devDependencies": {
-        "@backstage/plugin-scaffolder-react": "^1.16.0",
-        "@kubernetes/client-node": "^1.2.0",
+        "@backstage/plugin-scaffolder-react": "^1.18.0",
+        "@kubernetes/client-node": "^1.3.0",
         "@octokit/plugin-retry": "^8.0.1",
         "@octokit/plugin-throttling": "^11.0.1",
         "@octokit/rest": "^22.0.0",
-        "@playwright/test": "^1.52.0",
+        "@playwright/test": "^1.54.1",
         "@trivago/prettier-plugin-sort-imports": "^5.2.2",
-        "@types/node": "^20.12.14",
+        "@types/node": "^24.0.14",
         "@types/sodium-native": "^2.3.9",
-        "@typescript-eslint/eslint-plugin": "^7.18.0",
-        "@typescript-eslint/parser": "^7.18.0",
-        "axios": "^1.9.0",
-        "dotenv": "^16.5.0",
+        "@typescript-eslint/eslint-plugin": "^8.37.0",
+        "@typescript-eslint/parser": "^8.37.0",
+        "axios": "^1.10.0",
+        "dotenv": "^17.2.0",
         "eslint": "^8.57.0",
-        "eslint-config-prettier": "^9.1.0",
-        "eslint-plugin-prettier": "^5.2.0",
-        "glob": "^11.0.2",
-        "nock": "^14.0.4",
+        "eslint-config-prettier": "^10.1.5",
+        "eslint-plugin-prettier": "^5.5.1",
+        "glob": "^11.0.3",
+        "nock": "^14.0.5",
         "otplib": "^12.0.1",
         "pino": "^9.7.0",
         "pino-pretty": "^13.0.0",
-        "prettier": "^3.5.3",
+        "prettier": "^3.6.2",
         "rimraf": "^6.0.1",
         "ts-node": "^10.9.2",
         "typescript": "^5.8.3"
+    },
+    "engines": {
+        "node": ">=18.0.0",
+        "npm": ">=8.0.0"
     }
 }

package.json

@@ -1,8 +1,9 @@
-import { defineConfig, PlaywrightTestConfig, PlaywrightTestOptions, PlaywrightWorkerOptions } from '@playwright/test';
+import { defineConfig, PlaywrightTestConfig } from '@playwright/test';
 import { existsSync, readFileSync } from 'fs';
 import path from 'path';
-import { TestPlan } from './src/playwright/testplan';
+
 import { TestItem } from './src/playwright/testItem';
+import { TestPlan } from './src/playwright/testplan';
 
 // Extend Playwright types to include testItem
 declare module '@playwright/test' {
@@ -11,47 +12,89 @@ declare module '@playwright/test' {
   }
 }
 
-// Load the test plan for e2e tests
-const testPlanPath = process.env.TESTPLAN_PATH || path.resolve(process.cwd(), 'testplan.json');
-const testPlanData = JSON.parse(readFileSync(testPlanPath, 'utf-8'));
-const testPlan = new TestPlan(testPlanData);
+// Configuration constants
+const DEFAULT_TIMEOUT = 2100000; // 35 minutes
+const DEFAULT_WORKERS = 6;
+const DEFAULT_TESTPLAN_PATH = path.resolve(process.cwd(), 'testplan.json');
 
-// Create projects using the TestPlan class
-const e2eProjects = testPlan.getProjectConfigs().map(config => ({
-  name: config.name,
-  use: {
-    testItem: config.testItem,
-  },
-}));
+/**
+ * Load test plan configuration
+ */
+function loadTestPlan(): TestPlan {
+  const testPlanPath = process.env.TESTPLAN_PATH || DEFAULT_TESTPLAN_PATH;
+  
+  if (!existsSync(testPlanPath)) {
+    console.warn(`Test plan not found at ${testPlanPath}, using default configuration`);
+    return new TestPlan({ templates: [], tssc: [], tests: [] });
+  }
+  
+  try {
+    const testPlanData = JSON.parse(readFileSync(testPlanPath, 'utf-8'));
+    return new TestPlan(testPlanData);
+  } catch (error) {
+    console.error(`Failed to parse test plan: ${error}`);
+    throw error;
+  }
+}
 
-// Create ui projects for UI tests from exported test items
-let uiProjects: any[] = [];
-const exportedTestItemsPath = './tmp/test-items.json';
-if (existsSync(exportedTestItemsPath)) {
+/**
+ * Load exported test items for UI tests
+ */
+function loadUIProjects(): Array<{ name: string; testMatch: string; use: { testItem: TestItem } }> {
+  const exportedTestItemsPath = './tmp/test-items.json';
+  
+  if (!existsSync(exportedTestItemsPath)) {
+    return [];
+  }
+  
   try {
     const exportedData = JSON.parse(readFileSync(exportedTestItemsPath, 'utf-8'));
-    if (exportedData.testItems && Array.isArray(exportedData.testItems)) {
-      uiProjects = exportedData.testItems.map((itemData: any) => ({
-        name: `ui-${itemData.name}`,
-        testMatch: '**/ui.test.ts',
-        use: {
-          testItem: TestItem.fromJSON(itemData),
-        },
-      }));
+    
+    if (!exportedData.testItems || !Array.isArray(exportedData.testItems)) {
+      return [];
     }
+    
+    return exportedData.testItems.map((itemData: any) => ({
+      name: `ui-${(itemData as { name: string }).name}`,
+      testMatch: '**/ui.test.ts',
+      use: {
+        testItem: TestItem.fromJSON(itemData),
+      },
+    }));
   } catch (error) {
     console.warn('Could not load exported test items for UI tests:', error);
+    return [];
   }
 }
 
+// Load configurations
+const testPlan = loadTestPlan();
+const e2eProjects = testPlan.getProjectConfigs().map(config => ({
+  name: config.name,
+  use: {
+    testItem: config.testItem,
+  },
+}));
+
+const uiProjects = loadUIProjects();
 const allProjects = [...e2eProjects, ...uiProjects];
 
 export default defineConfig({
   testDir: './tests',
   testMatch: '**/*.test.ts',
-  workers: 6,
+  workers: DEFAULT_WORKERS,
+  timeout: DEFAULT_TIMEOUT,
+  
+  // Use specific projects or fallback to default
   projects: allProjects.length ? allProjects : [{ name: 'default' }],
-  reporter: [['html', { open: 'never' }], ['list']],
-  timeout: 2100000, // Default to 35 minutes (2100000ms)
+  
+  // Reporter configuration
+  reporter: [
+    ['html', { open: 'never', outputFolder: 'playwright-report' }],
+    ['list'],
+    ['junit', { outputFile: 'test-results/junit.xml' }],
+  ],
+  
+  // Global setup and teardown
   globalSetup: './global-setup.ts',
 });

playwright.config.ts

@@ -57,6 +57,8 @@ export class ArgoCD {
     const success = await this.argoCDClient.syncApplication(applicationName, this.NAMESPACE);
 
     if (!success) {
+      const status = await this.getApplicationStatus(environment);
+      console.log(status);
       throw new Error(`Failed to sync application ${applicationName}`);
     }
   }

src/rhtap/core/integration/cd/argocd.ts

@@ -247,4 +247,3 @@ export class TemplateFactory {
     }
   }
 }
-export { ContentModifications };