Serialiser_Engine: FromJson() to Trim the text before attempting to read it

[alelom]

Issues addressed by this PR

Closes #3109

The json text is trimmed before attempting to read it. See #3109 for a longer description.

This helps solving issues reading files with trailing/initial newlines/whitespaces, in particular for workflows that @pawelbaran highlighted in BHoM/File_Toolkit#161 (if you go there, make sure you read its closing comment BHoM/File_Toolkit#161 (comment)).

Test files

Attached in #3109.

Changelog

Additional comments

[adecler]

Gets the job done so happy to approve.

Note: I have only done a review based on the code but this PR is simple enough that I am confident it works and doesn't break anything.

[FraserGreenroyd]

Code changes make sense based on the issue described, happy to approve for merge and testing via alphas.

[FraserGreenroyd]

@BHoMBot check compliance
@BHoMBot check required

[bhombot-ci]

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

• check code-compliance
• check documentation-compliance
• check project-compliance
• check branch-compliance
• check dataset-compliance
• check copyright-compliance
• check code-compliance
• check documentation-compliance
• check project-compliance
• check core
• check null-handling
• check serialisation
• check versioning
• check installer

There are 19 requests in the queue ahead of you.

[bhombot-ci]

The check code-compliance has already been run previously and recorded as a successful check. This check has not been run again at this time.

[bhombot-ci]

The check documentation-compliance has already been run previously and recorded as a successful check. This check has not been run again at this time.

[FraserGreenroyd]

@BHoMBot check ready-to-merge

[bhombot-ci]

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

• check ready-to-merge

There are 1 requests in the queue ahead of you.

[FraserGreenroyd]

@BHoMBot this is a DevOps instruction. I am requesting neutral checks on: unit-tests

[bhombot-ci]

@FraserGreenroyd I have provided neutral checks to the checks requested. These checks will need to be run properly to obtain full results.