search

lichess-org / mobile

Lichess mobile app v2
GNU General Public License v3.0
1.36k stars 197 forks source link

[Study] add models for listing available studies #1002

Closed tom-anders closed 2 months ago

tom-anders commented 2 months ago

Do we need separate test for json parsing here? Or is it enough to have a end-to-end test for the final study list screen?

veloce commented 2 months ago

End to end tests are preferable of course. JSON test might help in case of complex JSON but not mandatory.

veloce commented 2 months ago

Some failing tests here @tom-anders

tom-anders commented 2 months ago

ah, classic unused imports, fixed now :+1:

tom-anders commented 2 months ago

oh looks like it's time to fix lints after the latest flutter update again though