gwen.input.data.readOnly

CHANGELOG

@@ -7,12 +7,14 @@
   - With GraalVM instead of Nashorn for JS evaluation
 - Use logback instead of log4j for logging
 - Support `empty` literal in DSLs wherever `blank` is accepted
-- Make all input data read only
+- Make all input data read only by default
 - Internal engine enhancements
   - Replace internal stack of data caches with a single data cache
   - Manage settings internally without exposing them as system properties
   - Implicits managed at lifecycle level
   - Improved scope boundaries
+- Add settings(s):
+  - `gwen.input.data.readOnly`
 - Add implicit variable(s):
   - `gwen.process.name`
 - Add environment variable(s)

src/main/resources/gwen.conf

@@ -80,6 +80,11 @@ gwen {
     }
     mode = "declarative"          # declarative|imperative
   }
+  input {
+    data {
+      readOnly = true
+    }
+  }
   launch {                        # Default launch options
     options {                     # See: https://gweninterpreter.org/docs/settings#launch-settings
       batch  = false

src/main/scala/gwen/core/GwenSettings.scala

@@ -76,6 +76,7 @@ object GwenSettings extends LazyLogging {
     `gwen.video.dir`
     `gwen.video.timeoutSecs`
     `gwen.logLevel.deprecations`
+    `gwen.input.data.readOnly`
   }
 
   /**
@@ -516,6 +517,14 @@ object GwenSettings extends LazyLogging {
     }
   }
 
+  /**
+    * Provides access to the `gwen.input.data.readOnly` setting used to control whether or not
+    * input data should be read only.
+    */
+  def `gwen.input.data.readOnly`: Boolean = {
+    Settings.getBoolean("gwen.input.data.readOnly")
+  }
+
   def `gwen.report.results.files`(options: GwenOptions): List[ResultFile] = {
     val fileSettings = Settings.findAll(_.startsWith(ResultFile.SettingsKey))
     fileSettings.filter(_._1.endsWith(".file")).keys.map(_.drop(ResultFile.SettingsKey.length + 1).dropRight(".file".length)).toList.sorted map { id => 

src/main/scala/gwen/core/eval/GwenREPL.scala

@@ -210,15 +210,15 @@ class GwenREPL[T <: EvalContext](val engine: EvalEngine[T], ctx: T) extends Impl
   }
 
   private def enteringLoop(): Unit = {
-    // repl always runs in imperative and lenient mode
     Settings.setLocal("gwen.feature.mode", FeatureMode.imperative.toString)
     Settings.setLocal("gwen.behavior.rules", BehaviorMode.lenient.toString)
+    Settings.setLocal("gwen.input.data.readOnly", false.toString)
 
   }
   private def exitingLoop(): Unit = {
-    // remove local imperative and lenient mode overrides
     Settings.clearLocal("gwen.feature.mode")
     Settings.clearLocal("gwen.behavior.rules")
+    Settings.clearLocal("gwen.input.data.readOnly")
   }
 
   /** Reads all input lines from the command line. */

src/main/scala/gwen/core/state/Mutability.scala

@@ -17,6 +17,7 @@
 package gwen.core.state
 
 import gwen.core.Errors
+import gwen.core.GwenSettings
 import gwen.core.state.Environment
 import gwen.core.node.gherkin.Annotations
 
@@ -28,11 +29,15 @@ trait Mutability {
 
     def checkMutability(name: String, scopedData: ScopedData): Unit = {
       scopedData.getOpt(key(name)).map(Annotations.valueOf) foreach { annotation =>
-        Errors.immutableModificationError(name, annotation)
+        if (GwenSettings.`gwen.input.data.readOnly`) {
+          Errors.immutableModificationError(name, annotation)
+        }
       }
     }
 
     def setReadOnly(name: String, annotation: Annotations, scopedData: ScopedData): Unit = {
-      scopedData.set(key(name), annotation.toString, force = true)
+      if (GwenSettings.`gwen.input.data.readOnly`) {
+        scopedData.set(key(name), annotation.toString, force = true)
+      }
     }
 }

src/test/scala/gwen/core/GwenSettingsTest.scala

@@ -76,5 +76,6 @@ class GwenSettingsTest extends BaseTest with Matchers {
     GwenSettings.`gwen.error.messages.inline.locators` should be (false)
     GwenSettings.`gwen.logLevel.deprecations` should be (Level.SEVERE)
     GwenSettings.`gwen.report.results.files`(GwenOptions()).length should be (0)
+    GwenSettings.`gwen.input.data.readOnly` should be (true)
   }
 }