Allow overriding env

This fixes #792.

Author: badeball

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 All notable changes to this project will be documented in this file.
 
+## Unreleased
+
+- Allow overriding env using tags, fixes [#792](https://github.com/badeball/cypress-cucumber-preprocessor/issues/792).
+
 ## v12.0.0
 
 Breaking changes:

features/issues/792.feature

@@ -0,0 +1,20 @@
+# https://github.com/badeball/cypress-cucumber-preprocessor/issues/792
+
+Feature: overriding env
+  Scenario: overriding env
+    Given a file named "cypress/e2e/foo/bar.feature" with:
+      """
+      @env(foo="bar")
+      Feature: a feature
+        Scenario: a scenario
+          Given a step
+      """
+    And a file named "cypress/e2e/foo/bar.js" with:
+      """
+      const { Given } = require("@badeball/cypress-cucumber-preprocessor");
+      Given("a step", () => {
+        expect(Cypress.env("foo")).to.equal("bar");
+      });
+      """
+    When I run cypress
+    Then it passes

lib/create-tests.ts

@@ -373,14 +373,20 @@ function createPickle(
     remainingSteps: [...steps],
   };
 
-  const env = { [INTERNAL_PROPERTY_NAME]: internalProperties };
+  const internalEnv = { [INTERNAL_PROPERTY_NAME]: internalProperties };
 
   const suiteOptions = tags
     .filter(looksLikeOptions)
     .map(tagToCypressOptions)
     .reduce(Object.assign, {});
 
-  it(scenarioName, { env, ...suiteOptions }, function () {
+  if (suiteOptions.env) {
+    Object.assign(suiteOptions.env, internalEnv);
+  } else {
+    suiteOptions.env = internalEnv;
+  }
+
+  it(scenarioName, suiteOptions, function () {
     const { remainingSteps, testCaseStartedId } = retrieveInternalProperties();
 
     assignRegistry(registry);