Fix #752 + tests

src/ScriptEngine/Compiler/Compiler.cs

@@ -622,21 +622,47 @@ private ParameterDefinition BuildParameterDefinition()
             if (_lastExtractedLexem.Token == Token.Equal)
             {
                 param.HasDefaultValue = true;
+                param.DefaultValueIndex = BuildDefaultParameterValue();
+            }
+
+            return param;
+        }
+
+        private int BuildDefaultParameterValue()
+        {
+            NextToken();
+
+            bool hasSign = false;
+            bool signIsMinus = _lastExtractedLexem.Token == Token.Minus;
+            if (signIsMinus || _lastExtractedLexem.Token == Token.Plus)
+            {
+                hasSign = true;
                 NextToken();
-                if (IsLiteral(ref _lastExtractedLexem))
-                {
-                    var cd = CreateConstDefinition(ref _lastExtractedLexem);
-                    var num = GetConstNumber(ref cd);
-                    param.DefaultValueIndex = num;
-                    NextToken();
-                }
-                else
+            }
+
+            if (IsLiteral(ref _lastExtractedLexem))
+            {
+                var cd = CreateConstDefinition(ref _lastExtractedLexem);
+                if (hasSign)
                 {
-                    throw CompilerException.UnexpectedOperation();
+                    if (_lastExtractedLexem.Type == LexemType.NumberLiteral && signIsMinus)
+                    {
+                        cd.Presentation = '-' + cd.Presentation;
+                    }
+                    else if (_lastExtractedLexem.Type == LexemType.StringLiteral
+                          || _lastExtractedLexem.Type == LexemType.DateLiteral)
+                    {
+                        throw CompilerException.NumberExpected();
+                    }
                 }
-            }
 
-            return param;
+                NextToken();
+                return GetConstNumber(ref cd);
+            }
+            else
+            {
+                throw CompilerException.LiteralExpected();
+            }
         }
 
         private void DispatchMethodBody()

src/ScriptEngine/Compiler/CompilerExceptions.cs

@@ -128,6 +128,16 @@ public static CompilerException ExpressionExpected()
             return new CompilerException("Ожидается выражение");
         }
 
+        public static CompilerException LiteralExpected()
+        {
+            return new CompilerException("Ожидается константа");
+        }
+
+        public static CompilerException NumberExpected()
+        {
+            return new CompilerException("Ожидается числовая константа");
+        }
+
     }
 
     public class ExtraClosedParenthesis : CompilerException

tests/engine-behaviors.os

@@ -46,7 +46,7 @@
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьПропускПараметров");
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьФинальнуюЗапятуюВОбъявленииМетода");
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьВозможностьРазыменованияПослеТернарногоОператора");
-
+	ВсеТесты.Добавить("ТестДолжен_ПроверитьМинусВЗначенииПарамераПоУмолчанию");
 
 	Возврат ВсеТесты;
 
@@ -524,3 +524,31 @@
 	Сценарий = ЗагрузитьСценарийИзСтроки(Код);
 
 КонецПроцедуры
+
+Процедура ТестДолжен_ПроверитьМинусВЗначенииПарамераПоУмолчанию() Экспорт
+
+	Сценарий = ЗагрузитьСценарийИзСтроки("Функция Метод( Парам = -1 ) Экспорт Возврат Парам; КонецФункции");
+	юТест.ПроверитьРавенство(Сценарий.Метод(), -1, "Параметр со значением по умолчанию -1");
+
+	Сценарий = ЗагрузитьСценарийИзСтроки("Функция Метод( Парам = - //перевод строки" " 2 ) Экспорт Возврат Парам; КонецФункции");
+	юТест.ПроверитьРавенство(Сценарий.Метод(), -2, "Параметр со значением по умолчанию -2");
+
+	Сценарий = ЗагрузитьСценарийИзСтроки("Функция Метод( Парам = -Истина ) Экспорт Возврат Парам; КонецФункции");
+	юТест.ПроверитьРавенство(Сценарий.Метод(), Истина, "Параметр со значением по умолчанию Истина");
+
+	Сценарий = ЗагрузитьСценарийИзСтроки("Функция Метод( Парам = -Неопределено ) Экспорт Возврат Парам; КонецФункции");
+	юТест.ПроверитьРавенство(Сценарий.Метод(), Неопределено, "Параметр со значением по умолчанию Неопределено");
+
+	Сценарий = ЗагрузитьСценарийИзСтроки("Функция Метод( Парам = -Null ) Экспорт Возврат Парам; КонецФункции");
+	юТест.ПроверитьРавенство(Сценарий.Метод(), Null, "Параметр со значением по умолчанию Null");
+
+	Ошибка = "Ожидается числовая константа";
+	юТест.ПроверитьКодСОшибкой("Процедура Метод( Парам = - ""1"" ) КонецПроцедуры", Ошибка );
+	юТест.ПроверитьКодСОшибкой("Процедура Метод( Парам = -'00010101') КонецПроцедуры", Ошибка );
+
+	Ошибка = "Ожидается константа";
+	юТест.ПроверитьКодСОшибкой("Процедура Метод( Парам = - п ) КонецПроцедуры", Ошибка );
+	юТест.ПроверитьКодСОшибкой("Процедура Метод( Парам = - -1 ) КонецПроцедуры", Ошибка );
+	юТест.ПроверитьКодСОшибкой("Процедура Метод( Парам = --1 ) КонецПроцедуры", Ошибка );
+
+КонецПроцедуры

tests/http.os

@@ -65,8 +65,8 @@
 
 	юТест.ПроверитьРавенство(200, Ответ.КодСостояния);
 
-	юТест.ПроверитьВхождение(ТелоОтвета, """p1"": ""v1""");
-	юТест.ПроверитьВхождение(ТелоОтвета, """p2"": ""v2""");
+	юТест.ПроверитьВхождение(ТелоОтвета, """p1"":""v1""");
+	юТест.ПроверитьВхождение(ТелоОтвета, """p2"":""v2""");
 
 КонецПроцедуры
 
@@ -165,7 +165,7 @@
 	ТелоОтвета = Ответ.ПолучитьТелоКакСтроку();
 
 	юТест.ПроверитьРавенство(200, Ответ.КодСостояния);
-	юТест.ПроверитьВхождение(ТелоОтвета, """p"": ""v""");
+	юТест.ПроверитьВхождение(ТелоОтвета, """p"":""v""");
 
 КонецПроцедуры
 

tests/start-all.cmd

@@ -8,8 +8,11 @@ set pathdir=%~dp0
 rem echo ᠬ� ���� %CD%
 rem echo �ਯ�� ���஢���� %pathdir%
 
+set progfiles=%ProgramFiles(x86)%
+if NOT EXIST ProgramFiles(x86) set progfiles=%ProgramFiles%
+
 @echo on
-"%ProgramFiles(x86)%\OneScript\bin\oscript.exe" "%pathdir%\testrunner.os" -runall %1 %2 %3 %4 %5
+"%progfiles%\OneScript\bin\oscript.exe" "%pathdir%testrunner.os" -runall %1 %2 %3 %4 %5
 @echo off
 
 rem echo ��� ������ %ERRORLEVEL%

tests/start.cmd

@@ -6,8 +6,11 @@ set pathdir=%~dp0
 rem echo ᠬ� ���� %CD%
 rem echo �ਯ�� ���஢���� %pathdir%
 
+set progfiles=%ProgramFiles(x86)%
+if NOT EXIST ProgramFiles(x86) set progfiles=%ProgramFiles%
+
 echo on
-"%ProgramFiles(x86)%\OneScript\oscript.exe" %pathdir%\testrunner.os -run %1 %2 %3 %4 %5
+"%progfiles%\OneScript\bin\oscript.exe" "%pathdir%testrunner.os" -run %1 %2 %3 %4 %5
 @echo off
 
 rem echo %ERRORLEVEL%

tests/testrunner.os

@@ -143,6 +143,21 @@
 	КонецЕсли;
 КонецПроцедуры
 
+Процедура ПроверитьКодСОшибкой( Код, Ошибка, ДопСообщениеОшибки = "" ) Экспорт
+	Попытка
+		Сценарий = ЗагрузитьСценарийИзСтроки(Код);
+		СообщениеОшибки = "Ожидали ошибку '"+Ошибка+"', но ее не было";
+	Исключение
+		ОписаниеОшибки = ИнформацияОбОшибке().Описание;
+		Если Найти(ОписаниеОшибки, Ошибка ) = 0 Тогда
+			СообщениеОшибки = "Ожидали ошибку '"+Ошибка+"', а была ошибка '"+ОписаниеОшибки+"'";
+		Иначе
+			Возврат;
+		КонецЕсли;
+	КонецПопытки;
+	ВызватьИсключение(СообщениеОшибки + ФорматДСО(ДопСообщениеОшибки));
+КонецПроцедуры
+
 Процедура ТестПройден() Экспорт
 КонецПроцедуры