Дмитрий Темин

В решении учителя метод BellState::tick() просто сменит стейт на clock, хотя должен ещё добавить минуту к текущему времени. Ведь будильник выключился спустя минуту самостоятельно. Можно дополнить тест соответствующей проверкой

``` public function testStartingBellIfAlarmOn1() { $clock = new AlarmClock(); $clock->longClickMode(); for ($i = 0; $i < 18 * 60; $i++) { $clock->tick(); } $this->assertTrue($clock->isAlarmTime()); $this->assertEquals('bell', $clock->getCurrentMode()); $clock->clickM(); $clock->clickH(); $clock->tick(); $this->assertEquals('clock', $clock->getCurrentMode()); $this->assertEquals(1, $clock->getMinutes()); }

2 1

Роман Ашиков

Приветствую, Дмитрий!

Спасибо за фидбек! Я поставил задачу. В ближайшее время проанализирую и, возможно, добавлю ваш тест в упражнение. :)

0

Роман Ашиков

Приветствую, Дмитрий!

Я добавил предложенный вами тест. Спасибо! Можете нажать на кнопку "Сброс", чтобы получить последнюю версию упражнения.

1

Есть вопрос или хотите участвовать в обсуждении?

Зарегистрируйтесь или войдите в свой аккаунт

Нажимая кнопку «Зарегистрироваться», вы даёте своё согласие на обработку персональных данных в соответствии с «Политикой конфиденциальности» и соглашаетесь с «Условиями оказания услуг». Защита от спама reCAPTCHA «Конфиденциальность» и «Условия использования».

Похожие вопросы

Евгений Вафиев 14 мая 2020 →

Добрый вечер. Задание конечно треш, 2 дня убил, но получил то, чтобы тесты прошли. Долго пытался перераспре...

Руслан Куга 19 марта 2020 →

https://ru.hexlet.io/code_reviews/233849 Когда делал, был уверен, что творю фигню полную. Ждал момента, ког...

Roman Kozlov 08 ноября 2019 →

Здравствуйте! У меня создается впечатление, что я делаю что-то не так. Как будто просто подгоняю решение по...

Алексей Кузин 05 ноября 2019 →

Не подскажите, пожалуйста, сложно даются уроки, долго вникаю, хочу чуть больше углубиться в полиморфизм, чт...

Василий Антуфьев 08 октября 2019 →

как метод tick для AlarmState работать должен? он менять часы alarmHours? не могу последнии два теста пройт...