[cmake][dune_add_test] Remove target dependency of expected to fail tests
requested to merge tkoch/dune-common:feature/fix-test-depend-on-target-for-expected-to-fail-tests into master
Since commit 27af4005 test depend on their target to correctly state "Not Run" when the target is missing instead of failed. This shouldn't be done for test that are expected to fail compiling though. This patch fixes that.
@joe I noticed a problem with the other fix too. Weirdly the CI didn't complain about this but you can see that the expected-to-fail-compiling tests are marked as "Not Run" now it in the logs. It seems weird that the pipeline still passed.