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

Labwork 2 done 3821Б1ПМ3 Качалин Алексей #100

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

Alicease
Copy link

@Vasilisk18 @LebedevIlyaG @DanyaMez Проверьте снова лабу, пожалуйста (первый раз закрыли в октябре)

@Alicease
Copy link
Author

Я замучался свою старую лабу с того года четвертовать, примите пожалуйста уже 🥺

@DanyaMez
Copy link

Проверь все четыре части еще 3 недели назад, все работает

@Vasilisk18
Copy link

Vasilisk18 commented Nov 11, 2022

Проверил, предыдущие тоже были проверены. Всё работает, все хорошо, тесты посмотрел.

@LebedevIlyaG
Copy link

Нет тестов, должны быть и тесты и пример, все в разных проектах.

Принято. 7 баллов.

@LebedevIlyaG
Copy link

Не понял о чем Вы, покажете на практике.

@Alicease
Copy link
Author

Нет тестов, должны быть и тесты и пример, все в разных проектах.

Принято. 7 баллов.

@LebedevIlyaG Добавил новые тесты с примером

Не понял о чем Вы, покажете на практике.

У меня закрыли пул реквесты (часть закрыл я, часть владелец репозитория), но ребята объяснили почему, вопросов нет

@LebedevIlyaG
Copy link

У Вас в тестах: ADD_FAILURE();
Т.е. их нет.
В этой и последующей лабораторной, тесты нужно писать самостоятельно.

@Alicease
Copy link
Author

Alicease commented Dec 8, 2022

У Вас в тестах: ADD_FAILURE(); Т.е. их нет. В этой и последующей лабораторной, тесты нужно писать самостоятельно.

Хорошо, в ближайшее время добавлю
Видимо файл, который я подгружал, был без изменений

@Alicease
Copy link
Author

У Вас в тестах: ADD_FAILURE(); Т.е. их нет. В этой и последующей лабораторной, тесты нужно писать самостоятельно.

Загрузил файлы с нового билда с тестами и скриншотами работы программы и скриншоты прохождения моих тестов

@LebedevIlyaG
Copy link

Принято 10 баллов.

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