Clean up code around test api keys and roles

[ashleysmithTTD]

What Changed:

• Exported the role objects from KeyHelper.spec.ts and import them into other key test files so we are not constantly redefining the API role objects
• Changed some naming of test variables where necessary to be more clear
• Extracted some variables out from the test block to be reused between tests in the same file

[alex-yau-ttd]

Looks great - though I do wonder if we could go a bit further - there's still a few instances where we are manually defining these in the code - mainly in stories but also in another spec file. Is this within scope here?

[ashleysmithTTD]

Looks great - though I do wonder if we could go a bit further - there's still a few instances where we are manually defining these in the code - mainly in stories but also in another spec file. Is this within scope here?

Yeah definitely - i'll go back and narrow it down some more