diff --git a/pkg/amqp-bunny/Tests/AmqpConsumerTest.php b/pkg/amqp-bunny/Tests/AmqpConsumerTest.php index a588c2e56..c2c694566 100644 --- a/pkg/amqp-bunny/Tests/AmqpConsumerTest.php +++ b/pkg/amqp-bunny/Tests/AmqpConsumerTest.php @@ -27,11 +27,6 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, AmqpConsumer::class); } - public function testCouldBeConstructedWithContextAndQueueAsArguments() - { - new AmqpConsumer($this->createContextMock(), new AmqpQueue('aName')); - } - public function testShouldReturnQueue() { $queue = new AmqpQueue('aName'); diff --git a/pkg/amqp-bunny/Tests/AmqpProducerTest.php b/pkg/amqp-bunny/Tests/AmqpProducerTest.php index e9244297c..17b390341 100644 --- a/pkg/amqp-bunny/Tests/AmqpProducerTest.php +++ b/pkg/amqp-bunny/Tests/AmqpProducerTest.php @@ -24,11 +24,6 @@ class AmqpProducerTest extends TestCase { use ClassExtensionTrait; - public function testCouldBeConstructedWithRequiredArguments() - { - new AmqpProducer($this->createBunnyChannelMock(), $this->createContextMock()); - } - public function testShouldImplementQueueInteropProducerInterface() { $this->assertClassImplements(Producer::class, AmqpProducer::class); diff --git a/pkg/amqp-bunny/Tests/AmqpSubscriptionConsumerTest.php b/pkg/amqp-bunny/Tests/AmqpSubscriptionConsumerTest.php index c1d056268..4bf08ded5 100644 --- a/pkg/amqp-bunny/Tests/AmqpSubscriptionConsumerTest.php +++ b/pkg/amqp-bunny/Tests/AmqpSubscriptionConsumerTest.php @@ -17,11 +17,6 @@ public function testShouldImplementQueueInteropSubscriptionConsumerInterface() $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class)); } - public function testCouldBeConstructedWithAmqpContextAsFirstArgument() - { - new AmqpSubscriptionConsumer($this->createAmqpContextMock()); - } - /** * @return AmqpContext|MockObject */ diff --git a/pkg/amqp-ext/Tests/AmqpConsumerTest.php b/pkg/amqp-ext/Tests/AmqpConsumerTest.php index 75c49cf01..ae803e047 100644 --- a/pkg/amqp-ext/Tests/AmqpConsumerTest.php +++ b/pkg/amqp-ext/Tests/AmqpConsumerTest.php @@ -19,11 +19,6 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, AmqpConsumer::class); } - public function testCouldBeConstructedWithContextAndQueueAsArguments() - { - new AmqpConsumer($this->createContext(), new AmqpQueue('aName')); - } - /** * @return MockObject|AmqpContext */ diff --git a/pkg/amqp-ext/Tests/AmqpContextTest.php b/pkg/amqp-ext/Tests/AmqpContextTest.php index 4c4ab4298..2b03bb3d2 100644 --- a/pkg/amqp-ext/Tests/AmqpContextTest.php +++ b/pkg/amqp-ext/Tests/AmqpContextTest.php @@ -28,18 +28,6 @@ public function testShouldImplementQueueInteropContextInterface() $this->assertClassImplements(Context::class, AmqpContext::class); } - public function testCouldBeConstructedWithExtChannelAsFirstArgument() - { - new AmqpContext($this->createExtChannelMock()); - } - - public function testCouldBeConstructedWithExtChannelCallbackFactoryAsFirstArgument() - { - new AmqpContext(function () { - return $this->createExtChannelMock(); - }); - } - public function testThrowIfNeitherCallbackNorExtChannelAsFirstArgument() { $this->expectException(\InvalidArgumentException::class); diff --git a/pkg/amqp-ext/Tests/AmqpSubscriptionConsumerTest.php b/pkg/amqp-ext/Tests/AmqpSubscriptionConsumerTest.php index 153fe275e..d71ddd776 100644 --- a/pkg/amqp-ext/Tests/AmqpSubscriptionConsumerTest.php +++ b/pkg/amqp-ext/Tests/AmqpSubscriptionConsumerTest.php @@ -17,11 +17,6 @@ public function testShouldImplementQueueInteropSubscriptionConsumerInterface() $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class)); } - public function testCouldBeConstructedWithAmqpContextAsFirstArgument() - { - new AmqpSubscriptionConsumer($this->createAmqpContextMock()); - } - /** * @return AmqpContext|MockObject */ diff --git a/pkg/async-event-dispatcher/Tests/AsyncProcessorTest.php b/pkg/async-event-dispatcher/Tests/AsyncProcessorTest.php index 7237c9ae2..019f9bcbe 100644 --- a/pkg/async-event-dispatcher/Tests/AsyncProcessorTest.php +++ b/pkg/async-event-dispatcher/Tests/AsyncProcessorTest.php @@ -24,11 +24,6 @@ public function testShouldImplementProcessorInterface() $this->assertClassImplements(Processor::class, AsyncProcessor::class); } - public function testCouldBeConstructedWithRegistryAndProxyEventDispatcher() - { - new AsyncProcessor($this->createRegistryMock(), $this->createProxyEventDispatcherMock()); - } - public function testRejectIfMessageMissingEventNameProperty() { $processor = new AsyncProcessor($this->createRegistryMock(), $this->createProxyEventDispatcherMock()); diff --git a/pkg/async-event-dispatcher/Tests/PhpSerializerEventTransformerTest.php b/pkg/async-event-dispatcher/Tests/PhpSerializerEventTransformerTest.php index 49f3a9371..498ca3ae9 100644 --- a/pkg/async-event-dispatcher/Tests/PhpSerializerEventTransformerTest.php +++ b/pkg/async-event-dispatcher/Tests/PhpSerializerEventTransformerTest.php @@ -21,11 +21,6 @@ public function testShouldImplementEventTransformerInterface() $this->assertClassImplements(EventTransformer::class, PhpSerializerEventTransformer::class); } - public function testCouldBeConstructedWithoutAnyArguments() - { - new PhpSerializerEventTransformer($this->createContextStub()); - } - public function testShouldReturnMessageWithPhpSerializedEventAsBodyOnToMessage() { $transformer = new PhpSerializerEventTransformer($this->createContextStub()); diff --git a/pkg/async-event-dispatcher/Tests/SimpleRegistryTest.php b/pkg/async-event-dispatcher/Tests/SimpleRegistryTest.php index 328ed1780..82e1e3fc7 100644 --- a/pkg/async-event-dispatcher/Tests/SimpleRegistryTest.php +++ b/pkg/async-event-dispatcher/Tests/SimpleRegistryTest.php @@ -18,11 +18,6 @@ public function testShouldImplementRegistryInterface() $this->assertClassImplements(Registry::class, SimpleRegistry::class); } - public function testCouldBeConstructedWithEventsMapAndTransformersMapAsArguments() - { - new SimpleRegistry([], []); - } - public function testShouldAllowGetTransportNameByEventName() { $registry = new SimpleRegistry([ diff --git a/pkg/dbal/Tests/DbalConsumerTest.php b/pkg/dbal/Tests/DbalConsumerTest.php index d36333c83..6538af11d 100644 --- a/pkg/dbal/Tests/DbalConsumerTest.php +++ b/pkg/dbal/Tests/DbalConsumerTest.php @@ -28,11 +28,6 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, DbalConsumer::class); } - public function testCouldBeConstructedWithRequiredArguments() - { - new DbalConsumer($this->createContextMock(), new DbalDestination('queue')); - } - public function testShouldReturnInstanceOfDestination() { $destination = new DbalDestination('queue'); diff --git a/pkg/dbal/Tests/DbalContextTest.php b/pkg/dbal/Tests/DbalContextTest.php index 12a13b21a..a1900b788 100644 --- a/pkg/dbal/Tests/DbalContextTest.php +++ b/pkg/dbal/Tests/DbalContextTest.php @@ -27,11 +27,6 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, DbalContext::class); } - public function testCouldBeConstructedWithRequiredArguments() - { - new DbalContext($this->createConnectionMock()); - } - public function testCouldBeConstructedWithEmptyConfiguration() { $factory = new DbalContext($this->createConnectionMock(), []); diff --git a/pkg/dbal/Tests/DbalProducerTest.php b/pkg/dbal/Tests/DbalProducerTest.php index 4dcb015e3..ec4d2043c 100644 --- a/pkg/dbal/Tests/DbalProducerTest.php +++ b/pkg/dbal/Tests/DbalProducerTest.php @@ -21,11 +21,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, DbalProducer::class); } - public function testCouldBeConstructedWithRequiredArguments() - { - new DbalProducer($this->createContextMock()); - } - public function testShouldThrowIfDestinationOfInvalidType() { $this->expectException(InvalidDestinationException::class); diff --git a/pkg/dbal/Tests/DbalSubscriptionConsumerTest.php b/pkg/dbal/Tests/DbalSubscriptionConsumerTest.php index a3fad6e66..bacbec127 100644 --- a/pkg/dbal/Tests/DbalSubscriptionConsumerTest.php +++ b/pkg/dbal/Tests/DbalSubscriptionConsumerTest.php @@ -25,11 +25,6 @@ public function testShouldImplementSubscriptionConsumerInterface() $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class)); } - public function testCouldBeConstructedWithDbalContextAsFirstArgument() - { - new DbalSubscriptionConsumer($this->createDbalContextMock()); - } - public function testShouldAddConsumerAndCallbackToSubscribersPropertyOnSubscribe() { $subscriptionConsumer = new DbalSubscriptionConsumer($this->createDbalContextMock()); @@ -66,6 +61,9 @@ public function testThrowsIfTrySubscribeAnotherConsumerToAlreadySubscribedQueue( $subscriptionConsumer->subscribe($barConsumer, $barCallback); } + /** + * @doesNotPerformAssertions + */ public function testShouldAllowSubscribeSameConsumerAndCallbackSecondTime() { $subscriptionConsumer = new DbalSubscriptionConsumer($this->createDbalContextMock()); diff --git a/pkg/dsn/Tests/DsnTest.php b/pkg/dsn/Tests/DsnTest.php index 0bcd7d43a..8bf4137ed 100644 --- a/pkg/dsn/Tests/DsnTest.php +++ b/pkg/dsn/Tests/DsnTest.php @@ -8,11 +8,6 @@ class DsnTest extends TestCase { - public function testCouldBeConstructedWithDsnAsFirstArgument() - { - Dsn::parseFirst('foo://localhost:1234'); - } - public function testThrowsIfSchemePartIsMissing() { $this->expectException(\LogicException::class); diff --git a/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrineClearIdentityMapExtensionTest.php b/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrineClearIdentityMapExtensionTest.php index 604012e78..7c5c2dd5d 100644 --- a/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrineClearIdentityMapExtensionTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrineClearIdentityMapExtensionTest.php @@ -16,11 +16,6 @@ class DoctrineClearIdentityMapExtensionTest extends TestCase { - public function testCouldBeConstructedWithRequiredArguments() - { - new DoctrineClearIdentityMapExtension($this->createRegistryMock()); - } - public function testShouldClearIdentityMap() { $manager = $this->createManagerMock(); diff --git a/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrinePingConnectionExtensionTest.php b/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrinePingConnectionExtensionTest.php index c530eeaa9..36df82e52 100644 --- a/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrinePingConnectionExtensionTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrinePingConnectionExtensionTest.php @@ -17,11 +17,6 @@ class DoctrinePingConnectionExtensionTest extends TestCase { - public function testCouldBeConstructedWithRequiredAttributes() - { - new DoctrinePingConnectionExtension($this->createRegistryMock()); - } - public function testShouldNotReconnectIfConnectionIsOK() { $connection = $this->createConnectionMock(); diff --git a/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/ResetServicesExtensionTest.php b/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/ResetServicesExtensionTest.php index 18e94f251..76fe29b94 100644 --- a/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/ResetServicesExtensionTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/ResetServicesExtensionTest.php @@ -16,10 +16,6 @@ class ResetServicesExtensionTest extends TestCase { - public function testCouldBeConstructedWithRequiredArguments() - { - new ResetServicesExtension($this->createResetterMock()); - } public function testItShouldResetServices() { diff --git a/pkg/enqueue-bundle/Tests/Unit/DependencyInjection/ConfigurationTest.php b/pkg/enqueue-bundle/Tests/Unit/DependencyInjection/ConfigurationTest.php index c9356f1ec..5330cde82 100644 --- a/pkg/enqueue-bundle/Tests/Unit/DependencyInjection/ConfigurationTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/DependencyInjection/ConfigurationTest.php @@ -25,11 +25,6 @@ public function testShouldBeFinal() $this->assertClassFinal(Configuration::class); } - public function testCouldBeConstructedWithDebugAsArgument() - { - new Configuration(true); - } - public function testShouldProcessSeveralTransports() { $configuration = new Configuration(true); diff --git a/pkg/enqueue-bundle/Tests/Unit/DependencyInjection/EnqueueExtensionTest.php b/pkg/enqueue-bundle/Tests/Unit/DependencyInjection/EnqueueExtensionTest.php index 3b8d0426d..6358bd24d 100644 --- a/pkg/enqueue-bundle/Tests/Unit/DependencyInjection/EnqueueExtensionTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/DependencyInjection/EnqueueExtensionTest.php @@ -30,11 +30,6 @@ public function testShouldBeFinal() $this->assertClassFinal(EnqueueExtension::class); } - public function testCouldBeConstructedWithoutAnyArguments() - { - new EnqueueExtension(); - } - public function testShouldRegisterConnectionFactory() { $container = $this->getContainerBuilder(true); diff --git a/pkg/enqueue-bundle/Tests/Unit/EnqueueBundleTest.php b/pkg/enqueue-bundle/Tests/Unit/EnqueueBundleTest.php index d93a1f1f9..7d5b0232b 100644 --- a/pkg/enqueue-bundle/Tests/Unit/EnqueueBundleTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/EnqueueBundleTest.php @@ -15,9 +15,4 @@ public function testShouldExtendBundleClass() { $this->assertClassExtends(Bundle::class, EnqueueBundle::class); } - - public function testCouldBeConstructedWithoutAnyArguments() - { - new EnqueueBundle(); - } } diff --git a/pkg/enqueue-bundle/Tests/Unit/Profiler/MessageQueueCollectorTest.php b/pkg/enqueue-bundle/Tests/Unit/Profiler/MessageQueueCollectorTest.php index 55f415d03..d6d638d75 100644 --- a/pkg/enqueue-bundle/Tests/Unit/Profiler/MessageQueueCollectorTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/Profiler/MessageQueueCollectorTest.php @@ -23,11 +23,6 @@ public function testShouldExtendDataCollectorClass() $this->assertClassExtends(DataCollector::class, MessageQueueCollector::class); } - public function testCouldBeConstructedWithEmptyConstructor() - { - new MessageQueueCollector(); - } - public function testShouldReturnExpectedName() { $collector = new MessageQueueCollector(); diff --git a/pkg/enqueue/Tests/ArrayProcessorRegistryTest.php b/pkg/enqueue/Tests/ArrayProcessorRegistryTest.php index 897276d62..93ad3ece1 100644 --- a/pkg/enqueue/Tests/ArrayProcessorRegistryTest.php +++ b/pkg/enqueue/Tests/ArrayProcessorRegistryTest.php @@ -18,11 +18,6 @@ public function testShouldImplementProcessorRegistryInterface() $this->assertClassImplements(ProcessorRegistryInterface::class, ArrayProcessorRegistry::class); } - public function testCouldBeConstructedWithoutAnyArgument() - { - new ArrayProcessorRegistry(); - } - public function testShouldThrowExceptionIfProcessorIsNotSet() { $registry = new ArrayProcessorRegistry(); diff --git a/pkg/enqueue/Tests/Client/ChainExtensionTest.php b/pkg/enqueue/Tests/Client/ChainExtensionTest.php index a59005bc1..0f42bcf18 100644 --- a/pkg/enqueue/Tests/Client/ChainExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ChainExtensionTest.php @@ -29,11 +29,6 @@ public function testShouldBeFinal() $this->assertClassFinal(ChainExtension::class); } - public function testCouldBeConstructedWithExtensionsArray() - { - new ChainExtension([$this->createExtension(), $this->createExtension()]); - } - public function testThrowIfArrayContainsNotExtension() { $this->expectException(\LogicException::class); diff --git a/pkg/enqueue/Tests/Client/ConsumptionExtension/DelayRedeliveredMessageExtensionTest.php b/pkg/enqueue/Tests/Client/ConsumptionExtension/DelayRedeliveredMessageExtensionTest.php index 0481d27d8..56b855f0c 100644 --- a/pkg/enqueue/Tests/Client/ConsumptionExtension/DelayRedeliveredMessageExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ConsumptionExtension/DelayRedeliveredMessageExtensionTest.php @@ -23,10 +23,6 @@ class DelayRedeliveredMessageExtensionTest extends TestCase { - public function testCouldBeConstructedWithRequiredArguments() - { - new DelayRedeliveredMessageExtension($this->createDriverMock(), 12345); - } public function testShouldSendDelayedMessageAndRejectOriginalMessage() { diff --git a/pkg/enqueue/Tests/Client/ConsumptionExtension/ExclusiveCommandExtensionTest.php b/pkg/enqueue/Tests/Client/ConsumptionExtension/ExclusiveCommandExtensionTest.php index dc2c00c96..b1e47c898 100644 --- a/pkg/enqueue/Tests/Client/ConsumptionExtension/ExclusiveCommandExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ConsumptionExtension/ExclusiveCommandExtensionTest.php @@ -34,11 +34,6 @@ public function testShouldBeFinal() $this->assertClassFinal(ExclusiveCommandExtension::class); } - public function testCouldBeConstructedWithDriverAsFirstArgument() - { - new ExclusiveCommandExtension($this->createDriverStub()); - } - public function testShouldDoNothingIfMessageHasTopicPropertySetOnPreReceive() { $message = new NullMessage(); diff --git a/pkg/enqueue/Tests/Client/ConsumptionExtension/FlushSpoolProducerExtensionTest.php b/pkg/enqueue/Tests/Client/ConsumptionExtension/FlushSpoolProducerExtensionTest.php index 63fb50c5a..6a782c524 100644 --- a/pkg/enqueue/Tests/Client/ConsumptionExtension/FlushSpoolProducerExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ConsumptionExtension/FlushSpoolProducerExtensionTest.php @@ -30,11 +30,6 @@ public function testShouldImplementEndExtensionInterface() $this->assertClassImplements(EndExtensionInterface::class, FlushSpoolProducerExtension::class); } - public function testCouldBeConstructedWithSpoolProducerAsFirstArgument() - { - new FlushSpoolProducerExtension($this->createSpoolProducerMock()); - } - public function testShouldFlushSpoolProducerOnEnd() { $producer = $this->createSpoolProducerMock(); diff --git a/pkg/enqueue/Tests/Client/ConsumptionExtension/LogExtensionTest.php b/pkg/enqueue/Tests/Client/ConsumptionExtension/LogExtensionTest.php index 063ea23af..db757676b 100644 --- a/pkg/enqueue/Tests/Client/ConsumptionExtension/LogExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ConsumptionExtension/LogExtensionTest.php @@ -55,11 +55,6 @@ public function testShouldSubClassOfLogExtension() $this->assertClassExtends(\Enqueue\Consumption\Extension\LogExtension::class, LogExtension::class); } - public function testCouldBeConstructedWithoutAnyArguments() - { - new LogExtension(); - } - public function testShouldLogStartOnStart() { $logger = $this->createLogger(); diff --git a/pkg/enqueue/Tests/Client/ConsumptionExtension/SetRouterPropertiesExtensionTest.php b/pkg/enqueue/Tests/Client/ConsumptionExtension/SetRouterPropertiesExtensionTest.php index e5015ff45..d521aefca 100644 --- a/pkg/enqueue/Tests/Client/ConsumptionExtension/SetRouterPropertiesExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ConsumptionExtension/SetRouterPropertiesExtensionTest.php @@ -27,11 +27,6 @@ public function testShouldImplementMessageReceivedExtensionInterface() $this->assertClassImplements(MessageReceivedExtensionInterface::class, SetRouterPropertiesExtension::class); } - public function testCouldBeConstructedWithRequiredArguments() - { - new SetRouterPropertiesExtension($this->createDriverMock()); - } - public function testShouldSetRouterProcessorPropertyIfNotSetAndOnRouterQueue() { $config = Config::create('test', '.', '', '', 'router-queue', '', 'router-processor-name'); diff --git a/pkg/enqueue/Tests/Client/ConsumptionExtension/SetupBrokerExtensionTest.php b/pkg/enqueue/Tests/Client/ConsumptionExtension/SetupBrokerExtensionTest.php index 4b241aa0e..fbd367975 100644 --- a/pkg/enqueue/Tests/Client/ConsumptionExtension/SetupBrokerExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ConsumptionExtension/SetupBrokerExtensionTest.php @@ -21,11 +21,6 @@ public function testShouldImplementStartExtensionInterface() $this->assertClassImplements(StartExtensionInterface::class, SetupBrokerExtension::class); } - public function testCouldBeConstructedWithRequiredArguments() - { - new SetupBrokerExtension($this->createDriverMock()); - } - public function testShouldSetupBroker() { $logger = new NullLogger(); diff --git a/pkg/enqueue/Tests/Client/DelegateProcessorTest.php b/pkg/enqueue/Tests/Client/DelegateProcessorTest.php index 40ef17989..9743cf4f3 100644 --- a/pkg/enqueue/Tests/Client/DelegateProcessorTest.php +++ b/pkg/enqueue/Tests/Client/DelegateProcessorTest.php @@ -13,11 +13,6 @@ class DelegateProcessorTest extends TestCase { - public function testCouldBeConstructedWithRequiredArguments() - { - new DelegateProcessor($this->createProcessorRegistryMock()); - } - public function testShouldThrowExceptionIfProcessorNameIsNotSet() { $this->expectException(\LogicException::class); diff --git a/pkg/enqueue/Tests/Client/DriverFactoryTest.php b/pkg/enqueue/Tests/Client/DriverFactoryTest.php index cce6311e5..ef504e0e0 100644 --- a/pkg/enqueue/Tests/Client/DriverFactoryTest.php +++ b/pkg/enqueue/Tests/Client/DriverFactoryTest.php @@ -61,11 +61,6 @@ public function testShouldBeFinal() $this->assertTrue($rc->isFinal()); } - public function testCouldBeConstructedWithoutAnyArguments() - { - new DriverFactory(); - } - public function testThrowIfPackageThatSupportSchemeNotInstalled() { $scheme = 'scheme5b7aa7d7cd213'; diff --git a/pkg/enqueue/Tests/Client/DriverPreSendTest.php b/pkg/enqueue/Tests/Client/DriverPreSendTest.php index 85686b4c1..32af2a81f 100644 --- a/pkg/enqueue/Tests/Client/DriverPreSendTest.php +++ b/pkg/enqueue/Tests/Client/DriverPreSendTest.php @@ -19,15 +19,6 @@ public function testShouldBeFinal() self::assertClassFinal(DriverPreSend::class); } - public function testCouldBeConstructedWithExpectedArguments() - { - new DriverPreSend( - new Message(), - $this->createProducerMock(), - $this->createDriverMock() - ); - } - public function testShouldAllowGetArgumentSetInConstructor() { $expectedMessage = new Message(); diff --git a/pkg/enqueue/Tests/Client/Extension/PrepareBodyExtensionTest.php b/pkg/enqueue/Tests/Client/Extension/PrepareBodyExtensionTest.php index 454034abb..d1c260388 100644 --- a/pkg/enqueue/Tests/Client/Extension/PrepareBodyExtensionTest.php +++ b/pkg/enqueue/Tests/Client/Extension/PrepareBodyExtensionTest.php @@ -22,11 +22,6 @@ public function testShouldImplementExtensionInterface() $this->assertTrue($rc->implementsInterface(PreSendCommandExtensionInterface::class)); } - public function testCouldConstructedWithoutAnyArguments() - { - new PrepareBodyExtension(); - } - /** * @dataProvider provideMessages * diff --git a/pkg/enqueue/Tests/Client/PostSendTest.php b/pkg/enqueue/Tests/Client/PostSendTest.php index 04927700c..ba51710e7 100644 --- a/pkg/enqueue/Tests/Client/PostSendTest.php +++ b/pkg/enqueue/Tests/Client/PostSendTest.php @@ -21,17 +21,6 @@ public function testShouldBeFinal() self::assertClassFinal(PostSend::class); } - public function testCouldBeConstructedWithExpectedArguments() - { - new PostSend( - new Message(), - $this->createProducerMock(), - $this->createDriverMock(), - $this->createDestinationMock(), - $this->createTransportMessageMock() - ); - } - public function testShouldAllowGetArgumentSetInConstructor() { $expectedMessage = new Message(); diff --git a/pkg/enqueue/Tests/Client/PreSendTest.php b/pkg/enqueue/Tests/Client/PreSendTest.php index 408be1c69..ae9400338 100644 --- a/pkg/enqueue/Tests/Client/PreSendTest.php +++ b/pkg/enqueue/Tests/Client/PreSendTest.php @@ -18,16 +18,6 @@ public function testShouldBeFinal() self::assertClassFinal(PreSend::class); } - public function testCouldBeConstructedWithExpectedArguments() - { - new PreSend( - 'aCommandOrTopic', - new Message(), - $this->createProducerMock(), - $this->createDriverMock() - ); - } - public function testShouldAllowGetArgumentSetInConstructor() { $expectedCommandOrTopic = 'theCommandOrTopic'; diff --git a/pkg/enqueue/Tests/Client/ProducerTest.php b/pkg/enqueue/Tests/Client/ProducerTest.php index 558e70ee6..9d4e2a304 100644 --- a/pkg/enqueue/Tests/Client/ProducerTest.php +++ b/pkg/enqueue/Tests/Client/ProducerTest.php @@ -24,20 +24,6 @@ public function testShouldBeFinal() self::assertClassFinal(Producer::class); } - public function testCouldBeConstructedWithRequiredArguments() - { - new Producer($this->createDriverMock(), $this->createRpcFactoryMock()); - } - - public function testCouldBeConstructedWithOptionalArguments() - { - new Producer( - $this->createDriverMock(), - $this->createRpcFactoryMock(), - $this->createMock(ExtensionInterface::class) - ); - } - /** * @return \PHPUnit\Framework\MockObject\MockObject */ diff --git a/pkg/enqueue/Tests/Client/SpoolProducerTest.php b/pkg/enqueue/Tests/Client/SpoolProducerTest.php index 024fd6da2..014fe4962 100644 --- a/pkg/enqueue/Tests/Client/SpoolProducerTest.php +++ b/pkg/enqueue/Tests/Client/SpoolProducerTest.php @@ -18,11 +18,6 @@ public function testShouldImplementProducerInterface() self::assertClassImplements(ProducerInterface::class, SpoolProducer::class); } - public function testCouldBeConstructedWithRealProducer() - { - new SpoolProducer($this->createProducerMock()); - } - public function testShouldQueueEventMessageOnSend() { $message = new Message(); diff --git a/pkg/enqueue/Tests/Client/TraceableProducerTest.php b/pkg/enqueue/Tests/Client/TraceableProducerTest.php index ac0a52961..ff042857c 100644 --- a/pkg/enqueue/Tests/Client/TraceableProducerTest.php +++ b/pkg/enqueue/Tests/Client/TraceableProducerTest.php @@ -18,11 +18,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(ProducerInterface::class, TraceableProducer::class); } - public function testCouldBeConstructedWithInternalMessageProducer() - { - new TraceableProducer($this->createProducerMock()); - } - public function testShouldPassAllArgumentsToInternalEventMessageProducerSendMethod() { $topic = 'theTopic'; diff --git a/pkg/enqueue/Tests/ConnectionFactoryFactoryTest.php b/pkg/enqueue/Tests/ConnectionFactoryFactoryTest.php index 3fb4fb57b..fe2de69a0 100644 --- a/pkg/enqueue/Tests/ConnectionFactoryFactoryTest.php +++ b/pkg/enqueue/Tests/ConnectionFactoryFactoryTest.php @@ -37,11 +37,9 @@ public function testShouldBeFinal() $this->assertTrue($rc->isFinal()); } - public function testCouldBeConstructedWithoutAnyArguments() - { - new ConnectionFactoryFactory(); - } - + /** + * @doesNotPerformAssertions + */ public function testShouldAcceptStringDSN() { $factory = new ConnectionFactoryFactory(); @@ -49,6 +47,9 @@ public function testShouldAcceptStringDSN() $factory->create('null:'); } + /** + * @doesNotPerformAssertions + */ public function testShouldAcceptArrayWithDsnKey() { $factory = new ConnectionFactoryFactory(); diff --git a/pkg/enqueue/Tests/Consumption/CallbackProcessorTest.php b/pkg/enqueue/Tests/Consumption/CallbackProcessorTest.php index f134cfe9f..86adbd3a9 100644 --- a/pkg/enqueue/Tests/Consumption/CallbackProcessorTest.php +++ b/pkg/enqueue/Tests/Consumption/CallbackProcessorTest.php @@ -18,12 +18,6 @@ public function testShouldImplementProcessorInterface() $this->assertClassImplements(Processor::class, CallbackProcessor::class); } - public function testCouldBeConstructedWithCallableAsArgument() - { - new CallbackProcessor(function () { - }); - } - public function testShouldCallCallbackAndProxyItsReturnedValue() { $expectedMessage = new NullMessage(); diff --git a/pkg/enqueue/Tests/Consumption/ChainExtensionTest.php b/pkg/enqueue/Tests/Consumption/ChainExtensionTest.php index 23bd4f979..198d00012 100644 --- a/pkg/enqueue/Tests/Consumption/ChainExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/ChainExtensionTest.php @@ -33,11 +33,6 @@ public function testShouldImplementExtensionInterface() $this->assertClassImplements(ExtensionInterface::class, ChainExtension::class); } - public function testCouldBeConstructedWithExtensionsArray() - { - new ChainExtension([$this->createExtension(), $this->createExtension()]); - } - public function testShouldProxyOnInitLoggerToAllInternalExtensions() { $context = new InitLogger(new NullLogger()); diff --git a/pkg/enqueue/Tests/Consumption/Exception/IllegalContextModificationExceptionTest.php b/pkg/enqueue/Tests/Consumption/Exception/IllegalContextModificationExceptionTest.php index 0885b500e..241f4adf9 100644 --- a/pkg/enqueue/Tests/Consumption/Exception/IllegalContextModificationExceptionTest.php +++ b/pkg/enqueue/Tests/Consumption/Exception/IllegalContextModificationExceptionTest.php @@ -20,9 +20,4 @@ public function testShouldExtendLogicException() { $this->assertClassExtends(\LogicException::class, IllegalContextModificationException::class); } - - public function testCouldBeConstructedWithoutAnyArguments() - { - new IllegalContextModificationException(); - } } diff --git a/pkg/enqueue/Tests/Consumption/Exception/InvalidArgumentExceptionTest.php b/pkg/enqueue/Tests/Consumption/Exception/InvalidArgumentExceptionTest.php index 296c76225..c1c5db362 100644 --- a/pkg/enqueue/Tests/Consumption/Exception/InvalidArgumentExceptionTest.php +++ b/pkg/enqueue/Tests/Consumption/Exception/InvalidArgumentExceptionTest.php @@ -21,11 +21,6 @@ public function testShouldExtendLogicException() $this->assertClassExtends(\LogicException::class, InvalidArgumentException::class); } - public function testCouldBeConstructedWithoutAnyArguments() - { - new InvalidArgumentException(); - } - public function testThrowIfAssertInstanceOfNotSameAsExpected() { $this->expectException(InvalidArgumentException::class); @@ -36,6 +31,9 @@ public function testThrowIfAssertInstanceOfNotSameAsExpected() InvalidArgumentException::assertInstanceOf(new \SplStack(), \SplQueue::class); } + /** + * @doesNotPerformAssertions + */ public function testShouldDoNothingIfAssertDestinationInstanceOfSameAsExpected() { InvalidArgumentException::assertInstanceOf(new \SplQueue(), \SplQueue::class); diff --git a/pkg/enqueue/Tests/Consumption/Exception/LogicExceptionTest.php b/pkg/enqueue/Tests/Consumption/Exception/LogicExceptionTest.php index ddd258098..2655609ae 100644 --- a/pkg/enqueue/Tests/Consumption/Exception/LogicExceptionTest.php +++ b/pkg/enqueue/Tests/Consumption/Exception/LogicExceptionTest.php @@ -20,9 +20,4 @@ public function testShouldExtendLogicException() { $this->assertClassExtends(\LogicException::class, LogicException::class); } - - public function testCouldBeConstructedWithoutAnyArguments() - { - new LogicException(); - } } diff --git a/pkg/enqueue/Tests/Consumption/Extension/LimitConsumedMessagesExtensionTest.php b/pkg/enqueue/Tests/Consumption/Extension/LimitConsumedMessagesExtensionTest.php index e1d4c50cd..137e30ba4 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/LimitConsumedMessagesExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/LimitConsumedMessagesExtensionTest.php @@ -16,11 +16,6 @@ class LimitConsumedMessagesExtensionTest extends TestCase { - public function testCouldBeConstructedWithRequiredArguments() - { - new LimitConsumedMessagesExtension(12345); - } - public function testOnPreConsumeShouldInterruptWhenLimitIsReached() { $logger = $this->createLoggerMock(); diff --git a/pkg/enqueue/Tests/Consumption/Extension/LimitConsumerMemoryExtensionTest.php b/pkg/enqueue/Tests/Consumption/Extension/LimitConsumerMemoryExtensionTest.php index 5a95486ec..15482997d 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/LimitConsumerMemoryExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/LimitConsumerMemoryExtensionTest.php @@ -17,11 +17,6 @@ class LimitConsumerMemoryExtensionTest extends TestCase { - public function testCouldBeConstructedWithRequiredArguments() - { - new LimitConsumerMemoryExtension(12345); - } - public function testShouldThrowExceptionIfMemoryLimitIsNotInt() { $this->expectException(\InvalidArgumentException::class); diff --git a/pkg/enqueue/Tests/Consumption/Extension/LimitConsumptionTimeExtensionTest.php b/pkg/enqueue/Tests/Consumption/Extension/LimitConsumptionTimeExtensionTest.php index fa730e384..fa6cb76a1 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/LimitConsumptionTimeExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/LimitConsumptionTimeExtensionTest.php @@ -17,11 +17,6 @@ class LimitConsumptionTimeExtensionTest extends TestCase { - public function testCouldBeConstructedWithRequiredArguments() - { - new LimitConsumptionTimeExtension(new \DateTime('+1 day')); - } - public function testOnPreConsumeShouldInterruptExecutionIfConsumptionTimeExceeded() { $context = new PreConsume( diff --git a/pkg/enqueue/Tests/Consumption/Extension/LogExtensionTest.php b/pkg/enqueue/Tests/Consumption/Extension/LogExtensionTest.php index 4bc4ae95f..006a2c549 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/LogExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/LogExtensionTest.php @@ -49,11 +49,6 @@ public function testShouldImplementPostMessageReceivedExtensionInterface() $this->assertClassImplements(PostMessageReceivedExtensionInterface::class, LogExtension::class); } - public function testCouldBeConstructedWithoutAnyArguments() - { - new LogExtension(); - } - public function testShouldLogStartOnStart() { $logger = $this->createLogger(); diff --git a/pkg/enqueue/Tests/Consumption/Extension/LoggerExtensionTest.php b/pkg/enqueue/Tests/Consumption/Extension/LoggerExtensionTest.php index f3641adf4..455468ed2 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/LoggerExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/LoggerExtensionTest.php @@ -20,11 +20,6 @@ public function testShouldImplementInitLoggerExtensionInterface() $this->assertClassImplements(InitLoggerExtensionInterface::class, LoggerExtension::class); } - public function testCouldBeConstructedWithLoggerAsFirstArgument() - { - new LoggerExtension($this->createLogger()); - } - public function testShouldSetLoggerToContextOnInitLogger() { $logger = $this->createLogger(); diff --git a/pkg/enqueue/Tests/Consumption/Extension/NicenessExtensionTest.php b/pkg/enqueue/Tests/Consumption/Extension/NicenessExtensionTest.php index a2937d0ca..734bc8417 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/NicenessExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/NicenessExtensionTest.php @@ -11,11 +11,6 @@ class NicenessExtensionTest extends TestCase { - public function testCouldBeConstructedWithRequiredArguments() - { - new NicenessExtension(0); - } - public function testShouldThrowExceptionOnInvalidArgument() { $this->expectException(\InvalidArgumentException::class); diff --git a/pkg/enqueue/Tests/Consumption/Extension/ReplyExtensionTest.php b/pkg/enqueue/Tests/Consumption/Extension/ReplyExtensionTest.php index 408895638..cb65816ce 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/ReplyExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/ReplyExtensionTest.php @@ -25,11 +25,6 @@ public function testShouldImplementPostMessageReceivedExtensionInterface() $this->assertClassImplements(PostMessageReceivedExtensionInterface::class, ReplyExtension::class); } - public function testCouldBeConstructedWithoutAnyArguments() - { - new ReplyExtension(); - } - public function testShouldDoNothingIfReceivedMessageNotHaveReplyToSet() { $extension = new ReplyExtension(); diff --git a/pkg/enqueue/Tests/Consumption/FallbackSubscriptionConsumerTest.php b/pkg/enqueue/Tests/Consumption/FallbackSubscriptionConsumerTest.php index 4f8ac7a55..5391f0c68 100644 --- a/pkg/enqueue/Tests/Consumption/FallbackSubscriptionConsumerTest.php +++ b/pkg/enqueue/Tests/Consumption/FallbackSubscriptionConsumerTest.php @@ -21,11 +21,6 @@ public function testShouldImplementSubscriptionConsumerInterface() $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class)); } - public function testCouldBeConstructedWithoutAnyArguments() - { - new FallbackSubscriptionConsumer(); - } - public function testShouldInitSubscribersPropertyWithEmptyArray() { $subscriptionConsumer = new FallbackSubscriptionConsumer(); @@ -69,6 +64,9 @@ public function testThrowsIfTrySubscribeAnotherConsumerToAlreadySubscribedQueue( $subscriptionConsumer->subscribe($barConsumer, $barCallback); } + /** + * @doesNotPerformAssertions + */ public function testShouldAllowSubscribeSameConsumerAndCallbackSecondTime() { $subscriptionConsumer = new FallbackSubscriptionConsumer(); diff --git a/pkg/enqueue/Tests/Consumption/QueueConsumerTest.php b/pkg/enqueue/Tests/Consumption/QueueConsumerTest.php index ecf7f9afa..06edab31b 100644 --- a/pkg/enqueue/Tests/Consumption/QueueConsumerTest.php +++ b/pkg/enqueue/Tests/Consumption/QueueConsumerTest.php @@ -39,21 +39,6 @@ class QueueConsumerTest extends TestCase { use ReadAttributeTrait; - public function testCouldBeConstructedWithAllArguments() - { - new QueueConsumer($this->createContextStub(), null, [], null, 0); - } - - public function testCouldBeConstructedWithContextOnly() - { - new QueueConsumer($this->createContextStub()); - } - - public function testCouldBeConstructedWithContextAndSingleExtension() - { - new QueueConsumer($this->createContextStub(), $this->createExtension()); - } - public function testShouldSetEmptyArrayToBoundProcessorsPropertyInConstructor() { $consumer = new QueueConsumer($this->createContextStub(), null, [], null, 0); diff --git a/pkg/enqueue/Tests/Router/RouteRecipientListProcessorTest.php b/pkg/enqueue/Tests/Router/RouteRecipientListProcessorTest.php index 8a2e089f7..ae878fc53 100644 --- a/pkg/enqueue/Tests/Router/RouteRecipientListProcessorTest.php +++ b/pkg/enqueue/Tests/Router/RouteRecipientListProcessorTest.php @@ -23,11 +23,6 @@ public function testShouldImplementProcessorInterface() $this->assertClassImplements(Processor::class, RouteRecipientListProcessor::class); } - public function testCouldBeConstructedWithRouterAsFirstArgument() - { - new RouteRecipientListProcessor($this->createRecipientListRouterMock()); - } - public function testShouldProduceRecipientsMessagesAndAckOriginalMessage() { $fooRecipient = new Recipient(new NullQueue('aName'), new NullMessage()); diff --git a/pkg/enqueue/Tests/Rpc/RpcClientTest.php b/pkg/enqueue/Tests/Rpc/RpcClientTest.php index e2035c4d5..db4813c88 100644 --- a/pkg/enqueue/Tests/Rpc/RpcClientTest.php +++ b/pkg/enqueue/Tests/Rpc/RpcClientTest.php @@ -15,11 +15,6 @@ class RpcClientTest extends TestCase { - public function testCouldBeConstructedWithContextAsFirstArgument() - { - new RpcClient($this->createContextMock()); - } - public function testShouldSetReplyToIfNotSet() { $context = new NullContext(); diff --git a/pkg/enqueue/Tests/Symfony/Client/ConsumeCommandTest.php b/pkg/enqueue/Tests/Symfony/Client/ConsumeCommandTest.php index 8076aaf76..3758ca96a 100644 --- a/pkg/enqueue/Tests/Symfony/Client/ConsumeCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/ConsumeCommandTest.php @@ -40,11 +40,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(ConsumeCommand::class); } - public function testCouldBeConstructedWithRequiredAttributes() - { - new ConsumeCommand($this->createMock(ContainerInterface::class), 'default'); - } - public function testShouldHaveAsCommandAttributeWithCommandName() { $commandClass = ConsumeCommand::class; diff --git a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/AnalyzeRouteCollectionPassTest.php b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/AnalyzeRouteCollectionPassTest.php index 8dcffe45a..568de6488 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/AnalyzeRouteCollectionPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/AnalyzeRouteCollectionPassTest.php @@ -23,11 +23,6 @@ public function testShouldBeFinal() $this->assertClassFinal(AnalyzeRouteCollectionPass::class); } - public function testCouldBeConstructedWithoutArguments() - { - new AnalyzeRouteCollectionPass(); - } - public function testThrowIfEnqueueClientsParameterNotSet() { $pass = new AnalyzeRouteCollectionPass(); diff --git a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildClientExtensionsPassTest.php b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildClientExtensionsPassTest.php index 90c7493c1..753790369 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildClientExtensionsPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildClientExtensionsPassTest.php @@ -25,11 +25,6 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildClientExtensionsPass::class); } - public function testCouldBeConstructedWithoutArguments() - { - new BuildClientExtensionsPass(); - } - public function testThrowIfEnqueueClientsParameterNotSet() { $pass = new BuildClientExtensionsPass(); diff --git a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildCommandSubscriberRoutesPassTest.php b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildCommandSubscriberRoutesPassTest.php index a2e964dcc..1a5f65a4f 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildCommandSubscriberRoutesPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildCommandSubscriberRoutesPassTest.php @@ -29,11 +29,6 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildCommandSubscriberRoutesPass::class); } - public function testCouldBeConstructedWithoutArguments() - { - new BuildCommandSubscriberRoutesPass(); - } - public function testThrowIfEnqueueClientsParameterNotSet() { $pass = new BuildCommandSubscriberRoutesPass(); diff --git a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildConsumptionExtensionsPassTest.php b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildConsumptionExtensionsPassTest.php index a94636261..c2975051b 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildConsumptionExtensionsPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildConsumptionExtensionsPassTest.php @@ -25,11 +25,6 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildConsumptionExtensionsPass::class); } - public function testCouldBeConstructedWithoutArguments() - { - new BuildConsumptionExtensionsPass(); - } - public function testThrowIfEnqueueClientsParameterNotSet() { $pass = new BuildConsumptionExtensionsPass(); diff --git a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRegistryPassTest.php b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRegistryPassTest.php index 8541f39bf..5c9ac4840 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRegistryPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRegistryPassTest.php @@ -26,11 +26,6 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildProcessorRegistryPass::class); } - public function testCouldBeConstructedWithoutArguments() - { - new BuildProcessorRegistryPass(); - } - public function testThrowIfEnqueueClientsParameterNotSet() { $pass = new BuildProcessorRegistryPass(); diff --git a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRoutesPassTest.php b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRoutesPassTest.php index b2839c466..0351c45f5 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRoutesPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRoutesPassTest.php @@ -25,11 +25,6 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildProcessorRoutesPass::class); } - public function testCouldBeConstructedWithoutArguments() - { - new BuildProcessorRoutesPass(); - } - public function testThrowIfEnqueueClientsParameterNotSet() { $pass = new BuildProcessorRoutesPass(); diff --git a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildTopicSubscriberRoutesPassTest.php b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildTopicSubscriberRoutesPassTest.php index af1c6a37a..b642b9001 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildTopicSubscriberRoutesPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildTopicSubscriberRoutesPassTest.php @@ -29,11 +29,6 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildTopicSubscriberRoutesPass::class); } - public function testCouldBeConstructedWithoutArguments() - { - new BuildTopicSubscriberRoutesPass(); - } - public function testThrowIfEnqueueClientsParameterNotSet() { $pass = new BuildTopicSubscriberRoutesPass(); diff --git a/pkg/enqueue/Tests/Symfony/Client/FlushSpoolProducerListenerTest.php b/pkg/enqueue/Tests/Symfony/Client/FlushSpoolProducerListenerTest.php index 3476c101e..539d332ee 100644 --- a/pkg/enqueue/Tests/Symfony/Client/FlushSpoolProducerListenerTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/FlushSpoolProducerListenerTest.php @@ -39,11 +39,6 @@ public function testShouldSubscribeOnConsoleTerminateEvent() $this->assertEquals('flushMessages', $events[ConsoleEvents::TERMINATE]); } - public function testCouldBeConstructedWithSpoolProducerAsFirstArgument() - { - new FlushSpoolProducerListener($this->createSpoolProducerMock()); - } - public function testShouldFlushSpoolProducerOnFlushMessagesCall() { $producerMock = $this->createSpoolProducerMock(); diff --git a/pkg/enqueue/Tests/Symfony/Client/ProduceCommandTest.php b/pkg/enqueue/Tests/Symfony/Client/ProduceCommandTest.php index 2efc4769f..daa909175 100644 --- a/pkg/enqueue/Tests/Symfony/Client/ProduceCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/ProduceCommandTest.php @@ -27,11 +27,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(ProduceCommand::class); } - public function testCouldBeConstructedWithContainerAsFirstArgument() - { - new ProduceCommand($this->createMock(ContainerInterface::class), 'default'); - } - public function testShouldHaveAsCommandAttributeWithCommandName() { $commandClass = ProduceCommand::class; diff --git a/pkg/enqueue/Tests/Symfony/Client/RoutesCommandTest.php b/pkg/enqueue/Tests/Symfony/Client/RoutesCommandTest.php index c2168c4f8..89bd7f745 100644 --- a/pkg/enqueue/Tests/Symfony/Client/RoutesCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/RoutesCommandTest.php @@ -29,11 +29,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(RoutesCommand::class); } - public function testCouldBeConstructedWithConfigAndRouteCollectionAsArguments() - { - new RoutesCommand($this->createMock(ContainerInterface::class), 'default'); - } - public function testShouldHaveAsCommandAttributeWithCommandName() { $commandClass = RoutesCommand::class; diff --git a/pkg/enqueue/Tests/Symfony/Client/SetupBrokerCommandTest.php b/pkg/enqueue/Tests/Symfony/Client/SetupBrokerCommandTest.php index 89ca379b5..c81c4e1b6 100644 --- a/pkg/enqueue/Tests/Symfony/Client/SetupBrokerCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/SetupBrokerCommandTest.php @@ -26,11 +26,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(SetupBrokerCommand::class); } - public function testCouldBeConstructedWithContainerAsFirstArgument() - { - new SetupBrokerCommand($this->createMock(ContainerInterface::class), 'default'); - } - public function testShouldHaveAsCommandAttributeWithCommandName() { $commandClass = SetupBrokerCommand::class; diff --git a/pkg/enqueue/Tests/Symfony/Client/SimpleConsumeCommandTest.php b/pkg/enqueue/Tests/Symfony/Client/SimpleConsumeCommandTest.php index 60d409325..21c491eb5 100644 --- a/pkg/enqueue/Tests/Symfony/Client/SimpleConsumeCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/SimpleConsumeCommandTest.php @@ -29,11 +29,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(SimpleConsumeCommand::class); } - public function testCouldBeConstructedWithRequiredAttributes() - { - new SimpleConsumeCommand($this->createQueueConsumerMock(), $this->createDriverStub(), $this->createDelegateProcessorMock()); - } - public function testShouldHaveExpectedOptions() { $command = new SimpleConsumeCommand($this->createQueueConsumerMock(), $this->createDriverStub(), $this->createDelegateProcessorMock()); diff --git a/pkg/enqueue/Tests/Symfony/Client/SimpleProduceCommandTest.php b/pkg/enqueue/Tests/Symfony/Client/SimpleProduceCommandTest.php index 2775870b6..3ff81bfd5 100644 --- a/pkg/enqueue/Tests/Symfony/Client/SimpleProduceCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/SimpleProduceCommandTest.php @@ -23,11 +23,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(SimpleProduceCommand::class); } - public function testCouldBeConstructedWithContainerAsFirstArgument() - { - new SimpleProduceCommand($this->createProducerMock()); - } - public function testShouldHaveExpectedOptions() { $command = new SimpleProduceCommand($this->createProducerMock()); diff --git a/pkg/enqueue/Tests/Symfony/Client/SimpleRoutesCommandTest.php b/pkg/enqueue/Tests/Symfony/Client/SimpleRoutesCommandTest.php index 356f3f9f4..20ee454cc 100644 --- a/pkg/enqueue/Tests/Symfony/Client/SimpleRoutesCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/SimpleRoutesCommandTest.php @@ -25,11 +25,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(SimpleRoutesCommand::class); } - public function testCouldBeConstructedWithConfigAndRouteCollectionAsArguments() - { - new SimpleRoutesCommand($this->createDriverMock()); - } - public function testShouldHaveCommandAliases() { $command = new SimpleRoutesCommand($this->createDriverMock()); diff --git a/pkg/enqueue/Tests/Symfony/Client/SimpleSetupBrokerCommandTest.php b/pkg/enqueue/Tests/Symfony/Client/SimpleSetupBrokerCommandTest.php index 91802ff11..3702dbf18 100644 --- a/pkg/enqueue/Tests/Symfony/Client/SimpleSetupBrokerCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/SimpleSetupBrokerCommandTest.php @@ -23,11 +23,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(SimpleSetupBrokerCommand::class); } - public function testCouldBeConstructedWithContainerAsFirstArgument() - { - new SimpleSetupBrokerCommand($this->createClientDriverMock()); - } - public function testShouldHaveCommandAliases() { $command = new SimpleSetupBrokerCommand($this->createClientDriverMock()); diff --git a/pkg/enqueue/Tests/Symfony/Consumption/ConfigurableConsumeCommandTest.php b/pkg/enqueue/Tests/Symfony/Consumption/ConfigurableConsumeCommandTest.php index c448c1cf5..251e264e2 100644 --- a/pkg/enqueue/Tests/Symfony/Consumption/ConfigurableConsumeCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Consumption/ConfigurableConsumeCommandTest.php @@ -33,11 +33,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(ConfigurableConsumeCommand::class); } - public function testCouldBeConstructedWithRequiredAttributes() - { - new ConfigurableConsumeCommand($this->createMock(ContainerInterface::class), 'default'); - } - public function testShouldHaveAsCommandAttributeWithCommandName() { $commandClass = ConfigurableConsumeCommand::class; diff --git a/pkg/enqueue/Tests/Symfony/Consumption/ConsumeCommandTest.php b/pkg/enqueue/Tests/Symfony/Consumption/ConsumeCommandTest.php index 3c16cde3d..f07bef03b 100644 --- a/pkg/enqueue/Tests/Symfony/Consumption/ConsumeCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Consumption/ConsumeCommandTest.php @@ -35,11 +35,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(ConsumeCommand::class); } - public function testCouldBeConstructedWithRequiredAttributes() - { - new ConsumeCommand($this->createMock(ContainerInterface::class), 'default'); - } - public function testShouldHaveAsCommandAttributeWithCommandName() { $commandClass = ConsumeCommand::class; diff --git a/pkg/enqueue/Tests/Symfony/Consumption/SimpleConsumeCommandTest.php b/pkg/enqueue/Tests/Symfony/Consumption/SimpleConsumeCommandTest.php index d54a70e58..eeb38bf19 100644 --- a/pkg/enqueue/Tests/Symfony/Consumption/SimpleConsumeCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Consumption/SimpleConsumeCommandTest.php @@ -24,11 +24,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(SimpleConsumeCommand::class); } - public function testCouldBeConstructedWithRequiredAttributes() - { - new SimpleConsumeCommand($this->createQueueConsumerMock()); - } - public function testShouldHaveExpectedOptions() { $command = new SimpleConsumeCommand($this->createQueueConsumerMock()); diff --git a/pkg/enqueue/Tests/Symfony/ContainerProcessorRegistryTest.php b/pkg/enqueue/Tests/Symfony/ContainerProcessorRegistryTest.php index e0dcf2e09..5504e8ef6 100644 --- a/pkg/enqueue/Tests/Symfony/ContainerProcessorRegistryTest.php +++ b/pkg/enqueue/Tests/Symfony/ContainerProcessorRegistryTest.php @@ -23,11 +23,6 @@ public function testShouldBeFinal() $this->assertClassFinal(ContainerProcessorRegistry::class); } - public function testCouldBeConstructedWithContainerAsFirstArgument() - { - new ContainerProcessorRegistry($this->createContainerMock()); - } - public function testShouldAllowGetProcessor() { $processorMock = $this->createProcessorMock(); diff --git a/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildConsumptionExtensionsPassTest.php b/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildConsumptionExtensionsPassTest.php index f16daf08a..bdccd338c 100644 --- a/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildConsumptionExtensionsPassTest.php +++ b/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildConsumptionExtensionsPassTest.php @@ -25,11 +25,6 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildConsumptionExtensionsPass::class); } - public function testCouldBeConstructedWithoutArguments() - { - new BuildConsumptionExtensionsPass(); - } - public function testThrowIfEnqueueTransportsParameterNotSet() { $pass = new BuildConsumptionExtensionsPass(); diff --git a/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildProcessorRegistryPassTest.php b/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildProcessorRegistryPassTest.php index 8d8edc537..134c216dc 100644 --- a/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildProcessorRegistryPassTest.php +++ b/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildProcessorRegistryPassTest.php @@ -26,11 +26,6 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildProcessorRegistryPass::class); } - public function testCouldBeConstructedWithoutArguments() - { - new BuildProcessorRegistryPass(); - } - public function testThrowIfEnqueueTransportsParameterNotSet() { $pass = new BuildProcessorRegistryPass(); diff --git a/pkg/enqueue/Tests/Symfony/LazyProducerTest.php b/pkg/enqueue/Tests/Symfony/LazyProducerTest.php index 1b15ce2ce..c8ba596a8 100644 --- a/pkg/enqueue/Tests/Symfony/LazyProducerTest.php +++ b/pkg/enqueue/Tests/Symfony/LazyProducerTest.php @@ -20,11 +20,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(ProducerInterface::class, LazyProducer::class); } - public function testCouldBeConstructedWithContainerAndServiceId() - { - new LazyProducer($this->createContainerMock(), 'realProducerId'); - } - public function testShouldNotCallRealProducerInConstructor() { $containerMock = $this->createContainerMock(); diff --git a/pkg/enqueue/Tests/Util/VarExportTest.php b/pkg/enqueue/Tests/Util/VarExportTest.php index 1d2384ac9..e8034a1c7 100644 --- a/pkg/enqueue/Tests/Util/VarExportTest.php +++ b/pkg/enqueue/Tests/Util/VarExportTest.php @@ -7,11 +7,6 @@ class VarExportTest extends TestCase { - public function testCouldBeConstructedWithValueAsArgument() - { - new VarExport('aVal'); - } - /** * @dataProvider provideValues * diff --git a/pkg/fs/Tests/FsConsumerTest.php b/pkg/fs/Tests/FsConsumerTest.php index 086bb55fe..67f03ae98 100644 --- a/pkg/fs/Tests/FsConsumerTest.php +++ b/pkg/fs/Tests/FsConsumerTest.php @@ -20,11 +20,6 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, FsConsumer::class); } - public function testCouldBeConstructedWithContextAndDestinationAndPreFetchCountAsArguments() - { - new FsConsumer($this->createContextMock(), new FsDestination(TempFile::generate()), 1); - } - public function testShouldReturnDestinationSetInConstructorOnGetQueue() { $destination = new FsDestination(TempFile::generate()); @@ -50,6 +45,9 @@ public function testShouldAllowGetPreviouslySetPreFetchCount() $this->assertSame(456, $consumer->getPreFetchCount()); } + /** + * @doesNotPerformAssertions + */ public function testShouldDoNothingOnAcknowledge() { $consumer = new FsConsumer($this->createContextMock(), new FsDestination(TempFile::generate()), 123); @@ -57,6 +55,9 @@ public function testShouldDoNothingOnAcknowledge() $consumer->acknowledge(new FsMessage()); } + /** + * @doesNotPerformAssertions + */ public function testShouldDoNothingOnReject() { $consumer = new FsConsumer($this->createContextMock(), new FsDestination(TempFile::generate()), 123); diff --git a/pkg/fs/Tests/FsContextTest.php b/pkg/fs/Tests/FsContextTest.php index 4bc05e9de..9d5a5f1fc 100644 --- a/pkg/fs/Tests/FsContextTest.php +++ b/pkg/fs/Tests/FsContextTest.php @@ -24,11 +24,6 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, FsContext::class); } - public function testCouldBeConstructedWithExpectedArguments() - { - new FsContext(sys_get_temp_dir(), 1, 0666, 100); - } - public function testShouldAllowCreateEmptyMessage() { $context = new FsContext(sys_get_temp_dir(), 1, 0666, 100); @@ -122,6 +117,9 @@ public function testShouldThrowIfNotFsDestinationGivenOnCreateConsumer() $this->assertInstanceOf(FsConsumer::class, $consumer); } + /** + * @doesNotPerformAssertions + */ public function testShouldCreateConsumer() { $tmpFile = new TempFile(sys_get_temp_dir().'/foo'); diff --git a/pkg/fs/Tests/FsProducerTest.php b/pkg/fs/Tests/FsProducerTest.php index 28700d52f..266854c7b 100644 --- a/pkg/fs/Tests/FsProducerTest.php +++ b/pkg/fs/Tests/FsProducerTest.php @@ -23,11 +23,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, FsProducer::class); } - public function testCouldBeConstructedWithContextAsFirstArgument() - { - new FsProducer($this->createContextMock()); - } - public function testThrowIfDestinationNotFsOnSend() { $producer = new FsProducer($this->createContextMock()); diff --git a/pkg/gearman/Tests/GearmanContextTest.php b/pkg/gearman/Tests/GearmanContextTest.php index f2b70d11b..8a36ad80b 100644 --- a/pkg/gearman/Tests/GearmanContextTest.php +++ b/pkg/gearman/Tests/GearmanContextTest.php @@ -24,11 +24,6 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, GearmanContext::class); } - public function testCouldBeConstructedWithConnectionConfigAsFirstArgument() - { - new GearmanContext(['host' => 'aHost', 'port' => 'aPort']); - } - public function testThrowNotImplementedOnCreateTemporaryQueue() { $context = new GearmanContext(['host' => 'aHost', 'port' => 'aPort']); diff --git a/pkg/gearman/Tests/GearmanProducerTest.php b/pkg/gearman/Tests/GearmanProducerTest.php index afb6181a3..2a7baa4de 100644 --- a/pkg/gearman/Tests/GearmanProducerTest.php +++ b/pkg/gearman/Tests/GearmanProducerTest.php @@ -18,11 +18,6 @@ class GearmanProducerTest extends TestCase use ClassExtensionTrait; use SkipIfGearmanExtensionIsNotInstalledTrait; - public function testCouldBeConstructedWithGearmanClientAsFirstArgument() - { - new GearmanProducer($this->createGearmanClientMock()); - } - public function testThrowIfDestinationInvalid() { $producer = new GearmanProducer($this->createGearmanClientMock()); diff --git a/pkg/job-queue/Tests/CalculateRootJobStatusProcessorTest.php b/pkg/job-queue/Tests/CalculateRootJobStatusProcessorTest.php index 2e5a6ac4c..8509f0544 100644 --- a/pkg/job-queue/Tests/CalculateRootJobStatusProcessorTest.php +++ b/pkg/job-queue/Tests/CalculateRootJobStatusProcessorTest.php @@ -17,16 +17,6 @@ class CalculateRootJobStatusProcessorTest extends \PHPUnit\Framework\TestCase { - public function testCouldBeConstructedWithRequiredArguments() - { - new CalculateRootJobStatusProcessor( - $this->createJobStorageMock(), - $this->createCalculateRootJobStatusCaseMock(), - $this->createProducerMock(), - $this->createLoggerMock() - ); - } - public function testShouldReturnSubscribedTopicNames() { $this->assertEquals( diff --git a/pkg/job-queue/Tests/CalculateRootJobStatusServiceTest.php b/pkg/job-queue/Tests/CalculateRootJobStatusServiceTest.php index f289c314c..695c08e6e 100644 --- a/pkg/job-queue/Tests/CalculateRootJobStatusServiceTest.php +++ b/pkg/job-queue/Tests/CalculateRootJobStatusServiceTest.php @@ -9,11 +9,6 @@ class CalculateRootJobStatusServiceTest extends \PHPUnit\Framework\TestCase { - public function testCouldBeConstructedWithRequiredArguments() - { - new CalculateRootJobStatusService($this->createJobStorageMock()); - } - public function stopStatusProvider() { return [ diff --git a/pkg/job-queue/Tests/DependentJobContextTest.php b/pkg/job-queue/Tests/DependentJobContextTest.php index 32340d687..35942a974 100644 --- a/pkg/job-queue/Tests/DependentJobContextTest.php +++ b/pkg/job-queue/Tests/DependentJobContextTest.php @@ -7,11 +7,6 @@ class DependentJobContextTest extends \PHPUnit\Framework\TestCase { - public function testCouldBeConstructedWithRequiredArguments() - { - new DependentJobContext(new Job()); - } - public function testShouldReturnJob() { $job = new Job(); diff --git a/pkg/job-queue/Tests/DependentJobServiceTest.php b/pkg/job-queue/Tests/DependentJobServiceTest.php index 149856308..e8809b4b0 100644 --- a/pkg/job-queue/Tests/DependentJobServiceTest.php +++ b/pkg/job-queue/Tests/DependentJobServiceTest.php @@ -10,11 +10,6 @@ class DependentJobServiceTest extends \PHPUnit\Framework\TestCase { - public function testCouldBeConstructedWithRequiredArguments() - { - new DependentJobService($this->createJobStorageMock()); - } - public function testShouldThrowIfJobIsNotRootJob() { $job = new Job(); diff --git a/pkg/job-queue/Tests/Doctrine/JobStorageTest.php b/pkg/job-queue/Tests/Doctrine/JobStorageTest.php index 38d72a81c..73f130d52 100644 --- a/pkg/job-queue/Tests/Doctrine/JobStorageTest.php +++ b/pkg/job-queue/Tests/Doctrine/JobStorageTest.php @@ -15,11 +15,6 @@ class JobStorageTest extends \PHPUnit\Framework\TestCase { - public function testCouldBeConstructedWithRequiredArguments() - { - new JobStorage($this->createDoctrineMock(), 'entity-class', 'unique_table'); - } - public function testShouldCreateJobObject() { $repository = $this->createRepositoryMock(); diff --git a/pkg/job-queue/Tests/JobProcessorTest.php b/pkg/job-queue/Tests/JobProcessorTest.php index 3f06bf8de..9f1c7b2fd 100644 --- a/pkg/job-queue/Tests/JobProcessorTest.php +++ b/pkg/job-queue/Tests/JobProcessorTest.php @@ -13,11 +13,6 @@ class JobProcessorTest extends TestCase { - public function testCouldBeCreatedWithRequiredArguments() - { - new JobProcessor($this->createJobStorage(), $this->createProducerMock()); - } - public function testCreateRootJobShouldThrowIfOwnerIdIsEmpty() { $processor = new JobProcessor($this->createJobStorage(), $this->createProducerMock()); diff --git a/pkg/mongodb/Tests/MongodbConsumerTest.php b/pkg/mongodb/Tests/MongodbConsumerTest.php index cd2ee06f8..c6a642500 100644 --- a/pkg/mongodb/Tests/MongodbConsumerTest.php +++ b/pkg/mongodb/Tests/MongodbConsumerTest.php @@ -25,11 +25,6 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, MongodbConsumer::class); } - public function testCouldBeConstructedWithRequiredArguments() - { - new MongodbConsumer($this->createContextMock(), new MongodbDestination('queue')); - } - public function testShouldReturnInstanceOfDestination() { $destination = new MongodbDestination('queue'); @@ -39,6 +34,9 @@ public function testShouldReturnInstanceOfDestination() $this->assertSame($destination, $consumer->getQueue()); } + /** + * @doesNotPerformAssertions + */ public function testCouldCallAcknowledgedMethod() { $consumer = new MongodbConsumer($this->createContextMock(), new MongodbDestination('queue')); diff --git a/pkg/mongodb/Tests/MongodbContextTest.php b/pkg/mongodb/Tests/MongodbContextTest.php index 94da3ae6a..8cdef79ff 100644 --- a/pkg/mongodb/Tests/MongodbContextTest.php +++ b/pkg/mongodb/Tests/MongodbContextTest.php @@ -29,11 +29,6 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, MongodbContext::class); } - public function testCouldBeConstructedWithRequiredArguments() - { - new MongodbContext($this->createClientMock()); - } - public function testCouldBeConstructedWithEmptyConfiguration() { $context = new MongodbContext($this->createClientMock(), []); diff --git a/pkg/mongodb/Tests/MongodbProducerTest.php b/pkg/mongodb/Tests/MongodbProducerTest.php index 8e8900cfd..6987b1a76 100644 --- a/pkg/mongodb/Tests/MongodbProducerTest.php +++ b/pkg/mongodb/Tests/MongodbProducerTest.php @@ -23,11 +23,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, MongodbProducer::class); } - public function testCouldBeConstructedWithRequiredArguments() - { - new MongodbProducer($this->createContextMock()); - } - public function testShouldThrowIfDestinationOfInvalidType() { $this->expectException(InvalidDestinationException::class); diff --git a/pkg/mongodb/Tests/MongodbSubscriptionConsumerTest.php b/pkg/mongodb/Tests/MongodbSubscriptionConsumerTest.php index 0fc5afae4..d982e0418 100644 --- a/pkg/mongodb/Tests/MongodbSubscriptionConsumerTest.php +++ b/pkg/mongodb/Tests/MongodbSubscriptionConsumerTest.php @@ -24,11 +24,6 @@ public function testShouldImplementSubscriptionConsumerInterface() $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class)); } - public function testCouldBeConstructedWithMongodbContextAsFirstArgument() - { - new MongodbSubscriptionConsumer($this->createMongodbContextMock()); - } - public function testShouldAddConsumerAndCallbackToSubscribersPropertyOnSubscribe() { $subscriptionConsumer = new MongodbSubscriptionConsumer($this->createMongodbContextMock()); @@ -65,6 +60,9 @@ public function testThrowsIfTrySubscribeAnotherConsumerToAlreadySubscribedQueue( $subscriptionConsumer->subscribe($barConsumer, $barCallback); } + /** + * @doesNotPerformAssertions + */ public function testShouldAllowSubscribeSameConsumerAndCallbackSecondTime() { $subscriptionConsumer = new MongodbSubscriptionConsumer($this->createMongodbContextMock()); diff --git a/pkg/null/Tests/NullConnectionFactoryTest.php b/pkg/null/Tests/NullConnectionFactoryTest.php index 80cfcc904..bbf377e85 100644 --- a/pkg/null/Tests/NullConnectionFactoryTest.php +++ b/pkg/null/Tests/NullConnectionFactoryTest.php @@ -17,11 +17,6 @@ public function testShouldImplementConnectionFactoryInterface() $this->assertClassImplements(ConnectionFactory::class, NullConnectionFactory::class); } - public function testCouldBeConstructedWithoutAnyArguments() - { - new NullConnectionFactory(); - } - public function testShouldReturnNullContextOnCreateContextCall() { $factory = new NullConnectionFactory(); diff --git a/pkg/null/Tests/NullConsumerTest.php b/pkg/null/Tests/NullConsumerTest.php index 6419e9c39..f4de53311 100644 --- a/pkg/null/Tests/NullConsumerTest.php +++ b/pkg/null/Tests/NullConsumerTest.php @@ -18,11 +18,6 @@ public function testShouldImplementMessageConsumerInterface() $this->assertClassImplements(Consumer::class, NullConsumer::class); } - public function testCouldBeConstructedWithQueueAsArgument() - { - new NullConsumer(new NullQueue('aName')); - } - public function testShouldAlwaysReturnNullOnReceive() { $consumer = new NullConsumer(new NullQueue('theQueueName')); @@ -41,6 +36,9 @@ public function testShouldAlwaysReturnNullOnReceiveNoWait() $this->assertNull($consumer->receiveNoWait()); } + /** + * @doesNotPerformAssertions + */ public function testShouldDoNothingOnAcknowledge() { $consumer = new NullConsumer(new NullQueue('theQueueName')); @@ -48,6 +46,9 @@ public function testShouldDoNothingOnAcknowledge() $consumer->acknowledge(new NullMessage()); } + /** + * @doesNotPerformAssertions + */ public function testShouldDoNothingOnReject() { $consumer = new NullConsumer(new NullQueue('theQueueName')); diff --git a/pkg/null/Tests/NullContextTest.php b/pkg/null/Tests/NullContextTest.php index 4e5150990..f0da566d2 100644 --- a/pkg/null/Tests/NullContextTest.php +++ b/pkg/null/Tests/NullContextTest.php @@ -21,11 +21,6 @@ public function testShouldImplementSessionInterface() $this->assertClassImplements(Context::class, NullContext::class); } - public function testCouldBeConstructedWithoutAnyArguments() - { - new NullContext(); - } - public function testShouldAllowCreateMessageWithoutAnyArguments() { $context = new NullContext(); diff --git a/pkg/null/Tests/NullProducerTest.php b/pkg/null/Tests/NullProducerTest.php index 289dab6c0..140d683ba 100644 --- a/pkg/null/Tests/NullProducerTest.php +++ b/pkg/null/Tests/NullProducerTest.php @@ -18,11 +18,9 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, NullProducer::class); } - public function testCouldBeConstructedWithoutAnyArguments() - { - new NullProducer(); - } - + /** + * @doesNotPerformAssertions + */ public function testShouldDoNothingOnSend() { $producer = new NullProducer(); diff --git a/pkg/null/Tests/NullQueueTest.php b/pkg/null/Tests/NullQueueTest.php index 0dfaf2f28..cb29ca180 100644 --- a/pkg/null/Tests/NullQueueTest.php +++ b/pkg/null/Tests/NullQueueTest.php @@ -16,11 +16,6 @@ public function testShouldImplementQueueInterface() $this->assertClassImplements(Queue::class, NullQueue::class); } - public function testCouldBeConstructedWithNameAsArgument() - { - new NullQueue('aName'); - } - public function testShouldAllowGetNameSetInConstructor() { $queue = new NullQueue('theName'); diff --git a/pkg/null/Tests/NullTopicTest.php b/pkg/null/Tests/NullTopicTest.php index a40545120..27c4b58de 100644 --- a/pkg/null/Tests/NullTopicTest.php +++ b/pkg/null/Tests/NullTopicTest.php @@ -16,11 +16,6 @@ public function testShouldImplementTopicInterface() $this->assertClassImplements(Topic::class, NullTopic::class); } - public function testCouldBeConstructedWithNameAsArgument() - { - new NullTopic('aName'); - } - public function testShouldAllowGetNameSetInConstructor() { $topic = new NullTopic('theName'); diff --git a/pkg/pheanstalk/Tests/PheanstalkConsumerTest.php b/pkg/pheanstalk/Tests/PheanstalkConsumerTest.php index c0138c726..944ac9b81 100644 --- a/pkg/pheanstalk/Tests/PheanstalkConsumerTest.php +++ b/pkg/pheanstalk/Tests/PheanstalkConsumerTest.php @@ -14,14 +14,6 @@ class PheanstalkConsumerTest extends TestCase { use ClassExtensionTrait; - public function testCouldBeConstructedWithDestinationAndPheanstalkAsArguments() - { - new PheanstalkConsumer( - new PheanstalkDestination('aQueueName'), - $this->createPheanstalkMock() - ); - } - public function testShouldReturnQueueSetInConstructor() { $destination = new PheanstalkDestination('aQueueName'); diff --git a/pkg/pheanstalk/Tests/PheanstalkContextTest.php b/pkg/pheanstalk/Tests/PheanstalkContextTest.php index 9f5ac5812..3b7bfbeb7 100644 --- a/pkg/pheanstalk/Tests/PheanstalkContextTest.php +++ b/pkg/pheanstalk/Tests/PheanstalkContextTest.php @@ -21,11 +21,6 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, PheanstalkContext::class); } - public function testCouldBeConstructedWithPheanstalkAsFirstArgument() - { - new PheanstalkContext($this->createPheanstalkMock()); - } - public function testThrowNotImplementedOnCreateTemporaryQueue() { $context = new PheanstalkContext($this->createPheanstalkMock()); diff --git a/pkg/pheanstalk/Tests/PheanstalkProducerTest.php b/pkg/pheanstalk/Tests/PheanstalkProducerTest.php index 7a736c09e..b9a69176c 100644 --- a/pkg/pheanstalk/Tests/PheanstalkProducerTest.php +++ b/pkg/pheanstalk/Tests/PheanstalkProducerTest.php @@ -18,11 +18,6 @@ class PheanstalkProducerTest extends TestCase { use ClassExtensionTrait; - public function testCouldBeConstructedWithPheanstalkAsFirstArgument() - { - new PheanstalkProducer($this->createPheanstalkMock()); - } - public function testThrowIfDestinationInvalid() { $producer = new PheanstalkProducer($this->createPheanstalkMock()); diff --git a/pkg/rdkafka/Tests/JsonSerializerTest.php b/pkg/rdkafka/Tests/JsonSerializerTest.php index d10479def..6c9bbef84 100644 --- a/pkg/rdkafka/Tests/JsonSerializerTest.php +++ b/pkg/rdkafka/Tests/JsonSerializerTest.php @@ -17,11 +17,6 @@ public function testShouldImplementSerializerInterface() $this->assertClassImplements(Serializer::class, JsonSerializer::class); } - public function testCouldBeConstructedWithoutAnyArguments() - { - new JsonSerializer(); - } - public function testShouldConvertMessageToJsonString() { $serializer = new JsonSerializer(); diff --git a/pkg/rdkafka/Tests/RdKafkaConsumerTest.php b/pkg/rdkafka/Tests/RdKafkaConsumerTest.php index 4f2b806c6..74e92954b 100644 --- a/pkg/rdkafka/Tests/RdKafkaConsumerTest.php +++ b/pkg/rdkafka/Tests/RdKafkaConsumerTest.php @@ -13,16 +13,6 @@ class RdKafkaConsumerTest extends TestCase { - public function testCouldBeConstructedWithRequiredArguments() - { - new RdKafkaConsumer( - $this->createKafkaConsumerMock(), - $this->createContextMock(), - new RdKafkaTopic(''), - $this->createSerializerMock() - ); - } - public function testShouldReturnQueueSetInConstructor() { $destination = new RdKafkaTopic(''); diff --git a/pkg/rdkafka/Tests/RdKafkaProducerTest.php b/pkg/rdkafka/Tests/RdKafkaProducerTest.php index ecfa4c6fc..6082968ad 100644 --- a/pkg/rdkafka/Tests/RdKafkaProducerTest.php +++ b/pkg/rdkafka/Tests/RdKafkaProducerTest.php @@ -17,11 +17,6 @@ class RdKafkaProducerTest extends TestCase { - public function testCouldBeConstructedWithKafkaProducerAndSerializerAsArguments() - { - new RdKafkaProducer($this->createKafkaProducerMock(), $this->createSerializerMock()); - } - public function testThrowIfDestinationInvalid() { $producer = new RdKafkaProducer($this->createKafkaProducerMock(), $this->createSerializerMock()); diff --git a/pkg/redis/Tests/RedisConsumerTest.php b/pkg/redis/Tests/RedisConsumerTest.php index 2908134fa..56373c18a 100644 --- a/pkg/redis/Tests/RedisConsumerTest.php +++ b/pkg/redis/Tests/RedisConsumerTest.php @@ -22,11 +22,6 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, RedisConsumer::class); } - public function testCouldBeConstructedWithContextAndDestinationAndPreFetchCountAsArguments() - { - new RedisConsumer($this->createContextMock(), new RedisDestination('aQueue')); - } - public function testShouldReturnDestinationSetInConstructorOnGetQueue() { $destination = new RedisDestination('aQueue'); diff --git a/pkg/redis/Tests/RedisContextTest.php b/pkg/redis/Tests/RedisContextTest.php index 0115f27b8..6395e954e 100644 --- a/pkg/redis/Tests/RedisContextTest.php +++ b/pkg/redis/Tests/RedisContextTest.php @@ -25,18 +25,6 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, RedisContext::class); } - public function testCouldBeConstructedWithRedisAsFirstArgument() - { - new RedisContext($this->createRedisMock(), 300); - } - - public function testCouldBeConstructedWithRedisFactoryAsFirstArgument() - { - new RedisContext(function () { - return $this->createRedisMock(); - }, 300); - } - public function testThrowIfNeitherRedisNorFactoryGiven() { $this->expectException(\InvalidArgumentException::class); diff --git a/pkg/redis/Tests/RedisProducerTest.php b/pkg/redis/Tests/RedisProducerTest.php index 53d5f19e3..40e03bae2 100644 --- a/pkg/redis/Tests/RedisProducerTest.php +++ b/pkg/redis/Tests/RedisProducerTest.php @@ -25,11 +25,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, RedisProducer::class); } - public function testCouldBeConstructedWithRedisAsFirstArgument() - { - new RedisProducer($this->createContextMock()); - } - public function testThrowIfDestinationNotRedisDestinationOnSend() { $producer = new RedisProducer($this->createContextMock()); diff --git a/pkg/redis/Tests/RedisSubscriptionConsumerTest.php b/pkg/redis/Tests/RedisSubscriptionConsumerTest.php index b3ba4d26b..8d00fcc14 100644 --- a/pkg/redis/Tests/RedisSubscriptionConsumerTest.php +++ b/pkg/redis/Tests/RedisSubscriptionConsumerTest.php @@ -22,11 +22,6 @@ public function testShouldImplementSubscriptionConsumerInterface() $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class)); } - public function testCouldBeConstructedWithRedisContextAsFirstArgument() - { - new RedisSubscriptionConsumer($this->createRedisContextMock()); - } - public function testShouldAddConsumerAndCallbackToSubscribersPropertyOnSubscribe() { $subscriptionConsumer = new RedisSubscriptionConsumer($this->createRedisContextMock()); @@ -63,6 +58,9 @@ public function testThrowsIfTrySubscribeAnotherConsumerToAlreadySubscribedQueue( $subscriptionConsumer->subscribe($barConsumer, $barCallback); } + /** + * @doesNotPerformAssertions + */ public function testShouldAllowSubscribeSameConsumerAndCallbackSecondTime() { $subscriptionConsumer = new RedisSubscriptionConsumer($this->createRedisContextMock()); diff --git a/pkg/redis/Tests/Spec/JsonSerializerTest.php b/pkg/redis/Tests/Spec/JsonSerializerTest.php index e374ea170..78a0aed86 100644 --- a/pkg/redis/Tests/Spec/JsonSerializerTest.php +++ b/pkg/redis/Tests/Spec/JsonSerializerTest.php @@ -20,11 +20,6 @@ public function testShouldImplementSerializerInterface() $this->assertClassImplements(Serializer::class, JsonSerializer::class); } - public function testCouldBeConstructedWithoutAnyArguments() - { - new JsonSerializer(); - } - public function testShouldConvertMessageToJsonString() { $serializer = new JsonSerializer(); diff --git a/pkg/sns/Tests/SnsProducerTest.php b/pkg/sns/Tests/SnsProducerTest.php index 5f8784905..1c6be7f85 100644 --- a/pkg/sns/Tests/SnsProducerTest.php +++ b/pkg/sns/Tests/SnsProducerTest.php @@ -27,11 +27,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, SnsProducer::class); } - public function testCouldBeConstructedWithRequiredArguments() - { - new SnsProducer($this->createSnsContextMock()); - } - public function testShouldThrowIfBodyOfInvalidType() { $this->expectException(InvalidMessageException::class); diff --git a/pkg/snsqs/Tests/SnsQsProducerTest.php b/pkg/snsqs/Tests/SnsQsProducerTest.php index aa3e28971..59798dc11 100644 --- a/pkg/snsqs/Tests/SnsQsProducerTest.php +++ b/pkg/snsqs/Tests/SnsQsProducerTest.php @@ -32,11 +32,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, SnsQsProducer::class); } - public function testCouldBeConstructedWithRequiredArguments() - { - new SnsQsProducer($this->createSnsContextMock(), $this->createSqsContextMock()); - } - public function testShouldThrowIfMessageIsInvalidType() { $this->expectException(InvalidMessageException::class); diff --git a/pkg/sqs/Tests/SqsConsumerTest.php b/pkg/sqs/Tests/SqsConsumerTest.php index 05fbb9b69..ef06c6157 100644 --- a/pkg/sqs/Tests/SqsConsumerTest.php +++ b/pkg/sqs/Tests/SqsConsumerTest.php @@ -24,11 +24,6 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, SqsConsumer::class); } - public function testCouldBeConstructedWithRequiredArguments() - { - new SqsConsumer($this->createContextMock(), new SqsDestination('queue')); - } - public function testShouldReturnInstanceOfDestination() { $destination = new SqsDestination('queue'); diff --git a/pkg/sqs/Tests/SqsContextTest.php b/pkg/sqs/Tests/SqsContextTest.php index 75970bbad..5081add41 100644 --- a/pkg/sqs/Tests/SqsContextTest.php +++ b/pkg/sqs/Tests/SqsContextTest.php @@ -25,11 +25,6 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, SqsContext::class); } - public function testCouldBeConstructedWithSqsClientAsFirstArgument() - { - new SqsContext($this->createSqsClientMock(), []); - } - public function testShouldAllowCreateEmptyMessage() { $context = new SqsContext($this->createSqsClientMock(), []); diff --git a/pkg/sqs/Tests/SqsProducerTest.php b/pkg/sqs/Tests/SqsProducerTest.php index 1bda62c48..35cb9850b 100644 --- a/pkg/sqs/Tests/SqsProducerTest.php +++ b/pkg/sqs/Tests/SqsProducerTest.php @@ -24,11 +24,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, SqsProducer::class); } - public function testCouldBeConstructedWithRequiredArguments() - { - new SqsProducer($this->createSqsContextMock()); - } - public function testShouldThrowIfBodyOfInvalidType() { $this->expectException(InvalidMessageException::class); diff --git a/pkg/stomp/Tests/BufferedStompClientTest.php b/pkg/stomp/Tests/BufferedStompClientTest.php index 684edf397..e4b6226e1 100644 --- a/pkg/stomp/Tests/BufferedStompClientTest.php +++ b/pkg/stomp/Tests/BufferedStompClientTest.php @@ -21,11 +21,6 @@ public function testShouldExtendLibStompClient() $this->assertClassExtends(Client::class, BufferedStompClient::class); } - public function testCouldBeConstructedWithRequiredArguments() - { - new BufferedStompClient('tcp://localhost:12345'); - } - public function testCouldGetBufferSizeValue() { $client = new BufferedStompClient('tcp://localhost:12345', 12345); diff --git a/pkg/stomp/Tests/StompConsumerTest.php b/pkg/stomp/Tests/StompConsumerTest.php index 916166657..d461284c9 100644 --- a/pkg/stomp/Tests/StompConsumerTest.php +++ b/pkg/stomp/Tests/StompConsumerTest.php @@ -25,11 +25,6 @@ public function testShouldImplementMessageConsumerInterface() $this->assertClassImplements(Consumer::class, StompConsumer::class); } - public function testCouldBeConstructedWithRequiredAttributes() - { - new StompConsumer($this->createStompClientMock(), $this->createDummyDestination()); - } - public function testCouldGetQueue() { $consumer = new StompConsumer($this->createStompClientMock(), $dest = $this->createDummyDestination()); diff --git a/pkg/stomp/Tests/StompContextTest.php b/pkg/stomp/Tests/StompContextTest.php index b0a41b64d..cfb9245dc 100644 --- a/pkg/stomp/Tests/StompContextTest.php +++ b/pkg/stomp/Tests/StompContextTest.php @@ -23,18 +23,6 @@ public function testShouldImplementSessionInterface() $this->assertClassImplements(Context::class, StompContext::class); } - public function testCouldBeCreatedWithRequiredArguments() - { - new StompContext($this->createStompClientMock(), ExtensionType::RABBITMQ); - } - - public function testCouldBeConstructedWithExtChannelCallbackFactoryAsFirstArgument() - { - new StompContext(function () { - return $this->createStompClientMock(); - }, ExtensionType::RABBITMQ); - } - public function testThrowIfNeitherCallbackNorExtChannelAsFirstArgument() { $this->expectException(\InvalidArgumentException::class); diff --git a/pkg/wamp/Tests/Spec/JsonSerializerTest.php b/pkg/wamp/Tests/Spec/JsonSerializerTest.php index eaa3c6bc2..f1c3e24d3 100644 --- a/pkg/wamp/Tests/Spec/JsonSerializerTest.php +++ b/pkg/wamp/Tests/Spec/JsonSerializerTest.php @@ -20,11 +20,6 @@ public function testShouldImplementSerializerInterface() $this->assertClassImplements(Serializer::class, JsonSerializer::class); } - public function testCouldBeConstructedWithoutAnyArguments() - { - new JsonSerializer(); - } - public function testShouldConvertMessageToJsonString() { $serializer = new JsonSerializer();