search

willstay / liga-internship

Задачи на занятиях
0 stars 0 forks source link

Тесты для changeTempoTest очень многословные #4

Open vladimirkoba opened 6 years ago

vladimirkoba commented 6 years ago

Посмотрел changeTempoTest и changeToneTest. Из-за того, что апи спроектированно не очень, пришлось городить такие костыли, типа измени темп, выгрузи файл, потом снова загрузи, найди трек с темпом и проверь его. Очень много ответственности.

Как минимум:

  1. Поиск трека с темпом нужно перенести в MidiTrack
  2. Хочется, чтобы changeTrack возвращал MidiTrack с измененным темпом, тогда его будет легко тестировать
  3. По-хорошему, в тесте не должно быть никаких операций ввода-вывода на диск, разве что загрузить какие-нибудь большие данные, необходимые для теста, типа json-а на 10 000 строк :)
willstay commented 6 years ago

Сократил размер тестов и убрал создание файла