search

AnyaGl / oop

Лабы по ООП
0 stars 0 forks source link

Замечания по программе Car #9

Open alexey-malov opened 4 years ago

alexey-malov commented 4 years ago
alexey-malov commented 4 years ago

image

alexey-malov commented 4 years ago

image

alexey-malov commented 4 years ago

https://github.com/AnyaGl/oop/blob/0015d81f41994edfb98ccb10ac19511dfc0a9e4b/labs/lab3/Car/Car.h#L11-L17

alexey-malov commented 4 years ago

на задний ход можно переключиться только на нулевой скорости; даже при движении назад на нейтральной передаче не допускается включать заднюю передачу.

alexey-malov commented 4 years ago
AnyaGl commented 4 years ago
  • [ ] Нет тестов класса "Автомобиль" (точнее, их запуск отключен)

А как можно запускать одновременно тесты и для Car, и для CarControl?

AnyaGl commented 4 years ago

Получается, из-за того, что тесты для Car отключены, показывает, что участки кода, которые Вы выше отправляли, не покрыты тестами?

alexey-malov commented 4 years ago

Получается, из-за того, что тесты для Car отключены, показывает, что участки кода, которые Вы выше отправляли, не покрыты тестами?

Это вы должны выяснить. Возможно, из-за того, что тесты отключены, они вообще не компилируются или вовсе не работают

AnyaGl commented 4 years ago

Получается, из-за того, что тесты для Car отключены, показывает, что участки кода, которые Вы выше отправляли, не покрыты тестами?

Это вы должны выяснить. Возможно, из-за того, что тесты отключены, они вообще не компилируются или вовсе не работают

Проверила. Если оба файла с тестами включены в сборку, то тесты не запускаются, поэтому я исключала по одному файлу и проверяла. Так получается, что весь код покрыт тестами. Как можно запустить сразу два теста, если они в одном проекте?

alexey-malov commented 4 years ago

Проверила. Если оба файла с тестами включены в сборку, то тесты не запускаются, поэтому я исключала по одному файлу и проверяла. Так получается, что весь код покрыт тестами. Как можно запустить сразу два теста, если они в одном проекте?

Почитайте инструкцию https://github.com/catchorg/Catch2/blob/master/docs/tutorial.md#scaling-up