refactor: Hubspot getMarshalledData -> getDataMarshaller

[laurenzlong]

No functional change, some refactors to make code easier to understand & reduce repetitive code:

• renamed getMarshalledData to getDataMarshaller to make it more obvious it's a higher order function
• revised getDataMarshaller to handle case when len(fields) is 0, so that GetRecordsFromIds doesn't need to implement it that logic
• also modified the test the check that full raw record is returned, I'm surprised it wasn't failing before

[Cobalt0s]

LGTM

[Cobalt0s]

@RajatPawar this PR is related to this function.

Hubspot is on the same list as Klaviyo/Atlassian/Keap, etc. The implementation would become more consize if this function would work with *ajson.Node which is easier for parsing into the struct or queried using custom jsonquery package.

Currently, common.ParseResult enforces the use of map[string]any, adding unnecessary complexity to the code for some connectors. Adding flexibility to either or would resolve this.

Once the decision is made regarding common.ParseResult, I will revisit Hubspot implementation to simplify it even further. Essentially we need to convert ajson.Node into struct {"id":"", "properties": {}}.