fix: GH-910 separate error handlers

[tomsaleeba]

This isn't a complete fix for #910 but it at least makes it so

1. the original error isn't masked
2. the promise is rejected so calling code can handle the error

I figured the ternary expression was only there because there was a single handler function.

Once I made this change, I could see that I was getting a transaction abort not an error. The distinction was hard to tell before just based on line numbers as they used the same function. The error for the aborted transaction is:

I don't know what's causing that, but that's not relevant to this PR. Although I'll happily accept any guesses on where to look for it.

CC @mimittqq who requested this in the linked issue too.

CC @tokmakoff

[tomsaleeba]

Actually, swapping PR to draft. I should've looked closer at my screenshot. I'm not catching that error after all.

I'll figure it out.