detect-engine: convert unittests to FAIL/PASS APIs v2

[ghostinthehive]

Make sure these boxes are signed before submitting your Pull Request -- thank you.

• I have read the contributing guide lines at https://redmine.openinfosecfoundation.org/projects/suricata/wiki/Contributing
• I have signed the Open Information Security Foundation contribution agreement at https://suricata-ids.org/about/contribution-agreement/
• I have updated the user guide (in doc/userguide/) to reflect the changes made (if applicable)

Link to redmine ticket:
https://redmine.openinfosecfoundation.org/issues/4024

Ticket: #4024
Prev-PR: #6523

Describe changes:

• Update DetectEngineTest* unittests to use FAIL/PASS APIs
• Structure the Tests to use a series of FAIL tests, and a final PASS at the end,
  if all FAIL tests aren't triggered.

[ghostinthehive]

walking through previous merged FAIL/PASS APIs commits at #6242
there was a change of the source file' copyright year in PR #6240 .

Asking if I should update detect-engine.c copyright year since it's 2007-2020?

[jufajardini]

walking through previous merged FAIL/PASS APIs commits at #6242 there was a change of the source file' copyright year in PR #6240 .

Asking if I should update detect-engine.c copyright year since it's 2007-2020?

Yes, please :) We'll usually update those as we work on them, so is indeed a good time for that.

[ghostinthehive]

Opened a PR-v3 #6532