Add tests and rename handler and data

[AurelienFT]

Important description

This PR improves readability of the AsyncMessage structure, fix potential panics.

Details :

• Fix potential panic behaviour : Denom value of 0 was accepted in deserialisation
• Fix problem function name in async message could be only u8::MAX size but the limit of length function is u16::MAX
• Unify data limits of async message that represent the parameters of the function call with the limits of parameters of a normal call. The values was differents.
• Wrong limit used for handler name in SetOrKeep serialisation
• Renamed handler -> function
• Renamed Data - > functions params
• Remove from bootstrap async pool variables unused
• Change excluded to included in max size of trigger datastore key to match the behaviour of the ledger datastore key limit

Follow up that needs to be done

In GRPC :

• Rename handler -> function
• Rename data -> function params

On a more general note, I think we should make a changelog for the mainnet so that we could add breaking changes (but useful to be done now otherwise it will be a bigger mess) like this change and #4409 for example. @Leo-Besancon and @damip I let you decide on that.

The messages.rs file now have 90% of line coverage and all min/max limit on serializer/deserializer are tested

[damip]

Very good work @AurelienFT ! Reviewing it

[Leo-Besancon]

Thanks for the small corrections, everything looks good!

[AurelienFT]

bors merge

[bors]

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

• build