search

LeastAuthority / destiny

Destiny – Cross-platform Magic Wormhole graphical client
MIT License
236 stars 13 forks source link

Missing Test automation #210

Open donpui opened 1 year ago

donpui commented 1 year ago

Destiny doesn't have any tests automation, end-2-end, cross-platform testing and till now mostly was tested manually.

However, it use w-w where main "business" logic sits and library has some tests.

Flutter reference on testing: https://docs.flutter.dev/testing

Probably we should explore how the best test flutter app and what kind of tests create here.

donpui commented 1 year ago

From my side, ideally we would need to have End-2-End tests which cover various platforms: Linux, Windows, MacOS, Android, iOS and even with different versions. We have already seen issues, that what is working on MacOs, might not work in Windows. Also there is big difference between Desktop and Mobile as there are some platform differences in code itself.

wuan commented 1 year ago

When we merge #209 we will have some widget and unit test. There is already one other test which is currently not running without errors and some integration tests (status needs to be checked). I propose to add disable or remove broken tests and run the tests we have as an additional step in the build pipeline.