Skip to content

Исправление тестов (манифесты, доступность пакетов) #173

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jun 10, 2020
Merged

Исправление тестов (манифесты, доступность пакетов) #173

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a73a18b
исправил тесты по манифестам
ovcharenko-di Jan 17, 2020
b172403
исправил тесты по проверке доступности пакетов
ovcharenko-di Jan 17, 2020
411c8cf
добавил очистку пробелов рядом с закрывающим тегом
ovcharenko-di Jan 20, 2020
bb8288e
тех. долг
ovcharenko-di Jan 24, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 5 additions & 6 deletions tests/download.os
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,11 +31,11 @@
СтрокаЗапуска = "python3 -m http.server";
Проц = СоздатьПроцесс(СтрокаЗапуска, КаталогСборки);
Проц.Запустить();
Приостановить(1000)
Приостановить(1000);
КонецПроцедуры

Процедура ПослеЗапускаТеста() Экспорт
//мВременныеФайлы.Удалить();

ПутьККаталогу = ОбъединитьПути(ТекущийКаталог(), "oscript_modules", "test");
Если ФС.КаталогСуществует(ПутьККаталогу) Тогда
УдалитьФайлы(ПутьККаталогу);
Expand Down Expand Up @@ -64,12 +64,12 @@

ФайлПакета = Новый Файл(ОбъединитьПути(КаталогСборки, "test-0.3.1.ospx"));
Утверждения.ПроверитьИстину(ФайлПакета.Существует(), "Файл пакета должен существовать");
СоздатьКаталог(ОбъединитьПути(КаталогСборки,"test"));
КопироватьФайл(ФайлПакета.ПолноеИмя, ОбъединитьПути(КаталогСборки,"test/test-0.3.1.ospx"));
СоздатьКаталог(ОбъединитьПути(КаталогСборки, "test"));
КопироватьФайл(ФайлПакета.ПолноеИмя, ОбъединитьПути(КаталогСборки, "test/test-0.3.1.ospx"));
КопироватьФайл(ФайлПакета.ПолноеИмя, ОбъединитьПути(КаталогСборки, "test/test.ospx"));


Запись = Новый ЗаписьТекста(ОбъединитьПути(КаталогСборки,"list.txt"));
Запись = Новый ЗаписьТекста(ОбъединитьПути(КаталогСборки, "list.txt"));
Запись.ЗаписатьСтроку("test");
Запись.Закрыть();

Expand Down Expand Up @@ -99,4 +99,3 @@
КонецПроцедуры

Лог = Логирование.ПолучитьЛог("oscript.app.opm");
//Лог.УстановитьУровень(УровниЛога.Отладка);
19 changes: 10 additions & 9 deletions tests/mft-serializer-test.os
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,7 @@
ЗаписатьМанифест(Запись, СоздатьТестовыйМанифест());

Результат = СтрЗаменить(Запись.Закрыть(), Символы.ВК+Символы.ПС, Символы.ПС);
Результат = СтрЗаменить(Результат, " />", "/>");

Copy link
Member

@EvilBeaver EvilBeaver Jan 23, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Зачем это?

Copy link
Contributor Author

@ovcharenko-di ovcharenko-di Jan 23, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@EvilBeaver, см. выше комментарий @nixel2007

Эталон =
"<?xml version=""1.0"" encoding=""utf-8""?>
Expand All @@ -71,17 +72,17 @@
| <engine-version>1.0</engine-version>
| <description>Это пакет для тестирования</description>
| <author-email>mail@server.com</author-email>
| <depends-on name=""asserts"" />
Copy link
Member

@nixel2007 nixel2007 Jan 17, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это изменение будет падать на одной OS и работать на другой. Linux и Windows формируют разные xml

Copy link
Contributor Author

@ovcharenko-di ovcharenko-di Jan 17, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тогда, пожалуй, следует 2 теста держать и пропускать один в зависимости от среды, верно?

Copy link
Member

@nixel2007 nixel2007 Jan 17, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В каком-то из соседних проектов вопрос решили через проверку по шаблону и \s* после кавычки.

Copy link
Contributor Author

@ovcharenko-di ovcharenko-di Jan 19, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

для инфо: нашел шаг в 1bdd
//Файл "folder0/file01.txt" содержит xml "Текст файла"

Copy link
Contributor Author

@ovcharenko-di ovcharenko-di Jan 20, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

сделал очень прямолинейно, но должно работать корректно

