Correction of #81 + Tests

Author: codingseb

CodingSeb.ExpressionEvaluator.Tests/ExpressionEvaluatorTests.cs

@@ -1509,6 +1509,48 @@ public void ExceptionThrowingEvaluation(ExpressionEvaluator evaluator, string ex
 
         #endregion
 
+        #region Bug corrections
+
+        /// <summary>
+        /// To correct #81 Exception is assigned to variable
+        /// With simple variable
+        /// </summary>
+        [Test]
+        [Category("Bug")]
+        [Category("#81")]
+        public void Evaluate_WithException_ThrowsExceptionAndDoesNotAssignItSimpleVariable()
+        {
+            ExpressionEvaluator evaluator = new ExpressionEvaluator();
+
+            evaluator.Variables.Add("exceptionGenerator", new ExceptionGenerator());
+            Assert.Throws<ExpressionEvaluatorSyntaxErrorException>(() => evaluator.Evaluate("result = exceptionGenerator.ThrowAnException()"));
+
+            evaluator.Variables.ContainsKey("result").ShouldBeFalse();
+        }
+
+        /// <summary>
+        /// To correct #81 Exception is assigned to variable
+        /// With InObject
+        /// </summary>
+        [Test]
+        [Category("Bug")]
+        [Category("#81")]
+        public void Evaluate_WithException_ThrowsExceptionAndDoesNotAssignItInObject()
+        {
+            ExpressionEvaluator evaluator = new ExpressionEvaluator();
+
+            ObjectContainer objectContainer = new ObjectContainer();
+
+            evaluator.Variables.Add("exceptionGenerator", new ExceptionGenerator());
+            evaluator.Variables.Add("objectContainer", objectContainer);
+            Assert.Throws<ExpressionEvaluatorSyntaxErrorException>(() => evaluator.Evaluate("objectContainer.AnObjectProperty = exceptionGenerator.ThrowAnException()"));
+
+            objectContainer.AnObjectProperty.ShouldBeOfType(typeof(int));
+            objectContainer.AnObjectProperty.ShouldBe(10);
+        }
+
+        #endregion
+
         #region EvaluateWithSpecificEvaluator
 
         #region TestCasesEvaluateWithSpecificEvaluator

CodingSeb.ExpressionEvaluator.Tests/TestsUtils/ExceptionGenerator.cs

@@ -0,0 +1,12 @@
+using System;
+
+namespace CodingSeb.ExpressionEvaluator.Tests
+{
+    public class ExceptionGenerator
+    {
+        public void ThrowAnException()
+        {
+            throw new Exception();
+        }
+    }
+}

CodingSeb.ExpressionEvaluator.Tests/TestsUtils/ObjectContainer.cs

@@ -0,0 +1,7 @@
+namespace CodingSeb.ExpressionEvaluator.Tests
+{
+    public class ObjectContainer
+    {
+        public object AnObjectProperty { get; set; } = 10;
+    }
+}

CodingSeb.ExpressionEvaluator/ExpressionEvaluator.cs

@@ -2198,6 +2198,9 @@ where method.GetParameters()[0].ParameterType == objType // static extMethod(thi
                                                 varValue = Evaluate(rightExpression);
                                             }
 
+                                            if (varValue is BubbleExceptionContainer exceptionContainer)
+                                                throw exceptionContainer.Exception;
+
                                             stack.Clear();
                                             stack.Push(varValue);
                                         }
@@ -2332,6 +2335,9 @@ where method.GetParameters()[0].ParameterType == objType // static extMethod(thi
                                         cusVarValueToPush = Evaluate(rightExpression);
                                     }
 
+                                    if (cusVarValueToPush is BubbleExceptionContainer exceptionContainer)
+                                        throw exceptionContainer.Exception;
+
                                     stack.Clear();
                                     stack.Push(cusVarValueToPush);
                                 }
@@ -2730,6 +2736,9 @@ protected virtual bool EvaluateIndexing(string expression, Stack<object> stack,
                             valueToPush = Evaluate(rightExpression);
                         }
 
+                        if (valueToPush is BubbleExceptionContainer exceptionContainer)
+                            throw exceptionContainer.Exception;
+
                         if (left is IDictionary<string, object> dictionaryLeft)
                             dictionaryLeft[right] = valueToPush;
                         else