Skip to content

Commit 4dbdbb5

Browse files
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)
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy