Werkspot / BingAdsApiBundle

Tests/Api/ClientTest.php

Doc Comments +1 added lines

@@ -61,6 +61,7 @@
     }
 
     /**
+     * @param integer $code
      * @return Mockery\MockInterface
      */
     private function runClientSoapException($code)

Api/Client.php

Spacing +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
     public function setConfig($config)
     {
         $this->config = $config;
-        $this->config['cache_dir'] = $this->config['cache_dir'] . '/' . 'BingAdsApiBundle'; //<-- important for the cache clear function
+        $this->config['cache_dir'] = $this->config['cache_dir'].'/'.'BingAdsApiBundle'; //<-- important for the cache clear function
         $this->config['csv']['fixHeader']['removeColumnHeader'] = true; //-- fix till i know how to do this
     }
 

Doc Comments +1 added lines, -1 removed lines

@@ -210,7 +210,7 @@
      * to request the report identifier. The identifier is used to check report generation status
      * before downloading the report.
      *
-     * @param mixed  $report
+     * @param ReportRequest  $report
      * @param string $name
      *
      * @return string ReportRequestId

Api/Helper/Csv.php

Spacing +1 added lines, -1 removed lines

@@ -87,7 +87,7 @@
 
         for ($i = 0; $i < count($array); ++$i) {
             if ($enclosure) {
-                $csvStr .= $enclosure . str_replace($enclosure, $enclosure . $enclosure, $array[$i]) . $enclosure;
+                $csvStr .= $enclosure.str_replace($enclosure, $enclosure.$enclosure, $array[$i]).$enclosure;
             } else {
                 $csvStr .= $array[$i];
             }

Tests/Api/Helper/FileTest.php

Spacing +8 added lines, -8 removed lines

@@ -25,10 +25,10 @@ 
 
     public function testGetFile()
     {
-        $nonExistingFile = ASSETS_DIR . 'iDoNotExist.txt';
-        $existingFile = ASSETS_DIR . 'report.csv';
+        $nonExistingFile = ASSETS_DIR.'iDoNotExist.txt';
+        $existingFile = ASSETS_DIR.'report.csv';
         $onlineFile = 'http://google.com/test.txt';
-        $file = ASSETS_DIR . 'example.txt';
+        $file = ASSETS_DIR.'example.txt';
 
         $mock = new MockHandler([new Response(200, [])]);
         $handler = HandlerStack::create($mock);
@@ -54,7 +54,7 @@ 
     public function testDownload()
     {
         $url = 'http://example.com';
-        $file = ASSETS_DIR . 'example.txt';
+        $file = ASSETS_DIR.'example.txt';
 
         $clientMock = Mockery::mock(Client::class);
         $clientMock
@@ -82,14 +82,14 @@ 
 
     public function testUnZip()
     {
-        $file = ASSETS_DIR . 'test.zip';
+        $file = ASSETS_DIR.'test.zip';
 
-        $this->fileSystem->copy(ASSETS_DIR . 'report.zip', $file);
+        $this->fileSystem->copy(ASSETS_DIR.'report.zip', $file);
 
         $fileHelper = new File();
         $files = $fileHelper->unZip($file);
 
-        $this->assertEquals($files, [ASSETS_DIR . '0039202.csv']);
+        $this->assertEquals($files, [ASSETS_DIR.'0039202.csv']);
 
         //-- Remove the files created by this test
         foreach ($files as $file) {
@@ -102,7 +102,7 @@ 
      */
     public function testCorruptUnZip()
     {
-        $file = ASSETS_DIR . 'corrupt.zip';
+        $file = ASSETS_DIR.'corrupt.zip';
         $fileHelper = new File();
         $fileHelper->unZip($file, false);
     }

Tests/Api/Report/GeoLocationPerformanceReportTest.php

Spacing +6 added lines, -6 removed lines

@@ -68,7 +68,7 @@ 
             $this->getTimeHelperMock()
         );
         $result = $apiClient->get(['TimePeriod', 'AccountName', 'AdGroupId'], 'GeoLocationPerformanceReport', ReportTimePeriod::LastWeek);
-        $this->assertEquals([ASSETS_DIR . 'report.csv'], $result);
+        $this->assertEquals([ASSETS_DIR.'report.csv'], $result);
     }
 
     public function testGeoLocationPerformanceReportMoveFile()
@@ -81,12 +81,12 @@ 
             $this->getCsvHelperMock(),
             $this->getTimeHelperMock()
         );
-        $result = $apiClient->get(['TimePeriod', 'AccountName', 'AdGroupId'], 'GeoLocationPerformanceReport', ReportTimePeriod::LastWeek, ASSETS_DIR . 'test.csv');
-        $this->assertEquals(ASSETS_DIR . 'test.csv', $result);
+        $result = $apiClient->get(['TimePeriod', 'AccountName', 'AdGroupId'], 'GeoLocationPerformanceReport', ReportTimePeriod::LastWeek, ASSETS_DIR.'test.csv');
+        $this->assertEquals(ASSETS_DIR.'test.csv', $result);
 
         //--Move File back
         $fileSystem = new Filesystem();
-        $fileSystem->rename(ASSETS_DIR . 'test.csv', ASSETS_DIR . 'report.csv');
+        $fileSystem->rename(ASSETS_DIR.'test.csv', ASSETS_DIR.'report.csv');
     }
 
     public function testGetRefreshToken()
@@ -224,7 +224,7 @@ 
             ->once()
             ->shouldReceive('unZip')
             ->with('/tmp/report.zip')
-            ->andReturn([ASSETS_DIR . 'report.csv'])
+            ->andReturn([ASSETS_DIR.'report.csv'])
             ->once();
 
         return $zipHelperMock;
@@ -235,7 +235,7 @@ 
      */
     private function getCsvHelperMock()
     {
-        $lines = file(ASSETS_DIR . 'report.csv');
+        $lines = file(ASSETS_DIR.'report.csv');
         $csvHelperMock = Mockery::mock(Helper\Csv::class);
         $csvHelperMock
             ->shouldReceive('removeHeaders')

Doc Comments +1 added lines, -1 removed lines

@@ -279,7 +279,7 @@
     }
 
     /**
-     * @param $code
+     * @param integer $code
      * @return SoapFault
      */
     private function generateSoapFault($code)

DependencyInjection/WerkspotBingAdsApiExtension.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
         $configuration = new Configuration();
         $config = $this->processConfiguration($configuration, $configs);
 
-        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
+        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
         $loader->load('services.yml');
 
         // Once the services definition are read, get your service and add a method call to setConfig()