Вынести в общий плагин проверку таблиц #49 #73
Conversation
| //позволяет сделать поиск в строке "ПроверяемаяСтрока" подстроки "Шаблон" | ||
| //при этом подстрока "Шаблон" может содержать символы * | ||
| //например СтрокаСоответствуетШаблону("Привет","*вет") | ||
| Функция СтрокаСоответствуетШаблону(ПроверяемаяСтрока, Знач Шаблон) Экспорт |
There was a problem hiding this comment.
Может быть, проверку регулярок все-таки выделить в отдельный плагин?
как я и планировал в Плагин "Регулярные выражения" #48
There was a problem hiding this comment.
Плагин универсальнім и кроссплатформеннім не получится сделать, имхо в целях рефакторинга віноса в плагин кода, лучше пусть будет так. Во всяком случаии можно будет сравинть если вдруг "кто-то" сделает изменения и потом надо будет синхронизировать.
There was a problem hiding this comment.
@pumbaEO Почему не получится?
Функция "соответствует регулярке" вроде как почти кроссплатформенной получилась, подавляющая часть регулярных выражений совпадает.
Хотелось бы хотя бы их в плагин выделить, все-таки эта функция явно не относится к сравнению таблиц и будет полезна в разных примерах.
Тем более, что в VB метод "соответствует регулярке" является экспортным и может где-то юзаться у клиентов продукта.
There was a problem hiding this comment.
Пока все не перешли на разработку тут, это будет очень трудно поддерживать. Я в основном все копипастом вырезаю и по возможности оставляю неотрефакториный код, что-бы потом понимать что и откуда взялось. Для полноценного плагина РаботаСРегулярнымиВыражениями, тут не хватает реализациий, пока только 1 кроссплатформенная проверка на соответствие шаблону и все.
There was a problem hiding this comment.
Так хотя бы эту одну проверку и вынести в плагин, уже было бы здорово.
По трудности с поддержкой, доработкой и мержем согласен.
будем думать по переводу всех участников.
По регулярке сделаю связь с #48
| Плагин = Плагин("СериализаторMXL"); | ||
| Данные = Плагин.СоздатьДанныеПоТабличномуДокументу(ТабличныйДокумент, РежимыЗагрузкиИлиИмяКолонкиЗамещения, ИмяКолонкиЗамещения); | ||
| Перем Серелизатор; | ||
| Серелизатор = Плагин("СериализаторMXL"); |
There was a problem hiding this comment.
опечатка Серелизатор.
Правильно Сериализатор
|
@pumbaEO Странная ошибка при сборке PR
Откуда появилось расхождение?? |
|
@pumbaEO Аналогичное расхождение тестов в ветке develop в последней сборке :(
|
|
@pumbaEO Что скажешь по последним ошибкам? |
|
SonarQube analysis reported 78 issues
Top 10 issues
|
|
По поводу того, что аллюр не показывает ошибки ничего странного нет, т.к в разных фичах есть сценарии названные одинаково и если посмотреть на отчет http://ci.silverbulleters.org/job/ADD%20test/view/change-requests/job/PR-73/lastCompletedBuild/allure/#suites/98d3104e051c652961429bf95fa0b5d6/96641aecef861eb/retries то можно увидеть что этот сценарий flaky |
реализация #49
Вынес код из ванессы в плагин.
В случаи отличия табличных документов по количеству все равно делаем сравнение и выводим сообщение, об ошибке.