35
21
32
Repository for the official website of Rayhatron - Web developer from Zimbabwe
A web app of a classic matching game where a player turns over two cards at a time in order to turn over a matching pair. The player uses their memory to remember where other cards they've turned over that didn't match are located, hence the name memory game.
Events
GH-5 Add support for running tests in batches
GH-5 Add support for running tests in batches to avoid quota limits and requests failing due to too many requests being sent to PageSpeed API.
Note, there are some changes related to variable naming which seem to not be in the main branch but are in production via tag 1.0.2 as seen in this commit. Please let me know how to properly reconcile this issue if those variable naming changes are to be removed from this PR.
Testing instructions
- Run the bulk-lighthouse script using the code in this PR and the tests should be run in batches provided your configuration file has a key
batchTestsset totrue.
Multiple test result failure
When the list of URLs to test is more than about 20, it seems that the tool starts to fail to retrieve the test result for a couple of URLs. Errors that are seen seem to point to the quota limit being exceeded or PageSpeed Insights API failing to process that many requests at one go.
A potential solution could be to allow the execution of the tests to be done in batches. From attempts at reducing the number of URLs to test, that figure of 20 is not absolute as tests could also fail at 15 URLs so it might be useful to have the size of a batch be configurable and also optional whether to execute in batches or not.
GH-3 Improve error handling and reporting when requests fail.
GH-3 Improve error handling and reporting when requests fail.
This PR improves error handling so that the test results message and table accurately reflect when a request to test a URL has failed or when a test result is below the threshold for a particular strategy.
Note, there are some changes related to variable naming which seem to not be in the main branch but are in production via tag 1.0.2 as seen in this commit. Please let me know how to properly reconcile this issue if those variable naming changes are to be removed from this PR.
Testing instructions
- Run the bulk-lighthouse script using the code in this PR and the summary message that is after the table of results should take into account failed requests. See issue description for more details.
Rayhatron create branch GH-3-error-response-does-not-trigger-failure
Switch to Full Site Editing theme
Merge pull request #7 from Rayhatron/full-site-editing-theme
Switch to Full Site Editing theme
Switch to Full Site Editing theme
Removed the classic style theme and created a basic full site editing theme.
Switch to Full Site Editing theme
Removed the classic style theme and created a basic full site editing theme.
