Moved error-lambda out, there's currently two predefined ones. One
that does allow failover and one that doesn't. This also kills the
newly introduced no-failover key. But it's for the better anyways.
This will make it much easier to actually do proper testing of
different cases in the code that simply wasn't possible before.
Also added a test of running the success lambda.
This was the first test I wrote, which also tested provider
priority. But now I have separate tests for provider priority so it
didn't need to be as complex anymore.
"test-helper.el" isn't a library intended to be loaded with `require`.
Instead it is loaded with `load`. Because of that, it is confusing to
provide a feature using `provide` anyway. And because around 70 other
packages come with a file named "test-helper.el" and several dozen of
those still provide the feature `test-helper`, this also leads to a,
admittedly somewhat theoretical, feature conflict.
This is also discussed at https://github.com/rejeep/ert-runner.el/issues/38.