added logic to process duplicates (#8)

Author: AlexGalichenko

.github/workflows/pull-request.yml

@@ -24,5 +24,5 @@ jobs:
         uses: mikepenz/action-junit-report@v4
         if: always()
         with:
-          report_paths: './test/results.xml'
+          report_paths: './test/report/results.xml'
           fail_on_failure: true

.gitignore

@@ -2,3 +2,4 @@ node_modules/
 adapter/
 playwright-report/
 test-results/
+test/report/

CHANGELOG.md

@@ -12,7 +12,8 @@ Check [Keep a Changelog](http://keepachangelog.com/) for recommendations on how
 
 ## [0.3.0]
 - :rocket: added _defineCucumber_ util function
-
+- :rocket: added logic to process duplicates
+  
 ## [0.2.0]
 - :rocket: updated fixture implementation
 

src/adapter.spec.ts

@@ -31,7 +31,7 @@ for (const feature of features) {
         test.beforeEach(world.init);
 
         for (const testCase of tests) {
-            const tag = testCase.tags.map(tag => tag.name);
+            const tag = testCase.tags.map((tag: { name: string }) => tag.name);
             test(testCase.name, { tag }, async () => {
                 for (const beforeHook of supportCodeLibrary.beforeTestCaseHookDefinitions) {
                     if (beforeHook.appliesToTestCase(testCase)) {

src/loader.ts

@@ -11,14 +11,29 @@ const builder = new AstBuilder(uuidFn)
 const matcher = new GherkinClassicTokenMatcher();
 const parser = new Parser(builder, matcher)
 
+function duplicates(tests: any[]) {
+    const counts: Record<string, number> = {};
+    return tests.map(item => {
+        const name = item.name;
+        if (!counts[name]) {
+            counts[name] = 0;
+        }
+        if (counts[name]) {
+            item.name = `${item.name} #${counts[item.name] + 1}`;
+        }
+        counts[item.name]++;
+        return item;
+    })
+}
+
 export function loadFeatures(globPattern: string[]) {
     const files = globSync(globPattern);
     return files.map(file => {
         const filePath = resolve(file);
         const gherkinDocument = parser.parse(readFileSync(filePath, 'utf-8'));
         return {
             feature: gherkinDocument.feature?.name,
-            tests: compile(gherkinDocument, file, uuidFn)
+            tests: duplicates(compile(gherkinDocument, file, uuidFn) as any)
         }
     });
 }

test/features/feature.feature

@@ -31,3 +31,9 @@ Feature: test feature
 
   Scenario: attach
     Given attach
+
+  Scenario: duplicate
+    Given log
+
+  Scenario: duplicate
+    Given log

test/features/feature2.feature

@@ -0,0 +1,39 @@
+Feature: test feature 2
+
+  Background:
+    Given simple step
+
+  Scenario: simple scenario
+    Given open 'https://google.com' url
+
+  Scenario Outline: simple scenario <example>
+    Given open '<example>' url
+
+    Examples:
+      | example                    |
+      | https://www.wikipedia.org/ |
+      | https://www.saucedemo.com/ |
+
+  Scenario: data table
+    Given data table step
+      | 1 |
+      | 2 |
+
+  Scenario: multiline
+    Given multiline step
+      """
+      first
+      second
+      """
+
+  Scenario: log
+    Given log
+
+  Scenario: attach
+    Given attach
+
+  Scenario: duplicate
+    Given log
+
+  Scenario: duplicate
+    Given log

test/playwright-e2e.config.ts

@@ -1,8 +1,11 @@
-import {defineConfig, devices} from '@playwright/test';
-import {resolve} from 'node:path';
+import { defineConfig, devices } from '@playwright/test';
+import { resolve } from 'node:path';
+import { defineCucumber } from '../src/defineCucumber';
 
-process.env.CONFIG = 'test/cucumber.ts';
-process.env.PROFILE = 'default';
+defineCucumber({
+    config: 'test/cucumber.ts',
+    profile: 'default'
+});
 
 /**
  * Read environment variables from file.
@@ -24,7 +27,10 @@ export default defineConfig({
     /* Opt out of parallel tests on CI. */
     workers: process.env.CI ? 1 : undefined,
     /* Reporter to use. See https://playwright.dev/docs/test-reporters */
-    reporter: [['junit', { outputFile: 'results.xml' }]],
+    reporter: [
+        ['html', { outputFolder: 'report' }],
+        ['junit', { outputFile: 'report/report.xml' }]
+    ],
     /* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */
     use: {
         /* Base URL to use in actions like `await page.goto('/')`. */

test/playwright.config.ts

@@ -21,7 +21,10 @@ export default defineConfig({
     /* Opt out of parallel tests on CI. */
     workers: process.env.CI ? 1 : 1,
     /* Reporter to use. See https://playwright.dev/docs/test-reporters */
-    reporter: 'html',
+    reporter: [
+        ['html', { outputFolder: 'report' }],
+        ['junit', { outputFile: 'report/report.xml' }]
+    ],
     /* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */
     use: {
         /* Base URL to use in actions like `await page.goto('/')`. */