Skip to content

Commit

Permalink
Merge pull request #609 from php-enqueue/fix-tests-08
Browse files Browse the repository at this point in the history
Fix Tests 0.8x
  • Loading branch information
makasim authored Nov 2, 2018
2 parents 3365fa7 + b462348 commit b0867d8
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions pkg/rdkafka/Tests/RdKafkaConsumerTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ public function testShouldReceiveFromQueueAndReturnNullIfNoMessageInQueue()
$kafkaConsumer = $this->createKafkaConsumerMock();
$kafkaConsumer
->expects($this->once())
->method('assign')
->method('subscribe')
;
$kafkaConsumer
->expects($this->once())
Expand Down Expand Up @@ -94,6 +94,8 @@ public function testShouldPassProperlyConfiguredTopicPartitionOnAssign()
$this->createSerializerMock()
);

$consumer->setOffset(12345);

$consumer->receive(1000);
$consumer->receive(1000);
$consumer->receive(1000);
Expand All @@ -109,7 +111,7 @@ public function testShouldSubscribeOnFirstReceiveOnly()
$kafkaConsumer = $this->createKafkaConsumerMock();
$kafkaConsumer
->expects($this->once())
->method('assign')
->method('subscribe')
;
$kafkaConsumer
->expects($this->any())
Expand Down Expand Up @@ -139,7 +141,7 @@ public function testThrowOnOffsetChangeAfterSubscribing()
$kafkaConsumer = $this->createKafkaConsumerMock();
$kafkaConsumer
->expects($this->once())
->method('assign')
->method('subscribe')
;
$kafkaConsumer
->expects($this->any())
Expand Down Expand Up @@ -174,7 +176,7 @@ public function testShouldReceiveFromQueueAndReturnMessageIfMessageInQueue()
$kafkaConsumer = $this->createKafkaConsumerMock();
$kafkaConsumer
->expects($this->once())
->method('assign')
->method('subscribe')
;
$kafkaConsumer
->expects($this->once())
Expand Down

0 comments on commit b0867d8

Please sign in to comment.