Merge pull request EvilBeaver#1604 from asosnoviy/fix/strreplase

EvilBeaver#1603 strReplace

Author: EvilBeaver

src/OneScript.Native/Runtime/BuiltInFunctions.cs

@@ -1,4 +1,4 @@
-/*----------------------------------------------------------
+/*----------------------------------------------------------
 This Source Code Form is subject to the terms of the
 Mozilla Public License, v.2.0. If a copy of the MPL
 was not distributed with this file, You can obtain one
@@ -178,7 +178,7 @@ public static int ChrCode(string strChar, int? position = null)
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
         [ContextMethod("СтрЗаменить", "StrReplace")]
         public static string StrReplace(string sourceString, string searchVal, string newVal) =>
-            sourceString.Replace(searchVal, newVal);
+            !string.IsNullOrEmpty(searchVal) ? sourceString.Replace(searchVal, newVal):sourceString;
 
         [ContextMethod("СтрПолучитьСтроку", "StrGetLine")]
         public static string StrGetLine(string strArg, int lineNumber)

tests/global-funcs.os

@@ -1,4 +1,4 @@
-///////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////
 //
 // Тест проверки поведения некоторых глобальных функций
 // 
@@ -102,6 +102,8 @@
 	ВсеТесты.Добавить("Тест_ДолженПроверитьНСтрВозвращаетПервуюСтроку");
 	ВсеТесты.Добавить("Тест_ДолженПроверитьПолучениеТекущейДатыВМиллисекундах");
 	ВсеТесты.Добавить("Тест_ДолженПроверитьОкруглениеТекущейДатыДоСекунд");
+
+	ВсеТесты.Добавить("Тест_ДолженПроверитьСтрЗаменить");
 	Возврат ВсеТесты;
 КонецФункции
 
@@ -1185,4 +1187,29 @@
 	ДатаИсходная = ТекущаяДата();
 	ДатаСозданнаяВручную = Дата(Формат(ДатаИсходная, "ДФ=ггггММддЧЧммсс"));
 	юТест.ПроверитьРавенство(ДатаИсходная, ДатаСозданнаяВручную);
-КонецПроцедуры
+КонецПроцедуры
+
+Процедура Тест_ДолженПроверитьСтрЗаменить() Экспорт
+
+	Тест = СтрЗаменить("123", "1", "4");
+	юТест.ПроверитьРавенство("423", Тест);
+
+	Тест = СтрЗаменить("", "1", "4");
+	юТест.ПроверитьРавенство("", Тест);
+
+	Тест = СтрЗаменить("123", "", "4");
+	юТест.ПроверитьРавенство("123", Тест);
+
+	Тест = СтрЗаменить("123", "1", "");
+	юТест.ПроверитьРавенство("23", Тест);
+
+	Тест = СтрЗаменить("123", "", "");
+	юТест.ПроверитьРавенство("123", Тест);
+	
+	Тест = СтрЗаменить("", "", "123");
+	юТест.ПроверитьРавенство("", Тест);
+
+	Тест = СтрЗаменить("", "", "");
+	юТест.ПроверитьРавенство("", Тест);
+
+КонецПроцедуры