Skip to content

Commit 52ac2dd

Browse files
committed
use assertCount where possible in tests
1 parent fd23046 commit 52ac2dd

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

tests/VIPSoft/CodeCoverageCommon/Model/AggregateTest.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ public function testGetCoverage()
2424
$coverage = $aggregate->getCoverage();
2525

2626
$this->assertTrue(is_array($coverage));
27-
$this->assertEquals(0, count($coverage));
27+
$this->assertCount(0, $coverage);
2828
}
2929

3030
public function testUpdateWhenEmpty()
@@ -35,7 +35,7 @@ public function testUpdateWhenEmpty()
3535
$coverage = $aggregate->getCoverage();
3636

3737
$this->assertTrue(is_array($coverage));
38-
$this->assertEquals(1, count($coverage));
38+
$this->assertCount(1, $coverage);
3939
$this->assertEquals(array(2 => 1), $coverage['test']);
4040
}
4141

@@ -48,7 +48,7 @@ public function testUpdateForCollision()
4848
$coverage = $aggregate->getCoverage();
4949

5050
$this->assertTrue(is_array($coverage));
51-
$this->assertEquals(2, count($coverage));
51+
$this->assertCount(2, $coverage);
5252
$this->assertEquals(array(2 => 1), $coverage['first']);
5353
$this->assertEquals(array(1 => -1), $coverage['second']);
5454
}
@@ -65,7 +65,7 @@ public function testUpdate($first, $second, $expected)
6565
$coverage = $aggregate->getCoverage();
6666

6767
$this->assertTrue(is_array($coverage));
68-
$this->assertEquals(1, count($coverage));
68+
$this->assertCount(1, $coverage);
6969
$this->assertEquals($expected, $coverage['test']);
7070
}
7171

0 commit comments

Comments
 (0)