Decaffeinate ci spec

[2colours]

No description provided.

[2colours]

Supersedes #21.

unused occurrances of request changed to _request, and one rather shouldn't require JSON files so reading it synchronously.

[confused-Techie]

@2colours One question I have concerns about, since the decaf PRs will be merged to main, will it cause issues that they are all separate PRs? Since these changes would be totally fine to be all on one PR.

[2colours]

@confused-Techie I'm doing it like this for one simple reason: this is how @GuilleW left it behind. I can just checkout the branch, rebase it to the new one, play around with the differences that happened on main in the meantime... but you are right, probably the end result should be a decaf branch where everything is decaffeinated. Which is, I think, where these PR's will lead.

[confused-Techie]

Overall this PR looks good, there's one change that should be made, as it's generally the more performant option, and another change I wanted to address prior to merging.

Otherwise the tests are currently running and fingers crossed they look good, appreciate this a ton @2colours and sorry this last one took a bit to get to

[confused-Techie]

With the important tests passing, and the changes here looking good I'll merge this one to the decaf branch.