Permalink
Browse files

Merge pull request #831 from wmde/PiwikVariableCollectorTest

Fixed risky test in PiwikVariableCollectorTest
  • Loading branch information...
2 parents e0814ae + 32a12d6 commit 76d31e4619e7b374695e43291eba3fcfff079418 @KaiNissen KaiNissen committed on GitHub Feb 15, 2017
Showing with 3 additions and 1 deletion.
  1. +3 −1 tests/Unit/Infrastructure/PiwikVariableCollectorTest.php
@@ -74,10 +74,12 @@ public function sessionAndDonationDataProvider() {
];
}
- private function assertCustomTrackingContainsEvents( $expectedEvents, $actualEvents ) {
+ private function assertCustomTrackingContainsEvents( array $expectedEvents, array $actualEvents ) {
foreach ( $expectedEvents as $event ) {
$this->assertContains( $event, $actualEvents );
}
+
+ $this->assertCount( count( $expectedEvents ), $actualEvents );
}
private function newDonationMock( string $amount, string $paymentType, int $paymentInterval ) {

0 comments on commit 76d31e4

Please sign in to comment.