Update evaluation hook tests

[carmocca]

What does this PR do?

• Merge tests
• Test arguments

Related to the Lightning hooks review: #7740

Before submitting

• Was this discussed/approved via a GitHub issue? (not for typos and docs)
• Did you read the contributor guideline, Pull Request section?
• Did you make sure your PR does only one thing, instead of bundling different changes together?
• [n/a] Did you make sure to update the documentation with your changes? (if necessary)
• Did you write any new necessary tests? (not for typos and docs)
• Did you verify new and existing tests pass locally with your changes?
• [n/a] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review

• Is this pull request ready for review? (if not, please submit in draft mode)
• Check that all items from Before submitting are resolved
• Make sure the title is self-explanatory and the description concisely explains the PR
• Add labels and milestones (and optionally projects) to the PR so it can be classified

[codecov]

Codecov Report

Merging #8013 (9dd95cc) into master (906de2a) will decrease coverage by 0%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #8013    +/-   ##
=======================================
- Coverage      92%     91%    -0%     
=======================================
  Files         207     210     +3     
  Lines       13389   13576   +187     
=======================================
+ Hits        12259   12411   +152     
- Misses       1130    1165    +35     

[awaelchli]

I like the explicit tests we had before more :)

[carmocca]

I like the explicit tests we had before more :)

I didn't change it to code-golf it. It has the advantage of easily checking any divergence between validate and test

[tchaton]

Same here. Definitely great value to validate validate vs test, but the structure seems more convoluted now.