Ensure errors in `ProjectSubscriptions::Create mutation is wrapped in a flat array
The following discussion from !133308 (merged) should be addressed:
-
@mcelicalderonG started a discussion: (+2 comments) @panoskanell I just realized something. This might be a problem.
response.message
will sometimes return an array, so in here you will have an array within an array. There's a convenient method on ServiceResponse whichwraps
whatever was sent asmessage
in the initializer. So you can safely do this hereresult(errors: response.errors)
The request spec didn't catch this as the returned object is stubbed and in that one you pass a single string as the
message
argument. It should be safer to always useresponse.errors
.Sorry I didn't catch this before, but if this gets merged, we will get a server error any time the service returns an error result (instead of getting the error string in the response)