Fixed in sdkman/sdkman-cli#1012

SDKMAN's install process does not let us choose a version. This can cause our pipelines to break at any point if SDKMAN makes a breaking change.

To reduce our dependence on SDKMAN's servers, I've vendored the install script so that it's more predictable. I've also cached the install so we don't need to keep pulling in the installation archive from their servers on every PR run.

These jobs utilise less than 50% of a large's resource. It's a waste. Go back to default (medium).

chore(ci): re-enable sdkman caching

chore(ci): reduce resource class

chore(ci): migrate to cimg images

The scanResult object was added to the json output where
it should only have been added to calculate the the
dockerfile warning string.
This change removes the scanResult from the json output
as it was not supposed to be exposed.

This refactor is done to not have a special case for
array vs single result when formatting the results
for JSON output.
Note: dataToSend will now take the jsonData after formatting
rather than the data before formatting. Since the formatting
only changes the vulnerbilities and removes the unwanted
scanResult, and since the vulnerbilities are removed from
dataToSend before it is used, this change will not affect the
CLI bahvior except for removing the unwanted scanResult.

Renaming the object as in IaC the logic does more than mapping
the results.

Rename test fixtures used in format-test-results to align
with the variable name change.
This is done in a separate commit so it's clearer in git
history that the files were changed and then renamed rather
than deleted and created new files.

fix: Remove scanResult from json output [MAGMA-1280]