| <depends-on name=""cmdline"" version=""&gt;=2.1"" version-max=""&lt;3.0"" />
| <depends-on name=""asserts""/>
| <depends-on name=""cmdline"" version=""&gt;=2.1"" version-max=""&lt;3.0""/>
| <executable>src/app1.os</executable>
| <executable>src/app2.os</executable>
| <include-content>src</include-content>
| <include-content>tests</include-content>
| <explicit-modules>
| <module name=""Модуль1"" src=""src/m1.os"" />
| <module name=""Модуль2"" src=""src/m2.os"" />
| <class name=""Класс1"" src=""src/class1.os"" />
| <class name=""Класс2"" src=""src/class2.os"" />
| <module name=""Модуль1"" src=""src/m1.os""/>
| <module name=""Модуль2"" src=""src/m2.os""/>
| <class name=""Класс1"" src=""src/class1.os""/>
| <class name=""Класс2"" src=""src/class2.os""/>
| </explicit-modules>
|</opm-metadata>";

Expand All @@ -98,6 +99,7 @@
Сериализатор.ЗаписатьМетаданныеВXML(Запись, СоздатьТестовыйМанифест());

Результат = СтрЗаменить(Запись.Закрыть(), Символы.ВК+Символы.ПС, Символы.ПС);
Результат = СтрЗаменить(Результат, " />", "/>");

Эталон =
"<?xml version=""1.0"" encoding=""utf-8""?>
Expand All @@ -108,8 +110,8 @@
| <engine-version>1.0</engine-version>
| <description>Это пакет для тестирования</description>
| <author-email>mail@server.com</author-email>
| <depends-on name=""asserts"" />
| <depends-on name=""cmdline"" version=""&gt;=2.1"" version-max=""&lt;3.0"" />
| <depends-on name=""asserts""/>
| <depends-on name=""cmdline"" version=""&gt;=2.1"" version-max=""&lt;3.0""/>
| <executable>src/app1.os</executable>
| <executable>src/app2.os</executable>
|</opm-metadata>";
Expand All @@ -118,7 +120,6 @@

КонецПроцедуры


Процедура ТестДолжен_ПрочитатьМанифестИзXML() Экспорт

Манифест = СоздатьТестовыйМанифест();
Expand Down
23 changes: 14 additions & 9 deletions tests/packagelist.os
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
СписокТестов = Новый Массив;

СписокТестов.Добавить("ТестДолжен_ПолучитьПакетыХаба");
СписокТестов.Добавить("ТестДолжен_РегистроНезависимостьПакетовХаба");
СписокТестов.Добавить("ТестДолжен_ПроверитьРегистроНезависимостьПакетовХаба");

Возврат СписокТестов;

Expand All @@ -28,19 +28,24 @@
СписокПакетовХаба = МенеджерПолучения.ПолучитьДоступныеПакеты();

Ожидаем.Что(СписокПакетовХаба.Количество()).Больше(1);
Ожидаем.Что(СписокПакетовХаба.Получить("gitsync")).Равно(Истина);
Ожидаем.Что(СписокПакетовХаба.Получить("opm")).Равно(Истина);
Ожидаем.Что(СписокПакетовХаба.Получить("someelsepackadge")).Равно(Неопределено);


НайденПакет_gitsync = МенеджерПолучения.ПакетДоступен("gitsync");
НайденПакет_opm = МенеджерПолучения.ПакетДоступен("opm");
НайденНесуществующийПакет = МенеджерПолучения.ПакетДоступен("someelsepackadge");

Ожидаем.Что(НайденПакет_gitsync).Равно(Истина);
Ожидаем.Что(НайденПакет_opm).Равно(Истина);
Ожидаем.Что(НайденНесуществующийПакет).Равно(Ложь);

КонецФункции

Функция ТестДолжен_РегистроНезависимостьПакетовХаба() Экспорт
Функция ТестДолжен_ПроверитьРегистроНезависимостьПакетовХаба() Экспорт

ИзмененноеИмяПакета = "PARSERFileV8i";

МенеджерПолучения = Новый МенеджерПолученияПакетов();
СписокПакетовХаба = МенеджерПолучения.ПолучитьДоступныеПакеты();
ПакетДоступен = МенеджерПолучения.ПакетДоступен(ИзмененноеИмяПакета);

Ожидаем.Что(СписокПакетовХаба.Количество()).Больше(1);
Ожидаем.Что(СписокПакетовХаба.Получить("ParserFileV8i")).Равно(Истина);
Ожидаем.Что(ПакетДоступен).Равно(Истина);

КонецФункции