Merge branch 'release/0.9.7'

oscript-library · Jul 24, 2017 · 5de9c8c · 5de9c8c
2 parents c9a2579 + b098edf
commit 5de9c8c
Show file tree

Hide file tree

Showing 5 changed files with 92 additions and 50 deletions.
diff --git a/.vscode/tasks.json b/.vscode/tasks.json
@@ -1,3 +1,11 @@
+// Available variables which can be used inside of strings.
+// ${workspaceRoot}: the root folder of the team
+// ${file}: the current opened file
+// ${fileBasename}: the current opened file's basename
+// ${fileDirname}: the current opened file's dirname
+// ${fileExtname}: the current opened file's extension
+// ${cwd}: the current working directory of the spawned process
+
 {
     "version": "2.0.0",
     "_runner": "terminal",
@@ -10,11 +18,11 @@
         "args": ["-c"]
     },
     "isShellCommand": true,
-    "showOutput": "silent",
+    // "showOutput": "silent",
     "_runner": "terminal",
     "tasks": [
         {
-            "taskName": "Testing project",
+            "taskName": "Testing project (1testrunner)",
             "args": [
                  "1testrunner",
                 "-runall",
@@ -36,17 +44,17 @@
             }
         },
         {
-            "taskName": "Testing current test-file",
+            "taskName": "Testing project (opm test)",
             "args": [
-                 "1testrunner",
-                "-run",
-                "${file}"
+                 "oscript",
+                 "src/opm.os",
+                "test"
             ],
             "echoCommand": true,
             "showOutput": "always",
             "suppressTaskName": true,
-            "isBuildCommand": false,
-            "isTestCommand": true,
+            // "isBuildCommand": false,
+            "isTestCommand": false,
             "problemMatcher": {
                 "fileLocation": "absolute",
                 "pattern": {
@@ -58,22 +66,33 @@
             }
         },
         {
-            "taskName": "Opm: package build",
+            "taskName": "Testing current test-file",
             "args": [
-                 "opm",
-                "build",
-                "${workspaceRoot}"
+                 "1testrunner",
+                "-run",
+                "${file}"
             ],
             "echoCommand": true,
             "showOutput": "always",
             "suppressTaskName": true,
-            "isBuildCommand": false
+            "isBuildCommand": false,
+            "isTestCommand": true,
+            "problemMatcher": {
+                "fileLocation": "absolute",
+                "pattern": {
+                    "regexp": "{Модуль\\s+(.+)\\s\\/\\s.*:\\s+(\\d+)\\s+\\/\\s+([^{]*)",
+                    "file": 1,
+                    "location": 2,
+                    "message": 3
+                }
+            }
         },
         {
             "taskName": "Exec all features",
             "args": [
                  "1bdd",
                 "${workspaceRoot}/features",
+                "-fail-fast", 
                 "-out", 
                 "${workspaceRoot}/exec.log"
             ],
@@ -145,20 +164,18 @@
             }
         },
         {
-            "taskName": "Exec feature + debug",
+            "taskName": "Generate feature steps",
             "args": [
                  "1bdd",
+                "gen",
                 "${file}",
-                "-fail-fast", 
-                "-verbose", 
-                "on", 
                 "-out", 
                 "${workspaceRoot}/exec.log"
             ],
             "echoCommand": true,
             "showOutput": "always",
             "suppressTaskName": true,
-            // "isBuildCommand": false,
+            "isBuildCommand": false,
             "isTestCommand": false,
             "problemMatcher": {
                 "fileLocation": "absolute",
@@ -171,28 +188,16 @@
             }
         },
         {
-            "taskName": "Generate feature steps",
+            "taskName": "Opm: package build",
             "args": [
-                 "1bdd",
-                "gen",
-                "${file}",
-                "-out", 
-                "${workspaceRoot}/exec.log"
+                "opm",
+                "build",
+                "${workspaceRoot}"
             ],
             "echoCommand": true,
             "showOutput": "always",
             "suppressTaskName": true,
-            "isBuildCommand": false,
-            "isTestCommand": false,
-            "problemMatcher": {
-                "fileLocation": "absolute",
-                "pattern": {
-                    "regexp": "{Модуль\\s+(.+)\\s\\/\\s.*:\\s+(\\d+)\\s+\\/\\s+([^{]*)",
-                    "file": 1,
-                    "location": 2,
-                    "message": 3
-                }
-            }
+            "isBuildCommand": false
         },
         {
             "taskName": "OneScript: compile",

diff --git a/features/step_definitions/Настройки.os b/features/step_definitions/Настройки.os
@@ -32,11 +32,11 @@
 //я читаю настройки из файла "opm.cfg"
 Процедура ЯЧитаюНастройкиИзФайла(Знач ПутьФайла) Экспорт
 	Файл = Новый Файл(ОбъединитьПути(ТекущийКаталог(), ПутьФайла));
-	Ожидаем.Что(Файл.Существует(), СтрШаблон("Ожидали, что файл <%1> существует, а это не так!", Файл.ПолноеИмя)).ЭтоИстина();
+	// Ожидаем.Что(Файл.Существует(), СтрШаблон("Ожидали, что файл <%1> существует, а это не так!", Файл.ПолноеИмя)).ЭтоИстина();
 
-	НастройкиПриложения.УстановитьФайлНастроек(Файл.ПолноеИмя);
+	НастройкиПриложенияOpm.УстановитьФайлНастроек(Файл.ПолноеИмя);
 
-	Настройки = НастройкиПриложения.Получить();
+	Настройки = НастройкиПриложенияOpm.Получить();
 	БДД.СохранитьВКонтекст(КлючКонтекста(), Настройки);
 КонецПроцедуры
 
@@ -76,13 +76,15 @@
 КонецФункции
 
 Функция КонвертироватьЗначениеПриНеобходимости(Знач ЗначениеНастройки)
-	ЗначениеНастройки = НРег(ЗначениеНастройки);
-	Если ЗначениеНастройки = "false" или ЗначениеНастройки = "ложь" Тогда
-		ЗначениеНастройки = Ложь;
-	ИначеЕсли ЗначениеНастройки = "true" или ЗначениеНастройки = "истина" Тогда
-		ЗначениеНастройки = Истина;
-	ИначеЕсли ЗначениеНастройки = "неопределено" Тогда
-		ЗначениеНастройки = Неопределено;
+	Если ТипЗнч(ЗначениеНастройки) = Тип("Строка") Тогда
+		ЗначениеНастройки = НРег(ЗначениеНастройки);
+		Если ЗначениеНастройки = "false" или ЗначениеНастройки = "ложь" Тогда
+			ЗначениеНастройки = Ложь;
+		ИначеЕсли ЗначениеНастройки = "true" или ЗначениеНастройки = "истина" Тогда
+			ЗначениеНастройки = Истина;
+		ИначеЕсли ЗначениеНастройки = "неопределено" Тогда
+			ЗначениеНастройки = Неопределено;
+		КонецЕсли;
 	КонецЕсли;
 	Возврат ЗначениеНастройки;
 КонецФункции

diff --git a/features/Настройки.feature b/features/Настройки.feature
@@ -21,7 +21,7 @@
     И значение настройки "Прокси.ИспользоватьПрокси" равно "false"
     И значение настройки "Прокси.ПроксиПоУмолчанию" равно "false"
     И значение настройки "Прокси.Сервер" равно ""
-    И значение настройки "Прокси.Порт" равно ""
+    И значение настройки "Прокси.Порт" равно 0
     И значение настройки "Прокси.Пользователь" равно ""
     И значение настройки "Прокси.Пароль" равно ""
     И значение настройки "Прокси.ИспользоватьАутентификациюОС" равно "false"
@@ -33,7 +33,19 @@
     И значение настройки "Прокси.ИспользоватьПрокси" равно "false"
     И значение настройки "Прокси.ПроксиПоУмолчанию" равно "Истина"
     И значение настройки "Прокси.Сервер" равно "Неопределено"
-    И значение настройки "Прокси.Порт" равно "Неопределено"
+    И значение настройки "Прокси.Порт" равно 0
+    И значение настройки "Прокси.Пользователь" равно "Неопределено"
+    И значение настройки "Прокси.Пароль" равно "Неопределено"
+    И значение настройки "Прокси.ИспользоватьАутентификациюОС" равно "false"
+
+Сценарий: Получение значений по умолчанию, если файл настроек отсутствует
+    Дано Файл "opm.cfg" не существует
+    Когда я читаю настройки из файла "opm.cfg"
+    Тогда значение настройки "СоздаватьShСкриптЗапуска" равно "false"
+    И значение настройки "Прокси.ИспользоватьПрокси" равно "false"
+    И значение настройки "Прокси.ПроксиПоУмолчанию" равно "Истина"
+    И значение настройки "Прокси.Сервер" равно "Неопределено"
+    И значение настройки "Прокси.Порт" равно 0
     И значение настройки "Прокси.Пользователь" равно "Неопределено"
     И значение настройки "Прокси.Пароль" равно "Неопределено"
     И значение настройки "Прокси.ИспользоватьАутентификациюОС" равно "false"
diff --git a/src/Модули/КонстантыOpm.os b/src/Модули/КонстантыOpm.os
@@ -23,4 +23,4 @@
 ПутьВЗапасномХранилище    = "/download/";
 ЛокальныйКаталогУстановкиПакетов = "oscript_modules";
 ИмяЛога 				   = "oscript.app.opm";
-ВерсияПродукта = "0.9.6";
+ВерсияПродукта = "0.9.7";
diff --git a/src/Модули/НастройкиПриложенияOpm.os b/src/Модули/НастройкиПриложенияOpm.os
@@ -43,6 +43,8 @@
 	// TODO сделать конвертацию терминов json в русские свойства настроек
 
 	мНастройки = ЗаполнитьНесуществующиеНастройкиПоУмолчанию(Настройки);
+
+	УстановитьЧисловоеЗначениеПортаСервера(мНастройки);
 КонецПроцедуры
 
 Функция ПрочитатьФайл(Знач Путь)
@@ -95,14 +97,35 @@
 	СтруктураПрокси.Вставить("ИспользоватьПрокси", Ложь);
 	СтруктураПрокси.Вставить("ПроксиПоУмолчанию", Истина);
 	СтруктураПрокси.Вставить("Сервер");
-	СтруктураПрокси.Вставить("Порт");
+	СтруктураПрокси.Вставить("Порт", 0);
 	СтруктураПрокси.Вставить("Пользователь");
 	СтруктураПрокси.Вставить("Пароль");
 	СтруктураПрокси.Вставить("ИспользоватьАутентификациюОС",Ложь);
 
 	Возврат СтруктураПрокси;
 КонецФункции	
 
+Процедура УстановитьЧисловоеЗначениеПортаСервера(Настройки)
+	Для каждого Настройка Из Настройки Цикл
+		Значение = Настройка.Значение;
+		Если НРег(Настройка.Ключ) = "порт" Тогда
+			Если Не ЗначениеЗаполнено(Значение) Тогда
+				Значение = 0;
+			Иначе
+				Значение = Число(Значение);
+			КонецЕсли;
+		Иначе
+
+			Если ТипЗнч(Значение) = Тип("Структура") Тогда
+				УстановитьЧисловоеЗначениеПортаСервера(Значение);
+			КонецЕсли;
+
+		КонецЕсли;
+
+		Настройки.Вставить(Настройка.Ключ, Значение);
+	КонецЦикла;
+КонецПроцедуры
+
 Процедура СохранитьНастройки(Знач Параметры) Экспорт
 	Получить();
 	ЗаполнитьНастройкиИзПараметров(Параметры);
@@ -128,7 +151,7 @@
 
 	Если мНастройки.Прокси.ПроксиПоУмолчанию Тогда
 			мНастройки.Прокси.Сервер       = "";
-			мНастройки.Прокси.Порт         = "";
+			мНастройки.Прокси.Порт         = 0;
 			мНастройки.Прокси.Пользователь = "";
 			мНастройки.Прокси.Пароль       = "";
 	КонецЕсли;