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

Enhanced A1-Test-App and A2-Test-App to automatically generate best download wait times #74

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

MonteCreasor
Copy link
Contributor

Added code to automatically adjust the waitForActivity() call for the DisplayImagesActivity. The calculation is based on the largest image in the image test set and takes into consideration the connection speed on each machine that it is run on. This solution does not require the students to ever have to make changes to waitTime constants. It has been tested on the fastest possible connection as well as the slowest possible connection (which takes roughly 10 minutes to download 6 images since I used GSM and GPRS, and I also tested it with EDGE speed).

I think you will get very few complaints with this design in the future. It is worth noting that Test4_SingleValidUrl is used to set the best wait time for the specific uses and their connection speed. The Images have also been changed: I remove 2 of the larger images that exceed 1000KB and replaced them with ones that are around 150KB (and those are to two largest now).

…it time for downloading multiple images depending on connection speeds and image sizes. This makes it unnecessary for students to ever need to adjust the wait time parameters for their specific hardware environment.
…it time for downloading multiple images depending on connection speeds and image sizes. This makes it unnecessary for students to ever need to adjust the wait time parameters for their specific hardware environment.
…it time for downloading multiple images depending on connection speeds and image sizes. This makes it unnecessary for students to ever need to adjust the wait time parameters for their specific hardware environment.
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.

1 participant