Skip to content

Commit 3ef3206

Browse files
committed
Removing constructor tests
Removing all tests that creates a new instance but doesn't call any methods nor preform any assertions.
1 parent 82d23fb commit 3ef3206

File tree

58 files changed

+0
-507
lines changed

Some content is hidden

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

58 files changed

+0
-507
lines changed

Tests/ArrayProcessorRegistryTest.php

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -18,14 +18,6 @@ public function testShouldImplementProcessorRegistryInterface()
1818
$this->assertClassImplements(ProcessorRegistryInterface::class, ArrayProcessorRegistry::class);
1919
}
2020

21-
/**
22-
* @doesNotPerformAssertions
23-
*/
24-
public function testCouldBeConstructedWithoutAnyArgument()
25-
{
26-
new ArrayProcessorRegistry();
27-
}
28-
2921
public function testShouldThrowExceptionIfProcessorIsNotSet()
3022
{
3123
$registry = new ArrayProcessorRegistry();

Tests/Client/ChainExtensionTest.php

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -29,14 +29,6 @@ public function testShouldBeFinal()
2929
$this->assertClassFinal(ChainExtension::class);
3030
}
3131

32-
/**
33-
* @doesNotPerformAssertions
34-
*/
35-
public function testCouldBeConstructedWithExtensionsArray()
36-
{
37-
new ChainExtension([$this->createExtension(), $this->createExtension()]);
38-
}
39-
4032
public function testThrowIfArrayContainsNotExtension()
4133
{
4234
$this->expectException(\LogicException::class);

Tests/Client/ConsumptionExtension/DelayRedeliveredMessageExtensionTest.php

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -23,13 +23,6 @@
2323

2424
class DelayRedeliveredMessageExtensionTest extends TestCase
2525
{
26-
/**
27-
* @doesNotPerformAssertions
28-
*/
29-
public function testCouldBeConstructedWithRequiredArguments()
30-
{
31-
new DelayRedeliveredMessageExtension($this->createDriverMock(), 12345);
32-
}
3326

3427
public function testShouldSendDelayedMessageAndRejectOriginalMessage()
3528
{

Tests/Client/ConsumptionExtension/ExclusiveCommandExtensionTest.php

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -34,14 +34,6 @@ public function testShouldBeFinal()
3434
$this->assertClassFinal(ExclusiveCommandExtension::class);
3535
}
3636

37-
/**
38-
* @doesNotPerformAssertions
39-
*/
40-
public function testCouldBeConstructedWithDriverAsFirstArgument()
41-
{
42-
new ExclusiveCommandExtension($this->createDriverStub());
43-
}
44-
4537
public function testShouldDoNothingIfMessageHasTopicPropertySetOnPreReceive()
4638
{
4739
$message = new NullMessage();

Tests/Client/ConsumptionExtension/FlushSpoolProducerExtensionTest.php

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -30,14 +30,6 @@ public function testShouldImplementEndExtensionInterface()
3030
$this->assertClassImplements(EndExtensionInterface::class, FlushSpoolProducerExtension::class);
3131
}
3232

33-
/**
34-
* @doesNotPerformAssertions
35-
*/
36-
public function testCouldBeConstructedWithSpoolProducerAsFirstArgument()
37-
{
38-
new FlushSpoolProducerExtension($this->createSpoolProducerMock());
39-
}
40-
4133
public function testShouldFlushSpoolProducerOnEnd()
4234
{
4335
$producer = $this->createSpoolProducerMock();

Tests/Client/ConsumptionExtension/LogExtensionTest.php

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -55,14 +55,6 @@ public function testShouldSubClassOfLogExtension()
5555
$this->assertClassExtends(\Enqueue\Consumption\Extension\LogExtension::class, LogExtension::class);
5656
}
5757

58-
/**
59-
* @doesNotPerformAssertions
60-
*/
61-
public function testCouldBeConstructedWithoutAnyArguments()
62-
{
63-
new LogExtension();
64-
}
65-
6658
public function testShouldLogStartOnStart()
6759
{
6860
$logger = $this->createLogger();

Tests/Client/ConsumptionExtension/SetRouterPropertiesExtensionTest.php

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -27,14 +27,6 @@ public function testShouldImplementMessageReceivedExtensionInterface()
2727
$this->assertClassImplements(MessageReceivedExtensionInterface::class, SetRouterPropertiesExtension::class);
2828
}
2929

30-
/**
31-
* @doesNotPerformAssertions
32-
*/
33-
public function testCouldBeConstructedWithRequiredArguments()
34-
{
35-
new SetRouterPropertiesExtension($this->createDriverMock());
36-
}
37-
3830
public function testShouldSetRouterProcessorPropertyIfNotSetAndOnRouterQueue()
3931
{
4032
$config = Config::create('test', '.', '', '', 'router-queue', '', 'router-processor-name');

Tests/Client/ConsumptionExtension/SetupBrokerExtensionTest.php

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -21,14 +21,6 @@ public function testShouldImplementStartExtensionInterface()
2121
$this->assertClassImplements(StartExtensionInterface::class, SetupBrokerExtension::class);
2222
}
2323

24-
/**
25-
* @doesNotPerformAssertions
26-
*/
27-
public function testCouldBeConstructedWithRequiredArguments()
28-
{
29-
new SetupBrokerExtension($this->createDriverMock());
30-
}
31-
3224
public function testShouldSetupBroker()
3325
{
3426
$logger = new NullLogger();

Tests/Client/DelegateProcessorTest.php

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -13,14 +13,6 @@
1313

1414
class DelegateProcessorTest extends TestCase
1515
{
16-
/**
17-
* @doesNotPerformAssertions
18-
*/
19-
public function testCouldBeConstructedWithRequiredArguments()
20-
{
21-
new DelegateProcessor($this->createProcessorRegistryMock());
22-
}
23-
2416
public function testShouldThrowExceptionIfProcessorNameIsNotSet()
2517
{
2618
$this->expectException(\LogicException::class);

Tests/Client/DriverFactoryTest.php

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -61,14 +61,6 @@ public function testShouldBeFinal()
6161
$this->assertTrue($rc->isFinal());
6262
}
6363

64-
/**
65-
* @doesNotPerformAssertions
66-
*/
67-
public function testCouldBeConstructedWithoutAnyArguments()
68-
{
69-
new DriverFactory();
70-
}
71-
7264
public function testThrowIfPackageThatSupportSchemeNotInstalled()
7365
{
7466
$scheme = 'scheme5b7aa7d7cd213';

0 commit comments

Comments
 (0)