fix EvilBeaver#1675: возвращаемый тип ЗагрузитьСценарий() и обработка ошибок

Author: Mr-Rm

src/OneScript.StandardLibrary/DynamicLoadingFunctions.cs

@@ -11,11 +11,13 @@ This Source Code Form is subject to the terms of the
 using OneScript.Exceptions;
 using OneScript.Execution;
 using OneScript.Language;
+using OneScript.Localization;
 using OneScript.StandardLibrary.Collections;
 using OneScript.StandardLibrary.NativeApi;
 using ScriptEngine;
 using ScriptEngine.Machine;
 using ScriptEngine.Machine.Contexts;
+using System;
 
 namespace OneScript.StandardLibrary
 {
@@ -48,19 +50,9 @@ public void AttachScript(IBslProcess process, string path, string typeName)
             {
                 _engine.AttachedScriptsFactory.AttachByPath(compiler, path, typeName, process);
             }
-            catch (SyntaxErrorException e)
-            {
-                // обернем в RuntimeException
-                throw new RuntimeException(
-                    Locale.NStr("ru = 'Ошибка компиляции подключаемого скрипта';en = 'Error compiling attached script'"),
-                    e);
-            }
-            catch (Compilation.CompilerException e)
-            {
-                // обернем в RuntimeException
-                throw new RuntimeException(
-                    Locale.NStr("ru = 'Ошибка компиляции подключаемого скрипта';en = 'Error compiling attached script'"),
-                    e);
+            catch (Exception e) when (e is SyntaxErrorException || e is CompilerException)
+            {
+                throw ScriptCompilingException(e);
             }
         }
 
@@ -97,20 +89,10 @@ public UserScriptContextInstance LoadScriptFromString(IBslProcess process,
                 try
                 {
                     return _engine.AttachedScriptsFactory.LoadFromString(compiler, code, process, extData);
-                }
-                catch (SyntaxErrorException e)
-                {
-                    // обернем в RuntimeException
-                    throw new RuntimeException(
-                        Locale.NStr("ru = 'Ошибка компиляции подключаемого скрипта';en = 'Error compiling attached script'"),
-                        e);
-                }
-                catch (CompilerException e)
-                {
-                    // обернем в RuntimeException
-                    throw new RuntimeException(
-                        Locale.NStr("ru = 'Ошибка компиляции подключаемого скрипта';en = 'Error compiling attached script'"),
-                        e);
+                }
+                catch (Exception e) when (e is SyntaxErrorException || e is CompilerException)
+                {
+                    throw ScriptCompilingException(e);
                 }
             }
         }
@@ -128,22 +110,28 @@ public UserScriptContextInstance LoadScriptFromString(IBslProcess process,
         /// // В коде скрипта somescript.os будет доступна глобальная переменная "ЧислоПи"
         /// Объект = ЗагрузитьСценарий("somescript.os", Контекст);</example>
         [ContextMethod("ЗагрузитьСценарий", "LoadScript")]
-        public IRuntimeContextInstance LoadScript(IBslProcess process, string path, StructureImpl externalContext = null)
-        {
+        public UserScriptContextInstance LoadScript(IBslProcess process, string path, StructureImpl externalContext = null)
+        {
             var compiler = _engine.GetCompilerService();
-            if(externalContext == null)
-                return _engine.AttachedScriptsFactory.LoadFromPath(compiler, path, process);
-            else
-            {
-                ExternalContextData extData = new ExternalContextData();
-
-                foreach (var item in externalContext)
+            try
+            {
+                if(externalContext == null)
+                    return _engine.AttachedScriptsFactory.LoadFromPath(compiler, path, process);
+                else
                 {
-                    extData.Add(item.Key.ToString()!, item.Value);
-                }
+                    ExternalContextData extData = new ExternalContextData();
 
-                return _engine.AttachedScriptsFactory.LoadFromPath(compiler, path, extData, process);
+                    foreach (var item in externalContext)
+                    {
+                        extData.Add(item.Key.ToString()!, item.Value);
+                    }
 
+                    return _engine.AttachedScriptsFactory.LoadFromPath(compiler, path, extData, process);
+                }
+            }
+            catch (Exception e) when (e is SyntaxErrorException || e is CompilerException)
+            {
+                throw ScriptCompilingException(e);
             }
         }
 
@@ -189,5 +177,12 @@ public bool AttachAddIn(string dllPath, string name = "", NativeApiEnums type =
                 return NativeApiFactory.Register(dllPath, name, _engine.TypeManager);
             }
         }
+
+        private static RuntimeException ScriptCompilingException(Exception e)
+        {
+            return new RuntimeException(BilingualString.Localize(
+                "Ошибка компиляции подключаемого сценария:\n",
+                "Error compiling attached script:\n") + e.Message);
+        }
     }
 }

tests/native-lib/test-native-use.os

@@ -15,7 +15,7 @@
     Тесты.Добавить("ТестДолжен_ПроверитьВызовМетодовСОшибочноПропущеннымиПараметрами");
     Тесты.Добавить("ТестДолжен_ПроверитьВызовМетодовСЛишнимиПараметрами");
     Тесты.Добавить("ТестДолжен_ПроверитьВызовКонструктораСПараметрамиПоУмолчанию");
-   
+    Тесты.Добавить("ТестДолжен_ПроверитьПодключениеСценарияСОбъявленнойПеременной");
     Возврат Тесты;
 
 КонецФункции
@@ -92,3 +92,10 @@
 Процедура ТестДолжен_ПроверитьВызовКонструктораСПараметрамиПоУмолчанию() Экспорт
     Сценарий = Новый ПараметрыКонструктораПоУмолчанию("Парам1");
 КонецПроцедуры
+
+Процедура ТестДолжен_ПроверитьПодключениеСценарияСОбъявленнойПеременной() Экспорт
+	Перем Сценарий;
+	Сценарий = ЗагрузитьСценарий(ОбъединитьПути(ТекущийСценарий().Каталог, "Классы", "ТестовыйСценарий.os"));
+	юТест.ПроверитьРавенство(Сценарий.Поле1, 1);
+КонецПроцедуры
+