sadr0b0t / yashlang

PeerTube and YouTube player for Android with local playlists and whitelisted recommendations
GNU General Public License v3.0
52 stars 3 forks source link

Тесты для логики выбора потоков в динамике (или на всю стейт-машину PlayerService) #193

Open sadr0b0t opened 1 year ago

sadr0b0t commented 1 year ago

Сейчас есть тесты на правильные приоритеты в StreamHelperTest. Но внутри PlayerService динамика выбора потоков размазана по множеству событий так, что нет уверенности, что там всё работает правильно, и эти дела довольно утомительно вручную проверять. Нужно как-то попробовать добавить к этой логике тесты. Возможно, не только к логике выбора потоков, а вообще ко всей стейт-манише PlayerService (раньше было в WatchVideoActivity). Хотябы частично.

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