Develop JavaScript alternative to vt-driver.php

bobbingwide commented 3 years ago

The Slog admin page has tabs to view reports, compare results etc. It doesn't yet have the ability to run performance tests. It would be nice to be able to drive the performance comparison testing directly from the Slog admin page.

Requirements

There would be two methods

Run the same query limit times.
Run a set of queries from a filtered file, performing up to limit times.

Proposed solution

MVP

[x] Develop a Driver tab that runs XMLHttpRequests to the server
[x] Support a loop that will run the request limit times
[x] Provide visual feedback so show that the loop is working and the results

Optional extras

[ ] Make it easily cancellable
[ ] Prevent the Run button from being pressed multiple times
[ ] Support loading a filtered file to provide the URLs for each request
[ ] Dynamically display the results in a chart
[x] Allow setting of the daily trace summary file name from the Driver tab

Assumptions, Limitations, Questions

It'll be easier to write using XMLHttpRequests than fetch().
I can write it using native JavaScript
No need to use React / Gutenberg - how possible would that be anyway?
No need for internationalization.
Logic only needed for admin.
It'll be easy to write as plain HTML and JavaScript then convert to WordPress
Could most of the driver logic be written as ALL JavaScript?
Is CSS needed to style the output?

bobbingwide commented 3 years ago

Perform request against another site

In order to be able to run the request against another site we have to be able to overcome blocking by CORS policy.

Access to XMLHttpRequest at 'https://oik-plugins.co.uk/' from origin 
'https://s.b' has been blocked by CORS policy: 
No 'Access-Control-Allow-Origin' header is present on the requested resource.
driver.html:104 An error occurred while transferring the file.

We should also be able to detect this message, or prevent the user from attempting to run a request against another site.

bobbingwide commented 3 years ago

I've done the tasks for the MVP.

bobbingwide commented 2 years ago

Allow setting of the daily trace summary file name from the Driver tab

See #25

bobbingwide / slog