bugfix: int32 is converted to incorrect string

[mekpavit]

Currently, stringify macro in api.pebble doesn't handle int32 Go type properly. It directly uses string(...) to convert int32 to string which will generate a UTF-8 character corresponding to the value of int instead. For example, string(57) will result in "9" instead of "57". This bug makes MessagingApiAPI.GetFollowers(start string, limit int32) to be nearly impossible to use.

To fix this, this PR changes add another condition to stringify macro for Integer type. With this change, it will use strconv.FormatInteger(int64(...), 10) instead of string(...) which yield a correct string value.

Apart from the bugfix, I also refactor api.pebble a bit to re-use stringify macro for any place required to use string type. And also add another condition to stringify macro to not doing an unnecessary string-conversion again for string type.

[CLAassistant]

All committers have signed the CLA.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (e159509) 55.19% compared to head (78ae4ba) 55.19%.

❗ Current head 78ae4ba differs from pull request most recent head 00c0394. Consider uploading reports for the commit 00c0394 to get more accurate results

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #416   +/-   ##
=======================================
  Coverage   55.19%   55.19%           
=======================================
  Files          87       87           
  Lines        5479     5479           
=======================================
  Hits         3024     3024           
  Misses       2233     2233           
  Partials      222      222           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Yang-33]

Good catch, thanks @mekpavit !

[Yang-33]

👍

[Yang-33]

This bug was fixed as v8.2.1 , thank you!