Add HTTP Range header tests for fetching ranges of pieces

filecoin-project / retrieval-load-testing

Other

4 stars 1 forks source link

Add HTTP Range header tests for fetching ranges of pieces #9

Closed kylehuntsman closed 1 year ago

kylehuntsman commented 1 year ago

Adds a second execution of tests that uses a series of HTTP Range header sizes and offsets to test piece range requests.

Notable Changes

Wrapped K6 http.get() to provide common functionality for both boost and raw fetch urls
Added checks for Content-Length values to prevent error spam when the Content-Length is either missing or invalid. This was occurring during timeouts and the manual cancellation of tests.
Added function documentation

Resolve issue #2

rvagg commented 1 year ago

Sorry to be so nitty, just seeing the inconsistencies. I think you should run standard over it to catch most of these. I have it in my editor so it complains at me as I go.

kylehuntsman commented 1 year ago

Sorry to be so nitty, just seeing the inconsistencies. I think you should run standard over it to catch most of these. I have it in my editor so it complains at me as I go.

Be nitty! I haven't had to write JS in awhile, so this was a good reminder about all the things I need to remember to look for.