Add acceptance test harness#927
Conversation
…lifecycle, wifi, and security on a live device
|
I think this is a good idea, but I think we should split this testsuite up into multiple based on required capabilities. It should be possible to run a subset of these tests without a working |
|
I would be happy to do that, although I would also like to get buy in from the rest of the team before I spend more time working on this. It's been sitting for a month and if there's no interest from the folks doing the merging then it's not worth putting more time into it. |
|
I also think its worthwhile to have acceptance tests but this PR is too big to be reasonably reviewed. Would it be possible to commit the main wiring first and then the tests in smaller chunks? |
|
Closing this for now but to be clear I really do want this code! I just want it in smaller chunks please! |
This is a set of tests to exercise the code against an actual device. It lets us catch regressions that other tests may not, as well as make it easier for devs to test that their code didn't cause any unintended side effects. The intent is to make it so that PRs are higher quality. Note the README.md it explains it a bit more detail.
Pull Request Checklist
cargo fmt.You must check one of: