test: fix test-domain-with-abort-on-uncaught-exception.js on AIX

[dnakamura]

Add SIGTRAP as an expected signal when running on AIX. To fix failing test. see #3239

backport of f45c315 for v0.12

[cjihrig]

LGTM if the CI is happy

[mhdawson]

@dnakamura, I think the original issue was seen on platforms other than AIX and was fixed so that it resolved those cases as well. Is the test in 0.12.X such that the fix was only needed for AIX ? Otherwise we should probably fix in a way similar to the original where it applies across platforms

[dnakamura]

Yeah, however the style of the test in this case is aimed more on a case by case basis. Not entirely sure on how to implement while keeping with the style of the test

[mhdawson]

Probably worth looking at the original issue to see what other platforms needed the same fix then at least we can include them in the if with AIX

[dnakamura]

Actually if those platforms indicate the signal in the return code, it should be simple

[mhdawson]

Ok, good to hear. I assume you'll update the patch to cover them

[mhdawson]

Devin do you have an idea when you'll have the update covering the other platforms ?

[mhdawson]

@dnakamura just wondering if you'll get to this soon

[jasnell]

@dnakamura ... ping

[rvagg]

ping, what's the status of this? we'll be at the end of v0.12 soon!

[mhdawson]

Going to ask @iwuzhere if he can take this one over

[imran-iq]

Shouldn't this already be fixed with 018e4e0?

Which looks like a back port 425a354 that should resolve the "other platforms" being mentioned

[mhdawson]

I'd agree with @iwuzhere I think this can just be closed. I'll close if anybody disagrees please re-open.