Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Automated test - Validated #210

Merged
merged 9 commits into from
Jun 30, 2021
Merged

Conversation

c19x
Copy link
Contributor

@c19x c19x commented Jun 16, 2021

  • Demo app enables remote control by automated test server
  • Supports start/stop/upload commands to automate existing test procedure
  • Validated on Pixel 3

Signed-off-by: c19x support@c19x.org

Closes #209

c19x added 8 commits June 16, 2021 23:58
- Demo app enables remote control by automated test server
- Supports start/stop/upload commands to automate existing test procedure
- Validated on Pixel 3

Signed-off-by: c19x <support@c19x.org>
- Enables remote reset of components, including in-memory data, stored log files, and UI
- Supports remote commands "start,stop,clear,upload(file)"
- Validated on Pixel 3
- Set AppDelegate.automatedTestServer from null to test server "http://address:port" to enable remote control

Signed-off-by: c19x <support@c19x.org>
- Reduced reliance on static for AutomatedTestClient

Signed-off-by: c19x <support@c19x.org>
- Minor update to ensure detection.csv log is generated on SensorArray state update
- Necessary for automated testing where phone may be reset remotely and restarted
- Validated on Pixel3

Signed-off-by: c19x <support@c19x.org>
- Passes all unit tests

Signed-off-by: c19x <support@c19x.org>
- Build histogram of time duration to connect and process device over BLE
- Histogram is stored on CSV file in debug mode
- Experiment data using 10 devices show reduction of connect timeout from 12 to 6 seconds (or even 4 seconds is acceptable)
- Validated on Pixel 2, Pixel 3, A10, A20, Note8, S9+ with 4 iPhones

Signed-off-by: c19x <support@c19x.org>
- Moved to Datatype.Distribution
- Rewritten fixed tests for Distribution
- Improved documentation
- Improved tests for Analysis.Aggregate.Gaussian
- Passed all unit tests

Signed-off-by: c19x <support@c19x.org>
Signed-off-by: c19x <support@c19x.org>
@adamfowleruk
Copy link
Collaborator

@c19x please resolve conflict and push another commit.

@adamfowleruk adamfowleruk merged commit f121558 into theheraldproject:develop Jun 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

On-device automated test
2 participants