Added handling of no cases provided for ResultRequest.

[KalleEdstroem]

Fix bug for no results when no load cases is specified.

Issues addressed by this PR

Closes #373

When no loadcases is specified no result was returned without a description or warning for why. This PR sees to fix this bug by providing the results from all loadcases when no specific cases is specified.

This is in line with descriptions in the ...ResultRequest method in the Structure_oM
https://github.com/BHoM/BHoM/blob/7a13095ab3de961715b4a6c8c18b60771e624f2f/Structure_oM/Requests/BarResultRequest.cs#L43-L46

Test files

Grasshopper script, lusas .mdl file , and related lusas files provided below.
Download all files to the same location and run the script and lusas as usual.

https://burohappold.sharepoint.com/:f:/r/sites/BHoM/02_Current/12_Scripts/01_Issue/BHoM/Lusas_Toolkit/%23373%20Misleading%20hint%20in%20ResultRequest%20component?csf=1&web=1&e=BRp3ut

Changelog

• Fixed bug whereby if users did not provide any LoadcaseIds, no results were returned. Now if no LoadcaseIds are provided, all LoadcaseIds will be used to return results (of all types)

Additional comments

[peterjamesnugent]

Tested using the script and works as intended. Given it's implemented in the GetLoadcaseIds method it will be utilised for all result calls.

[peterjamesnugent]

@BHoMBot check required
@BHoMBot check copyright-compliance
@BHoMBot check ready-to-merge

[bhombot-ci]

@peterjamesnugent to confirm, the following actions are now queued:

• check code-compliance
• check documentation-compliance
• check project-compliance
• check core
• check null-handling
• check serialisation
• check versioning
• check installer
• check copyright-compliance
• check ready-to-merge

[FraserGreenroyd]

@BHoMBot check ready-to-merge

[bhombot-ci]

@FraserGreenroyd to confirm, the following actions are now queued:

• check ready-to-merge