f1ad5aed

By: Tom Sydney Kerckhove <syd@cs-syd.eu>

Use expectFailing instead of try @SomeException for negative cases

The negative-case tests assert that runSqitchPostgresqlChecks /
runSqitchPersistentChecks throws on a deliberately broken fixture. The
previous shape was 'try @SomeException ... case ... of Left _ -> ...':
broad, swallows any exception, and obscures the intent.

Wrap the negative-case describe blocks with expectFailing instead. The
test body just calls the library function directly; sydtest counts a
failure of the inner test as success. Reads as the literal intent.

Timeline