fix UTs and lint issue

Author: kobenguyent

.github/workflows/html-reporter-tests.yml

@@ -15,7 +15,7 @@ const output = require('../output')
 const Codecept = require('../codecept')
 
 const defaultConfig = {
-  output: (typeof global !== 'undefined' && global.output_dir) ? global.output_dir : './output',
+  output: typeof global !== 'undefined' && global.output_dir ? global.output_dir : './output',
   reportFileName: 'report.html',
   includeArtifacts: true,
   showSteps: true,
@@ -89,7 +89,7 @@ module.exports = function (config) {
 
   // Track overall test execution
   event.dispatcher.on(event.all.before, () => {
-  reportData.startTime = (new Date()).toISOString()
+    reportData.startTime = new Date().toISOString()
     output.print('HTML Reporter: Starting HTML report generation...')
   })
 
@@ -145,7 +145,7 @@ module.exports = function (config) {
     if (step.htmlReporterStartTime) {
       step.duration = Date.now() - step.htmlReporterStartTime
     }
-    
+
     // Serialize args immediately to preserve them through worker serialization
     let serializedArgs = []
     if (step.args && Array.isArray(step.args)) {
@@ -164,7 +164,7 @@ module.exports = function (config) {
         }
       })
     }
-    
+
     currentTestSteps.push({
       name: step.name,
       actor: step.actor,
@@ -246,7 +246,7 @@ module.exports = function (config) {
 
     // Check if this test already exists in reportData.tests (from a previous retry)
     const existingTestIndex = reportData.tests.findIndex(t => t.id === testId)
-  const hasFailedBefore = existingTestIndex >= 0 && reportData.tests[existingTestIndex] && reportData.tests[existingTestIndex].state === 'failed'
+    const hasFailedBefore = existingTestIndex >= 0 && reportData.tests[existingTestIndex] && reportData.tests[existingTestIndex].state === 'failed'
     const currentlyFailed = test.state === 'failed'
 
     // Debug artifacts collection (but don't process them yet - screenshots may not be ready)
@@ -256,7 +256,7 @@ module.exports = function (config) {
     // This ensures the feature name is preserved when test data is JSON stringified
     const parentTitle = test.parent?.title || test.suite?.title || (suite && suite.title) || null
     const suiteTitle = test.suite?.title || (suite && suite.title) || null
-    
+
     const testData = {
       ...test,
       id: testId,
@@ -279,7 +279,7 @@ module.exports = function (config) {
 
     if (existingTestIndex >= 0) {
       // Update existing test with final result (including failed state)
-  if (existingTestIndex >= 0) reportData.tests[existingTestIndex] = testData
+      if (existingTestIndex >= 0) reportData.tests[existingTestIndex] = testData
       output.debug(`HTML Reporter: Updated existing test - ${test.title}, Final state: ${test.state}`)
     } else {
       // Add new test
@@ -329,9 +329,9 @@ module.exports = function (config) {
   })
 
   // Generate final report
-  event.dispatcher.on(event.all.result, async (result) => {
-  reportData.endTime = (new Date()).toISOString()
-  reportData.duration = (new Date(reportData.endTime).getTime() - new Date(reportData.startTime).getTime())
+  event.dispatcher.on(event.all.result, async result => {
+    reportData.endTime = new Date().toISOString()
+    reportData.duration = new Date(reportData.endTime).getTime() - new Date(reportData.startTime).getTime()
 
     // Process artifacts now that all async tasks (including screenshots) are complete
     output.debug(`HTML Reporter: Processing artifacts for ${reportData.tests.length} tests after all async tasks complete`)
@@ -382,10 +382,9 @@ module.exports = function (config) {
       .map(t => {
         const testName = t.title || 'Unknown Test'
         // Try to get feature name from BDD, preserved titles (worker mode), or direct access
-        let featureName = t.feature?.name || t.parentTitle || t.suiteTitle || 
-                          t.parent?.title || t.suite?.title || 'Unknown Feature';
+        let featureName = t.feature?.name || t.parentTitle || t.suiteTitle || t.parent?.title || t.suite?.title || 'Unknown Feature'
         if (featureName === 'Unknown Feature' && t.suite && t.suite.feature && t.suite.feature.name) {
-          featureName = t.suite.feature.name;
+          featureName = t.suite.feature.name
         }
 
         if (t.err) {
@@ -976,9 +975,7 @@ module.exports = function (config) {
       .map(test => {
         const statusClass = test.state || 'unknown'
         // Use preserved parent/suite titles (for worker mode) or fallback to direct access
-        const feature = test.isBdd && test.feature ? test.feature.name : 
-                        test.parentTitle || test.suiteTitle || 
-                        test.parent?.title || test.suite?.title || 'Unknown Feature';
+        const feature = test.isBdd && test.feature ? test.feature.name : test.parentTitle || test.suiteTitle || test.parent?.title || test.suite?.title || 'Unknown Feature'
         // Always try to show steps if available, even for unknown feature
         const steps = config.showSteps && test.steps ? (test.isBdd ? generateBddStepsHtml(test.steps) : generateStepsHtml(test.steps)) : ''
         const featureDetails = test.isBdd && test.feature ? generateBddFeatureHtml(test.feature) : ''