From a2177a568596fe17fe464aa370c7cb75f969b08d Mon Sep 17 00:00:00 2001 From: James Read Date: Sat, 11 Jan 2025 20:47:49 +0000 Subject: [PATCH 1/2] Fixing risky tests Makring tests that have no assertions with the annotation to suppress phpunits warning around risky tests closes 1370 --- pkg/amqp-bunny/Tests/AmqpConsumerTest.php | 3 +++ pkg/amqp-bunny/Tests/AmqpProducerTest.php | 4 ++++ pkg/amqp-bunny/Tests/AmqpSubscriptionConsumerTest.php | 3 +++ pkg/amqp-ext/Tests/AmqpConsumerTest.php | 3 +++ pkg/amqp-ext/Tests/AmqpContextTest.php | 6 ++++++ pkg/amqp-ext/Tests/AmqpSubscriptionConsumerTest.php | 3 +++ pkg/async-event-dispatcher/Tests/AsyncProcessorTest.php | 3 +++ .../Tests/PhpSerializerEventTransformerTest.php | 3 +++ pkg/async-event-dispatcher/Tests/SimpleRegistryTest.php | 3 +++ pkg/dbal/Tests/DbalConsumerTest.php | 3 +++ pkg/dbal/Tests/DbalContextTest.php | 3 +++ pkg/dbal/Tests/DbalProducerTest.php | 3 +++ pkg/dbal/Tests/DbalSubscriptionConsumerTest.php | 6 ++++++ pkg/dsn/Tests/DsnTest.php | 3 +++ .../Extension/DoctrineClearIdentityMapExtensionTest.php | 3 +++ .../Extension/DoctrinePingConnectionExtensionTest.php | 3 +++ .../Consumption/Extension/ResetServicesExtensionTest.php | 3 +++ .../Tests/Unit/DependencyInjection/ConfigurationTest.php | 3 +++ .../Unit/DependencyInjection/EnqueueExtensionTest.php | 3 +++ pkg/enqueue-bundle/Tests/Unit/EnqueueBundleTest.php | 3 +++ .../Tests/Unit/Profiler/MessageQueueCollectorTest.php | 3 +++ pkg/enqueue/Tests/ArrayProcessorRegistryTest.php | 3 +++ pkg/enqueue/Tests/Client/ChainExtensionTest.php | 3 +++ .../DelayRedeliveredMessageExtensionTest.php | 3 +++ .../ExclusiveCommandExtensionTest.php | 3 +++ .../FlushSpoolProducerExtensionTest.php | 3 +++ .../Client/ConsumptionExtension/LogExtensionTest.php | 3 +++ .../SetRouterPropertiesExtensionTest.php | 3 +++ .../ConsumptionExtension/SetupBrokerExtensionTest.php | 3 +++ pkg/enqueue/Tests/Client/DelegateProcessorTest.php | 3 +++ pkg/enqueue/Tests/Client/DriverFactoryTest.php | 3 +++ pkg/enqueue/Tests/Client/DriverPreSendTest.php | 3 +++ .../Tests/Client/Extension/PrepareBodyExtensionTest.php | 3 +++ pkg/enqueue/Tests/Client/PostSendTest.php | 3 +++ pkg/enqueue/Tests/Client/PreSendTest.php | 3 +++ pkg/enqueue/Tests/Client/ProducerTest.php | 6 ++++++ pkg/enqueue/Tests/Client/SpoolProducerTest.php | 3 +++ pkg/enqueue/Tests/Client/TraceableProducerTest.php | 3 +++ pkg/enqueue/Tests/ConnectionFactoryFactoryTest.php | 9 +++++++++ pkg/enqueue/Tests/Consumption/CallbackProcessorTest.php | 3 +++ pkg/enqueue/Tests/Consumption/ChainExtensionTest.php | 3 +++ .../IllegalContextModificationExceptionTest.php | 3 +++ .../Exception/InvalidArgumentExceptionTest.php | 6 ++++++ .../Tests/Consumption/Exception/LogicExceptionTest.php | 3 +++ .../Extension/LimitConsumedMessagesExtensionTest.php | 3 +++ .../Extension/LimitConsumerMemoryExtensionTest.php | 3 +++ .../Extension/LimitConsumptionTimeExtensionTest.php | 3 +++ .../Tests/Consumption/Extension/LogExtensionTest.php | 3 +++ .../Tests/Consumption/Extension/LoggerExtensionTest.php | 3 +++ .../Consumption/Extension/NicenessExtensionTest.php | 3 +++ .../Tests/Consumption/Extension/ReplyExtensionTest.php | 3 +++ .../Consumption/FallbackSubscriptionConsumerTest.php | 6 ++++++ pkg/enqueue/Tests/Consumption/QueueConsumerTest.php | 9 +++++++++ .../Tests/Router/RouteRecipientListProcessorTest.php | 3 +++ pkg/enqueue/Tests/Rpc/RpcClientTest.php | 3 +++ pkg/enqueue/Tests/Symfony/Client/ConsumeCommandTest.php | 3 +++ .../AnalyzeRouteCollectionPassTest.php | 3 +++ .../BuildClientExtensionsPassTest.php | 3 +++ .../BuildCommandSubscriberRoutesPassTest.php | 3 +++ .../BuildConsumptionExtensionsPassTest.php | 3 +++ .../BuildProcessorRegistryPassTest.php | 3 +++ .../DependencyInjection/BuildProcessorRoutesPassTest.php | 3 +++ .../BuildTopicSubscriberRoutesPassTest.php | 3 +++ .../Symfony/Client/FlushSpoolProducerListenerTest.php | 3 +++ pkg/enqueue/Tests/Symfony/Client/ProduceCommandTest.php | 3 +++ pkg/enqueue/Tests/Symfony/Client/RoutesCommandTest.php | 3 +++ .../Tests/Symfony/Client/SetupBrokerCommandTest.php | 3 +++ .../Tests/Symfony/Client/SimpleConsumeCommandTest.php | 3 +++ .../Tests/Symfony/Client/SimpleProduceCommandTest.php | 3 +++ .../Tests/Symfony/Client/SimpleRoutesCommandTest.php | 3 +++ .../Symfony/Client/SimpleSetupBrokerCommandTest.php | 3 +++ .../Consumption/ConfigurableConsumeCommandTest.php | 3 +++ .../Tests/Symfony/Consumption/ConsumeCommandTest.php | 3 +++ .../Symfony/Consumption/SimpleConsumeCommandTest.php | 3 +++ .../Tests/Symfony/ContainerProcessorRegistryTest.php | 3 +++ .../BuildConsumptionExtensionsPassTest.php | 3 +++ .../BuildProcessorRegistryPassTest.php | 3 +++ pkg/enqueue/Tests/Symfony/LazyProducerTest.php | 3 +++ pkg/enqueue/Tests/Util/VarExportTest.php | 3 +++ pkg/fs/Tests/FsConsumerTest.php | 9 +++++++++ pkg/fs/Tests/FsContextTest.php | 6 ++++++ pkg/fs/Tests/FsProducerTest.php | 3 +++ pkg/gearman/Tests/GearmanContextTest.php | 3 +++ pkg/gearman/Tests/GearmanProducerTest.php | 3 +++ .../Tests/CalculateRootJobStatusProcessorTest.php | 3 +++ .../Tests/CalculateRootJobStatusServiceTest.php | 3 +++ pkg/job-queue/Tests/DependentJobContextTest.php | 3 +++ pkg/job-queue/Tests/DependentJobServiceTest.php | 3 +++ pkg/job-queue/Tests/Doctrine/JobStorageTest.php | 3 +++ pkg/job-queue/Tests/JobProcessorTest.php | 3 +++ pkg/mongodb/Tests/MongodbConsumerTest.php | 6 ++++++ pkg/mongodb/Tests/MongodbContextTest.php | 3 +++ pkg/mongodb/Tests/MongodbProducerTest.php | 3 +++ pkg/mongodb/Tests/MongodbSubscriptionConsumerTest.php | 6 ++++++ pkg/null/Tests/NullConnectionFactoryTest.php | 3 +++ pkg/null/Tests/NullConsumerTest.php | 9 +++++++++ pkg/null/Tests/NullContextTest.php | 3 +++ pkg/null/Tests/NullProducerTest.php | 6 ++++++ pkg/null/Tests/NullQueueTest.php | 3 +++ pkg/null/Tests/NullTopicTest.php | 3 +++ pkg/pheanstalk/Tests/PheanstalkConsumerTest.php | 3 +++ pkg/pheanstalk/Tests/PheanstalkContextTest.php | 3 +++ pkg/pheanstalk/Tests/PheanstalkProducerTest.php | 3 +++ pkg/rdkafka/Tests/JsonSerializerTest.php | 3 +++ pkg/rdkafka/Tests/RdKafkaConsumerTest.php | 3 +++ pkg/rdkafka/Tests/RdKafkaProducerTest.php | 3 +++ pkg/redis/Tests/RedisConsumerTest.php | 3 +++ pkg/redis/Tests/RedisContextTest.php | 6 ++++++ pkg/redis/Tests/RedisProducerTest.php | 3 +++ pkg/redis/Tests/RedisSubscriptionConsumerTest.php | 6 ++++++ pkg/redis/Tests/Spec/JsonSerializerTest.php | 3 +++ pkg/sns/Tests/SnsProducerTest.php | 3 +++ pkg/snsqs/Tests/SnsQsProducerTest.php | 3 +++ pkg/sqs/Tests/SqsConsumerTest.php | 3 +++ pkg/sqs/Tests/SqsContextTest.php | 3 +++ pkg/sqs/Tests/SqsProducerTest.php | 3 +++ pkg/stomp/Tests/BufferedStompClientTest.php | 3 +++ pkg/stomp/Tests/StompConsumerTest.php | 3 +++ pkg/stomp/Tests/StompContextTest.php | 6 ++++++ pkg/wamp/Tests/Spec/JsonSerializerTest.php | 3 +++ 120 files changed, 421 insertions(+) diff --git a/pkg/amqp-bunny/Tests/AmqpConsumerTest.php b/pkg/amqp-bunny/Tests/AmqpConsumerTest.php index a588c2e56..da6c40652 100644 --- a/pkg/amqp-bunny/Tests/AmqpConsumerTest.php +++ b/pkg/amqp-bunny/Tests/AmqpConsumerTest.php @@ -27,6 +27,9 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, AmqpConsumer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithContextAndQueueAsArguments() { new AmqpConsumer($this->createContextMock(), new AmqpQueue('aName')); diff --git a/pkg/amqp-bunny/Tests/AmqpProducerTest.php b/pkg/amqp-bunny/Tests/AmqpProducerTest.php index e9244297c..a7ca055fd 100644 --- a/pkg/amqp-bunny/Tests/AmqpProducerTest.php +++ b/pkg/amqp-bunny/Tests/AmqpProducerTest.php @@ -24,6 +24,10 @@ class AmqpProducerTest extends TestCase { use ClassExtensionTrait; + /** + * @doesNotPerformAssertions + */ + public function testCouldBeConstructedWithRequiredArguments() { new AmqpProducer($this->createBunnyChannelMock(), $this->createContextMock()); diff --git a/pkg/amqp-bunny/Tests/AmqpSubscriptionConsumerTest.php b/pkg/amqp-bunny/Tests/AmqpSubscriptionConsumerTest.php index c1d056268..cc2ac9a54 100644 --- a/pkg/amqp-bunny/Tests/AmqpSubscriptionConsumerTest.php +++ b/pkg/amqp-bunny/Tests/AmqpSubscriptionConsumerTest.php @@ -17,6 +17,9 @@ public function testShouldImplementQueueInteropSubscriptionConsumerInterface() $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class)); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithAmqpContextAsFirstArgument() { new AmqpSubscriptionConsumer($this->createAmqpContextMock()); diff --git a/pkg/amqp-ext/Tests/AmqpConsumerTest.php b/pkg/amqp-ext/Tests/AmqpConsumerTest.php index 75c49cf01..470c49c86 100644 --- a/pkg/amqp-ext/Tests/AmqpConsumerTest.php +++ b/pkg/amqp-ext/Tests/AmqpConsumerTest.php @@ -19,6 +19,9 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, AmqpConsumer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithContextAndQueueAsArguments() { new AmqpConsumer($this->createContext(), new AmqpQueue('aName')); diff --git a/pkg/amqp-ext/Tests/AmqpContextTest.php b/pkg/amqp-ext/Tests/AmqpContextTest.php index 4c4ab4298..4b42827ba 100644 --- a/pkg/amqp-ext/Tests/AmqpContextTest.php +++ b/pkg/amqp-ext/Tests/AmqpContextTest.php @@ -28,11 +28,17 @@ public function testShouldImplementQueueInteropContextInterface() $this->assertClassImplements(Context::class, AmqpContext::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithExtChannelAsFirstArgument() { new AmqpContext($this->createExtChannelMock()); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithExtChannelCallbackFactoryAsFirstArgument() { new AmqpContext(function () { diff --git a/pkg/amqp-ext/Tests/AmqpSubscriptionConsumerTest.php b/pkg/amqp-ext/Tests/AmqpSubscriptionConsumerTest.php index 153fe275e..8ba08161b 100644 --- a/pkg/amqp-ext/Tests/AmqpSubscriptionConsumerTest.php +++ b/pkg/amqp-ext/Tests/AmqpSubscriptionConsumerTest.php @@ -17,6 +17,9 @@ public function testShouldImplementQueueInteropSubscriptionConsumerInterface() $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class)); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithAmqpContextAsFirstArgument() { new AmqpSubscriptionConsumer($this->createAmqpContextMock()); diff --git a/pkg/async-event-dispatcher/Tests/AsyncProcessorTest.php b/pkg/async-event-dispatcher/Tests/AsyncProcessorTest.php index 7237c9ae2..9bf294194 100644 --- a/pkg/async-event-dispatcher/Tests/AsyncProcessorTest.php +++ b/pkg/async-event-dispatcher/Tests/AsyncProcessorTest.php @@ -24,6 +24,9 @@ public function testShouldImplementProcessorInterface() $this->assertClassImplements(Processor::class, AsyncProcessor::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRegistryAndProxyEventDispatcher() { 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..04d9f2dc6 100644 --- a/pkg/async-event-dispatcher/Tests/PhpSerializerEventTransformerTest.php +++ b/pkg/async-event-dispatcher/Tests/PhpSerializerEventTransformerTest.php @@ -21,6 +21,9 @@ public function testShouldImplementEventTransformerInterface() $this->assertClassImplements(EventTransformer::class, PhpSerializerEventTransformer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutAnyArguments() { new PhpSerializerEventTransformer($this->createContextStub()); diff --git a/pkg/async-event-dispatcher/Tests/SimpleRegistryTest.php b/pkg/async-event-dispatcher/Tests/SimpleRegistryTest.php index 328ed1780..52d6f1694 100644 --- a/pkg/async-event-dispatcher/Tests/SimpleRegistryTest.php +++ b/pkg/async-event-dispatcher/Tests/SimpleRegistryTest.php @@ -18,6 +18,9 @@ public function testShouldImplementRegistryInterface() $this->assertClassImplements(Registry::class, SimpleRegistry::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithEventsMapAndTransformersMapAsArguments() { new SimpleRegistry([], []); diff --git a/pkg/dbal/Tests/DbalConsumerTest.php b/pkg/dbal/Tests/DbalConsumerTest.php index d36333c83..7c2bbbc03 100644 --- a/pkg/dbal/Tests/DbalConsumerTest.php +++ b/pkg/dbal/Tests/DbalConsumerTest.php @@ -28,6 +28,9 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, DbalConsumer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new DbalConsumer($this->createContextMock(), new DbalDestination('queue')); diff --git a/pkg/dbal/Tests/DbalContextTest.php b/pkg/dbal/Tests/DbalContextTest.php index 12a13b21a..6c0cc2878 100644 --- a/pkg/dbal/Tests/DbalContextTest.php +++ b/pkg/dbal/Tests/DbalContextTest.php @@ -27,6 +27,9 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, DbalContext::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new DbalContext($this->createConnectionMock()); diff --git a/pkg/dbal/Tests/DbalProducerTest.php b/pkg/dbal/Tests/DbalProducerTest.php index 4dcb015e3..f7800bc56 100644 --- a/pkg/dbal/Tests/DbalProducerTest.php +++ b/pkg/dbal/Tests/DbalProducerTest.php @@ -21,6 +21,9 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, DbalProducer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new DbalProducer($this->createContextMock()); diff --git a/pkg/dbal/Tests/DbalSubscriptionConsumerTest.php b/pkg/dbal/Tests/DbalSubscriptionConsumerTest.php index a3fad6e66..9c372de82 100644 --- a/pkg/dbal/Tests/DbalSubscriptionConsumerTest.php +++ b/pkg/dbal/Tests/DbalSubscriptionConsumerTest.php @@ -25,6 +25,9 @@ public function testShouldImplementSubscriptionConsumerInterface() $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class)); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithDbalContextAsFirstArgument() { new DbalSubscriptionConsumer($this->createDbalContextMock()); @@ -66,6 +69,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..f693fce04 100644 --- a/pkg/dsn/Tests/DsnTest.php +++ b/pkg/dsn/Tests/DsnTest.php @@ -8,6 +8,9 @@ class DsnTest extends TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithDsnAsFirstArgument() { Dsn::parseFirst('foo://localhost:1234'); diff --git a/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrineClearIdentityMapExtensionTest.php b/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrineClearIdentityMapExtensionTest.php index 604012e78..bf8c9a8b3 100644 --- a/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrineClearIdentityMapExtensionTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrineClearIdentityMapExtensionTest.php @@ -16,6 +16,9 @@ class DoctrineClearIdentityMapExtensionTest extends TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new DoctrineClearIdentityMapExtension($this->createRegistryMock()); diff --git a/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrinePingConnectionExtensionTest.php b/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrinePingConnectionExtensionTest.php index c530eeaa9..b842b766d 100644 --- a/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrinePingConnectionExtensionTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrinePingConnectionExtensionTest.php @@ -17,6 +17,9 @@ class DoctrinePingConnectionExtensionTest extends TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredAttributes() { new DoctrinePingConnectionExtension($this->createRegistryMock()); diff --git a/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/ResetServicesExtensionTest.php b/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/ResetServicesExtensionTest.php index 18e94f251..1feba7a9f 100644 --- a/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/ResetServicesExtensionTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/ResetServicesExtensionTest.php @@ -16,6 +16,9 @@ class ResetServicesExtensionTest extends TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new ResetServicesExtension($this->createResetterMock()); diff --git a/pkg/enqueue-bundle/Tests/Unit/DependencyInjection/ConfigurationTest.php b/pkg/enqueue-bundle/Tests/Unit/DependencyInjection/ConfigurationTest.php index c9356f1ec..746874017 100644 --- a/pkg/enqueue-bundle/Tests/Unit/DependencyInjection/ConfigurationTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/DependencyInjection/ConfigurationTest.php @@ -25,6 +25,9 @@ public function testShouldBeFinal() $this->assertClassFinal(Configuration::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithDebugAsArgument() { 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..9a20f9966 100644 --- a/pkg/enqueue-bundle/Tests/Unit/DependencyInjection/EnqueueExtensionTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/DependencyInjection/EnqueueExtensionTest.php @@ -30,6 +30,9 @@ public function testShouldBeFinal() $this->assertClassFinal(EnqueueExtension::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutAnyArguments() { new EnqueueExtension(); diff --git a/pkg/enqueue-bundle/Tests/Unit/EnqueueBundleTest.php b/pkg/enqueue-bundle/Tests/Unit/EnqueueBundleTest.php index d93a1f1f9..13776ede4 100644 --- a/pkg/enqueue-bundle/Tests/Unit/EnqueueBundleTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/EnqueueBundleTest.php @@ -16,6 +16,9 @@ public function testShouldExtendBundleClass() $this->assertClassExtends(Bundle::class, EnqueueBundle::class); } + /** + * @doesNotPerformAssertions + */ 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..12700be60 100644 --- a/pkg/enqueue-bundle/Tests/Unit/Profiler/MessageQueueCollectorTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/Profiler/MessageQueueCollectorTest.php @@ -23,6 +23,9 @@ public function testShouldExtendDataCollectorClass() $this->assertClassExtends(DataCollector::class, MessageQueueCollector::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithEmptyConstructor() { new MessageQueueCollector(); diff --git a/pkg/enqueue/Tests/ArrayProcessorRegistryTest.php b/pkg/enqueue/Tests/ArrayProcessorRegistryTest.php index 897276d62..c0937547d 100644 --- a/pkg/enqueue/Tests/ArrayProcessorRegistryTest.php +++ b/pkg/enqueue/Tests/ArrayProcessorRegistryTest.php @@ -18,6 +18,9 @@ public function testShouldImplementProcessorRegistryInterface() $this->assertClassImplements(ProcessorRegistryInterface::class, ArrayProcessorRegistry::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutAnyArgument() { new ArrayProcessorRegistry(); diff --git a/pkg/enqueue/Tests/Client/ChainExtensionTest.php b/pkg/enqueue/Tests/Client/ChainExtensionTest.php index a59005bc1..c5dc60451 100644 --- a/pkg/enqueue/Tests/Client/ChainExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ChainExtensionTest.php @@ -29,6 +29,9 @@ public function testShouldBeFinal() $this->assertClassFinal(ChainExtension::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithExtensionsArray() { new ChainExtension([$this->createExtension(), $this->createExtension()]); diff --git a/pkg/enqueue/Tests/Client/ConsumptionExtension/DelayRedeliveredMessageExtensionTest.php b/pkg/enqueue/Tests/Client/ConsumptionExtension/DelayRedeliveredMessageExtensionTest.php index 0481d27d8..6d502461f 100644 --- a/pkg/enqueue/Tests/Client/ConsumptionExtension/DelayRedeliveredMessageExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ConsumptionExtension/DelayRedeliveredMessageExtensionTest.php @@ -23,6 +23,9 @@ class DelayRedeliveredMessageExtensionTest extends TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new DelayRedeliveredMessageExtension($this->createDriverMock(), 12345); diff --git a/pkg/enqueue/Tests/Client/ConsumptionExtension/ExclusiveCommandExtensionTest.php b/pkg/enqueue/Tests/Client/ConsumptionExtension/ExclusiveCommandExtensionTest.php index dc2c00c96..9a2270d4a 100644 --- a/pkg/enqueue/Tests/Client/ConsumptionExtension/ExclusiveCommandExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ConsumptionExtension/ExclusiveCommandExtensionTest.php @@ -34,6 +34,9 @@ public function testShouldBeFinal() $this->assertClassFinal(ExclusiveCommandExtension::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithDriverAsFirstArgument() { new ExclusiveCommandExtension($this->createDriverStub()); diff --git a/pkg/enqueue/Tests/Client/ConsumptionExtension/FlushSpoolProducerExtensionTest.php b/pkg/enqueue/Tests/Client/ConsumptionExtension/FlushSpoolProducerExtensionTest.php index 63fb50c5a..d69969b78 100644 --- a/pkg/enqueue/Tests/Client/ConsumptionExtension/FlushSpoolProducerExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ConsumptionExtension/FlushSpoolProducerExtensionTest.php @@ -30,6 +30,9 @@ public function testShouldImplementEndExtensionInterface() $this->assertClassImplements(EndExtensionInterface::class, FlushSpoolProducerExtension::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithSpoolProducerAsFirstArgument() { new FlushSpoolProducerExtension($this->createSpoolProducerMock()); diff --git a/pkg/enqueue/Tests/Client/ConsumptionExtension/LogExtensionTest.php b/pkg/enqueue/Tests/Client/ConsumptionExtension/LogExtensionTest.php index 063ea23af..fe9e6ab0c 100644 --- a/pkg/enqueue/Tests/Client/ConsumptionExtension/LogExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ConsumptionExtension/LogExtensionTest.php @@ -55,6 +55,9 @@ public function testShouldSubClassOfLogExtension() $this->assertClassExtends(\Enqueue\Consumption\Extension\LogExtension::class, LogExtension::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutAnyArguments() { new LogExtension(); diff --git a/pkg/enqueue/Tests/Client/ConsumptionExtension/SetRouterPropertiesExtensionTest.php b/pkg/enqueue/Tests/Client/ConsumptionExtension/SetRouterPropertiesExtensionTest.php index e5015ff45..5337d0495 100644 --- a/pkg/enqueue/Tests/Client/ConsumptionExtension/SetRouterPropertiesExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ConsumptionExtension/SetRouterPropertiesExtensionTest.php @@ -27,6 +27,9 @@ public function testShouldImplementMessageReceivedExtensionInterface() $this->assertClassImplements(MessageReceivedExtensionInterface::class, SetRouterPropertiesExtension::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new SetRouterPropertiesExtension($this->createDriverMock()); diff --git a/pkg/enqueue/Tests/Client/ConsumptionExtension/SetupBrokerExtensionTest.php b/pkg/enqueue/Tests/Client/ConsumptionExtension/SetupBrokerExtensionTest.php index 4b241aa0e..a72ae389a 100644 --- a/pkg/enqueue/Tests/Client/ConsumptionExtension/SetupBrokerExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ConsumptionExtension/SetupBrokerExtensionTest.php @@ -21,6 +21,9 @@ public function testShouldImplementStartExtensionInterface() $this->assertClassImplements(StartExtensionInterface::class, SetupBrokerExtension::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new SetupBrokerExtension($this->createDriverMock()); diff --git a/pkg/enqueue/Tests/Client/DelegateProcessorTest.php b/pkg/enqueue/Tests/Client/DelegateProcessorTest.php index 40ef17989..f9bd83ebf 100644 --- a/pkg/enqueue/Tests/Client/DelegateProcessorTest.php +++ b/pkg/enqueue/Tests/Client/DelegateProcessorTest.php @@ -13,6 +13,9 @@ class DelegateProcessorTest extends TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new DelegateProcessor($this->createProcessorRegistryMock()); diff --git a/pkg/enqueue/Tests/Client/DriverFactoryTest.php b/pkg/enqueue/Tests/Client/DriverFactoryTest.php index cce6311e5..605358aa3 100644 --- a/pkg/enqueue/Tests/Client/DriverFactoryTest.php +++ b/pkg/enqueue/Tests/Client/DriverFactoryTest.php @@ -61,6 +61,9 @@ public function testShouldBeFinal() $this->assertTrue($rc->isFinal()); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutAnyArguments() { new DriverFactory(); diff --git a/pkg/enqueue/Tests/Client/DriverPreSendTest.php b/pkg/enqueue/Tests/Client/DriverPreSendTest.php index 85686b4c1..f3c1ee375 100644 --- a/pkg/enqueue/Tests/Client/DriverPreSendTest.php +++ b/pkg/enqueue/Tests/Client/DriverPreSendTest.php @@ -19,6 +19,9 @@ public function testShouldBeFinal() self::assertClassFinal(DriverPreSend::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithExpectedArguments() { new DriverPreSend( diff --git a/pkg/enqueue/Tests/Client/Extension/PrepareBodyExtensionTest.php b/pkg/enqueue/Tests/Client/Extension/PrepareBodyExtensionTest.php index 454034abb..81b29e801 100644 --- a/pkg/enqueue/Tests/Client/Extension/PrepareBodyExtensionTest.php +++ b/pkg/enqueue/Tests/Client/Extension/PrepareBodyExtensionTest.php @@ -22,6 +22,9 @@ public function testShouldImplementExtensionInterface() $this->assertTrue($rc->implementsInterface(PreSendCommandExtensionInterface::class)); } + /** + * @doesNotPerformAssertions + */ public function testCouldConstructedWithoutAnyArguments() { new PrepareBodyExtension(); diff --git a/pkg/enqueue/Tests/Client/PostSendTest.php b/pkg/enqueue/Tests/Client/PostSendTest.php index 04927700c..9eb845e49 100644 --- a/pkg/enqueue/Tests/Client/PostSendTest.php +++ b/pkg/enqueue/Tests/Client/PostSendTest.php @@ -21,6 +21,9 @@ public function testShouldBeFinal() self::assertClassFinal(PostSend::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithExpectedArguments() { new PostSend( diff --git a/pkg/enqueue/Tests/Client/PreSendTest.php b/pkg/enqueue/Tests/Client/PreSendTest.php index 408be1c69..40dffafa2 100644 --- a/pkg/enqueue/Tests/Client/PreSendTest.php +++ b/pkg/enqueue/Tests/Client/PreSendTest.php @@ -18,6 +18,9 @@ public function testShouldBeFinal() self::assertClassFinal(PreSend::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithExpectedArguments() { new PreSend( diff --git a/pkg/enqueue/Tests/Client/ProducerTest.php b/pkg/enqueue/Tests/Client/ProducerTest.php index 558e70ee6..38dd1f075 100644 --- a/pkg/enqueue/Tests/Client/ProducerTest.php +++ b/pkg/enqueue/Tests/Client/ProducerTest.php @@ -24,11 +24,17 @@ public function testShouldBeFinal() self::assertClassFinal(Producer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new Producer($this->createDriverMock(), $this->createRpcFactoryMock()); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithOptionalArguments() { new Producer( diff --git a/pkg/enqueue/Tests/Client/SpoolProducerTest.php b/pkg/enqueue/Tests/Client/SpoolProducerTest.php index 024fd6da2..511b91edb 100644 --- a/pkg/enqueue/Tests/Client/SpoolProducerTest.php +++ b/pkg/enqueue/Tests/Client/SpoolProducerTest.php @@ -18,6 +18,9 @@ public function testShouldImplementProducerInterface() self::assertClassImplements(ProducerInterface::class, SpoolProducer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRealProducer() { new SpoolProducer($this->createProducerMock()); diff --git a/pkg/enqueue/Tests/Client/TraceableProducerTest.php b/pkg/enqueue/Tests/Client/TraceableProducerTest.php index ac0a52961..3ce23f89a 100644 --- a/pkg/enqueue/Tests/Client/TraceableProducerTest.php +++ b/pkg/enqueue/Tests/Client/TraceableProducerTest.php @@ -18,6 +18,9 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(ProducerInterface::class, TraceableProducer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithInternalMessageProducer() { new TraceableProducer($this->createProducerMock()); diff --git a/pkg/enqueue/Tests/ConnectionFactoryFactoryTest.php b/pkg/enqueue/Tests/ConnectionFactoryFactoryTest.php index 3fb4fb57b..c522c6334 100644 --- a/pkg/enqueue/Tests/ConnectionFactoryFactoryTest.php +++ b/pkg/enqueue/Tests/ConnectionFactoryFactoryTest.php @@ -37,11 +37,17 @@ public function testShouldBeFinal() $this->assertTrue($rc->isFinal()); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutAnyArguments() { new ConnectionFactoryFactory(); } + /** + * @doesNotPerformAssertions + */ public function testShouldAcceptStringDSN() { $factory = new ConnectionFactoryFactory(); @@ -49,6 +55,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..9cafdb885 100644 --- a/pkg/enqueue/Tests/Consumption/CallbackProcessorTest.php +++ b/pkg/enqueue/Tests/Consumption/CallbackProcessorTest.php @@ -18,6 +18,9 @@ public function testShouldImplementProcessorInterface() $this->assertClassImplements(Processor::class, CallbackProcessor::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithCallableAsArgument() { new CallbackProcessor(function () { diff --git a/pkg/enqueue/Tests/Consumption/ChainExtensionTest.php b/pkg/enqueue/Tests/Consumption/ChainExtensionTest.php index 23bd4f979..c19cc2f20 100644 --- a/pkg/enqueue/Tests/Consumption/ChainExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/ChainExtensionTest.php @@ -33,6 +33,9 @@ public function testShouldImplementExtensionInterface() $this->assertClassImplements(ExtensionInterface::class, ChainExtension::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithExtensionsArray() { new ChainExtension([$this->createExtension(), $this->createExtension()]); diff --git a/pkg/enqueue/Tests/Consumption/Exception/IllegalContextModificationExceptionTest.php b/pkg/enqueue/Tests/Consumption/Exception/IllegalContextModificationExceptionTest.php index 0885b500e..5de5c2f05 100644 --- a/pkg/enqueue/Tests/Consumption/Exception/IllegalContextModificationExceptionTest.php +++ b/pkg/enqueue/Tests/Consumption/Exception/IllegalContextModificationExceptionTest.php @@ -21,6 +21,9 @@ public function testShouldExtendLogicException() $this->assertClassExtends(\LogicException::class, IllegalContextModificationException::class); } + /** + * @doesNotPerformAssertions + */ 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..cc2ca3122 100644 --- a/pkg/enqueue/Tests/Consumption/Exception/InvalidArgumentExceptionTest.php +++ b/pkg/enqueue/Tests/Consumption/Exception/InvalidArgumentExceptionTest.php @@ -21,6 +21,9 @@ public function testShouldExtendLogicException() $this->assertClassExtends(\LogicException::class, InvalidArgumentException::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutAnyArguments() { new InvalidArgumentException(); @@ -36,6 +39,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..f7a68c9a9 100644 --- a/pkg/enqueue/Tests/Consumption/Exception/LogicExceptionTest.php +++ b/pkg/enqueue/Tests/Consumption/Exception/LogicExceptionTest.php @@ -21,6 +21,9 @@ public function testShouldExtendLogicException() $this->assertClassExtends(\LogicException::class, LogicException::class); } + /** + * @doesNotPerformAssertions + */ 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..15d221409 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/LimitConsumedMessagesExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/LimitConsumedMessagesExtensionTest.php @@ -16,6 +16,9 @@ class LimitConsumedMessagesExtensionTest extends TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new LimitConsumedMessagesExtension(12345); diff --git a/pkg/enqueue/Tests/Consumption/Extension/LimitConsumerMemoryExtensionTest.php b/pkg/enqueue/Tests/Consumption/Extension/LimitConsumerMemoryExtensionTest.php index 5a95486ec..22d33826e 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/LimitConsumerMemoryExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/LimitConsumerMemoryExtensionTest.php @@ -17,6 +17,9 @@ class LimitConsumerMemoryExtensionTest extends TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new LimitConsumerMemoryExtension(12345); diff --git a/pkg/enqueue/Tests/Consumption/Extension/LimitConsumptionTimeExtensionTest.php b/pkg/enqueue/Tests/Consumption/Extension/LimitConsumptionTimeExtensionTest.php index fa730e384..fbd61d48f 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/LimitConsumptionTimeExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/LimitConsumptionTimeExtensionTest.php @@ -17,6 +17,9 @@ class LimitConsumptionTimeExtensionTest extends TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new LimitConsumptionTimeExtension(new \DateTime('+1 day')); diff --git a/pkg/enqueue/Tests/Consumption/Extension/LogExtensionTest.php b/pkg/enqueue/Tests/Consumption/Extension/LogExtensionTest.php index 4bc4ae95f..f529dcd4c 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/LogExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/LogExtensionTest.php @@ -49,6 +49,9 @@ public function testShouldImplementPostMessageReceivedExtensionInterface() $this->assertClassImplements(PostMessageReceivedExtensionInterface::class, LogExtension::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutAnyArguments() { new LogExtension(); diff --git a/pkg/enqueue/Tests/Consumption/Extension/LoggerExtensionTest.php b/pkg/enqueue/Tests/Consumption/Extension/LoggerExtensionTest.php index f3641adf4..227c0830f 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/LoggerExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/LoggerExtensionTest.php @@ -20,6 +20,9 @@ public function testShouldImplementInitLoggerExtensionInterface() $this->assertClassImplements(InitLoggerExtensionInterface::class, LoggerExtension::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithLoggerAsFirstArgument() { new LoggerExtension($this->createLogger()); diff --git a/pkg/enqueue/Tests/Consumption/Extension/NicenessExtensionTest.php b/pkg/enqueue/Tests/Consumption/Extension/NicenessExtensionTest.php index a2937d0ca..15c004db4 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/NicenessExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/NicenessExtensionTest.php @@ -11,6 +11,9 @@ class NicenessExtensionTest extends TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new NicenessExtension(0); diff --git a/pkg/enqueue/Tests/Consumption/Extension/ReplyExtensionTest.php b/pkg/enqueue/Tests/Consumption/Extension/ReplyExtensionTest.php index 408895638..3963e94e6 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/ReplyExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/ReplyExtensionTest.php @@ -25,6 +25,9 @@ public function testShouldImplementPostMessageReceivedExtensionInterface() $this->assertClassImplements(PostMessageReceivedExtensionInterface::class, ReplyExtension::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutAnyArguments() { new ReplyExtension(); diff --git a/pkg/enqueue/Tests/Consumption/FallbackSubscriptionConsumerTest.php b/pkg/enqueue/Tests/Consumption/FallbackSubscriptionConsumerTest.php index 4f8ac7a55..aecf71628 100644 --- a/pkg/enqueue/Tests/Consumption/FallbackSubscriptionConsumerTest.php +++ b/pkg/enqueue/Tests/Consumption/FallbackSubscriptionConsumerTest.php @@ -21,6 +21,9 @@ public function testShouldImplementSubscriptionConsumerInterface() $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class)); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutAnyArguments() { new FallbackSubscriptionConsumer(); @@ -69,6 +72,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..ce72cb9d9 100644 --- a/pkg/enqueue/Tests/Consumption/QueueConsumerTest.php +++ b/pkg/enqueue/Tests/Consumption/QueueConsumerTest.php @@ -39,16 +39,25 @@ class QueueConsumerTest extends TestCase { use ReadAttributeTrait; + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithAllArguments() { new QueueConsumer($this->createContextStub(), null, [], null, 0); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithContextOnly() { new QueueConsumer($this->createContextStub()); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithContextAndSingleExtension() { new QueueConsumer($this->createContextStub(), $this->createExtension()); diff --git a/pkg/enqueue/Tests/Router/RouteRecipientListProcessorTest.php b/pkg/enqueue/Tests/Router/RouteRecipientListProcessorTest.php index 8a2e089f7..7e38f9616 100644 --- a/pkg/enqueue/Tests/Router/RouteRecipientListProcessorTest.php +++ b/pkg/enqueue/Tests/Router/RouteRecipientListProcessorTest.php @@ -23,6 +23,9 @@ public function testShouldImplementProcessorInterface() $this->assertClassImplements(Processor::class, RouteRecipientListProcessor::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRouterAsFirstArgument() { new RouteRecipientListProcessor($this->createRecipientListRouterMock()); diff --git a/pkg/enqueue/Tests/Rpc/RpcClientTest.php b/pkg/enqueue/Tests/Rpc/RpcClientTest.php index e2035c4d5..e8a3720b1 100644 --- a/pkg/enqueue/Tests/Rpc/RpcClientTest.php +++ b/pkg/enqueue/Tests/Rpc/RpcClientTest.php @@ -15,6 +15,9 @@ class RpcClientTest extends TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithContextAsFirstArgument() { new RpcClient($this->createContextMock()); diff --git a/pkg/enqueue/Tests/Symfony/Client/ConsumeCommandTest.php b/pkg/enqueue/Tests/Symfony/Client/ConsumeCommandTest.php index 8076aaf76..17be30aed 100644 --- a/pkg/enqueue/Tests/Symfony/Client/ConsumeCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/ConsumeCommandTest.php @@ -40,6 +40,9 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(ConsumeCommand::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredAttributes() { new ConsumeCommand($this->createMock(ContainerInterface::class), 'default'); diff --git a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/AnalyzeRouteCollectionPassTest.php b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/AnalyzeRouteCollectionPassTest.php index 8dcffe45a..fc3051b97 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/AnalyzeRouteCollectionPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/AnalyzeRouteCollectionPassTest.php @@ -23,6 +23,9 @@ public function testShouldBeFinal() $this->assertClassFinal(AnalyzeRouteCollectionPass::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutArguments() { new AnalyzeRouteCollectionPass(); diff --git a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildClientExtensionsPassTest.php b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildClientExtensionsPassTest.php index 90c7493c1..baae3c768 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildClientExtensionsPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildClientExtensionsPassTest.php @@ -25,6 +25,9 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildClientExtensionsPass::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutArguments() { new BuildClientExtensionsPass(); diff --git a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildCommandSubscriberRoutesPassTest.php b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildCommandSubscriberRoutesPassTest.php index a2e964dcc..7317a8ef8 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildCommandSubscriberRoutesPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildCommandSubscriberRoutesPassTest.php @@ -29,6 +29,9 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildCommandSubscriberRoutesPass::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutArguments() { new BuildCommandSubscriberRoutesPass(); diff --git a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildConsumptionExtensionsPassTest.php b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildConsumptionExtensionsPassTest.php index a94636261..64966d602 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildConsumptionExtensionsPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildConsumptionExtensionsPassTest.php @@ -25,6 +25,9 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildConsumptionExtensionsPass::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutArguments() { new BuildConsumptionExtensionsPass(); diff --git a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRegistryPassTest.php b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRegistryPassTest.php index 8541f39bf..98679abbd 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRegistryPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRegistryPassTest.php @@ -26,6 +26,9 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildProcessorRegistryPass::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutArguments() { new BuildProcessorRegistryPass(); diff --git a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRoutesPassTest.php b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRoutesPassTest.php index b2839c466..d4ac29803 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRoutesPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRoutesPassTest.php @@ -25,6 +25,9 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildProcessorRoutesPass::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutArguments() { new BuildProcessorRoutesPass(); diff --git a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildTopicSubscriberRoutesPassTest.php b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildTopicSubscriberRoutesPassTest.php index af1c6a37a..0ada6064a 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildTopicSubscriberRoutesPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildTopicSubscriberRoutesPassTest.php @@ -29,6 +29,9 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildTopicSubscriberRoutesPass::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutArguments() { new BuildTopicSubscriberRoutesPass(); diff --git a/pkg/enqueue/Tests/Symfony/Client/FlushSpoolProducerListenerTest.php b/pkg/enqueue/Tests/Symfony/Client/FlushSpoolProducerListenerTest.php index 3476c101e..feae3b52f 100644 --- a/pkg/enqueue/Tests/Symfony/Client/FlushSpoolProducerListenerTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/FlushSpoolProducerListenerTest.php @@ -39,6 +39,9 @@ public function testShouldSubscribeOnConsoleTerminateEvent() $this->assertEquals('flushMessages', $events[ConsoleEvents::TERMINATE]); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithSpoolProducerAsFirstArgument() { new FlushSpoolProducerListener($this->createSpoolProducerMock()); diff --git a/pkg/enqueue/Tests/Symfony/Client/ProduceCommandTest.php b/pkg/enqueue/Tests/Symfony/Client/ProduceCommandTest.php index 2efc4769f..b6a1d18a4 100644 --- a/pkg/enqueue/Tests/Symfony/Client/ProduceCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/ProduceCommandTest.php @@ -27,6 +27,9 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(ProduceCommand::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithContainerAsFirstArgument() { new ProduceCommand($this->createMock(ContainerInterface::class), 'default'); diff --git a/pkg/enqueue/Tests/Symfony/Client/RoutesCommandTest.php b/pkg/enqueue/Tests/Symfony/Client/RoutesCommandTest.php index c2168c4f8..8adf71064 100644 --- a/pkg/enqueue/Tests/Symfony/Client/RoutesCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/RoutesCommandTest.php @@ -29,6 +29,9 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(RoutesCommand::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithConfigAndRouteCollectionAsArguments() { new RoutesCommand($this->createMock(ContainerInterface::class), 'default'); diff --git a/pkg/enqueue/Tests/Symfony/Client/SetupBrokerCommandTest.php b/pkg/enqueue/Tests/Symfony/Client/SetupBrokerCommandTest.php index 89ca379b5..53a8d4494 100644 --- a/pkg/enqueue/Tests/Symfony/Client/SetupBrokerCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/SetupBrokerCommandTest.php @@ -26,6 +26,9 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(SetupBrokerCommand::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithContainerAsFirstArgument() { new SetupBrokerCommand($this->createMock(ContainerInterface::class), 'default'); diff --git a/pkg/enqueue/Tests/Symfony/Client/SimpleConsumeCommandTest.php b/pkg/enqueue/Tests/Symfony/Client/SimpleConsumeCommandTest.php index 60d409325..97f1c6c58 100644 --- a/pkg/enqueue/Tests/Symfony/Client/SimpleConsumeCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/SimpleConsumeCommandTest.php @@ -29,6 +29,9 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(SimpleConsumeCommand::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredAttributes() { 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..50d46c96d 100644 --- a/pkg/enqueue/Tests/Symfony/Client/SimpleProduceCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/SimpleProduceCommandTest.php @@ -23,6 +23,9 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(SimpleProduceCommand::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithContainerAsFirstArgument() { new SimpleProduceCommand($this->createProducerMock()); diff --git a/pkg/enqueue/Tests/Symfony/Client/SimpleRoutesCommandTest.php b/pkg/enqueue/Tests/Symfony/Client/SimpleRoutesCommandTest.php index 356f3f9f4..71bb5cf6b 100644 --- a/pkg/enqueue/Tests/Symfony/Client/SimpleRoutesCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/SimpleRoutesCommandTest.php @@ -25,6 +25,9 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(SimpleRoutesCommand::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithConfigAndRouteCollectionAsArguments() { new SimpleRoutesCommand($this->createDriverMock()); diff --git a/pkg/enqueue/Tests/Symfony/Client/SimpleSetupBrokerCommandTest.php b/pkg/enqueue/Tests/Symfony/Client/SimpleSetupBrokerCommandTest.php index 91802ff11..d3e8f2170 100644 --- a/pkg/enqueue/Tests/Symfony/Client/SimpleSetupBrokerCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/SimpleSetupBrokerCommandTest.php @@ -23,6 +23,9 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(SimpleSetupBrokerCommand::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithContainerAsFirstArgument() { new SimpleSetupBrokerCommand($this->createClientDriverMock()); diff --git a/pkg/enqueue/Tests/Symfony/Consumption/ConfigurableConsumeCommandTest.php b/pkg/enqueue/Tests/Symfony/Consumption/ConfigurableConsumeCommandTest.php index c448c1cf5..82342e4d6 100644 --- a/pkg/enqueue/Tests/Symfony/Consumption/ConfigurableConsumeCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Consumption/ConfigurableConsumeCommandTest.php @@ -33,6 +33,9 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(ConfigurableConsumeCommand::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredAttributes() { new ConfigurableConsumeCommand($this->createMock(ContainerInterface::class), 'default'); diff --git a/pkg/enqueue/Tests/Symfony/Consumption/ConsumeCommandTest.php b/pkg/enqueue/Tests/Symfony/Consumption/ConsumeCommandTest.php index 3c16cde3d..b56ef68cc 100644 --- a/pkg/enqueue/Tests/Symfony/Consumption/ConsumeCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Consumption/ConsumeCommandTest.php @@ -35,6 +35,9 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(ConsumeCommand::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredAttributes() { new ConsumeCommand($this->createMock(ContainerInterface::class), 'default'); diff --git a/pkg/enqueue/Tests/Symfony/Consumption/SimpleConsumeCommandTest.php b/pkg/enqueue/Tests/Symfony/Consumption/SimpleConsumeCommandTest.php index d54a70e58..7648587d6 100644 --- a/pkg/enqueue/Tests/Symfony/Consumption/SimpleConsumeCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Consumption/SimpleConsumeCommandTest.php @@ -24,6 +24,9 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(SimpleConsumeCommand::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredAttributes() { new SimpleConsumeCommand($this->createQueueConsumerMock()); diff --git a/pkg/enqueue/Tests/Symfony/ContainerProcessorRegistryTest.php b/pkg/enqueue/Tests/Symfony/ContainerProcessorRegistryTest.php index e0dcf2e09..1370b4df6 100644 --- a/pkg/enqueue/Tests/Symfony/ContainerProcessorRegistryTest.php +++ b/pkg/enqueue/Tests/Symfony/ContainerProcessorRegistryTest.php @@ -23,6 +23,9 @@ public function testShouldBeFinal() $this->assertClassFinal(ContainerProcessorRegistry::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithContainerAsFirstArgument() { new ContainerProcessorRegistry($this->createContainerMock()); diff --git a/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildConsumptionExtensionsPassTest.php b/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildConsumptionExtensionsPassTest.php index f16daf08a..c558aaddb 100644 --- a/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildConsumptionExtensionsPassTest.php +++ b/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildConsumptionExtensionsPassTest.php @@ -25,6 +25,9 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildConsumptionExtensionsPass::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutArguments() { new BuildConsumptionExtensionsPass(); diff --git a/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildProcessorRegistryPassTest.php b/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildProcessorRegistryPassTest.php index 8d8edc537..e3a333055 100644 --- a/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildProcessorRegistryPassTest.php +++ b/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildProcessorRegistryPassTest.php @@ -26,6 +26,9 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildProcessorRegistryPass::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutArguments() { new BuildProcessorRegistryPass(); diff --git a/pkg/enqueue/Tests/Symfony/LazyProducerTest.php b/pkg/enqueue/Tests/Symfony/LazyProducerTest.php index 1b15ce2ce..7fbe714ac 100644 --- a/pkg/enqueue/Tests/Symfony/LazyProducerTest.php +++ b/pkg/enqueue/Tests/Symfony/LazyProducerTest.php @@ -20,6 +20,9 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(ProducerInterface::class, LazyProducer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithContainerAndServiceId() { new LazyProducer($this->createContainerMock(), 'realProducerId'); diff --git a/pkg/enqueue/Tests/Util/VarExportTest.php b/pkg/enqueue/Tests/Util/VarExportTest.php index 1d2384ac9..a8fa92f20 100644 --- a/pkg/enqueue/Tests/Util/VarExportTest.php +++ b/pkg/enqueue/Tests/Util/VarExportTest.php @@ -7,6 +7,9 @@ class VarExportTest extends TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithValueAsArgument() { new VarExport('aVal'); diff --git a/pkg/fs/Tests/FsConsumerTest.php b/pkg/fs/Tests/FsConsumerTest.php index 086bb55fe..044d05ae2 100644 --- a/pkg/fs/Tests/FsConsumerTest.php +++ b/pkg/fs/Tests/FsConsumerTest.php @@ -20,6 +20,9 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, FsConsumer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithContextAndDestinationAndPreFetchCountAsArguments() { new FsConsumer($this->createContextMock(), new FsDestination(TempFile::generate()), 1); @@ -50,6 +53,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 +63,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..ed1aec87b 100644 --- a/pkg/fs/Tests/FsContextTest.php +++ b/pkg/fs/Tests/FsContextTest.php @@ -24,6 +24,9 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, FsContext::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithExpectedArguments() { new FsContext(sys_get_temp_dir(), 1, 0666, 100); @@ -122,6 +125,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..1aa25fa3e 100644 --- a/pkg/fs/Tests/FsProducerTest.php +++ b/pkg/fs/Tests/FsProducerTest.php @@ -23,6 +23,9 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, FsProducer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithContextAsFirstArgument() { new FsProducer($this->createContextMock()); diff --git a/pkg/gearman/Tests/GearmanContextTest.php b/pkg/gearman/Tests/GearmanContextTest.php index f2b70d11b..deafe303b 100644 --- a/pkg/gearman/Tests/GearmanContextTest.php +++ b/pkg/gearman/Tests/GearmanContextTest.php @@ -24,6 +24,9 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, GearmanContext::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithConnectionConfigAsFirstArgument() { new GearmanContext(['host' => 'aHost', 'port' => 'aPort']); diff --git a/pkg/gearman/Tests/GearmanProducerTest.php b/pkg/gearman/Tests/GearmanProducerTest.php index afb6181a3..537a0550b 100644 --- a/pkg/gearman/Tests/GearmanProducerTest.php +++ b/pkg/gearman/Tests/GearmanProducerTest.php @@ -18,6 +18,9 @@ class GearmanProducerTest extends TestCase use ClassExtensionTrait; use SkipIfGearmanExtensionIsNotInstalledTrait; + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithGearmanClientAsFirstArgument() { new GearmanProducer($this->createGearmanClientMock()); diff --git a/pkg/job-queue/Tests/CalculateRootJobStatusProcessorTest.php b/pkg/job-queue/Tests/CalculateRootJobStatusProcessorTest.php index 2e5a6ac4c..edb489474 100644 --- a/pkg/job-queue/Tests/CalculateRootJobStatusProcessorTest.php +++ b/pkg/job-queue/Tests/CalculateRootJobStatusProcessorTest.php @@ -17,6 +17,9 @@ class CalculateRootJobStatusProcessorTest extends \PHPUnit\Framework\TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new CalculateRootJobStatusProcessor( diff --git a/pkg/job-queue/Tests/CalculateRootJobStatusServiceTest.php b/pkg/job-queue/Tests/CalculateRootJobStatusServiceTest.php index f289c314c..05badc3e7 100644 --- a/pkg/job-queue/Tests/CalculateRootJobStatusServiceTest.php +++ b/pkg/job-queue/Tests/CalculateRootJobStatusServiceTest.php @@ -9,6 +9,9 @@ class CalculateRootJobStatusServiceTest extends \PHPUnit\Framework\TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new CalculateRootJobStatusService($this->createJobStorageMock()); diff --git a/pkg/job-queue/Tests/DependentJobContextTest.php b/pkg/job-queue/Tests/DependentJobContextTest.php index 32340d687..7e517e705 100644 --- a/pkg/job-queue/Tests/DependentJobContextTest.php +++ b/pkg/job-queue/Tests/DependentJobContextTest.php @@ -7,6 +7,9 @@ class DependentJobContextTest extends \PHPUnit\Framework\TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new DependentJobContext(new Job()); diff --git a/pkg/job-queue/Tests/DependentJobServiceTest.php b/pkg/job-queue/Tests/DependentJobServiceTest.php index 149856308..a524ab255 100644 --- a/pkg/job-queue/Tests/DependentJobServiceTest.php +++ b/pkg/job-queue/Tests/DependentJobServiceTest.php @@ -10,6 +10,9 @@ class DependentJobServiceTest extends \PHPUnit\Framework\TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new DependentJobService($this->createJobStorageMock()); diff --git a/pkg/job-queue/Tests/Doctrine/JobStorageTest.php b/pkg/job-queue/Tests/Doctrine/JobStorageTest.php index 38d72a81c..e37a2a599 100644 --- a/pkg/job-queue/Tests/Doctrine/JobStorageTest.php +++ b/pkg/job-queue/Tests/Doctrine/JobStorageTest.php @@ -15,6 +15,9 @@ class JobStorageTest extends \PHPUnit\Framework\TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new JobStorage($this->createDoctrineMock(), 'entity-class', 'unique_table'); diff --git a/pkg/job-queue/Tests/JobProcessorTest.php b/pkg/job-queue/Tests/JobProcessorTest.php index 3f06bf8de..882678bf2 100644 --- a/pkg/job-queue/Tests/JobProcessorTest.php +++ b/pkg/job-queue/Tests/JobProcessorTest.php @@ -13,6 +13,9 @@ class JobProcessorTest extends TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeCreatedWithRequiredArguments() { new JobProcessor($this->createJobStorage(), $this->createProducerMock()); diff --git a/pkg/mongodb/Tests/MongodbConsumerTest.php b/pkg/mongodb/Tests/MongodbConsumerTest.php index cd2ee06f8..35dfba377 100644 --- a/pkg/mongodb/Tests/MongodbConsumerTest.php +++ b/pkg/mongodb/Tests/MongodbConsumerTest.php @@ -25,6 +25,9 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, MongodbConsumer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new MongodbConsumer($this->createContextMock(), new MongodbDestination('queue')); @@ -39,6 +42,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..830a6dd55 100644 --- a/pkg/mongodb/Tests/MongodbContextTest.php +++ b/pkg/mongodb/Tests/MongodbContextTest.php @@ -29,6 +29,9 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, MongodbContext::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new MongodbContext($this->createClientMock()); diff --git a/pkg/mongodb/Tests/MongodbProducerTest.php b/pkg/mongodb/Tests/MongodbProducerTest.php index 8e8900cfd..6b8b1b2e8 100644 --- a/pkg/mongodb/Tests/MongodbProducerTest.php +++ b/pkg/mongodb/Tests/MongodbProducerTest.php @@ -23,6 +23,9 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, MongodbProducer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new MongodbProducer($this->createContextMock()); diff --git a/pkg/mongodb/Tests/MongodbSubscriptionConsumerTest.php b/pkg/mongodb/Tests/MongodbSubscriptionConsumerTest.php index 0fc5afae4..5ff1687d8 100644 --- a/pkg/mongodb/Tests/MongodbSubscriptionConsumerTest.php +++ b/pkg/mongodb/Tests/MongodbSubscriptionConsumerTest.php @@ -24,6 +24,9 @@ public function testShouldImplementSubscriptionConsumerInterface() $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class)); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithMongodbContextAsFirstArgument() { new MongodbSubscriptionConsumer($this->createMongodbContextMock()); @@ -65,6 +68,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..8c2599591 100644 --- a/pkg/null/Tests/NullConnectionFactoryTest.php +++ b/pkg/null/Tests/NullConnectionFactoryTest.php @@ -17,6 +17,9 @@ public function testShouldImplementConnectionFactoryInterface() $this->assertClassImplements(ConnectionFactory::class, NullConnectionFactory::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutAnyArguments() { new NullConnectionFactory(); diff --git a/pkg/null/Tests/NullConsumerTest.php b/pkg/null/Tests/NullConsumerTest.php index 6419e9c39..5db15ae95 100644 --- a/pkg/null/Tests/NullConsumerTest.php +++ b/pkg/null/Tests/NullConsumerTest.php @@ -18,6 +18,9 @@ public function testShouldImplementMessageConsumerInterface() $this->assertClassImplements(Consumer::class, NullConsumer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithQueueAsArgument() { new NullConsumer(new NullQueue('aName')); @@ -41,6 +44,9 @@ public function testShouldAlwaysReturnNullOnReceiveNoWait() $this->assertNull($consumer->receiveNoWait()); } + /** + * @doesNotPerformAssertions + */ public function testShouldDoNothingOnAcknowledge() { $consumer = new NullConsumer(new NullQueue('theQueueName')); @@ -48,6 +54,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..257eee84b 100644 --- a/pkg/null/Tests/NullContextTest.php +++ b/pkg/null/Tests/NullContextTest.php @@ -21,6 +21,9 @@ public function testShouldImplementSessionInterface() $this->assertClassImplements(Context::class, NullContext::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutAnyArguments() { new NullContext(); diff --git a/pkg/null/Tests/NullProducerTest.php b/pkg/null/Tests/NullProducerTest.php index 289dab6c0..149310498 100644 --- a/pkg/null/Tests/NullProducerTest.php +++ b/pkg/null/Tests/NullProducerTest.php @@ -18,11 +18,17 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, NullProducer::class); } + /** + * @doesNotPerformAssertions + */ 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..cace66439 100644 --- a/pkg/null/Tests/NullQueueTest.php +++ b/pkg/null/Tests/NullQueueTest.php @@ -16,6 +16,9 @@ public function testShouldImplementQueueInterface() $this->assertClassImplements(Queue::class, NullQueue::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithNameAsArgument() { new NullQueue('aName'); diff --git a/pkg/null/Tests/NullTopicTest.php b/pkg/null/Tests/NullTopicTest.php index a40545120..017df3236 100644 --- a/pkg/null/Tests/NullTopicTest.php +++ b/pkg/null/Tests/NullTopicTest.php @@ -16,6 +16,9 @@ public function testShouldImplementTopicInterface() $this->assertClassImplements(Topic::class, NullTopic::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithNameAsArgument() { new NullTopic('aName'); diff --git a/pkg/pheanstalk/Tests/PheanstalkConsumerTest.php b/pkg/pheanstalk/Tests/PheanstalkConsumerTest.php index c0138c726..b78f3be23 100644 --- a/pkg/pheanstalk/Tests/PheanstalkConsumerTest.php +++ b/pkg/pheanstalk/Tests/PheanstalkConsumerTest.php @@ -14,6 +14,9 @@ class PheanstalkConsumerTest extends TestCase { use ClassExtensionTrait; + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithDestinationAndPheanstalkAsArguments() { new PheanstalkConsumer( diff --git a/pkg/pheanstalk/Tests/PheanstalkContextTest.php b/pkg/pheanstalk/Tests/PheanstalkContextTest.php index 9f5ac5812..e122872c3 100644 --- a/pkg/pheanstalk/Tests/PheanstalkContextTest.php +++ b/pkg/pheanstalk/Tests/PheanstalkContextTest.php @@ -21,6 +21,9 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, PheanstalkContext::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithPheanstalkAsFirstArgument() { new PheanstalkContext($this->createPheanstalkMock()); diff --git a/pkg/pheanstalk/Tests/PheanstalkProducerTest.php b/pkg/pheanstalk/Tests/PheanstalkProducerTest.php index 7a736c09e..b6d6dff03 100644 --- a/pkg/pheanstalk/Tests/PheanstalkProducerTest.php +++ b/pkg/pheanstalk/Tests/PheanstalkProducerTest.php @@ -18,6 +18,9 @@ class PheanstalkProducerTest extends TestCase { use ClassExtensionTrait; + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithPheanstalkAsFirstArgument() { new PheanstalkProducer($this->createPheanstalkMock()); diff --git a/pkg/rdkafka/Tests/JsonSerializerTest.php b/pkg/rdkafka/Tests/JsonSerializerTest.php index d10479def..32805ca16 100644 --- a/pkg/rdkafka/Tests/JsonSerializerTest.php +++ b/pkg/rdkafka/Tests/JsonSerializerTest.php @@ -17,6 +17,9 @@ public function testShouldImplementSerializerInterface() $this->assertClassImplements(Serializer::class, JsonSerializer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutAnyArguments() { new JsonSerializer(); diff --git a/pkg/rdkafka/Tests/RdKafkaConsumerTest.php b/pkg/rdkafka/Tests/RdKafkaConsumerTest.php index 4f2b806c6..471e31a91 100644 --- a/pkg/rdkafka/Tests/RdKafkaConsumerTest.php +++ b/pkg/rdkafka/Tests/RdKafkaConsumerTest.php @@ -13,6 +13,9 @@ class RdKafkaConsumerTest extends TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new RdKafkaConsumer( diff --git a/pkg/rdkafka/Tests/RdKafkaProducerTest.php b/pkg/rdkafka/Tests/RdKafkaProducerTest.php index ecfa4c6fc..25451f5b4 100644 --- a/pkg/rdkafka/Tests/RdKafkaProducerTest.php +++ b/pkg/rdkafka/Tests/RdKafkaProducerTest.php @@ -17,6 +17,9 @@ class RdKafkaProducerTest extends TestCase { + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithKafkaProducerAndSerializerAsArguments() { new RdKafkaProducer($this->createKafkaProducerMock(), $this->createSerializerMock()); diff --git a/pkg/redis/Tests/RedisConsumerTest.php b/pkg/redis/Tests/RedisConsumerTest.php index 2908134fa..659060fa8 100644 --- a/pkg/redis/Tests/RedisConsumerTest.php +++ b/pkg/redis/Tests/RedisConsumerTest.php @@ -22,6 +22,9 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, RedisConsumer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithContextAndDestinationAndPreFetchCountAsArguments() { new RedisConsumer($this->createContextMock(), new RedisDestination('aQueue')); diff --git a/pkg/redis/Tests/RedisContextTest.php b/pkg/redis/Tests/RedisContextTest.php index 0115f27b8..597b71554 100644 --- a/pkg/redis/Tests/RedisContextTest.php +++ b/pkg/redis/Tests/RedisContextTest.php @@ -25,11 +25,17 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, RedisContext::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRedisAsFirstArgument() { new RedisContext($this->createRedisMock(), 300); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRedisFactoryAsFirstArgument() { new RedisContext(function () { diff --git a/pkg/redis/Tests/RedisProducerTest.php b/pkg/redis/Tests/RedisProducerTest.php index 53d5f19e3..345c632b5 100644 --- a/pkg/redis/Tests/RedisProducerTest.php +++ b/pkg/redis/Tests/RedisProducerTest.php @@ -25,6 +25,9 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, RedisProducer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRedisAsFirstArgument() { new RedisProducer($this->createContextMock()); diff --git a/pkg/redis/Tests/RedisSubscriptionConsumerTest.php b/pkg/redis/Tests/RedisSubscriptionConsumerTest.php index b3ba4d26b..f27cb5155 100644 --- a/pkg/redis/Tests/RedisSubscriptionConsumerTest.php +++ b/pkg/redis/Tests/RedisSubscriptionConsumerTest.php @@ -22,6 +22,9 @@ public function testShouldImplementSubscriptionConsumerInterface() $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class)); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRedisContextAsFirstArgument() { new RedisSubscriptionConsumer($this->createRedisContextMock()); @@ -63,6 +66,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..21bc8d836 100644 --- a/pkg/redis/Tests/Spec/JsonSerializerTest.php +++ b/pkg/redis/Tests/Spec/JsonSerializerTest.php @@ -20,6 +20,9 @@ public function testShouldImplementSerializerInterface() $this->assertClassImplements(Serializer::class, JsonSerializer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutAnyArguments() { new JsonSerializer(); diff --git a/pkg/sns/Tests/SnsProducerTest.php b/pkg/sns/Tests/SnsProducerTest.php index 5f8784905..7001784dc 100644 --- a/pkg/sns/Tests/SnsProducerTest.php +++ b/pkg/sns/Tests/SnsProducerTest.php @@ -27,6 +27,9 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, SnsProducer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new SnsProducer($this->createSnsContextMock()); diff --git a/pkg/snsqs/Tests/SnsQsProducerTest.php b/pkg/snsqs/Tests/SnsQsProducerTest.php index aa3e28971..033bfdfd4 100644 --- a/pkg/snsqs/Tests/SnsQsProducerTest.php +++ b/pkg/snsqs/Tests/SnsQsProducerTest.php @@ -32,6 +32,9 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, SnsQsProducer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new SnsQsProducer($this->createSnsContextMock(), $this->createSqsContextMock()); diff --git a/pkg/sqs/Tests/SqsConsumerTest.php b/pkg/sqs/Tests/SqsConsumerTest.php index 05fbb9b69..b05ce246d 100644 --- a/pkg/sqs/Tests/SqsConsumerTest.php +++ b/pkg/sqs/Tests/SqsConsumerTest.php @@ -24,6 +24,9 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, SqsConsumer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new SqsConsumer($this->createContextMock(), new SqsDestination('queue')); diff --git a/pkg/sqs/Tests/SqsContextTest.php b/pkg/sqs/Tests/SqsContextTest.php index 75970bbad..bfa2163bf 100644 --- a/pkg/sqs/Tests/SqsContextTest.php +++ b/pkg/sqs/Tests/SqsContextTest.php @@ -25,6 +25,9 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, SqsContext::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithSqsClientAsFirstArgument() { new SqsContext($this->createSqsClientMock(), []); diff --git a/pkg/sqs/Tests/SqsProducerTest.php b/pkg/sqs/Tests/SqsProducerTest.php index 1bda62c48..1db3632d0 100644 --- a/pkg/sqs/Tests/SqsProducerTest.php +++ b/pkg/sqs/Tests/SqsProducerTest.php @@ -24,6 +24,9 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, SqsProducer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new SqsProducer($this->createSqsContextMock()); diff --git a/pkg/stomp/Tests/BufferedStompClientTest.php b/pkg/stomp/Tests/BufferedStompClientTest.php index 684edf397..3a2183b48 100644 --- a/pkg/stomp/Tests/BufferedStompClientTest.php +++ b/pkg/stomp/Tests/BufferedStompClientTest.php @@ -21,6 +21,9 @@ public function testShouldExtendLibStompClient() $this->assertClassExtends(Client::class, BufferedStompClient::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredArguments() { new BufferedStompClient('tcp://localhost:12345'); diff --git a/pkg/stomp/Tests/StompConsumerTest.php b/pkg/stomp/Tests/StompConsumerTest.php index 916166657..2d3997677 100644 --- a/pkg/stomp/Tests/StompConsumerTest.php +++ b/pkg/stomp/Tests/StompConsumerTest.php @@ -25,6 +25,9 @@ public function testShouldImplementMessageConsumerInterface() $this->assertClassImplements(Consumer::class, StompConsumer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithRequiredAttributes() { new StompConsumer($this->createStompClientMock(), $this->createDummyDestination()); diff --git a/pkg/stomp/Tests/StompContextTest.php b/pkg/stomp/Tests/StompContextTest.php index b0a41b64d..8f8fea6bb 100644 --- a/pkg/stomp/Tests/StompContextTest.php +++ b/pkg/stomp/Tests/StompContextTest.php @@ -23,11 +23,17 @@ public function testShouldImplementSessionInterface() $this->assertClassImplements(Context::class, StompContext::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeCreatedWithRequiredArguments() { new StompContext($this->createStompClientMock(), ExtensionType::RABBITMQ); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithExtChannelCallbackFactoryAsFirstArgument() { new StompContext(function () { diff --git a/pkg/wamp/Tests/Spec/JsonSerializerTest.php b/pkg/wamp/Tests/Spec/JsonSerializerTest.php index eaa3c6bc2..0b0dbae4f 100644 --- a/pkg/wamp/Tests/Spec/JsonSerializerTest.php +++ b/pkg/wamp/Tests/Spec/JsonSerializerTest.php @@ -20,6 +20,9 @@ public function testShouldImplementSerializerInterface() $this->assertClassImplements(Serializer::class, JsonSerializer::class); } + /** + * @doesNotPerformAssertions + */ public function testCouldBeConstructedWithoutAnyArguments() { new JsonSerializer(); From 4c232c2e47fb3d459ae5a1891df3fee4aaac60a2 Mon Sep 17 00:00:00 2001 From: James Read Date: Wed, 15 Jan 2025 21:17:06 +0000 Subject: [PATCH 2/2] Removing constructor tests Removing all tests that creates a new instance but doesn't call any methods nor preform any assertions. --- pkg/amqp-bunny/Tests/AmqpConsumerTest.php | 8 ------- pkg/amqp-bunny/Tests/AmqpProducerTest.php | 9 ------- .../Tests/AmqpSubscriptionConsumerTest.php | 8 ------- pkg/amqp-ext/Tests/AmqpConsumerTest.php | 8 ------- pkg/amqp-ext/Tests/AmqpContextTest.php | 18 -------------- .../Tests/AmqpSubscriptionConsumerTest.php | 8 ------- .../Tests/AsyncProcessorTest.php | 8 ------- .../PhpSerializerEventTransformerTest.php | 8 ------- .../Tests/SimpleRegistryTest.php | 8 ------- pkg/dbal/Tests/DbalConsumerTest.php | 8 ------- pkg/dbal/Tests/DbalContextTest.php | 8 ------- pkg/dbal/Tests/DbalProducerTest.php | 8 ------- .../Tests/DbalSubscriptionConsumerTest.php | 8 ------- pkg/dsn/Tests/DsnTest.php | 8 ------- .../DoctrineClearIdentityMapExtensionTest.php | 8 ------- .../DoctrinePingConnectionExtensionTest.php | 8 ------- .../Extension/ResetServicesExtensionTest.php | 7 ------ .../DependencyInjection/ConfigurationTest.php | 8 ------- .../EnqueueExtensionTest.php | 8 ------- .../Tests/Unit/EnqueueBundleTest.php | 8 ------- .../Profiler/MessageQueueCollectorTest.php | 8 ------- .../Tests/ArrayProcessorRegistryTest.php | 8 ------- .../Tests/Client/ChainExtensionTest.php | 8 ------- .../DelayRedeliveredMessageExtensionTest.php | 7 ------ .../ExclusiveCommandExtensionTest.php | 8 ------- .../FlushSpoolProducerExtensionTest.php | 8 ------- .../ConsumptionExtension/LogExtensionTest.php | 8 ------- .../SetRouterPropertiesExtensionTest.php | 8 ------- .../SetupBrokerExtensionTest.php | 8 ------- .../Tests/Client/DelegateProcessorTest.php | 8 ------- .../Tests/Client/DriverFactoryTest.php | 8 ------- .../Tests/Client/DriverPreSendTest.php | 12 ---------- .../Extension/PrepareBodyExtensionTest.php | 8 ------- pkg/enqueue/Tests/Client/PostSendTest.php | 14 ----------- pkg/enqueue/Tests/Client/PreSendTest.php | 13 ---------- pkg/enqueue/Tests/Client/ProducerTest.php | 20 ---------------- .../Tests/Client/SpoolProducerTest.php | 8 ------- .../Tests/Client/TraceableProducerTest.php | 8 ------- .../Tests/ConnectionFactoryFactoryTest.php | 8 ------- .../Consumption/CallbackProcessorTest.php | 9 ------- .../Tests/Consumption/ChainExtensionTest.php | 8 ------- ...llegalContextModificationExceptionTest.php | 8 ------- .../InvalidArgumentExceptionTest.php | 8 ------- .../Exception/LogicExceptionTest.php | 8 ------- .../LimitConsumedMessagesExtensionTest.php | 8 ------- .../LimitConsumerMemoryExtensionTest.php | 8 ------- .../LimitConsumptionTimeExtensionTest.php | 8 ------- .../Extension/LogExtensionTest.php | 8 ------- .../Extension/LoggerExtensionTest.php | 8 ------- .../Extension/NicenessExtensionTest.php | 8 ------- .../Extension/ReplyExtensionTest.php | 8 ------- .../FallbackSubscriptionConsumerTest.php | 8 ------- .../Tests/Consumption/QueueConsumerTest.php | 24 ------------------- .../RouteRecipientListProcessorTest.php | 8 ------- pkg/enqueue/Tests/Rpc/RpcClientTest.php | 8 ------- .../Symfony/Client/ConsumeCommandTest.php | 8 ------- .../AnalyzeRouteCollectionPassTest.php | 8 ------- .../BuildClientExtensionsPassTest.php | 8 ------- .../BuildCommandSubscriberRoutesPassTest.php | 8 ------- .../BuildConsumptionExtensionsPassTest.php | 8 ------- .../BuildProcessorRegistryPassTest.php | 8 ------- .../BuildProcessorRoutesPassTest.php | 8 ------- .../BuildTopicSubscriberRoutesPassTest.php | 8 ------- .../Client/FlushSpoolProducerListenerTest.php | 8 ------- .../Symfony/Client/ProduceCommandTest.php | 8 ------- .../Symfony/Client/RoutesCommandTest.php | 8 ------- .../Symfony/Client/SetupBrokerCommandTest.php | 8 ------- .../Client/SimpleConsumeCommandTest.php | 8 ------- .../Client/SimpleProduceCommandTest.php | 8 ------- .../Client/SimpleRoutesCommandTest.php | 8 ------- .../Client/SimpleSetupBrokerCommandTest.php | 8 ------- .../ConfigurableConsumeCommandTest.php | 8 ------- .../Consumption/ConsumeCommandTest.php | 8 ------- .../Consumption/SimpleConsumeCommandTest.php | 8 ------- .../ContainerProcessorRegistryTest.php | 8 ------- .../BuildConsumptionExtensionsPassTest.php | 8 ------- .../BuildProcessorRegistryPassTest.php | 8 ------- .../Tests/Symfony/LazyProducerTest.php | 8 ------- pkg/enqueue/Tests/Util/VarExportTest.php | 8 ------- pkg/fs/Tests/FsConsumerTest.php | 8 ------- pkg/fs/Tests/FsContextTest.php | 8 ------- pkg/fs/Tests/FsProducerTest.php | 8 ------- pkg/gearman/Tests/GearmanContextTest.php | 8 ------- pkg/gearman/Tests/GearmanProducerTest.php | 8 ------- .../CalculateRootJobStatusProcessorTest.php | 13 ---------- .../CalculateRootJobStatusServiceTest.php | 8 ------- .../Tests/DependentJobContextTest.php | 8 ------- .../Tests/DependentJobServiceTest.php | 8 ------- .../Tests/Doctrine/JobStorageTest.php | 8 ------- pkg/job-queue/Tests/JobProcessorTest.php | 8 ------- pkg/mongodb/Tests/MongodbConsumerTest.php | 8 ------- pkg/mongodb/Tests/MongodbContextTest.php | 8 ------- pkg/mongodb/Tests/MongodbProducerTest.php | 8 ------- .../Tests/MongodbSubscriptionConsumerTest.php | 8 ------- pkg/null/Tests/NullConnectionFactoryTest.php | 8 ------- pkg/null/Tests/NullConsumerTest.php | 8 ------- pkg/null/Tests/NullContextTest.php | 8 ------- pkg/null/Tests/NullProducerTest.php | 8 ------- pkg/null/Tests/NullQueueTest.php | 8 ------- pkg/null/Tests/NullTopicTest.php | 8 ------- .../Tests/PheanstalkConsumerTest.php | 11 --------- .../Tests/PheanstalkContextTest.php | 8 ------- .../Tests/PheanstalkProducerTest.php | 8 ------- pkg/rdkafka/Tests/JsonSerializerTest.php | 8 ------- pkg/rdkafka/Tests/RdKafkaConsumerTest.php | 13 ---------- pkg/rdkafka/Tests/RdKafkaProducerTest.php | 8 ------- pkg/redis/Tests/RedisConsumerTest.php | 8 ------- pkg/redis/Tests/RedisContextTest.php | 18 -------------- pkg/redis/Tests/RedisProducerTest.php | 8 ------- .../Tests/RedisSubscriptionConsumerTest.php | 8 ------- pkg/redis/Tests/Spec/JsonSerializerTest.php | 8 ------- pkg/sns/Tests/SnsProducerTest.php | 8 ------- pkg/snsqs/Tests/SnsQsProducerTest.php | 8 ------- pkg/sqs/Tests/SqsConsumerTest.php | 8 ------- pkg/sqs/Tests/SqsContextTest.php | 8 ------- pkg/sqs/Tests/SqsProducerTest.php | 8 ------- pkg/stomp/Tests/BufferedStompClientTest.php | 8 ------- pkg/stomp/Tests/StompConsumerTest.php | 8 ------- pkg/stomp/Tests/StompContextTest.php | 18 -------------- pkg/wamp/Tests/Spec/JsonSerializerTest.php | 8 ------- 120 files changed, 1046 deletions(-) diff --git a/pkg/amqp-bunny/Tests/AmqpConsumerTest.php b/pkg/amqp-bunny/Tests/AmqpConsumerTest.php index da6c40652..c2c694566 100644 --- a/pkg/amqp-bunny/Tests/AmqpConsumerTest.php +++ b/pkg/amqp-bunny/Tests/AmqpConsumerTest.php @@ -27,14 +27,6 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, AmqpConsumer::class); } - /** - * @doesNotPerformAssertions - */ - 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 a7ca055fd..17b390341 100644 --- a/pkg/amqp-bunny/Tests/AmqpProducerTest.php +++ b/pkg/amqp-bunny/Tests/AmqpProducerTest.php @@ -24,15 +24,6 @@ class AmqpProducerTest extends TestCase { use ClassExtensionTrait; - /** - * @doesNotPerformAssertions - */ - - 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 cc2ac9a54..4bf08ded5 100644 --- a/pkg/amqp-bunny/Tests/AmqpSubscriptionConsumerTest.php +++ b/pkg/amqp-bunny/Tests/AmqpSubscriptionConsumerTest.php @@ -17,14 +17,6 @@ public function testShouldImplementQueueInteropSubscriptionConsumerInterface() $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class)); } - /** - * @doesNotPerformAssertions - */ - 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 470c49c86..ae803e047 100644 --- a/pkg/amqp-ext/Tests/AmqpConsumerTest.php +++ b/pkg/amqp-ext/Tests/AmqpConsumerTest.php @@ -19,14 +19,6 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, AmqpConsumer::class); } - /** - * @doesNotPerformAssertions - */ - 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 4b42827ba..2b03bb3d2 100644 --- a/pkg/amqp-ext/Tests/AmqpContextTest.php +++ b/pkg/amqp-ext/Tests/AmqpContextTest.php @@ -28,24 +28,6 @@ public function testShouldImplementQueueInteropContextInterface() $this->assertClassImplements(Context::class, AmqpContext::class); } - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithExtChannelAsFirstArgument() - { - new AmqpContext($this->createExtChannelMock()); - } - - /** - * @doesNotPerformAssertions - */ - 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 8ba08161b..d71ddd776 100644 --- a/pkg/amqp-ext/Tests/AmqpSubscriptionConsumerTest.php +++ b/pkg/amqp-ext/Tests/AmqpSubscriptionConsumerTest.php @@ -17,14 +17,6 @@ public function testShouldImplementQueueInteropSubscriptionConsumerInterface() $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class)); } - /** - * @doesNotPerformAssertions - */ - 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 9bf294194..019f9bcbe 100644 --- a/pkg/async-event-dispatcher/Tests/AsyncProcessorTest.php +++ b/pkg/async-event-dispatcher/Tests/AsyncProcessorTest.php @@ -24,14 +24,6 @@ public function testShouldImplementProcessorInterface() $this->assertClassImplements(Processor::class, AsyncProcessor::class); } - /** - * @doesNotPerformAssertions - */ - 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 04d9f2dc6..498ca3ae9 100644 --- a/pkg/async-event-dispatcher/Tests/PhpSerializerEventTransformerTest.php +++ b/pkg/async-event-dispatcher/Tests/PhpSerializerEventTransformerTest.php @@ -21,14 +21,6 @@ public function testShouldImplementEventTransformerInterface() $this->assertClassImplements(EventTransformer::class, PhpSerializerEventTransformer::class); } - /** - * @doesNotPerformAssertions - */ - 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 52d6f1694..82e1e3fc7 100644 --- a/pkg/async-event-dispatcher/Tests/SimpleRegistryTest.php +++ b/pkg/async-event-dispatcher/Tests/SimpleRegistryTest.php @@ -18,14 +18,6 @@ public function testShouldImplementRegistryInterface() $this->assertClassImplements(Registry::class, SimpleRegistry::class); } - /** - * @doesNotPerformAssertions - */ - 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 7c2bbbc03..6538af11d 100644 --- a/pkg/dbal/Tests/DbalConsumerTest.php +++ b/pkg/dbal/Tests/DbalConsumerTest.php @@ -28,14 +28,6 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, DbalConsumer::class); } - /** - * @doesNotPerformAssertions - */ - 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 6c0cc2878..a1900b788 100644 --- a/pkg/dbal/Tests/DbalContextTest.php +++ b/pkg/dbal/Tests/DbalContextTest.php @@ -27,14 +27,6 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, DbalContext::class); } - /** - * @doesNotPerformAssertions - */ - 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 f7800bc56..ec4d2043c 100644 --- a/pkg/dbal/Tests/DbalProducerTest.php +++ b/pkg/dbal/Tests/DbalProducerTest.php @@ -21,14 +21,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, DbalProducer::class); } - /** - * @doesNotPerformAssertions - */ - 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 9c372de82..bacbec127 100644 --- a/pkg/dbal/Tests/DbalSubscriptionConsumerTest.php +++ b/pkg/dbal/Tests/DbalSubscriptionConsumerTest.php @@ -25,14 +25,6 @@ public function testShouldImplementSubscriptionConsumerInterface() $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class)); } - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithDbalContextAsFirstArgument() - { - new DbalSubscriptionConsumer($this->createDbalContextMock()); - } - public function testShouldAddConsumerAndCallbackToSubscribersPropertyOnSubscribe() { $subscriptionConsumer = new DbalSubscriptionConsumer($this->createDbalContextMock()); diff --git a/pkg/dsn/Tests/DsnTest.php b/pkg/dsn/Tests/DsnTest.php index f693fce04..8bf4137ed 100644 --- a/pkg/dsn/Tests/DsnTest.php +++ b/pkg/dsn/Tests/DsnTest.php @@ -8,14 +8,6 @@ class DsnTest extends TestCase { - /** - * @doesNotPerformAssertions - */ - 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 bf8c9a8b3..7c5c2dd5d 100644 --- a/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrineClearIdentityMapExtensionTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrineClearIdentityMapExtensionTest.php @@ -16,14 +16,6 @@ class DoctrineClearIdentityMapExtensionTest extends TestCase { - /** - * @doesNotPerformAssertions - */ - 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 b842b766d..36df82e52 100644 --- a/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrinePingConnectionExtensionTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/DoctrinePingConnectionExtensionTest.php @@ -17,14 +17,6 @@ class DoctrinePingConnectionExtensionTest extends TestCase { - /** - * @doesNotPerformAssertions - */ - 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 1feba7a9f..76fe29b94 100644 --- a/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/ResetServicesExtensionTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/Consumption/Extension/ResetServicesExtensionTest.php @@ -16,13 +16,6 @@ class ResetServicesExtensionTest extends TestCase { - /** - * @doesNotPerformAssertions - */ - 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 746874017..5330cde82 100644 --- a/pkg/enqueue-bundle/Tests/Unit/DependencyInjection/ConfigurationTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/DependencyInjection/ConfigurationTest.php @@ -25,14 +25,6 @@ public function testShouldBeFinal() $this->assertClassFinal(Configuration::class); } - /** - * @doesNotPerformAssertions - */ - 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 9a20f9966..6358bd24d 100644 --- a/pkg/enqueue-bundle/Tests/Unit/DependencyInjection/EnqueueExtensionTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/DependencyInjection/EnqueueExtensionTest.php @@ -30,14 +30,6 @@ public function testShouldBeFinal() $this->assertClassFinal(EnqueueExtension::class); } - /** - * @doesNotPerformAssertions - */ - 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 13776ede4..7d5b0232b 100644 --- a/pkg/enqueue-bundle/Tests/Unit/EnqueueBundleTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/EnqueueBundleTest.php @@ -15,12 +15,4 @@ public function testShouldExtendBundleClass() { $this->assertClassExtends(Bundle::class, EnqueueBundle::class); } - - /** - * @doesNotPerformAssertions - */ - 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 12700be60..d6d638d75 100644 --- a/pkg/enqueue-bundle/Tests/Unit/Profiler/MessageQueueCollectorTest.php +++ b/pkg/enqueue-bundle/Tests/Unit/Profiler/MessageQueueCollectorTest.php @@ -23,14 +23,6 @@ public function testShouldExtendDataCollectorClass() $this->assertClassExtends(DataCollector::class, MessageQueueCollector::class); } - /** - * @doesNotPerformAssertions - */ - 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 c0937547d..93ad3ece1 100644 --- a/pkg/enqueue/Tests/ArrayProcessorRegistryTest.php +++ b/pkg/enqueue/Tests/ArrayProcessorRegistryTest.php @@ -18,14 +18,6 @@ public function testShouldImplementProcessorRegistryInterface() $this->assertClassImplements(ProcessorRegistryInterface::class, ArrayProcessorRegistry::class); } - /** - * @doesNotPerformAssertions - */ - 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 c5dc60451..0f42bcf18 100644 --- a/pkg/enqueue/Tests/Client/ChainExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ChainExtensionTest.php @@ -29,14 +29,6 @@ public function testShouldBeFinal() $this->assertClassFinal(ChainExtension::class); } - /** - * @doesNotPerformAssertions - */ - 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 6d502461f..56b855f0c 100644 --- a/pkg/enqueue/Tests/Client/ConsumptionExtension/DelayRedeliveredMessageExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ConsumptionExtension/DelayRedeliveredMessageExtensionTest.php @@ -23,13 +23,6 @@ class DelayRedeliveredMessageExtensionTest extends TestCase { - /** - * @doesNotPerformAssertions - */ - 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 9a2270d4a..b1e47c898 100644 --- a/pkg/enqueue/Tests/Client/ConsumptionExtension/ExclusiveCommandExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ConsumptionExtension/ExclusiveCommandExtensionTest.php @@ -34,14 +34,6 @@ public function testShouldBeFinal() $this->assertClassFinal(ExclusiveCommandExtension::class); } - /** - * @doesNotPerformAssertions - */ - 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 d69969b78..6a782c524 100644 --- a/pkg/enqueue/Tests/Client/ConsumptionExtension/FlushSpoolProducerExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ConsumptionExtension/FlushSpoolProducerExtensionTest.php @@ -30,14 +30,6 @@ public function testShouldImplementEndExtensionInterface() $this->assertClassImplements(EndExtensionInterface::class, FlushSpoolProducerExtension::class); } - /** - * @doesNotPerformAssertions - */ - 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 fe9e6ab0c..db757676b 100644 --- a/pkg/enqueue/Tests/Client/ConsumptionExtension/LogExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ConsumptionExtension/LogExtensionTest.php @@ -55,14 +55,6 @@ public function testShouldSubClassOfLogExtension() $this->assertClassExtends(\Enqueue\Consumption\Extension\LogExtension::class, LogExtension::class); } - /** - * @doesNotPerformAssertions - */ - 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 5337d0495..d521aefca 100644 --- a/pkg/enqueue/Tests/Client/ConsumptionExtension/SetRouterPropertiesExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ConsumptionExtension/SetRouterPropertiesExtensionTest.php @@ -27,14 +27,6 @@ public function testShouldImplementMessageReceivedExtensionInterface() $this->assertClassImplements(MessageReceivedExtensionInterface::class, SetRouterPropertiesExtension::class); } - /** - * @doesNotPerformAssertions - */ - 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 a72ae389a..fbd367975 100644 --- a/pkg/enqueue/Tests/Client/ConsumptionExtension/SetupBrokerExtensionTest.php +++ b/pkg/enqueue/Tests/Client/ConsumptionExtension/SetupBrokerExtensionTest.php @@ -21,14 +21,6 @@ public function testShouldImplementStartExtensionInterface() $this->assertClassImplements(StartExtensionInterface::class, SetupBrokerExtension::class); } - /** - * @doesNotPerformAssertions - */ - 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 f9bd83ebf..9743cf4f3 100644 --- a/pkg/enqueue/Tests/Client/DelegateProcessorTest.php +++ b/pkg/enqueue/Tests/Client/DelegateProcessorTest.php @@ -13,14 +13,6 @@ class DelegateProcessorTest extends TestCase { - /** - * @doesNotPerformAssertions - */ - 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 605358aa3..ef504e0e0 100644 --- a/pkg/enqueue/Tests/Client/DriverFactoryTest.php +++ b/pkg/enqueue/Tests/Client/DriverFactoryTest.php @@ -61,14 +61,6 @@ public function testShouldBeFinal() $this->assertTrue($rc->isFinal()); } - /** - * @doesNotPerformAssertions - */ - 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 f3c1ee375..32af2a81f 100644 --- a/pkg/enqueue/Tests/Client/DriverPreSendTest.php +++ b/pkg/enqueue/Tests/Client/DriverPreSendTest.php @@ -19,18 +19,6 @@ public function testShouldBeFinal() self::assertClassFinal(DriverPreSend::class); } - /** - * @doesNotPerformAssertions - */ - 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 81b29e801..d1c260388 100644 --- a/pkg/enqueue/Tests/Client/Extension/PrepareBodyExtensionTest.php +++ b/pkg/enqueue/Tests/Client/Extension/PrepareBodyExtensionTest.php @@ -22,14 +22,6 @@ public function testShouldImplementExtensionInterface() $this->assertTrue($rc->implementsInterface(PreSendCommandExtensionInterface::class)); } - /** - * @doesNotPerformAssertions - */ - public function testCouldConstructedWithoutAnyArguments() - { - new PrepareBodyExtension(); - } - /** * @dataProvider provideMessages * diff --git a/pkg/enqueue/Tests/Client/PostSendTest.php b/pkg/enqueue/Tests/Client/PostSendTest.php index 9eb845e49..ba51710e7 100644 --- a/pkg/enqueue/Tests/Client/PostSendTest.php +++ b/pkg/enqueue/Tests/Client/PostSendTest.php @@ -21,20 +21,6 @@ public function testShouldBeFinal() self::assertClassFinal(PostSend::class); } - /** - * @doesNotPerformAssertions - */ - 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 40dffafa2..ae9400338 100644 --- a/pkg/enqueue/Tests/Client/PreSendTest.php +++ b/pkg/enqueue/Tests/Client/PreSendTest.php @@ -18,19 +18,6 @@ public function testShouldBeFinal() self::assertClassFinal(PreSend::class); } - /** - * @doesNotPerformAssertions - */ - 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 38dd1f075..9d4e2a304 100644 --- a/pkg/enqueue/Tests/Client/ProducerTest.php +++ b/pkg/enqueue/Tests/Client/ProducerTest.php @@ -24,26 +24,6 @@ public function testShouldBeFinal() self::assertClassFinal(Producer::class); } - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithRequiredArguments() - { - new Producer($this->createDriverMock(), $this->createRpcFactoryMock()); - } - - /** - * @doesNotPerformAssertions - */ - 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 511b91edb..014fe4962 100644 --- a/pkg/enqueue/Tests/Client/SpoolProducerTest.php +++ b/pkg/enqueue/Tests/Client/SpoolProducerTest.php @@ -18,14 +18,6 @@ public function testShouldImplementProducerInterface() self::assertClassImplements(ProducerInterface::class, SpoolProducer::class); } - /** - * @doesNotPerformAssertions - */ - 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 3ce23f89a..ff042857c 100644 --- a/pkg/enqueue/Tests/Client/TraceableProducerTest.php +++ b/pkg/enqueue/Tests/Client/TraceableProducerTest.php @@ -18,14 +18,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(ProducerInterface::class, TraceableProducer::class); } - /** - * @doesNotPerformAssertions - */ - 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 c522c6334..fe2de69a0 100644 --- a/pkg/enqueue/Tests/ConnectionFactoryFactoryTest.php +++ b/pkg/enqueue/Tests/ConnectionFactoryFactoryTest.php @@ -37,14 +37,6 @@ public function testShouldBeFinal() $this->assertTrue($rc->isFinal()); } - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithoutAnyArguments() - { - new ConnectionFactoryFactory(); - } - /** * @doesNotPerformAssertions */ diff --git a/pkg/enqueue/Tests/Consumption/CallbackProcessorTest.php b/pkg/enqueue/Tests/Consumption/CallbackProcessorTest.php index 9cafdb885..86adbd3a9 100644 --- a/pkg/enqueue/Tests/Consumption/CallbackProcessorTest.php +++ b/pkg/enqueue/Tests/Consumption/CallbackProcessorTest.php @@ -18,15 +18,6 @@ public function testShouldImplementProcessorInterface() $this->assertClassImplements(Processor::class, CallbackProcessor::class); } - /** - * @doesNotPerformAssertions - */ - 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 c19cc2f20..198d00012 100644 --- a/pkg/enqueue/Tests/Consumption/ChainExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/ChainExtensionTest.php @@ -33,14 +33,6 @@ public function testShouldImplementExtensionInterface() $this->assertClassImplements(ExtensionInterface::class, ChainExtension::class); } - /** - * @doesNotPerformAssertions - */ - 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 5de5c2f05..241f4adf9 100644 --- a/pkg/enqueue/Tests/Consumption/Exception/IllegalContextModificationExceptionTest.php +++ b/pkg/enqueue/Tests/Consumption/Exception/IllegalContextModificationExceptionTest.php @@ -20,12 +20,4 @@ public function testShouldExtendLogicException() { $this->assertClassExtends(\LogicException::class, IllegalContextModificationException::class); } - - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithoutAnyArguments() - { - new IllegalContextModificationException(); - } } diff --git a/pkg/enqueue/Tests/Consumption/Exception/InvalidArgumentExceptionTest.php b/pkg/enqueue/Tests/Consumption/Exception/InvalidArgumentExceptionTest.php index cc2ca3122..c1c5db362 100644 --- a/pkg/enqueue/Tests/Consumption/Exception/InvalidArgumentExceptionTest.php +++ b/pkg/enqueue/Tests/Consumption/Exception/InvalidArgumentExceptionTest.php @@ -21,14 +21,6 @@ public function testShouldExtendLogicException() $this->assertClassExtends(\LogicException::class, InvalidArgumentException::class); } - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithoutAnyArguments() - { - new InvalidArgumentException(); - } - public function testThrowIfAssertInstanceOfNotSameAsExpected() { $this->expectException(InvalidArgumentException::class); diff --git a/pkg/enqueue/Tests/Consumption/Exception/LogicExceptionTest.php b/pkg/enqueue/Tests/Consumption/Exception/LogicExceptionTest.php index f7a68c9a9..2655609ae 100644 --- a/pkg/enqueue/Tests/Consumption/Exception/LogicExceptionTest.php +++ b/pkg/enqueue/Tests/Consumption/Exception/LogicExceptionTest.php @@ -20,12 +20,4 @@ public function testShouldExtendLogicException() { $this->assertClassExtends(\LogicException::class, LogicException::class); } - - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithoutAnyArguments() - { - new LogicException(); - } } diff --git a/pkg/enqueue/Tests/Consumption/Extension/LimitConsumedMessagesExtensionTest.php b/pkg/enqueue/Tests/Consumption/Extension/LimitConsumedMessagesExtensionTest.php index 15d221409..137e30ba4 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/LimitConsumedMessagesExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/LimitConsumedMessagesExtensionTest.php @@ -16,14 +16,6 @@ class LimitConsumedMessagesExtensionTest extends TestCase { - /** - * @doesNotPerformAssertions - */ - 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 22d33826e..15482997d 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/LimitConsumerMemoryExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/LimitConsumerMemoryExtensionTest.php @@ -17,14 +17,6 @@ class LimitConsumerMemoryExtensionTest extends TestCase { - /** - * @doesNotPerformAssertions - */ - 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 fbd61d48f..fa6cb76a1 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/LimitConsumptionTimeExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/LimitConsumptionTimeExtensionTest.php @@ -17,14 +17,6 @@ class LimitConsumptionTimeExtensionTest extends TestCase { - /** - * @doesNotPerformAssertions - */ - 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 f529dcd4c..006a2c549 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/LogExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/LogExtensionTest.php @@ -49,14 +49,6 @@ public function testShouldImplementPostMessageReceivedExtensionInterface() $this->assertClassImplements(PostMessageReceivedExtensionInterface::class, LogExtension::class); } - /** - * @doesNotPerformAssertions - */ - 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 227c0830f..455468ed2 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/LoggerExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/LoggerExtensionTest.php @@ -20,14 +20,6 @@ public function testShouldImplementInitLoggerExtensionInterface() $this->assertClassImplements(InitLoggerExtensionInterface::class, LoggerExtension::class); } - /** - * @doesNotPerformAssertions - */ - 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 15c004db4..734bc8417 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/NicenessExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/NicenessExtensionTest.php @@ -11,14 +11,6 @@ class NicenessExtensionTest extends TestCase { - /** - * @doesNotPerformAssertions - */ - 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 3963e94e6..cb65816ce 100644 --- a/pkg/enqueue/Tests/Consumption/Extension/ReplyExtensionTest.php +++ b/pkg/enqueue/Tests/Consumption/Extension/ReplyExtensionTest.php @@ -25,14 +25,6 @@ public function testShouldImplementPostMessageReceivedExtensionInterface() $this->assertClassImplements(PostMessageReceivedExtensionInterface::class, ReplyExtension::class); } - /** - * @doesNotPerformAssertions - */ - 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 aecf71628..5391f0c68 100644 --- a/pkg/enqueue/Tests/Consumption/FallbackSubscriptionConsumerTest.php +++ b/pkg/enqueue/Tests/Consumption/FallbackSubscriptionConsumerTest.php @@ -21,14 +21,6 @@ public function testShouldImplementSubscriptionConsumerInterface() $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class)); } - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithoutAnyArguments() - { - new FallbackSubscriptionConsumer(); - } - public function testShouldInitSubscribersPropertyWithEmptyArray() { $subscriptionConsumer = new FallbackSubscriptionConsumer(); diff --git a/pkg/enqueue/Tests/Consumption/QueueConsumerTest.php b/pkg/enqueue/Tests/Consumption/QueueConsumerTest.php index ce72cb9d9..06edab31b 100644 --- a/pkg/enqueue/Tests/Consumption/QueueConsumerTest.php +++ b/pkg/enqueue/Tests/Consumption/QueueConsumerTest.php @@ -39,30 +39,6 @@ class QueueConsumerTest extends TestCase { use ReadAttributeTrait; - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithAllArguments() - { - new QueueConsumer($this->createContextStub(), null, [], null, 0); - } - - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithContextOnly() - { - new QueueConsumer($this->createContextStub()); - } - - /** - * @doesNotPerformAssertions - */ - 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 7e38f9616..ae878fc53 100644 --- a/pkg/enqueue/Tests/Router/RouteRecipientListProcessorTest.php +++ b/pkg/enqueue/Tests/Router/RouteRecipientListProcessorTest.php @@ -23,14 +23,6 @@ public function testShouldImplementProcessorInterface() $this->assertClassImplements(Processor::class, RouteRecipientListProcessor::class); } - /** - * @doesNotPerformAssertions - */ - 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 e8a3720b1..db4813c88 100644 --- a/pkg/enqueue/Tests/Rpc/RpcClientTest.php +++ b/pkg/enqueue/Tests/Rpc/RpcClientTest.php @@ -15,14 +15,6 @@ class RpcClientTest extends TestCase { - /** - * @doesNotPerformAssertions - */ - 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 17be30aed..3758ca96a 100644 --- a/pkg/enqueue/Tests/Symfony/Client/ConsumeCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/ConsumeCommandTest.php @@ -40,14 +40,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(ConsumeCommand::class); } - /** - * @doesNotPerformAssertions - */ - 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 fc3051b97..568de6488 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/AnalyzeRouteCollectionPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/AnalyzeRouteCollectionPassTest.php @@ -23,14 +23,6 @@ public function testShouldBeFinal() $this->assertClassFinal(AnalyzeRouteCollectionPass::class); } - /** - * @doesNotPerformAssertions - */ - 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 baae3c768..753790369 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildClientExtensionsPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildClientExtensionsPassTest.php @@ -25,14 +25,6 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildClientExtensionsPass::class); } - /** - * @doesNotPerformAssertions - */ - 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 7317a8ef8..1a5f65a4f 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildCommandSubscriberRoutesPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildCommandSubscriberRoutesPassTest.php @@ -29,14 +29,6 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildCommandSubscriberRoutesPass::class); } - /** - * @doesNotPerformAssertions - */ - 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 64966d602..c2975051b 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildConsumptionExtensionsPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildConsumptionExtensionsPassTest.php @@ -25,14 +25,6 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildConsumptionExtensionsPass::class); } - /** - * @doesNotPerformAssertions - */ - 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 98679abbd..5c9ac4840 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRegistryPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRegistryPassTest.php @@ -26,14 +26,6 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildProcessorRegistryPass::class); } - /** - * @doesNotPerformAssertions - */ - 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 d4ac29803..0351c45f5 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRoutesPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildProcessorRoutesPassTest.php @@ -25,14 +25,6 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildProcessorRoutesPass::class); } - /** - * @doesNotPerformAssertions - */ - 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 0ada6064a..b642b9001 100644 --- a/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildTopicSubscriberRoutesPassTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/DependencyInjection/BuildTopicSubscriberRoutesPassTest.php @@ -29,14 +29,6 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildTopicSubscriberRoutesPass::class); } - /** - * @doesNotPerformAssertions - */ - 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 feae3b52f..539d332ee 100644 --- a/pkg/enqueue/Tests/Symfony/Client/FlushSpoolProducerListenerTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/FlushSpoolProducerListenerTest.php @@ -39,14 +39,6 @@ public function testShouldSubscribeOnConsoleTerminateEvent() $this->assertEquals('flushMessages', $events[ConsoleEvents::TERMINATE]); } - /** - * @doesNotPerformAssertions - */ - 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 b6a1d18a4..daa909175 100644 --- a/pkg/enqueue/Tests/Symfony/Client/ProduceCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/ProduceCommandTest.php @@ -27,14 +27,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(ProduceCommand::class); } - /** - * @doesNotPerformAssertions - */ - 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 8adf71064..89bd7f745 100644 --- a/pkg/enqueue/Tests/Symfony/Client/RoutesCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/RoutesCommandTest.php @@ -29,14 +29,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(RoutesCommand::class); } - /** - * @doesNotPerformAssertions - */ - 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 53a8d4494..c81c4e1b6 100644 --- a/pkg/enqueue/Tests/Symfony/Client/SetupBrokerCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/SetupBrokerCommandTest.php @@ -26,14 +26,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(SetupBrokerCommand::class); } - /** - * @doesNotPerformAssertions - */ - 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 97f1c6c58..21c491eb5 100644 --- a/pkg/enqueue/Tests/Symfony/Client/SimpleConsumeCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/SimpleConsumeCommandTest.php @@ -29,14 +29,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(SimpleConsumeCommand::class); } - /** - * @doesNotPerformAssertions - */ - 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 50d46c96d..3ff81bfd5 100644 --- a/pkg/enqueue/Tests/Symfony/Client/SimpleProduceCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/SimpleProduceCommandTest.php @@ -23,14 +23,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(SimpleProduceCommand::class); } - /** - * @doesNotPerformAssertions - */ - 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 71bb5cf6b..20ee454cc 100644 --- a/pkg/enqueue/Tests/Symfony/Client/SimpleRoutesCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/SimpleRoutesCommandTest.php @@ -25,14 +25,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(SimpleRoutesCommand::class); } - /** - * @doesNotPerformAssertions - */ - 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 d3e8f2170..3702dbf18 100644 --- a/pkg/enqueue/Tests/Symfony/Client/SimpleSetupBrokerCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Client/SimpleSetupBrokerCommandTest.php @@ -23,14 +23,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(SimpleSetupBrokerCommand::class); } - /** - * @doesNotPerformAssertions - */ - 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 82342e4d6..251e264e2 100644 --- a/pkg/enqueue/Tests/Symfony/Consumption/ConfigurableConsumeCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Consumption/ConfigurableConsumeCommandTest.php @@ -33,14 +33,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(ConfigurableConsumeCommand::class); } - /** - * @doesNotPerformAssertions - */ - 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 b56ef68cc..f07bef03b 100644 --- a/pkg/enqueue/Tests/Symfony/Consumption/ConsumeCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Consumption/ConsumeCommandTest.php @@ -35,14 +35,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(ConsumeCommand::class); } - /** - * @doesNotPerformAssertions - */ - 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 7648587d6..eeb38bf19 100644 --- a/pkg/enqueue/Tests/Symfony/Consumption/SimpleConsumeCommandTest.php +++ b/pkg/enqueue/Tests/Symfony/Consumption/SimpleConsumeCommandTest.php @@ -24,14 +24,6 @@ public function testShouldNotBeFinal() $this->assertClassNotFinal(SimpleConsumeCommand::class); } - /** - * @doesNotPerformAssertions - */ - 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 1370b4df6..5504e8ef6 100644 --- a/pkg/enqueue/Tests/Symfony/ContainerProcessorRegistryTest.php +++ b/pkg/enqueue/Tests/Symfony/ContainerProcessorRegistryTest.php @@ -23,14 +23,6 @@ public function testShouldBeFinal() $this->assertClassFinal(ContainerProcessorRegistry::class); } - /** - * @doesNotPerformAssertions - */ - 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 c558aaddb..bdccd338c 100644 --- a/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildConsumptionExtensionsPassTest.php +++ b/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildConsumptionExtensionsPassTest.php @@ -25,14 +25,6 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildConsumptionExtensionsPass::class); } - /** - * @doesNotPerformAssertions - */ - 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 e3a333055..134c216dc 100644 --- a/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildProcessorRegistryPassTest.php +++ b/pkg/enqueue/Tests/Symfony/DependencyInjection/BuildProcessorRegistryPassTest.php @@ -26,14 +26,6 @@ public function testShouldBeFinal() $this->assertClassFinal(BuildProcessorRegistryPass::class); } - /** - * @doesNotPerformAssertions - */ - 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 7fbe714ac..c8ba596a8 100644 --- a/pkg/enqueue/Tests/Symfony/LazyProducerTest.php +++ b/pkg/enqueue/Tests/Symfony/LazyProducerTest.php @@ -20,14 +20,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(ProducerInterface::class, LazyProducer::class); } - /** - * @doesNotPerformAssertions - */ - 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 a8fa92f20..e8034a1c7 100644 --- a/pkg/enqueue/Tests/Util/VarExportTest.php +++ b/pkg/enqueue/Tests/Util/VarExportTest.php @@ -7,14 +7,6 @@ class VarExportTest extends TestCase { - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithValueAsArgument() - { - new VarExport('aVal'); - } - /** * @dataProvider provideValues * diff --git a/pkg/fs/Tests/FsConsumerTest.php b/pkg/fs/Tests/FsConsumerTest.php index 044d05ae2..67f03ae98 100644 --- a/pkg/fs/Tests/FsConsumerTest.php +++ b/pkg/fs/Tests/FsConsumerTest.php @@ -20,14 +20,6 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, FsConsumer::class); } - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithContextAndDestinationAndPreFetchCountAsArguments() - { - new FsConsumer($this->createContextMock(), new FsDestination(TempFile::generate()), 1); - } - public function testShouldReturnDestinationSetInConstructorOnGetQueue() { $destination = new FsDestination(TempFile::generate()); diff --git a/pkg/fs/Tests/FsContextTest.php b/pkg/fs/Tests/FsContextTest.php index ed1aec87b..9d5a5f1fc 100644 --- a/pkg/fs/Tests/FsContextTest.php +++ b/pkg/fs/Tests/FsContextTest.php @@ -24,14 +24,6 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, FsContext::class); } - /** - * @doesNotPerformAssertions - */ - 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); diff --git a/pkg/fs/Tests/FsProducerTest.php b/pkg/fs/Tests/FsProducerTest.php index 1aa25fa3e..266854c7b 100644 --- a/pkg/fs/Tests/FsProducerTest.php +++ b/pkg/fs/Tests/FsProducerTest.php @@ -23,14 +23,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, FsProducer::class); } - /** - * @doesNotPerformAssertions - */ - 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 deafe303b..8a36ad80b 100644 --- a/pkg/gearman/Tests/GearmanContextTest.php +++ b/pkg/gearman/Tests/GearmanContextTest.php @@ -24,14 +24,6 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, GearmanContext::class); } - /** - * @doesNotPerformAssertions - */ - 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 537a0550b..2a7baa4de 100644 --- a/pkg/gearman/Tests/GearmanProducerTest.php +++ b/pkg/gearman/Tests/GearmanProducerTest.php @@ -18,14 +18,6 @@ class GearmanProducerTest extends TestCase use ClassExtensionTrait; use SkipIfGearmanExtensionIsNotInstalledTrait; - /** - * @doesNotPerformAssertions - */ - 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 edb489474..8509f0544 100644 --- a/pkg/job-queue/Tests/CalculateRootJobStatusProcessorTest.php +++ b/pkg/job-queue/Tests/CalculateRootJobStatusProcessorTest.php @@ -17,19 +17,6 @@ class CalculateRootJobStatusProcessorTest extends \PHPUnit\Framework\TestCase { - /** - * @doesNotPerformAssertions - */ - 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 05badc3e7..695c08e6e 100644 --- a/pkg/job-queue/Tests/CalculateRootJobStatusServiceTest.php +++ b/pkg/job-queue/Tests/CalculateRootJobStatusServiceTest.php @@ -9,14 +9,6 @@ class CalculateRootJobStatusServiceTest extends \PHPUnit\Framework\TestCase { - /** - * @doesNotPerformAssertions - */ - 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 7e517e705..35942a974 100644 --- a/pkg/job-queue/Tests/DependentJobContextTest.php +++ b/pkg/job-queue/Tests/DependentJobContextTest.php @@ -7,14 +7,6 @@ class DependentJobContextTest extends \PHPUnit\Framework\TestCase { - /** - * @doesNotPerformAssertions - */ - 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 a524ab255..e8809b4b0 100644 --- a/pkg/job-queue/Tests/DependentJobServiceTest.php +++ b/pkg/job-queue/Tests/DependentJobServiceTest.php @@ -10,14 +10,6 @@ class DependentJobServiceTest extends \PHPUnit\Framework\TestCase { - /** - * @doesNotPerformAssertions - */ - 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 e37a2a599..73f130d52 100644 --- a/pkg/job-queue/Tests/Doctrine/JobStorageTest.php +++ b/pkg/job-queue/Tests/Doctrine/JobStorageTest.php @@ -15,14 +15,6 @@ class JobStorageTest extends \PHPUnit\Framework\TestCase { - /** - * @doesNotPerformAssertions - */ - 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 882678bf2..9f1c7b2fd 100644 --- a/pkg/job-queue/Tests/JobProcessorTest.php +++ b/pkg/job-queue/Tests/JobProcessorTest.php @@ -13,14 +13,6 @@ class JobProcessorTest extends TestCase { - /** - * @doesNotPerformAssertions - */ - 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 35dfba377..c6a642500 100644 --- a/pkg/mongodb/Tests/MongodbConsumerTest.php +++ b/pkg/mongodb/Tests/MongodbConsumerTest.php @@ -25,14 +25,6 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, MongodbConsumer::class); } - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithRequiredArguments() - { - new MongodbConsumer($this->createContextMock(), new MongodbDestination('queue')); - } - public function testShouldReturnInstanceOfDestination() { $destination = new MongodbDestination('queue'); diff --git a/pkg/mongodb/Tests/MongodbContextTest.php b/pkg/mongodb/Tests/MongodbContextTest.php index 830a6dd55..8cdef79ff 100644 --- a/pkg/mongodb/Tests/MongodbContextTest.php +++ b/pkg/mongodb/Tests/MongodbContextTest.php @@ -29,14 +29,6 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, MongodbContext::class); } - /** - * @doesNotPerformAssertions - */ - 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 6b8b1b2e8..6987b1a76 100644 --- a/pkg/mongodb/Tests/MongodbProducerTest.php +++ b/pkg/mongodb/Tests/MongodbProducerTest.php @@ -23,14 +23,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, MongodbProducer::class); } - /** - * @doesNotPerformAssertions - */ - 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 5ff1687d8..d982e0418 100644 --- a/pkg/mongodb/Tests/MongodbSubscriptionConsumerTest.php +++ b/pkg/mongodb/Tests/MongodbSubscriptionConsumerTest.php @@ -24,14 +24,6 @@ public function testShouldImplementSubscriptionConsumerInterface() $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class)); } - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithMongodbContextAsFirstArgument() - { - new MongodbSubscriptionConsumer($this->createMongodbContextMock()); - } - public function testShouldAddConsumerAndCallbackToSubscribersPropertyOnSubscribe() { $subscriptionConsumer = new MongodbSubscriptionConsumer($this->createMongodbContextMock()); diff --git a/pkg/null/Tests/NullConnectionFactoryTest.php b/pkg/null/Tests/NullConnectionFactoryTest.php index 8c2599591..bbf377e85 100644 --- a/pkg/null/Tests/NullConnectionFactoryTest.php +++ b/pkg/null/Tests/NullConnectionFactoryTest.php @@ -17,14 +17,6 @@ public function testShouldImplementConnectionFactoryInterface() $this->assertClassImplements(ConnectionFactory::class, NullConnectionFactory::class); } - /** - * @doesNotPerformAssertions - */ - 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 5db15ae95..f4de53311 100644 --- a/pkg/null/Tests/NullConsumerTest.php +++ b/pkg/null/Tests/NullConsumerTest.php @@ -18,14 +18,6 @@ public function testShouldImplementMessageConsumerInterface() $this->assertClassImplements(Consumer::class, NullConsumer::class); } - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithQueueAsArgument() - { - new NullConsumer(new NullQueue('aName')); - } - public function testShouldAlwaysReturnNullOnReceive() { $consumer = new NullConsumer(new NullQueue('theQueueName')); diff --git a/pkg/null/Tests/NullContextTest.php b/pkg/null/Tests/NullContextTest.php index 257eee84b..f0da566d2 100644 --- a/pkg/null/Tests/NullContextTest.php +++ b/pkg/null/Tests/NullContextTest.php @@ -21,14 +21,6 @@ public function testShouldImplementSessionInterface() $this->assertClassImplements(Context::class, NullContext::class); } - /** - * @doesNotPerformAssertions - */ - 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 149310498..140d683ba 100644 --- a/pkg/null/Tests/NullProducerTest.php +++ b/pkg/null/Tests/NullProducerTest.php @@ -18,14 +18,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, NullProducer::class); } - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithoutAnyArguments() - { - new NullProducer(); - } - /** * @doesNotPerformAssertions */ diff --git a/pkg/null/Tests/NullQueueTest.php b/pkg/null/Tests/NullQueueTest.php index cace66439..cb29ca180 100644 --- a/pkg/null/Tests/NullQueueTest.php +++ b/pkg/null/Tests/NullQueueTest.php @@ -16,14 +16,6 @@ public function testShouldImplementQueueInterface() $this->assertClassImplements(Queue::class, NullQueue::class); } - /** - * @doesNotPerformAssertions - */ - 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 017df3236..27c4b58de 100644 --- a/pkg/null/Tests/NullTopicTest.php +++ b/pkg/null/Tests/NullTopicTest.php @@ -16,14 +16,6 @@ public function testShouldImplementTopicInterface() $this->assertClassImplements(Topic::class, NullTopic::class); } - /** - * @doesNotPerformAssertions - */ - 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 b78f3be23..944ac9b81 100644 --- a/pkg/pheanstalk/Tests/PheanstalkConsumerTest.php +++ b/pkg/pheanstalk/Tests/PheanstalkConsumerTest.php @@ -14,17 +14,6 @@ class PheanstalkConsumerTest extends TestCase { use ClassExtensionTrait; - /** - * @doesNotPerformAssertions - */ - 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 e122872c3..3b7bfbeb7 100644 --- a/pkg/pheanstalk/Tests/PheanstalkContextTest.php +++ b/pkg/pheanstalk/Tests/PheanstalkContextTest.php @@ -21,14 +21,6 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, PheanstalkContext::class); } - /** - * @doesNotPerformAssertions - */ - 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 b6d6dff03..b9a69176c 100644 --- a/pkg/pheanstalk/Tests/PheanstalkProducerTest.php +++ b/pkg/pheanstalk/Tests/PheanstalkProducerTest.php @@ -18,14 +18,6 @@ class PheanstalkProducerTest extends TestCase { use ClassExtensionTrait; - /** - * @doesNotPerformAssertions - */ - 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 32805ca16..6c9bbef84 100644 --- a/pkg/rdkafka/Tests/JsonSerializerTest.php +++ b/pkg/rdkafka/Tests/JsonSerializerTest.php @@ -17,14 +17,6 @@ public function testShouldImplementSerializerInterface() $this->assertClassImplements(Serializer::class, JsonSerializer::class); } - /** - * @doesNotPerformAssertions - */ - 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 471e31a91..74e92954b 100644 --- a/pkg/rdkafka/Tests/RdKafkaConsumerTest.php +++ b/pkg/rdkafka/Tests/RdKafkaConsumerTest.php @@ -13,19 +13,6 @@ class RdKafkaConsumerTest extends TestCase { - /** - * @doesNotPerformAssertions - */ - 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 25451f5b4..6082968ad 100644 --- a/pkg/rdkafka/Tests/RdKafkaProducerTest.php +++ b/pkg/rdkafka/Tests/RdKafkaProducerTest.php @@ -17,14 +17,6 @@ class RdKafkaProducerTest extends TestCase { - /** - * @doesNotPerformAssertions - */ - 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 659060fa8..56373c18a 100644 --- a/pkg/redis/Tests/RedisConsumerTest.php +++ b/pkg/redis/Tests/RedisConsumerTest.php @@ -22,14 +22,6 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, RedisConsumer::class); } - /** - * @doesNotPerformAssertions - */ - 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 597b71554..6395e954e 100644 --- a/pkg/redis/Tests/RedisContextTest.php +++ b/pkg/redis/Tests/RedisContextTest.php @@ -25,24 +25,6 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, RedisContext::class); } - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithRedisAsFirstArgument() - { - new RedisContext($this->createRedisMock(), 300); - } - - /** - * @doesNotPerformAssertions - */ - 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 345c632b5..40e03bae2 100644 --- a/pkg/redis/Tests/RedisProducerTest.php +++ b/pkg/redis/Tests/RedisProducerTest.php @@ -25,14 +25,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, RedisProducer::class); } - /** - * @doesNotPerformAssertions - */ - 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 f27cb5155..8d00fcc14 100644 --- a/pkg/redis/Tests/RedisSubscriptionConsumerTest.php +++ b/pkg/redis/Tests/RedisSubscriptionConsumerTest.php @@ -22,14 +22,6 @@ public function testShouldImplementSubscriptionConsumerInterface() $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class)); } - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithRedisContextAsFirstArgument() - { - new RedisSubscriptionConsumer($this->createRedisContextMock()); - } - public function testShouldAddConsumerAndCallbackToSubscribersPropertyOnSubscribe() { $subscriptionConsumer = new RedisSubscriptionConsumer($this->createRedisContextMock()); diff --git a/pkg/redis/Tests/Spec/JsonSerializerTest.php b/pkg/redis/Tests/Spec/JsonSerializerTest.php index 21bc8d836..78a0aed86 100644 --- a/pkg/redis/Tests/Spec/JsonSerializerTest.php +++ b/pkg/redis/Tests/Spec/JsonSerializerTest.php @@ -20,14 +20,6 @@ public function testShouldImplementSerializerInterface() $this->assertClassImplements(Serializer::class, JsonSerializer::class); } - /** - * @doesNotPerformAssertions - */ - 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 7001784dc..1c6be7f85 100644 --- a/pkg/sns/Tests/SnsProducerTest.php +++ b/pkg/sns/Tests/SnsProducerTest.php @@ -27,14 +27,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, SnsProducer::class); } - /** - * @doesNotPerformAssertions - */ - 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 033bfdfd4..59798dc11 100644 --- a/pkg/snsqs/Tests/SnsQsProducerTest.php +++ b/pkg/snsqs/Tests/SnsQsProducerTest.php @@ -32,14 +32,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, SnsQsProducer::class); } - /** - * @doesNotPerformAssertions - */ - 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 b05ce246d..ef06c6157 100644 --- a/pkg/sqs/Tests/SqsConsumerTest.php +++ b/pkg/sqs/Tests/SqsConsumerTest.php @@ -24,14 +24,6 @@ public function testShouldImplementConsumerInterface() $this->assertClassImplements(Consumer::class, SqsConsumer::class); } - /** - * @doesNotPerformAssertions - */ - 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 bfa2163bf..5081add41 100644 --- a/pkg/sqs/Tests/SqsContextTest.php +++ b/pkg/sqs/Tests/SqsContextTest.php @@ -25,14 +25,6 @@ public function testShouldImplementContextInterface() $this->assertClassImplements(Context::class, SqsContext::class); } - /** - * @doesNotPerformAssertions - */ - 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 1db3632d0..35cb9850b 100644 --- a/pkg/sqs/Tests/SqsProducerTest.php +++ b/pkg/sqs/Tests/SqsProducerTest.php @@ -24,14 +24,6 @@ public function testShouldImplementProducerInterface() $this->assertClassImplements(Producer::class, SqsProducer::class); } - /** - * @doesNotPerformAssertions - */ - 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 3a2183b48..e4b6226e1 100644 --- a/pkg/stomp/Tests/BufferedStompClientTest.php +++ b/pkg/stomp/Tests/BufferedStompClientTest.php @@ -21,14 +21,6 @@ public function testShouldExtendLibStompClient() $this->assertClassExtends(Client::class, BufferedStompClient::class); } - /** - * @doesNotPerformAssertions - */ - 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 2d3997677..d461284c9 100644 --- a/pkg/stomp/Tests/StompConsumerTest.php +++ b/pkg/stomp/Tests/StompConsumerTest.php @@ -25,14 +25,6 @@ public function testShouldImplementMessageConsumerInterface() $this->assertClassImplements(Consumer::class, StompConsumer::class); } - /** - * @doesNotPerformAssertions - */ - 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 8f8fea6bb..cfb9245dc 100644 --- a/pkg/stomp/Tests/StompContextTest.php +++ b/pkg/stomp/Tests/StompContextTest.php @@ -23,24 +23,6 @@ public function testShouldImplementSessionInterface() $this->assertClassImplements(Context::class, StompContext::class); } - /** - * @doesNotPerformAssertions - */ - public function testCouldBeCreatedWithRequiredArguments() - { - new StompContext($this->createStompClientMock(), ExtensionType::RABBITMQ); - } - - /** - * @doesNotPerformAssertions - */ - 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 0b0dbae4f..f1c3e24d3 100644 --- a/pkg/wamp/Tests/Spec/JsonSerializerTest.php +++ b/pkg/wamp/Tests/Spec/JsonSerializerTest.php @@ -20,14 +20,6 @@ public function testShouldImplementSerializerInterface() $this->assertClassImplements(Serializer::class, JsonSerializer::class); } - /** - * @doesNotPerformAssertions - */ - public function testCouldBeConstructedWithoutAnyArguments() - { - new JsonSerializer(); - } - public function testShouldConvertMessageToJsonString() { $serializer = new JsonSerializer();