Add test case for CcpCrashOnPurpose#32
Conversation
Good to have a test for this because its behaviour may change depending on the platform it runs on.
ccp-serpent
left a comment
There was a problem hiding this comment.
I see no problem with this addition in isolation.
Before accepting I'm curious about your opinion on whether something like this should exist in Exefile instead? That way, we could potentially extend our code coverage to our crashpad integration while achieving the same results as outlined in this PR.
This isn't testing crash dump functionality, this is only testing that core's ability to force a crash works. Adding relevant tests for the crashpad integration is a good idea, but should happen in addition to this. |
Good to have a test for this because its behaviour may change depending on the platform it runs on.