Fix existing tests and include them in CI

LeastAuthority / destiny

Destiny – Cross-platform Magic Wormhole graphical client

MIT License

257 stars 14 forks source link

Closed wuan closed 1 year ago

wuan commented 1 year ago

This should fix #214

[ ] Description accurately reflects what changes are being made.
[ ] Description explains why the changes are being made (or references an issue containing one).
[ ] The PR appropriately sized.
[ ] New code has enough tests.
[ ] New code has enough documentation to answer "how do I use it?" and "what does it do?".
[ ] Existing documentation is up-to-date, if impacted.

donpui commented 1 year ago

Good. I have also started some integration tests work on. You can check here: https://github.com/LeastAuthority/destiny/blob/integration-tests/integration_test/simple_file_transfer_test.dart (reuse or avoid duplication). There are some challenges with file picker and outside interface, but if works, should allow run on emulators (and check minimum/maximum sdk compatibilities). Old integration tests seems useless now.