Each unit test in /query/conditionals.php is being called twice.

[tyxla]

When testing a particular test under /tests/phpunit/tests/query/conditionals.php, that test is being called twice. For example, you can try the following command:

phpunit --filter "test_post_comments_feed_with_no_comments"

(the test_post_comments_feed_with_no_comments test is defined under Tests_Query_Conditionals in /tests/phpunit/tests/query/conditionals.php).

[tyxla]

After investigating I discovered that this has is caused by tests/phpunit/tests/query/verboseRewriteRules.php. Extending Tests_Query_Conditionals causes its tests to be called twice.

This file appears to have been created in 915/tests.

Since this file contains no tests, it does nothing so it can be removed.

Not sure if a patch is needed for that, but attaching one anyway.

[tyxla]

Removing tests/phpunit/tests/query/verboseRewriteRules.php to avoid duplicate calling of query/conditional tests.

[boonebgorges]

it does nothing so it can be removed.

This is not correct. Tests_Query_VerbosePageRules extends Tests_Query_Conditionals, which means it runs all the test methods of the latter class. And the two are not redundant, since they run with different permalink structures. I'm not sure what the original purpose of testing these specific permalink structures was - maybe @dd32 can chime in - but I don't think we're looking at a bug.

[tyxla]

Makes sense to me, thanks for shedding light on that.
Guess this is done intentionally, so that ticket can be closed indeed.

[dd32]

Indeed this is by design.

Different permalink formats could act differently to one another, so it's testing various angles.

I suspect this might be from prior to our Verbose rewrite rule cleanups, so it might not still apply for the original purpose, but there's no harm in keeping them both IMHO, as it provides better coverage.