-
Notifications
You must be signed in to change notification settings - Fork 5
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.
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
УтвержденияТаблиц #12
The head ref may contain hidden characters: "\u0423\u0442\u0432\u0435\u0440\u0436\u0434\u0435\u043D\u0438\u044F\u0422\u0430\u0431\u043B\u0438\u0446"
УтвержденияТаблиц #12
Conversation
Сделай ребейз, пожалуйста. И меня смущает, что текущий мастер бранч проходит тесты без ошибок. Значит, тестов на то, что ты пофиксил, не хватает. Можешь докинуть? |
Функционал работал правильно, поэтому я тесты и не доработал. Просто при неравенстве таблиц отображение отличий могло не сформироваться корректно. Напилить тестов, что в определённых ситуациях исключения выбрасываются с нужным текстом? И зачем ребейз? Я текущий мастер смержил в ветку. |
@artbear подтверди, пожалуйста, что все хорошо и можно вливать (или нельзя) |
смержил ПР |
Доработал ситуации, когда нужно сравнивать таблицы, но структуры таблиц (разный состав/количество колонок) или количество строк в таблицах не равны.
Теперь падать не должно, такие ситуации корректно обрабатываются.
Но теперь просится метод Содержит() для таблиц.