Improve PHPUnit fixtures

tests/Devices/CachedCollectionTest.php

@@ -21,15 +21,15 @@ class CachedCollectionTest extends TestCase
     /** @var CachedCollection */
     private $cachedCollection;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         $this->collection = Mockery::mock(CollectionInterface::class);
         $this->cache = Mockery::mock(CacheInterface::class);
         $this->cachedCollection = new CachedCollection($this->collection, $this->cache);
     }
 
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         Mockery::close();
     }

tests/Devices/CollectionTest.php

@@ -17,14 +17,14 @@ class CollectionTest extends TestCase
     /** @var Collection */
     private $collection;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         $this->factory = Mockery::mock(FactoryInterface::class);
         $this->collection = new Collection($this->factory);
     }
 
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         Mockery::close();
     }

tests/Devices/DiscoveryTest.php

@@ -21,7 +21,7 @@ class DiscoveryTest extends TestCase
     /** @var Intruder */
     private $discovery;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         $this->collection = Mockery::mock(CollectionInterface::class);
         $this->collection->shouldReceive("getLogger")->with()->andReturn(new NullLogger());
@@ -31,7 +31,7 @@ public function setUp(): void
     }
 
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         Mockery::close();
     }

tests/Devices/FactoryTest.php

@@ -11,7 +11,7 @@
 
 class FactoryTest extends TestCase
 {
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         Mockery::close();
     }

tests/LiveTest.php

@@ -13,7 +13,7 @@ abstract class LiveTest extends TestCase
     /** @var NetworkInterface */
     protected $network;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         $this->network = new Network();
 

tests/MockTest.php

@@ -16,12 +16,12 @@ abstract class MockTest extends TestCase
     /** @var NetworkInterface&MockInterface */
     protected $network;
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         Mockery::close();
     }
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         $this->network = Mockery::mock(NetworkInterface::class);
 

tests/NetworkTest.php

@@ -20,14 +20,14 @@ class NetworkTest extends MockTest
     private $collection;
 
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         $this->collection = Mockery::mock(CollectionInterface::class);
         $this->network = new Network($this->collection);
     }
 
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         Mockery::close();
     }

tests/PlaylistLiveTest.php

@@ -14,7 +14,7 @@ class PlaylistLiveTest extends LiveTest
     /** @var string */
     protected $playlistName = "phpunit-test";
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 
@@ -26,7 +26,7 @@ public function setUp(): void
     }
 
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         $this->playlist->delete();
     }

tests/QueueLiveTest.php

@@ -18,7 +18,7 @@ class QueueLiveTest extends LiveTest
     /** @var ControllerStateInterface */
     protected $state;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 
@@ -35,7 +35,7 @@ public function setUp(): void
     }
 
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         if ($this->controller) {
             $this->controller->restoreState($this->state);

tests/QueueTest.php

@@ -22,7 +22,7 @@ class QueueTest extends MockTest
     /** @var Queue */
     protected $queue;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 
@@ -31,7 +31,7 @@ public function setUp(): void
         $this->queue = new Queue($this->controller);
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         Mockery::close();
     }

tests/Services/RadioTest.php

@@ -19,7 +19,7 @@ class RadioTest extends MockTest
     private $radio;
 
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 

tests/SpeakerLiveTest.php

@@ -13,7 +13,7 @@ class SpeakerLiveTest extends LiveTest
     /** @var SpeakerInterface */
     protected $speaker;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
         $speakers = $this->network->getSpeakers();

tests/SpeakerTest.php

@@ -16,15 +16,15 @@ class SpeakerTest extends MockTest
     /** @var Speaker */
     protected $speaker;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 
         $this->device = $this->getDevice();
         $this->speaker = $this->getSpeaker($this->device);
     }
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         Mockery::close();
     }

tests/StateTest.php

@@ -10,7 +10,7 @@
 
 class StateTest extends TrackTest
 {
-    public function setUp(): void
+    protected function setUp(): void
     {
         $controller = Mockery::mock(ControllerInterface::class);
         $controller->shouldReceive("getIp")->andReturn("192.168.0.66");

tests/Tracks/FactoryTest.php

@@ -19,7 +19,7 @@ class FactoryTest extends MockTest
     /** @var Factory */
     private $factory;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 

tests/Tracks/TrackTest.php

@@ -40,7 +40,7 @@ class TrackTest extends TestCase
     /** @var TrackInterface */
     protected $track2;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         $controller = Mockery::mock(ControllerInterface::class);
         $controller->shouldReceive("getIp")->with()->andReturn("192.168.0.66");
@@ -53,7 +53,7 @@ public function setUp(): void
     }
 
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         Mockery::close();
     }

tests/Utils/DirectoryTest.php

@@ -15,13 +15,13 @@ class DirectoryTest extends TestCase
     private $filesystem;
 
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         $this->filesystem = Mockery::mock(FilesystemInterface::class);
     }
 
 
-    public function tearDown(): void
+    protected function tearDown(): void
     {
         Mockery::close();
     }