Исправление тестов (манифесты, доступность пакетов) #173
Conversation
ovcharenko-di
commented
Jan 17, 2020
ovcharenko-di
commented
- тесты по манифестам падали из-за лишних пробелов в эталонных строках
- тест по проверке регистронезависимости пакетов, по сути, ничего не проверял т.к. искал пакет в кеше по "правильному" имени
- исправил тесты по поиску доступных пакетов
|
Коммитов многовото. Как будто отбранчевался не от девелопа |
| | <engine-version>1.0</engine-version> | ||
| | <description>Это пакет для тестирования</description> | ||
| | <author-email>mail@server.com</author-email> | ||
| | <depends-on name=""asserts"" /> |
There was a problem hiding this comment.
Это изменение будет падать на одной OS и работать на другой. Linux и Windows формируют разные xml
There was a problem hiding this comment.
Тогда, пожалуй, следует 2 теста держать и пропускать один в зависимости от среды, верно?
There was a problem hiding this comment.
В каком-то из соседних проектов вопрос решили через проверку по шаблону и \s* после кавычки.
There was a problem hiding this comment.
для инфо: нашел шаг в 1bdd
//Файл "folder0/file01.txt" содержит xml "Текст файла"
There was a problem hiding this comment.
сделал очень прямолинейно, но должно работать корректно
ovcharenko-di
force-pushed
the
fix/tests
branch
from
407b13a to
b172403
Compare
|
наверное, надо еще перезапустить билд |
|
@ovcharenko-di влей/сделай ребейз девелопа, пожалуйста. |
|
|
||
| Результат = СтрЗаменить(Запись.Закрыть(), Символы.ВК+Символы.ПС, Символы.ПС); | ||
| Результат = СтрЗаменить(Результат, " />", "/>"); | ||
|
|
There was a problem hiding this comment.
@EvilBeaver, см. выше комментарий @nixel2007
|
@ovcharenko-di тесты упали, посмотришь? |
@nixel2007, не понимаю, почему этот тест может падать.
|
|
@ovcharenko-di Доделаешь ПР ? |
Знать бы, что там не так на CI. У меня локально все ок |
5 participants
