facebook / flipper / 11742194002 / 3

51%

main: 51%

push

github

Client error message

Summary:
See previous diff in stack for context, serialization can fail and when it does we just send an empty string payload to flipper which is v confusing.

Instead we now do proper error handling

A note about the use of try catch around nsjson serialization i know this is not typical but the previous approach was to use the validator function to see if data could be serialized. this was returning false before but the problem is we have idea what the error is. Unfortunatley despite taking an error parm the NSJSONSerialization dataWithJSONObject function throws an exception so im force to catch it to get a useful error message

Differential Revision: D65605276

fbshipit-source-id: 506ecc48b

3292 of 7365 branches covered (44.7%)

Branch coverage included in aggregate %.

8124 of 15013 relevant lines covered (54.11%)

84.92 hits per line