rattfieldnz / safe-urls

tests/TestCase.php

Spacing +1 added lines, -1 removed lines

@@ -15,6 +15,6 @@
      */
     protected function getPackageProviders($app)
     {
-        return [SafeUrlsServiceProvider::class];
+        return [ SafeUrlsServiceProvider::class ];
     }
 }

src/Libraries/Config/Config.php

Spacing +3 added lines, -3 removed lines

@@ -18,13 +18,13 @@
     public const DEFAULT_TIMEOUT = 10;
 
     // Default threat types.
-    public const DEFAULT_THREAT_TYPES = ['THREAT_TYPE_UNSPECIFIED'];
+    public const DEFAULT_THREAT_TYPES = [ 'THREAT_TYPE_UNSPECIFIED' ];
 
     // Default threst entry types.
-    public const DEFAULT_THREAT_ENTRY_TYPES = ['THREAT_ENTRY_TYPE_UNSPECIFIED'];
+    public const DEFAULT_THREAT_ENTRY_TYPES = [ 'THREAT_ENTRY_TYPE_UNSPECIFIED' ];
 
     // Default threat platform types.
-    public const DEFAULT_THREAT_PLATFORM_TYPES = ['PLATFORM_TYPE_UNSPECIFIED'];
+    public const DEFAULT_THREAT_PLATFORM_TYPES = [ 'PLATFORM_TYPE_UNSPECIFIED' ];
 
     /**
      * Retrieve the Google API key.

tests/Libraries/Data/DataTest.php

Spacing +11 added lines, -11 removed lines

@@ -10,11 +10,11 @@ 
 class DataTest extends TestCase
 {
     private $_formattedUrls = [
-        ['url' => 'https://www.google.com'],
-        ['url' => 'https://github.com'],
-        ['url' => 'https://github.styleci.io'],
-        ['url' => 'https://travis-ci.org'],
-        ['url' => 'https://packagist.org'],
+        [ 'url' => 'https://www.google.com' ],
+        [ 'url' => 'https://github.com' ],
+        [ 'url' => 'https://github.styleci.io' ],
+        [ 'url' => 'https://travis-ci.org' ],
+        [ 'url' => 'https://packagist.org' ],
     ];
 
     private $_urls = [
@@ -52,7 +52,7 @@ 
 
     public function testPayloadNoUrls()
     {
-        $urls = [];
+        $urls = [ ];
 
         $expected = self::_payload($urls);
 
@@ -70,10 +70,10 @@ 
 
     public function testFormatUrlsSingle()
     {
-        $url = ['https://www.google.com'];
+        $url = [ 'https://www.google.com' ];
 
         $expected = [
-            ['url' => 'https://www.google.com'],
+            [ 'url' => 'https://www.google.com' ],
         ];
 
         $actual = Data::formatUrls($url);
@@ -82,15 +82,15 @@ 
 
     public function testFormatUrlsNone()
     {
-        $urls = [];
+        $urls = [ ];
 
-        $expected = [];
+        $expected = [ ];
 
         $actual = Data::formatUrls($urls);
         $this->assertEquals($expected, $actual);
     }
 
-    private static function _payload(array $urls, array $threatTypes = [], array $platformTypes = [], array $threatEntryTypes = []): array
+    private static function _payload(array $urls, array $threatTypes = [ ], array $platformTypes = [ ], array $threatEntryTypes = [ ]): array
     {
         return [
             'client' => [

src/Libraries/Data/Data.php

Spacing +3 added lines, -3 removed lines

@@ -26,7 +26,7 @@ 
      *
      * @return array The generated payload.
      */
-    public static function payload(array $urls, array $threatTypes = [], array $platformTypes = [], array $threatEntryTypes = []): array
+    public static function payload(array $urls, array $threatTypes = [ ], array $platformTypes = [ ], array $threatEntryTypes = [ ]): array
     {
         return [
             'client' => [
@@ -51,9 +51,9 @@ 
      */
     public static function formatUrls(array $urls): array
     {
-        $formattedUrls = [];
+        $formattedUrls = [ ];
         foreach ($urls as $url) {
-            $formattedUrls[] = ['url' => $url];
+            $formattedUrls[ ] = [ 'url' => $url ];
         }
 
         return $formattedUrls;

src/SafeUrlsServiceProvider.php

Spacing +6 added lines, -6 removed lines

@@ -44,7 +44,7 @@ 
     public function register()
     {
         // Register the service the package provides.
-        $this->app->singleton('safe-urls', function () {
+        $this->app->singleton('safe-urls', function() {
             return new SafeUrls();
         });
     }
@@ -56,7 +56,7 @@ 
      */
     public function provides()
     {
-        return ['safe-urls'];
+        return [ 'safe-urls' ];
     }
 
     /**
@@ -72,7 +72,7 @@ 
         ], 'safe-urls');
 
         // Registering package commands.
-        $this->commands(['safe-urls']);
+        $this->commands([ 'safe-urls' ]);
     }
 
     /**
@@ -85,8 +85,8 @@ 
      */
     protected function mergeConfigFrom($path, $key)
     {
-        $config = $this->app['config']->get($key, []);
-        $this->app['config']->set($key, $this->mergeConfig($config, require $path));
+        $config = $this->app[ 'config' ]->get($key, [ ]);
+        $this->app[ 'config' ]->set($key, $this->mergeConfig($config, require $path));
     }
 
     /**
@@ -110,7 +110,7 @@ 
             if (is_numeric($key)) {
                 continue;
             }
-            $array[$key] = $this->mergeConfig($value, $merging[$key]);
+            $array[ $key ] = $this->mergeConfig($value, $merging[ $key ]);
         }
 
         return $array;

tests/Libraries/Traits/StaticCallingTraitTest.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
             $testClass->callStatic(
                 $className, $methodName, $this->urlsToTest
             ), true
-        )['status'];
+        )[ 'status' ];
 
         $this->assertEquals($expected, $actual);
     }

tests/SafeUrlsTest.php

Spacing +11 added lines, -11 removed lines

@@ -43,12 +43,12 @@ 
         $urlToRemove = 'https://testsafebrowsing.appspot.com/s/phishing.html';
 
         $expected = [
-            $this->urlsToTest[1],
-            $this->urlsToTest[2],
-            $this->urlsToTest[3],
-            $this->urlsToTest[4],
+            $this->urlsToTest[ 1 ],
+            $this->urlsToTest[ 2 ],
+            $this->urlsToTest[ 3 ],
+            $this->urlsToTest[ 4 ],
         ];
-        $this->safeUrls->remove([$urlToRemove]);
+        $this->safeUrls->remove([ $urlToRemove ]);
 
         $actual = $this->safeUrls->getCurrentUrls(true);
 
@@ -59,7 +59,7 @@ 
     {
         $this->safeUrls->add($this->urlsToTest);
 
-        $expected = [];
+        $expected = [ ];
         $this->safeUrls->remove($this->urlsToTest);
         $actual = $this->safeUrls->getCurrentUrls(true);
 
@@ -76,8 +76,8 @@ 
 
         $expected = '{ "status": 200, "response": { "matches": [ { "threatType": "SOCIAL_ENGINEERING", "platformType": "ANY_PLATFORM", "threat": { "url": "https://testsafebrowsing.appspot.com/s/phishing.html" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "MALWARE", "platformType": "ANY_PLATFORM", "threat": { "url": "https://testsafebrowsing.appspot.com/s/malware.html" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "MALWARE", "platformType": "ANY_PLATFORM", "threat": { "url": "http://testsafebrowsing.appspot.com/apiv4/ANY_PLATFORM/MALWARE/URL/" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "SOCIAL_ENGINEERING", "platformType": "ANY_PLATFORM", "threat": { "url": "http://testsafebrowsing.appspot.com/apiv4/ANY_PLATFORM/SOCIAL_ENGINEERING/URL/" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "MALWARE", "platformType": "ANY_PLATFORM", "threat": { "url": "http://malware.testing.google.test/testing/malware/" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "SOCIAL_ENGINEERING", "platformType": "ANY_PLATFORM", "threat": { "url": "http://malware.testing.google.test/testing/malware/" }, "cacheDuration": "300s", "threatEntryType": "URL" } ] } }';
 
-        $expected = json_decode($expected, true)['status'];
-        $actual = json_decode((string) $this->safeUrls->execute()->getResults(), true)['status'];
+        $expected = json_decode($expected, true)[ 'status' ];
+        $actual = json_decode((string) $this->safeUrls->execute()->getResults(), true)[ 'status' ];
 
         $this->assertEquals($expected, $actual);
     }
@@ -90,9 +90,9 @@ 
     public function testCheck()
     {
         $expected = '{ "status": 200, "response": { "matches": [ { "threatType": "SOCIAL_ENGINEERING", "platformType": "ANY_PLATFORM", "threat": { "url": "https://testsafebrowsing.appspot.com/s/phishing.html" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "MALWARE", "platformType": "ANY_PLATFORM", "threat": { "url": "https://testsafebrowsing.appspot.com/s/malware.html" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "MALWARE", "platformType": "ANY_PLATFORM", "threat": { "url": "http://testsafebrowsing.appspot.com/apiv4/ANY_PLATFORM/MALWARE/URL/" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "SOCIAL_ENGINEERING", "platformType": "ANY_PLATFORM", "threat": { "url": "http://testsafebrowsing.appspot.com/apiv4/ANY_PLATFORM/SOCIAL_ENGINEERING/URL/" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "MALWARE", "platformType": "ANY_PLATFORM", "threat": { "url": "http://malware.testing.google.test/testing/malware/" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "SOCIAL_ENGINEERING", "platformType": "ANY_PLATFORM", "threat": { "url": "http://malware.testing.google.test/testing/malware/" }, "cacheDuration": "300s", "threatEntryType": "URL" } ] } }';
-        $expected = json_decode($expected, true)['status'];
+        $expected = json_decode($expected, true)[ 'status' ];
 
-        $actual = json_decode($this->safeUrls::check($this->urlsToTest), true)['status'];
+        $actual = json_decode($this->safeUrls::check($this->urlsToTest), true)[ 'status' ];
 
         $this->assertEquals($expected, $actual);
     }
@@ -105,7 +105,7 @@ 
         $this->safeUrls->add($this->urlsToTest);
 
         $safeUrlsMock = $this->getMockBuilder(SafeUrls::class)
-            ->setMethods(['callStatic'])
+            ->setMethods([ 'callStatic' ])
             ->getMock();
 
         /* @scrutinizer ignore-call */

src/SafeUrls.php

Spacing +3 added lines, -3 removed lines

@@ -35,10 +35,10 @@ 
     public function __construct()
     {
         // Initialise the list of urls as an empty array.
-        $this->urls = [];
+        $this->urls = [ ];
 
         // Initialise the list of urls as an empty array.
-        $this->results = [];
+        $this->results = [ ];
     }
 
     /**
@@ -140,7 +140,7 @@ 
         //$this->results = '{ "status": 200, "response": { "matches": [ { "threatType": "SOCIAL_ENGINEERING", "platformType": "ANY_PLATFORM", "threat": { "url": "https://testsafebrowsing.appspot.com/s/phishing.html" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "MALWARE", "platformType": "ANY_PLATFORM", "threat": { "url": "https://testsafebrowsing.appspot.com/s/malware.html" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "MALWARE", "platformType": "ANY_PLATFORM", "threat": { "url": "http://testsafebrowsing.appspot.com/apiv4/ANY_PLATFORM/MALWARE/URL/" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "SOCIAL_ENGINEERING", "platformType": "ANY_PLATFORM", "threat": { "url": "http://testsafebrowsing.appspot.com/apiv4/ANY_PLATFORM/SOCIAL_ENGINEERING/URL/" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "MALWARE", "platformType": "ANY_PLATFORM", "threat": { "url": "http://malware.testing.google.test/testing/malware/" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "SOCIAL_ENGINEERING", "platformType": "ANY_PLATFORM", "threat": { "url": "http://malware.testing.google.test/testing/malware/" }, "cacheDuration": "300s", "threatEntryType": "URL" } ] } }';
         //$data = json_decode((string)$this->results);
 
-        $data = json_decode((string)$this->results );
+        $data = json_decode((string) $this->results);
         $response = empty($data->response) ? null : $data->response;
         if (empty($response)) {
             return false;