diff options
| author | Thiemo Kreuz <thiemo.kreuz@wikimedia.de> | 2021-04-22 10:40:46 +0200 |
|---|---|---|
| committer | Thiemo Kreuz <thiemo.kreuz@wikimedia.de> | 2021-04-23 11:58:58 +0200 |
| commit | b95a07380a64e72581a8bf41f466f58a1901ecb1 (patch) | |
| tree | c4c271c16b01c10685f4bef807b71d1e2a41bf2d /tests/phpunit/unit/includes/api/ApiUnblockTest.php | |
| parent | 791aa5e30666ab10736266e1d04c506aec771d96 (diff) | |
| download | mediawikicore-b95a07380a64e72581a8bf41f466f58a1901ecb1.tar.gz mediawikicore-b95a07380a64e72581a8bf41f466f58a1901ecb1.zip | |
Remove meaningless ->expects( $this->any() ) from all tests
It is not entirely meaningless. It might be an indicator that
the number of calls to a method is intentionally unlimited.
This is similar to e.g. an @inheritDoc PHPDoc comment that
marks a method as being "intentionally undocumented".
However, what's the meaning of being "intentionally
unconstrained"? Let's just not have any constraint then.
I feel all these ->expects( $this->any() ) bloat the test
code so much that it's never worth it.
Change-Id: I9925e7706bd03e1666f6eb0b284cb42b0dd3be23
Diffstat (limited to 'tests/phpunit/unit/includes/api/ApiUnblockTest.php')
0 files changed, 0 insertions, 0 deletions