Closed jonas-m- closed 5 years ago
@jonas-m- would you add some unit tests?
@jonas-m- would you add some unit tests?
Sure - will do
@jonas-m- sorry again for the delays, finally can handle the PR. Regarding your question about orchestrator and test services apks - seems to me that it's enough to point users that they need to setup it correctly. And we can improve that in future, as well.
Weird enough that issue with orchestrator not printing output is resolved but issue https://issuetracker.google.com/issues/64486480 is not closed)
Thanks a lot for contribution!
This fixes #90, closes #131 and closes #135
Tested in production - works perfectly, also the report generation
One point of uncertainty - the requirement to install test orchestrator and test services APKs before running the tests. I'm not convinced that this should be something
composer
itself should be concerned with, as those APKs just need to be installed, no parsing is done on them otherwise (as opposed to normal app/test APKs)The installation can either be done manually (as described in the official docs) OR, if used with Gradle - with a custom task, something like this: