Passed
Branch master (86ee69)
by Rafal
01:59
created
src/Application/SpotifyWebApiPhp/SpotifyWebApiInterface.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
      * @param array $options
17 17
      * @return bool
18 18
      */
19
-    public function addPlaylistTracks($playlistId, array $tracks, array $options = []) : bool;
19
+    public function addPlaylistTracks($playlistId, array $tracks, array $options = [ ]) : bool;
20 20
 
21 21
     /**
22 22
      * @param string $playlistId
@@ -30,14 +30,14 @@  discard block
 block discarded – undo
30 30
      * @param array $options
31 31
      * @return PlaylistDataProvider
32 32
      */
33
-    public function getPlaylist(string $playlistId, array $options = []) : PlaylistDataProvider;
33
+    public function getPlaylist(string $playlistId, array $options = [ ]) : PlaylistDataProvider;
34 34
 
35 35
     /**
36 36
      * @param $playlistId
37 37
      * @param array $options
38 38
      * @return object
39 39
      */
40
-    public function getPlaylistTracks(string $playlistId, array $options = []);
40
+    public function getPlaylistTracks(string $playlistId, array $options = [ ]);
41 41
 
42 42
     /**
43 43
      * @param string $userId
@@ -46,12 +46,12 @@  discard block
 block discarded – undo
46 46
      * @return PlaylistDataProvider
47 47
      * @throws PlaylistNotFound
48 48
      */
49
-    public function getUserPlaylistByName(string $userId, string $playlistName, array $options = []): PlaylistDataProvider;
49
+    public function getUserPlaylistByName(string $userId, string $playlistName, array $options = [ ]): PlaylistDataProvider;
50 50
 
51 51
     /**
52 52
      * @param TrackSearchRequestDataProvider $trackSearchRequest
53 53
      * @param array $options
54 54
      * @return TracksSearchDataProvider
55 55
      */
56
-    public function searchTrack(TrackSearchRequestDataProvider $trackSearchRequest, array $options = []): TracksSearchDataProvider;
56
+    public function searchTrack(TrackSearchRequestDataProvider $trackSearchRequest, array $options = [ ]): TracksSearchDataProvider;
57 57
 }
58 58
\ No newline at end of file
Please login to merge, or discard this patch.
src/Application/SpotifyWebApiPhp/SpotifyWebApi.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
      * @param array $options
37 37
      * @return bool
38 38
      */
39
-    public function addPlaylistTracks($playlistId, array $tracks, array $options = []): bool
39
+    public function addPlaylistTracks($playlistId, array $tracks, array $options = [ ]): bool
40 40
     {
41 41
         return $this->baseSpotifyWebAPI->addPlaylistTracks($playlistId, $tracks, $options);
42 42
     }
@@ -48,18 +48,18 @@  discard block
 block discarded – undo
48 48
      */
49 49
     public function deletePlaylistTracks(string $playlistId, array $tracksInfo): bool
50 50
     {
51
-        $tracksToDelete = [];
51
+        $tracksToDelete = [ ];
52 52
         foreach ($tracksInfo as $deleteTrackInfoDataProvider) {
53 53
             if (!$deleteTrackInfoDataProvider instanceof DeleteTrackInfoDataProvider) {
54 54
                 throw new \RuntimeException(
55 55
                     sprintf('tracksInfo should be instanceof class: %s', DeleteTrackInfoDataProvider::class)
56 56
                 );
57 57
             }
58
-            $tracksToDelete[] = $deleteTrackInfoDataProvider->toArray();
58
+            $tracksToDelete[ ] = $deleteTrackInfoDataProvider->toArray();
59 59
         }
60 60
 
61 61
         $this->baseSpotifyWebAPI->setReturnType(Request::RETURN_OBJECT);
62
-        $delete = (bool)$this->baseSpotifyWebAPI->deletePlaylistTracks($playlistId, $tracksToDelete);
62
+        $delete = (bool) $this->baseSpotifyWebAPI->deletePlaylistTracks($playlistId, $tracksToDelete);
63 63
         $this->baseSpotifyWebAPI->setReturnType(Request::RETURN_ASSOC);
64 64
 
65 65
         return $delete;
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
      * @param array $options
71 71
      * @return PlaylistDataProvider
72 72
      */
73
-    public function getPlaylist(string $playlistId, array $options = []): PlaylistDataProvider
73
+    public function getPlaylist(string $playlistId, array $options = [ ]): PlaylistDataProvider
74 74
     {
75 75
         $jsonObjectResult = $this->baseSpotifyWebAPI->getPlaylist($playlistId, $options);
76 76
 
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
      * @return PlaylistDataProvider
88 88
      * @throws PlaylistNotFound
89 89
      */
90
-    public function getUserPlaylistByName(string $userId, string $playlistName, array $options = []): PlaylistDataProvider
90
+    public function getUserPlaylistByName(string $userId, string $playlistName, array $options = [ ]): PlaylistDataProvider
91 91
     {
92 92
         $userPlaylistsDataProvider = $this->getUserPlaylists($userId, $options);
93 93
         $playlists = $userPlaylistsDataProvider->getItems();
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
105 105
      * @param array $options
106 106
      * @return PlaylistTracksDataProvider
107 107
      */
108
-    public function getPlaylistTracks(string $playlistId, array $options = []): PlaylistTracksDataProvider
108
+    public function getPlaylistTracks(string $playlistId, array $options = [ ]): PlaylistTracksDataProvider
109 109
     {
110 110
         $jsonObjectResult = $this->baseSpotifyWebAPI->getPlaylistTracks($playlistId, $options);
111 111
 
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
      * @param array $options
121 121
      * @return TracksSearchDataProvider
122 122
      */
123
-    public function searchTrack(TrackSearchRequestDataProvider $trackSearchRequest, array $options = []): TracksSearchDataProvider
123
+    public function searchTrack(TrackSearchRequestDataProvider $trackSearchRequest, array $options = [ ]): TracksSearchDataProvider
124 124
     {
125 125
         $jsonObjectResult = $this->search(
126 126
             sprintf(
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
         );
135 135
 
136 136
         $tracksSearchDataProvider = new TracksSearchDataProvider();
137
-        $tracksSearchDataProvider->fromArray($jsonObjectResult[$trackSearchRequest->getResultType()]);
137
+        $tracksSearchDataProvider->fromArray($jsonObjectResult[ $trackSearchRequest->getResultType() ]);
138 138
 
139 139
         return $tracksSearchDataProvider;
140 140
     }
@@ -145,9 +145,9 @@  discard block
 block discarded – undo
145 145
      * @param array $options
146 146
      * @return array
147 147
      */
148
-    private function search(string $query, array $type, array $options = []): array
148
+    private function search(string $query, array $type, array $options = [ ]): array
149 149
     {
150
-        return (array)$this->baseSpotifyWebAPI->search($query, $type, $options);
150
+        return (array) $this->baseSpotifyWebAPI->search($query, $type, $options);
151 151
     }
152 152
 
153 153
     /**
@@ -155,9 +155,9 @@  discard block
 block discarded – undo
155 155
      * @param array $options
156 156
      * @return UserPlaylistsDataProvider
157 157
      */
158
-    private function getUserPlaylists(string $userId, array $options = []): UserPlaylistsDataProvider
158
+    private function getUserPlaylists(string $userId, array $options = [ ]): UserPlaylistsDataProvider
159 159
     {
160
-        $userPlaylistInfo = (array)$this->baseSpotifyWebAPI->getUserPlaylists($userId, $options);
160
+        $userPlaylistInfo = (array) $this->baseSpotifyWebAPI->getUserPlaylists($userId, $options);
161 161
 
162 162
         $userPlaylistsDataProvider = new UserPlaylistsDataProvider();
163 163
         $userPlaylistsDataProvider->fromArray($userPlaylistInfo);
Please login to merge, or discard this patch.
test/Integration/Application/SpotifyWebApiPhp/SpotifyWebApiTest.php 1 patch
Spacing   +26 added lines, -26 removed lines patch added patch discarded remove patch
@@ -62,10 +62,10 @@  discard block
 block discarded – undo
62 62
     public function testGetUserPlaylists(): void
63 63
     {
64 64
         $spotifyPlaylist = $this->spotifyWebApi->getUserPlaylistByName(
65
-            static::spotifyInfo['user'],
66
-            static::spotifyInfo['playlistName']
65
+            static::spotifyInfo[ 'user' ],
66
+            static::spotifyInfo[ 'playlistName' ]
67 67
         );
68
-        $this->assertSame(static::spotifyInfo['playlistId'], $spotifyPlaylist->getId());
68
+        $this->assertSame(static::spotifyInfo[ 'playlistId' ], $spotifyPlaylist->getId());
69 69
     }
70 70
 
71 71
     public function testGetUserPlaylistsNotFoundUser(): void
@@ -77,18 +77,18 @@  discard block
 block discarded – undo
77 77
     public function testGetUserPlaylistsNotFoundPlayList(): void
78 78
     {
79 79
         $this->expectException(PlaylistNotFound::class);
80
-        $this->spotifyWebApi->getUserPlaylistByName(static::spotifyInfo['user'], 'uniTest-Not_FOUND-Playlist');
80
+        $this->spotifyWebApi->getUserPlaylistByName(static::spotifyInfo[ 'user' ], 'uniTest-Not_FOUND-Playlist');
81 81
     }
82 82
 
83 83
     public function testGetPlaylist(): void
84 84
     {
85 85
         $spotifyPlayList = $this->spotifyWebApi->getPlaylist(
86
-            static::spotifyInfo['playlistId']
86
+            static::spotifyInfo[ 'playlistId' ]
87 87
         );
88
-        $this->assertSame(static::spotifyInfo['playlistId'], $spotifyPlayList->getId());
89
-        $this->assertSame(static::spotifyInfo['playlistName'], $spotifyPlayList->getName());
88
+        $this->assertSame(static::spotifyInfo[ 'playlistId' ], $spotifyPlayList->getId());
89
+        $this->assertSame(static::spotifyInfo[ 'playlistName' ], $spotifyPlayList->getName());
90 90
         $this->assertTrue($spotifyPlayList->getPublic());
91
-        $this->assertSame('U2', $spotifyPlayList->getTracks()->getItems()[0]->getTrack()->getArtists()[0]->getName());
91
+        $this->assertSame('U2', $spotifyPlayList->getTracks()->getItems()[ 0 ]->getTrack()->getArtists()[ 0 ]->getName());
92 92
     }
93 93
 
94 94
     public function testGetNoExistUserPlaylist(): void
@@ -111,18 +111,18 @@  discard block
 block discarded – undo
111 111
     {
112 112
         $trackSearchRequestDataProvider = new TrackSearchRequestDataProvider();
113 113
         $trackSearchRequestDataProvider->setArtist(
114
-            static::spotifySong['artist']
114
+            static::spotifySong[ 'artist' ]
115 115
         );
116 116
         $trackSearchRequestDataProvider->setTrack(
117
-            static::spotifySong['track']
117
+            static::spotifySong[ 'track' ]
118 118
         );
119 119
         $searchResult = $this->spotifyWebApi->searchTrack($trackSearchRequestDataProvider);
120 120
 
121 121
         $this->assertGreaterThan(0, $searchResult->getItems());
122 122
         $this->assertSame(
123
-            strtolower(static::spotifySong['track']),
123
+            strtolower(static::spotifySong[ 'track' ]),
124 124
             strtolower(
125
-                $searchResult->getItems()[0]->getName()
125
+                $searchResult->getItems()[ 0 ]->getName()
126 126
             )
127 127
         );
128 128
     }
@@ -144,27 +144,27 @@  discard block
 block discarded – undo
144 144
     public function testAddPlaylistTracks()
145 145
     {
146 146
         $addResult = $this->spotifyWebApi->addPlaylistTracks(
147
-            static::spotifyInfo['playlistId'],
147
+            static::spotifyInfo[ 'playlistId' ],
148 148
             [
149
-                static::spotifySong['trackId']
149
+                static::spotifySong[ 'trackId' ]
150 150
             ]
151 151
         );
152 152
         $this->assertTrue($addResult);
153 153
 
154 154
         $spotifyPlayList = $this->spotifyWebApi->getPlaylistTracks(
155
-            static::spotifyInfo['playlistId']
155
+            static::spotifyInfo[ 'playlistId' ]
156 156
         );
157 157
 
158 158
         $this->assertNotEmpty($spotifyPlayList->getItems());
159 159
         $this->assertNotEmpty($spotifyPlayList->getTotal());
160
-        $this->assertSame(static::spotifySong['trackId'], $spotifyPlayList->getItems()[1]->getTrack()->getId());
160
+        $this->assertSame(static::spotifySong[ 'trackId' ], $spotifyPlayList->getItems()[ 1 ]->getTrack()->getId());
161 161
     }
162 162
 
163 163
     public function testAddPlaylistTracksNotExistTrackId()
164 164
     {
165 165
         try {
166 166
             $this->spotifyWebApi->addPlaylistTracks(
167
-                static::spotifyInfo['playlistId'],
167
+                static::spotifyInfo[ 'playlistId' ],
168 168
                 [
169 169
                     'uniTest-Not_FOUND--ID'
170 170
                 ]
@@ -183,24 +183,24 @@  discard block
 block discarded – undo
183 183
     public function testDeleteUserPlaylistTracks()
184 184
     {
185 185
         $spotifyPlayList = $this->spotifyWebApi->getPlaylistTracks(
186
-            static::spotifyInfo['playlistId']
186
+            static::spotifyInfo[ 'playlistId' ]
187 187
         );
188 188
 
189 189
         $this->assertCount(2, $spotifyPlayList->getItems());
190 190
         $this->assertSame(2, $spotifyPlayList->getTotal());
191 191
 
192 192
         $deleteTrack = new DeleteTrackInfoDataProvider();
193
-        $deleteTrack->setId(static::spotifySong['trackId']);
193
+        $deleteTrack->setId(static::spotifySong[ 'trackId' ]);
194 194
 
195 195
         $result = $this->spotifyWebApi->deletePlaylistTracks(
196
-            static::spotifyInfo['playlistId'],
197
-            [$deleteTrack]
196
+            static::spotifyInfo[ 'playlistId' ],
197
+            [ $deleteTrack ]
198 198
         );
199 199
 
200 200
         $this->assertTrue($result);
201 201
 
202 202
         $spotifyPlayList = $this->spotifyWebApi->getPlaylistTracks(
203
-            static::spotifyInfo['playlistId']
203
+            static::spotifyInfo[ 'playlistId' ]
204 204
         );
205 205
 
206 206
         $this->assertCount(1, $spotifyPlayList->getItems());
@@ -213,8 +213,8 @@  discard block
 block discarded – undo
213 213
             $deleteTrack = new DeleteTrackInfoDataProvider();
214 214
             $deleteTrack->setId('uniTest-Not_FOUND--ID');
215 215
             $this->spotifyWebApi->deletePlaylistTracks(
216
-                static::spotifyInfo['playlistId'],
217
-                [$deleteTrack]
216
+                static::spotifyInfo[ 'playlistId' ],
217
+                [ $deleteTrack ]
218 218
             );
219 219
         } catch (\SpotifyWebAPI\SpotifyWebAPIException $e) {
220 220
             $this->assertSame(
@@ -231,8 +231,8 @@  discard block
 block discarded – undo
231 231
     {
232 232
         $this->expectException(\RuntimeException::class);
233 233
         $this->spotifyWebApi->deletePlaylistTracks(
234
-            static::spotifyInfo['playlistId'],
235
-            ['id' => 'hahaha']
234
+            static::spotifyInfo[ 'playlistId' ],
235
+            [ 'id' => 'hahaha' ]
236 236
         );
237 237
     }
238 238
 }
Please login to merge, or discard this patch.