Some tests for OptionInlineNamespacesEvaluationRule + refactoring

Author: codingseb

CodingSeb.ExpressionEvaluator.Tests/ExpressionEvaluatorTests.cs

@@ -1530,6 +1530,46 @@ public static IEnumerable<TestCaseData> TestCasesForExceptionThrowingEvaluation
 
                 #endregion
 
+                #region OptionInlineNamespacesEvaluationRule
+
+                evaluator = new ExpressionEvaluator()
+                {
+                    OptionInlineNamespacesEvaluationRule = InlineNamespacesEvaluationRule.BlockAll
+                };
+
+                yield return new TestCaseData(evaluator, "new System.Collections.Generic.List<string>(){\"-\"}", typeof(ExpressionEvaluatorSyntaxErrorException))
+                    .SetCategory("Options")
+                    .SetCategory("InlineNamespacesEvaluationRule.BlockAll")
+                    .SetCategory("OptionInlineNamespacesEvaluationRule");
+                
+                evaluator = new ExpressionEvaluator()
+                {
+                    OptionInlineNamespacesEvaluationRule = InlineNamespacesEvaluationRule.AllowOnlyInlineNamespacesList
+                };
+
+                evaluator.InlineNamespacesList = evaluator.Namespaces.ToList();
+
+                evaluator.InlineNamespacesList.Remove("System.Collections.Generic");
+
+                yield return new TestCaseData(evaluator, "new System.Collections.Generic.List<string>()", typeof(ExpressionEvaluatorSyntaxErrorException))
+                    .SetCategory("Options")
+                    .SetCategory("InlineNamespacesEvaluationRule.AllowOnlyInlineNamespacesList")
+                    .SetCategory("OptionInlineNamespacesEvaluationRule");
+                
+                evaluator = new ExpressionEvaluator()
+                {
+                    OptionInlineNamespacesEvaluationRule = InlineNamespacesEvaluationRule.BlockOnlyInlineNamespacesList
+                };
+
+                evaluator.InlineNamespacesList = evaluator.Namespaces.ToList();
+                
+                yield return new TestCaseData(evaluator, "new System.Collections.Generic.List<string>(){ \"\" }", typeof(ExpressionEvaluatorSyntaxErrorException))
+                    .SetCategory("Options")
+                    .SetCategory("InlineNamespacesEvaluationRule.BlockOnlyInlineNamespacesList")
+                    .SetCategory("OptionInlineNamespacesEvaluationRule");
+
+                #endregion
+
                 #endregion
 
                 #region TypesToBlock

CodingSeb.ExpressionEvaluator/ExpressionEvaluator.cs

@@ -14,7 +14,6 @@
 using System.Dynamic;
 using System.Globalization;
 using System.Linq;
-using System.Linq.Expressions;
 using System.Reflection;
 using System.Runtime.CompilerServices;
 using System.Runtime.InteropServices;
@@ -567,15 +566,9 @@ public bool OptionCaseSensitiveEvaluationActive
         /// </summary>
         public bool OptionVariablesPersistenceCustomComparer { get; set; }
 
-        private StringComparison StringComparisonForCasing { get; set; } = StringComparison.Ordinal;
+        protected StringComparison StringComparisonForCasing { get; set; } = StringComparison.Ordinal;
 
-        protected StringComparer StringComparerForCasing
-        {
-            get
-            {
-                return OptionCaseSensitiveEvaluationActive ? StringComparer.Ordinal : StringComparer.OrdinalIgnoreCase;
-            }
-        }
+        protected StringComparer StringComparerForCasing => OptionCaseSensitiveEvaluationActive ? StringComparer.Ordinal : StringComparer.OrdinalIgnoreCase;
 
         /// <summary>
         /// If <c>true</c> all numbers without decimal and suffixes evaluations will be done as double
@@ -618,10 +611,7 @@ public CultureInfo CultureInfoForNumberParsing
         /// </summary>
         public string OptionNumberParsingDecimalSeparator
         {
-            get
-            {
-                return optionNumberParsingDecimalSeparator;
-            }
+            get => optionNumberParsingDecimalSeparator;
 
             set
             {
@@ -644,10 +634,7 @@ public string OptionNumberParsingDecimalSeparator
         /// </summary>
         public string OptionNumberParsingThousandSeparator
         {
-            get
-            {
-                return optionNumberParsingThousandSeparator;
-            }
+            get => optionNumberParsingThousandSeparator;
 
             set
             {