MathematicaEvaluator should get immutable and convert it to mutable to avoid out-side interference.

Author: tomerghelber

src/main/scala/com/github/tomerghelber/mathematica/eval/MathematicaEvaluator.scala

@@ -9,9 +9,15 @@ import scala.collection.mutable
  * @author user
  * @since 01-Nov-19
  */
-class MathematicaEvaluator(globalEnvironment: mutable.Map[String, ASTNode]= mutable.Map.empty[String, ASTNode])
+class MathematicaEvaluator private (globalEnvironment: mutable.Map[String, ASTNode])
   extends LazyLogging {
 
+  /* --- constructors --- */
+
+  def this(startEnvironment: Map[String, ASTNode]=Map.empty) = {
+    this(mutable.Map[String, ASTNode](startEnvironment.toSeq:_*))
+  }
+
   /* --- publics --- */
 
   def eval(node: ASTNode): ASTNode = {

src/test/scala/com/github/tomerghelber/mathematica/eval/MathematicaEvaluatorSpec.scala

@@ -7,8 +7,6 @@ import org.scalatestplus.scalacheck.ScalaCheckPropertyChecks
 import com.github.tomerghelber.mathematica.ast.{FunctionNode, NumberNode, SymbolNode}
 import com.github.tomerghelber.mathematica.parser.MathematicaParser
 
-import scala.collection.mutable
-
 /**
  * @author user
  * @since 01-Nov-19
@@ -35,7 +33,7 @@ class MathematicaEvaluatorSpec extends FunSpec with Matchers with ScalaCheckProp
 
     it("symbol evaluated") {
       forAll { (symbol: SymbolNode, expected: NumberNode) =>
-        val eval = new MathematicaEvaluator(mutable.Map((symbol.value, expected)))
+        val eval = new MathematicaEvaluator(Map((symbol.value, expected)))
         val actual = eval.eval(symbol)
         actual shouldBe expected
       }