Apply fixes from StyleCI (koel#799)

PCanyi · Aug 31, 2018 · 40d4174 · 40d4174
1 parent d88dd79
commit 40d4174
Show file tree

Hide file tree

Showing 7 changed files with 11 additions and 10 deletions.
diff --git a/app/Services/ApiClient.php b/app/Services/ApiClient.php
@@ -4,8 +4,8 @@
 
 use GuzzleHttp\Client;
 use GuzzleHttp\Exception\ClientException;
-use Illuminate\Log\Logger;
 use Illuminate\Contracts\Cache\Repository as Cache;
+use Illuminate\Log\Logger;
 use InvalidArgumentException;
 use SimpleXMLElement;
 
@@ -71,7 +71,7 @@ public function request(string $method, string $uri, bool $appendKey = true, arr
         } catch (ClientException $e) {
             $this->logger->error($e);
 
-            return null;
+            return;
         }
     }
 

diff --git a/app/Services/ApiConsumerInterface.php b/app/Services/ApiConsumerInterface.php
@@ -5,6 +5,8 @@
 interface ApiConsumerInterface
 {
     public function getEndpoint(): ?string;
+
     public function getKey(): ?string;
+
     public function getSecret(): ?string;
 }
diff --git a/app/Services/YouTubeService.php b/app/Services/YouTubeService.php
@@ -43,7 +43,7 @@ public function searchVideosRelatedToSong(Song $song, string $pageToken = '')
     public function search(string $q, string $pageToken = '', int $perPage = 10)
     {
         if (!$this->enabled()) {
-            return null;
+            return;
         }
 
         $uri = sprintf('search?part=snippet&type=video&maxResults=%s&pageToken=%s&q=%s',

diff --git a/tests/CreatesApplication.php b/tests/CreatesApplication.php
@@ -3,7 +3,6 @@
 namespace Tests;
 
 use App\Models\User;
-
 use Illuminate\Contracts\Console\Kernel as Artisan;
 use Illuminate\Foundation\Application;
 

diff --git a/tests/Feature/TestCase.php b/tests/Feature/TestCase.php
@@ -56,28 +56,28 @@ protected function createSampleMediaSet(): void
     protected function getAsUser($url, $user = null): self
     {
         return $this->get($url, [
-            'Authorization' => 'Bearer ' . $this->generateJwtToken($user),
+            'Authorization' => 'Bearer '.$this->generateJwtToken($user),
         ]);
     }
 
     protected function deleteAsUser($url, $data = [], $user = null): self
     {
         return $this->delete($url, $data, [
-            'Authorization' => 'Bearer ' . $this->generateJwtToken($user),
+            'Authorization' => 'Bearer '.$this->generateJwtToken($user),
         ]);
     }
 
     protected function postAsUser($url, $data, $user = null): self
     {
         return $this->post($url, $data, [
-            'Authorization' => 'Bearer ' . $this->generateJwtToken($user),
+            'Authorization' => 'Bearer '.$this->generateJwtToken($user),
         ]);
     }
 
     protected function putAsUser($url, $data, $user = null): self
     {
         return $this->put($url, $data, [
-            'Authorization' => 'Bearer ' . $this->generateJwtToken($user),
+            'Authorization' => 'Bearer '.$this->generateJwtToken($user),
         ]);
     }
 

diff --git a/tests/Integration/Factories/StreamerFactoryTest.php b/tests/Integration/Factories/StreamerFactoryTest.php
@@ -10,8 +10,8 @@
 use App\Services\Streamers\XAccelRedirectStreamer;
 use App\Services\Streamers\XSendFileStreamer;
 use App\Services\TranscodingService;
-use Tests\TestCase;
 use phpmock\mockery\PHPMockery;
+use Tests\TestCase;
 
 class StreamerFactoryTest extends TestCase
 {

diff --git a/tests/Unit/Services/ApiClientTest.php b/tests/Unit/Services/ApiClientTest.php
@@ -28,7 +28,7 @@ public function setUp()
     {
         parent::setUp();
 
-        /**
+        /*
          * @var Client client
          * @var Cache cache
          * @var Logger logger