Skip to content
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

Merged

Conversation

Golovanoff
Copy link
Contributor

Доработал ситуации, когда нужно сравнивать таблицы, но структуры таблиц (разный состав/количество колонок) или количество строк в таблицах не равны.

Теперь падать не должно, такие ситуации корректно обрабатываются.
Но теперь просится метод Содержит() для таблиц.

@nixel2007
Copy link
Member

Сделай ребейз, пожалуйста.

И меня смущает, что текущий мастер бранч проходит тесты без ошибок. Значит, тестов на то, что ты пофиксил, не хватает. Можешь докинуть?

@Golovanoff
Copy link
Contributor Author

Сделай ребейз, пожалуйста.

И меня смущает, что текущий мастер бранч проходит тесты без ошибок. Значит, тестов на то, что ты пофиксил, не хватает. Можешь докинуть?

Функционал работал правильно, поэтому я тесты и не доработал. Просто при неравенстве таблиц отображение отличий могло не сформироваться корректно.

Напилить тестов, что в определённых ситуациях исключения выбрасываются с нужным текстом?

И зачем ребейз? Я текущий мастер смержил в ветку.

@nixel2007 nixel2007 closed this Dec 7, 2024
@nixel2007 nixel2007 reopened this Dec 7, 2024
src/bdd-asserts-class.os Outdated Show resolved Hide resolved
@EvilBeaver
Copy link
Member

@artbear подтверди, пожалуйста, что все хорошо и можно вливать (или нельзя)

@artbear artbear merged commit 3c52f58 into oscript-library:master Dec 28, 2024
2 checks passed
@artbear
Copy link
Member

artbear commented Dec 28, 2024

@artbear подтверди, пожалуйста, что все хорошо и можно вливать (или нельзя)

смержил ПР

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants