Skip to content

Commit 4dbdbb5

Browse files
makasimmakasim
authored
Merge pull request #1371 from JimTools/bugfix/risky-tests
Fixing risky tests
2 parents 48d3303 + 4c232c2 commit 4dbdbb5

File tree

120 files changed

+42
-667
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

120 files changed

+42
-667
lines changed

‎pkg/amqp-bunny/Tests/AmqpConsumerTest.php

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -27,11 +27,6 @@ public function testShouldImplementConsumerInterface()
2727
$this->assertClassImplements(Consumer::class, AmqpConsumer::class);
2828
}
2929

30-
public function testCouldBeConstructedWithContextAndQueueAsArguments()
31-
{
32-
new AmqpConsumer($this->createContextMock(), new AmqpQueue('aName'));
33-
}
34-
3530
public function testShouldReturnQueue()
3631
{
3732
$queue = new AmqpQueue('aName');

‎pkg/amqp-bunny/Tests/AmqpProducerTest.php

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,6 @@ class AmqpProducerTest extends TestCase
2424
{
2525
use ClassExtensionTrait;
2626

27-
public function testCouldBeConstructedWithRequiredArguments()
28-
{
29-
new AmqpProducer($this->createBunnyChannelMock(), $this->createContextMock());
30-
}
31-
3227
public function testShouldImplementQueueInteropProducerInterface()
3328
{
3429
$this->assertClassImplements(Producer::class, AmqpProducer::class);

‎pkg/amqp-bunny/Tests/AmqpSubscriptionConsumerTest.php

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,6 @@ public function testShouldImplementQueueInteropSubscriptionConsumerInterface()
1717
$this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class));
1818
}
1919

20-
public function testCouldBeConstructedWithAmqpContextAsFirstArgument()
21-
{
22-
new AmqpSubscriptionConsumer($this->createAmqpContextMock());
23-
}
24-
2520
/**
2621
* @return AmqpContext|MockObject
2722
*/

‎pkg/amqp-ext/Tests/AmqpConsumerTest.php

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -19,11 +19,6 @@ public function testShouldImplementConsumerInterface()
1919
$this->assertClassImplements(Consumer::class, AmqpConsumer::class);
2020
}
2121

22-
public function testCouldBeConstructedWithContextAndQueueAsArguments()
23-
{
24-
new AmqpConsumer($this->createContext(), new AmqpQueue('aName'));
25-
}
26-
2722
/**
2823
* @return MockObject|AmqpContext
2924
*/

‎pkg/amqp-ext/Tests/AmqpContextTest.php

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -28,18 +28,6 @@ public function testShouldImplementQueueInteropContextInterface()
2828
$this->assertClassImplements(Context::class, AmqpContext::class);
2929
}
3030

31-
public function testCouldBeConstructedWithExtChannelAsFirstArgument()
32-
{
33-
new AmqpContext($this->createExtChannelMock());
34-
}
35-
36-
public function testCouldBeConstructedWithExtChannelCallbackFactoryAsFirstArgument()
37-
{
38-
new AmqpContext(function () {
39-
return $this->createExtChannelMock();
40-
});
41-
}
42-
4331
public function testThrowIfNeitherCallbackNorExtChannelAsFirstArgument()
4432
{
4533
$this->expectException(\InvalidArgumentException::class);

‎pkg/amqp-ext/Tests/AmqpSubscriptionConsumerTest.php

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,6 @@ public function testShouldImplementQueueInteropSubscriptionConsumerInterface()
1717
$this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class));
1818
}
1919

20-
public function testCouldBeConstructedWithAmqpContextAsFirstArgument()
21-
{
22-
new AmqpSubscriptionConsumer($this->createAmqpContextMock());
23-
}
24-
2520
/**
2621
* @return AmqpContext|MockObject
2722
*/

‎pkg/async-event-dispatcher/Tests/AsyncProcessorTest.php

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,6 @@ public function testShouldImplementProcessorInterface()
2424
$this->assertClassImplements(Processor::class, AsyncProcessor::class);
2525
}
2626

27-
public function testCouldBeConstructedWithRegistryAndProxyEventDispatcher()
28-
{
29-
new AsyncProcessor($this->createRegistryMock(), $this->createProxyEventDispatcherMock());
30-
}
31-
3227
public function testRejectIfMessageMissingEventNameProperty()
3328
{
3429
$processor = new AsyncProcessor($this->createRegistryMock(), $this->createProxyEventDispatcherMock());

‎pkg/async-event-dispatcher/Tests/PhpSerializerEventTransformerTest.php

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21,11 +21,6 @@ public function testShouldImplementEventTransformerInterface()
2121
$this->assertClassImplements(EventTransformer::class, PhpSerializerEventTransformer::class);
2222
}
2323

24-
public function testCouldBeConstructedWithoutAnyArguments()
25-
{
26-
new PhpSerializerEventTransformer($this->createContextStub());
27-
}
28-
2924
public function testShouldReturnMessageWithPhpSerializedEventAsBodyOnToMessage()
3025
{
3126
$transformer = new PhpSerializerEventTransformer($this->createContextStub());

‎pkg/async-event-dispatcher/Tests/SimpleRegistryTest.php

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -18,11 +18,6 @@ public function testShouldImplementRegistryInterface()
1818
$this->assertClassImplements(Registry::class, SimpleRegistry::class);
1919
}
2020

21-
public function testCouldBeConstructedWithEventsMapAndTransformersMapAsArguments()
22-
{
23-
new SimpleRegistry([], []);
24-
}
25-
2621
public function testShouldAllowGetTransportNameByEventName()
2722
{
2823
$registry = new SimpleRegistry([

‎pkg/dbal/Tests/DbalConsumerTest.php

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,11 +28,6 @@ public function testShouldImplementConsumerInterface()
2828
$this->assertClassImplements(Consumer::class, DbalConsumer::class);
2929
}
3030

31-
public function testCouldBeConstructedWithRequiredArguments()
32-
{
33-
new DbalConsumer($this->createContextMock(), new DbalDestination('queue'));
34-
}
35-
3631
public function testShouldReturnInstanceOfDestination()
3732
{
3833
$destination = new DbalDestination('queue');

0 commit comments

Comments
 (0)