Remove PASS_IF macro from the FAIL/PASS API
According to the discussion in this PR https://github.com/OISF/suricata/pull/5485
it was agreed that having a PASS_IF macro could lead to unittests checks that would
not have the PASS as the final call, or to the test-writing logic having to be reverted
to write the PASS_IF, after all the FAIL_IF checks.
Adding a ticket to keep this well documented/tracked.