Added a tests for variables and functions with names containing Unicode characters

Taritsyn · Taritsyn · commit 1b54aeb9a58b · 2018-11-28T16:39:46.000+03:00
diff --git a/build/common.props b/build/common.props
@@ -3,7 +3,7 @@
 		<Copyright>Copyright © 2012-2018 Andrey Taritsyn</Copyright>
 	</PropertyGroup>
 
-	<PropertyGroup Condition=" '$(TargetFramework)' == 'net40-client' Or '$(TargetFramework)' == 'net45' Or '$(TargetFramework)' == 'net451' ">
+	<PropertyGroup Condition=" '$(TargetFramework)' == 'net40-client' Or '$(TargetFramework)' == 'net45' Or '$(TargetFramework)' == 'net451' Or '$(TargetFramework)' == 'net46' ">
 		<DefineConstants>$(DefineConstants);NETFULL</DefineConstants>
 	</PropertyGroup>
 
diff --git a/test/MsieJavaScriptEngine.Test.Common/CommonTestsBase.cs b/test/MsieJavaScriptEngine.Test.Common/CommonTestsBase.cs
@@ -638,6 +638,29 @@ public virtual void CallingOfFunctionWithManyParametersAndUnicodeStringResultIsC
 			Assert.AreEqual("Привет, Петя!", output);
 		}
 
+		[Test]
+		public virtual void CallingOfFunctionWithNameContainingUnicodeCharactersIsCorrect()
+		{
+			// Arrange
+			const string functionCode = @"function сумма(число1, число2) {
+	var результат = число1 + число2;
+
+	return результат;
+}";
+
+			// Act
+			int output;
+
+			using (var jsEngine = CreateJsEngine())
+			{
+				jsEngine.Execute(functionCode);
+				output = jsEngine.CallFunction<int>("сумма", 678, 711);
+			}
+
+			// Assert
+			Assert.AreEqual(1389, output);
+		}
+
 		#endregion
 
 		#region Getting, setting and removing variables
@@ -916,6 +939,30 @@ public virtual void RemovingVariableIsCorrect()
 			Assert.IsFalse(variableAfterRemovingExists);
 		}
 
+		[Test]
+		public virtual void RemovingVariableWithNameContainingUnicodeCharactersIsCorrect()
+		{
+			// Arrange
+			const string variableName = "цена";
+			const double input = 6780.00;
+
+			// Act
+			bool variableBeforeRemovingExists;
+			bool variableAfterRemovingExists;
+
+			using (var jsEngine = CreateJsEngine())
+			{
+				jsEngine.SetVariableValue(variableName, input);
+				variableBeforeRemovingExists = jsEngine.HasVariable(variableName);
+				jsEngine.RemoveVariable(variableName);
+				variableAfterRemovingExists = jsEngine.HasVariable(variableName);
+			}
+
+			// Assert
+			Assert.IsTrue(variableBeforeRemovingExists);
+			Assert.IsFalse(variableAfterRemovingExists);
+		}
+
 		#endregion
 
 		#region Script interruption
