route4me / route4me-php-sdk

UnitTestFiles/Test/OptimizationProblemUntTests.php

Spacing +4 added lines, -4 removed lines

@@ -27,9 +27,9 @@ 
 
     public static function setUpBeforeClass()
     {
-        if (Constants::API_KEY == Constants::DEMO_API_KEY) {
-            $className = str_replace('UnitTestFiles\\Test\\','',get_class());
-            self::markTestSkipped( 'PHPUnit will skip '. $className ." class - it impossible run with demo API key" );
+        if (Constants::API_KEY==Constants::DEMO_API_KEY) {
+            $className = str_replace('UnitTestFiles\\Test\\', '', get_class());
+            self::markTestSkipped('PHPUnit will skip '.$className." class - it impossible run with demo API key");
         }
 
         Route4Me::setApiKey(Constants::API_KEY);
@@ -263,7 +263,7 @@ 
 
         $this->assertNotNull($result);
         $this->assertTrue($result instanceof OptimizationProblem);
-        $this->assertTrue(sizeof($result->addresses)==$initialAddresses+2);
+        $this->assertTrue(sizeof($result->addresses)==$initialAddresses + 2);
     }
 
     public function testGetRandomOptimizationId()

UnitTestFiles/Test/RouteExampleTests.php

Spacing +11 added lines, -11 removed lines

@@ -36,9 +36,9 @@ 
 
     public static function setUpBeforeClass()
     {
-        if (Constants::API_KEY == Constants::DEMO_API_KEY) {
-            $className = str_replace('UnitTestFiles\\Test\\','',get_class());
-            self::markTestSkipped( 'PHPUnit will skip '. $className ." class - it impossible run with demo API key" );
+        if (Constants::API_KEY==Constants::DEMO_API_KEY) {
+            $className = str_replace('UnitTestFiles\\Test\\', '', get_class());
+            self::markTestSkipped('PHPUnit will skip '.$className." class - it impossible run with demo API key");
         }
 
         Route4Me::setApiKey(Constants::API_KEY);
@@ -187,7 +187,7 @@ 
 
     public function testGetScheduleCalendar()
     {
-        $now            = new \DateTime();
+        $now = new \DateTime();
 
         $schedCalendarParams = new ScheduleCalendarParameters();
 
@@ -241,7 +241,7 @@ 
         $this->assertTrue(isset($result['status']), "Cannot merge the routes");
         $this->assertTrue($result['status'], "Cannot merge the routes");
         $this->assertTrue(isset($result['optimization_problem_id']), "Cannot merge the routes");
-        $this->assertEquals(32,strlen($result['optimization_problem_id']), "Cannot merge the routes");
+        $this->assertEquals(32, strlen($result['optimization_problem_id']), "Cannot merge the routes");
 
         self::$removedOptimizationIDs[] = $result['optimization_problem_id'];
     }
@@ -286,7 +286,7 @@ 
 
         self::$createdProblems[] = $problems;
 
-        $routes=$problems->getRoutes();
+        $routes = $problems->getRoutes();
         $this->assertTrue(sizeof($routes)==2, "The generated optimization hasn't two routes");
 
         $totalTripDistanceByDuration = $routes[0]->trip_distance + $routes[1]->trip_distance;
@@ -325,9 +325,9 @@ 
         $problems = OptimizationProblem::optimize($optimizationParams);
         $this->assertTrue(!is_null($problems), "Cannot generate an optimization with the distance priority fine-tuning");
 
-        self::$createdProblems[]=$problems;
+        self::$createdProblems[] = $problems;
 
-        $routes=$problems->getRoutes();
+        $routes = $problems->getRoutes();
         $this->assertTrue(sizeof($routes)==2, "The generated optimization hasn't two routes");
 
         $totalTripDistanceByDistance = $routes[0]->trip_distance + $routes[1]->trip_distance;
@@ -355,7 +355,7 @@ 
             'device_type'               => DeviceType::WEB,
             'optimize'                  => OptimizationType::TIME,
             'metric'                    => Metric::MATRIX,
-            'route_max_duration'        => 86400 ,
+            'route_max_duration'        => 86400,
             'travel_mode'               => TravelMode::DRIVING,
             'vehicle_capacity'          => 50,
             'vehicle_max_distance_mi'   => 10000,
@@ -626,7 +626,7 @@ 
         $this->assertNotNull($problem->addresses);
 
         $firstAddress = $problem->routes[0]->addresses[0];
-        $lastAddress = $problem->routes[0]->addresses[sizeof($problem->routes[0]->addresses)-1];
+        $lastAddress = $problem->routes[0]->addresses[sizeof($problem->routes[0]->addresses) - 1];
 
         $this->assertEquals($firstAddress->route_destination_id, $lastAddress->route_destination_id);
     }
@@ -818,7 +818,7 @@ 
             }
 
             if (sizeof(self::$removedOptimizationIDs)>0) {
-                $optimizationProblemIDs = array_merge($optimizationProblemIDs,self::$removedOptimizationIDs);
+                $optimizationProblemIDs = array_merge($optimizationProblemIDs, self::$removedOptimizationIDs);
             }
 
             $params = [

UnitTestFiles/Test/RouteTests.php

Spacing +14 added lines, -14 removed lines

@@ -180,7 +180,7 @@ 
         $users = $member->getUsers();
 
         assert(!is_null($users), 'Cannot retrieve list of the users');
-        assert(2 == sizeof($users), 'Cannot retrieve list of the users');
+        assert(2==sizeof($users), 'Cannot retrieve list of the users');
         assert(isset($users['results']), 'Cannot retrieve list of the users');
         assert(isset($users['total']), 'Cannot retrieve list of the users');
 
@@ -211,7 +211,7 @@ 
         $route->update();
 
         $this->assertNotNull($route);
-        $this->assertInstanceOf(Route::class,$route);
+        $this->assertInstanceOf(Route::class, $route);
 
         //endregion
 
@@ -219,10 +219,10 @@ 
 
         $route = new Route();
 
-        $assignedRoute= $route->getRoutes(['route_id' => $routeId]);
+        $assignedRoute = $route->getRoutes(['route_id' => $routeId]);
 
         $this->assertNotNull($assignedRoute);
-        $this->assertInstanceOf(Route::class,$assignedRoute);
+        $this->assertInstanceOf(Route::class, $assignedRoute);
         $this->assertEquals($randomUserID, $assignedRoute->parameters->member_id);
 
         //endregion
@@ -276,7 +276,7 @@ 
             $removedVehicleIDs[] = $result['vehicle_id'];
         }
 
-        $randomIndex = rand(0, sizeof($response)-1);
+        $randomIndex = rand(0, sizeof($response) - 1);
         $vehicleId = $response[$randomIndex]['vehicle_id'];
 
         //endregion
@@ -302,7 +302,7 @@ 
         $route->update();
 
         $this->assertNotNull($route);
-        $this->assertInstanceOf(Route::class,$route);
+        $this->assertInstanceOf(Route::class, $route);
 
         //endregion
 
@@ -310,10 +310,10 @@ 
 
         $route = new Route();
 
-        $assignedRoute= $route->getRoutes(['route_id' => $routeId]);
+        $assignedRoute = $route->getRoutes(['route_id' => $routeId]);
 
         $this->assertNotNull($assignedRoute);
-        $this->assertInstanceOf(Route::class,$assignedRoute);
+        $this->assertInstanceOf(Route::class, $assignedRoute);
         $this->assertEquals($vehicleId, $assignedRoute->parameters->vehicle_id);
 
         //endregion
@@ -376,7 +376,7 @@ 
         $params->route_id           = $route_id;
         $params->route_path_output  = 'Points';
 
-        $routeResult = (array) $route->getRoutePoints($params->toArray());
+        $routeResult = (array)$route->getRoutePoints($params->toArray());
 
         $this->assertNotNull($routeResult);
         $this->assertTrue(is_array($routeResult));
@@ -449,8 +449,8 @@ 
         foreach ($routeResults as $routeResult) {
 
             $this->assertTrue(
-                $routeResult->parameters->route_date - $tz >= $startDateUnix &&
-                $routeResult->parameters->route_date - $tz <= $endtDateUnix
+                $routeResult->parameters->route_date - $tz>=$startDateUnix &&
+                $routeResult->parameters->route_date - $tz<=$endtDateUnix
             );
         }
     }
@@ -500,7 +500,7 @@ 
 
         foreach (self::$testRoutes as $route1) {
             if (isset($route1->destination_count)) {
-                if ($route1->destination_count > 4) {
+                if ($route1->destination_count>4) {
                     $selectedRoute = $route->getRoutes(['route_id' => $route1->route_id]);
                     break;
                 }
@@ -656,7 +656,7 @@ 
         $this->assertInstanceOf(Route::class, Route::fromArray($result));
         $this->assertNull($result['optimization_problem_id']);
 
-        $removedRouteIDs[]= $result['route_id'];
+        $removedRouteIDs[] = $result['route_id'];
     }
 
     public function testUpdateRouteAvoidanceZone()
@@ -773,7 +773,7 @@ 
             }
 
             if (sizeof(self::$removedOptimizationIDs)>0) {
-                $optimizationProblemIDs = array_merge($optimizationProblemIDs,self::$removedOptimizationIDs);
+                $optimizationProblemIDs = array_merge($optimizationProblemIDs, self::$removedOptimizationIDs);
             }
 
             $params = [

UnitTestFiles/Test/TerritoryTests.php

Spacing +3 added lines, -3 removed lines

@@ -119,7 +119,7 @@ 
         $this->assertInstanceOf(Territory::class, Territory::fromArray($response[0]));
     }
 
-    public function testGetTerritory(){
+    public function testGetTerritory() {
         $territory = new Territory();
 
         $params = [
@@ -132,7 +132,7 @@ 
         $this->assertTrue(is_array($response));
         $this->assertInstanceOf(Territory::class, Territory::fromArray($response));
         $this->assertTrue(isset($response['territory_id']));
-        $this->assertEquals(self::$createdTerritories[0],$response['territory_id']);
+        $this->assertEquals(self::$createdTerritories[0], $response['territory_id']);
     }
 
     public function testCreateCircleTerritory()
@@ -281,7 +281,7 @@ 
     {
         $territory = new Territory();
 
-        $territoryId = self::$createdTerritories[sizeof(self::$createdTerritories)-1];
+        $territoryId = self::$createdTerritories[sizeof(self::$createdTerritories) - 1];
 
         $response = $territory->deleteTerritory($territoryId);
 

Braces +3 added lines, -1 removed lines

@@ -294,7 +294,9 @@
 
     public static function tearDownAfterClass()
     {
-        if (sizeof(self::$createdTerritories)<1) return;
+        if (sizeof(self::$createdTerritories)<1) {
+        	return;
+        }
 
         $territory = new Territory();
 

UnitTestFiles/Test/AddressNoteUnitTests.php

Spacing +11 added lines, -11 removed lines

@@ -37,9 +37,9 @@ 
 
     public static function setUpBeforeClass()
     {
-        if (Constants::API_KEY == Constants::DEMO_API_KEY) {
-            $className = str_replace('UnitTestFiles\\Test\\','',get_class());
-            self::markTestSkipped( 'PHPUnit will skip '. $className ." class - it impossible run with demo API key" );
+        if (Constants::API_KEY==Constants::DEMO_API_KEY) {
+            $className = str_replace('UnitTestFiles\\Test\\', '', get_class());
+            self::markTestSkipped('PHPUnit will skip '.$className." class - it impossible run with demo API key");
         }
 
         Route4Me::setApiKey(Constants::API_KEY);
@@ -135,8 +135,8 @@ 
         $customNoteTypes = $addressNote->getAllCustomNoteTypes();
 
         foreach ($customNoteTypes as $custNote) {
-            if ($custNote['note_custom_type'] == 'Unit Test To Do') {
-                self::$createdCustomNotes[]='Unit Test To Do';
+            if ($custNote['note_custom_type']=='Unit Test To Do') {
+                self::$createdCustomNotes[] = 'Unit Test To Do';
             }
         }
 
@@ -158,7 +158,7 @@ 
                 && $customNoteTypeResult['affected']!=null
                 && $customNoteTypeResult['affected']==1
             ) {
-                self::$createdCustomNotes[]='Unit Test To Do';
+                self::$createdCustomNotes[] = 'Unit Test To Do';
             }
         }
     }
@@ -214,10 +214,10 @@ 
         );
 
         $this->assertNotNull($addressWithNotes);
-        $this->assertTrue(sizeof( $addressWithNotes->notes)>0);
+        $this->assertTrue(sizeof($addressWithNotes->notes)>0);
         $this->assertContainsOnlyInstancesOf(AddressNote::class, [$firstNote]);
         $this->assertEquals('Test Address Note', $firstNote->contents);
-        $this->assertEquals('dropoff',$firstNote->activity_type);
+        $this->assertEquals('dropoff', $firstNote->activity_type);
     }
 
     public function testAddAddressNote()
@@ -296,14 +296,14 @@ 
 
         $customNoteType = $addressNote->getCustomNoteTypeByKey(self::$createdCustomNotes[0]);
 
-        $this->assertEquals(self::$createdCustomNotes[0],$customNoteType['note_custom_type']);
+        $this->assertEquals(self::$createdCustomNotes[0], $customNoteType['note_custom_type']);
     }
 
     public function testGetAllCustomNoteTypes()
     {
         $addressNote = new AddressNote();
 
-        $customNoteTypes =  $addressNote->getAllCustomNoteTypes();
+        $customNoteTypes = $addressNote->getAllCustomNoteTypes();
 
         $firstCustomNoteType = CustomNoteTypeResponse::fromArray($customNoteTypes[0]);
 
@@ -370,7 +370,7 @@ 
         $customNote = CustomNoteType::fromArray($note->custom_types[0]);
 
         $this->assertContainsOnlyInstancesOf(CustomNoteType::class, [$customNote]);
-        $this->assertEquals('Pickup package',$customNote->note_custom_value);
+        $this->assertEquals('Pickup package', $customNote->note_custom_value);
     }
 
     public function testRemoveCustomNoteType()

UnitTestFiles/Test/ActivityParametersUntTests.php

Spacing +3 added lines, -3 removed lines

@@ -19,9 +19,9 @@
 
     public static function setUpBeforeClass()
     {
-        if (Constants::API_KEY == Constants::DEMO_API_KEY) {
-            $className = str_replace('UnitTestFiles\\Test\\','',get_class());
-            self::markTestSkipped( 'PHPUnit will skip '. $className ." class - it impossible run with demo API key" );
+        if (Constants::API_KEY==Constants::DEMO_API_KEY) {
+            $className = str_replace('UnitTestFiles\\Test\\', '', get_class());
+            self::markTestSkipped('PHPUnit will skip '.$className." class - it impossible run with demo API key");
         }
 
         Route4Me::setApiKey(Constants::API_KEY);

UnitTestFiles/Test/V5/RouteTests.php

Spacing +12 added lines, -12 removed lines

@@ -302,7 +302,7 @@ 
         $result = $route->duplicateRoute($routeIDs);
 
         $this->assertNotNull($result);
-        $this->assertInstanceOf(RouteDuplicateResponse::class, RouteDuplicateResponse::fromArray( $result));
+        $this->assertInstanceOf(RouteDuplicateResponse::class, RouteDuplicateResponse::fromArray($result));
         $this->assertTrue($result['status']);
     }
 
@@ -313,9 +313,9 @@ 
         $result = $route->getRouteDataTableConfig();
 
         $this->assertNotNull($result);
-        $this->assertInstanceOf(  RouteDataTableConfigResponse::class, RouteDataTableConfigResponse::fromArray( $result));
-        $this->assertInstanceOf(  ApiCapabilities::class, ApiCapabilities::fromArray( $result['api_capabilities']));
-        $this->assertInstanceOf(  ApiPreferences::class, ApiPreferences::fromArray( $result['api_preferences']));
+        $this->assertInstanceOf(RouteDataTableConfigResponse::class, RouteDataTableConfigResponse::fromArray($result));
+        $this->assertInstanceOf(ApiCapabilities::class, ApiCapabilities::fromArray($result['api_capabilities']));
+        $this->assertInstanceOf(ApiPreferences::class, ApiPreferences::fromArray($result['api_preferences']));
     }
 
     public function testRouteDataTableFallbackConfig()
@@ -325,14 +325,14 @@ 
         $result = $route->getRouteDataTableFallbackConfig();
 
         $this->assertNotNull($result);
-        $this->assertInstanceOf(  RouteDataTableConfigResponse::class, RouteDataTableConfigResponse::fromArray( $result));
-        $this->assertInstanceOf(  ApiCapabilities::class, ApiCapabilities::fromArray( $result['api_capabilities']));
-        $this->assertInstanceOf(  ApiPreferences::class, ApiPreferences::fromArray( $result['api_preferences']));
+        $this->assertInstanceOf(RouteDataTableConfigResponse::class, RouteDataTableConfigResponse::fromArray($result));
+        $this->assertInstanceOf(ApiCapabilities::class, ApiCapabilities::fromArray($result['api_capabilities']));
+        $this->assertInstanceOf(ApiPreferences::class, ApiPreferences::fromArray($result['api_preferences']));
     }
 
     public function testUpdateRouteParameters()
     {
-        $routeQueryParams= new RouteParametersQuery();
+        $routeQueryParams = new RouteParametersQuery();
         $routeQueryParams->route_id = self::$testRoutes[sizeof(self::$testRoutes) - 1]->route_id;
         $routeQueryParams->original = 0;
 
@@ -349,21 +349,21 @@ 
         $result = $route->updateRouteParameters($routeQueryParams->toArray(), $routeParams->toArray());
 
         $this->assertNotNull($result);
-        $this->assertInstanceOf(  Route::class, Route::fromArray( $result));
+        $this->assertInstanceOf(Route::class, Route::fromArray($result));
     }
 
     public function  testDeleteRoutes()
     {
         $routeToDeleteID = self::$testRoutes[sizeof(self::$testRoutes) - 1]->route_id;
 
-        $routeIDs = [ $routeToDeleteID ];
+        $routeIDs = [$routeToDeleteID];
 
         $route = new Route();
 
         $result = $route->deleteRoutes($routeIDs);
 
         $this->assertNotNull($result);
-        $this->assertInstanceOf(RoutesDeleteResponse::class, RoutesDeleteResponse::fromArray( $result));
+        $this->assertInstanceOf(RoutesDeleteResponse::class, RoutesDeleteResponse::fromArray($result));
         $this->assertTrue($result['deleted']);
 
         self::$testRoutes = array_pop(self::$testRoutes);
@@ -381,7 +381,7 @@ 
             }
 
             if (sizeof(self::$removedOptimizationIDs)>0) {
-                $optimizationProblemIDs = array_merge($optimizationProblemIDs,self::$removedOptimizationIDs);
+                $optimizationProblemIDs = array_merge($optimizationProblemIDs, self::$removedOptimizationIDs);
             }
 
             $params = [

UnitTestFiles/Test/AvoidanceZoneUnitTests.php

Spacing +4 added lines, -4 removed lines

@@ -108,7 +108,7 @@ 
 
         $this->assertNotNull($avoidanceZones);
         $this->assertTrue(is_array($avoidanceZones));
-        $first=AvoidanceZone::fromArray($avoidanceZones[0]);
+        $first = AvoidanceZone::fromArray($avoidanceZones[0]);
         $this->assertContainsOnlyInstancesOf('Route4Me\AvoidanceZone', [$first]);
     }
 
@@ -159,7 +159,7 @@ 
 
     public function testDeleteAvoidanceZone()
     {
-        $territoryId = $this->createdAvoidanceZones[sizeof($this->createdAvoidanceZones)-1]->territory_id;
+        $territoryId = $this->createdAvoidanceZones[sizeof($this->createdAvoidanceZones) - 1]->territory_id;
         $result = $this->avoidanceZone->deleteAvoidanceZone($territoryId);
         //$result = AvoidanceZone::deleteAvoidanceZone($territoryId);
 
@@ -185,8 +185,8 @@ 
 
         $this->assertNotNull($updatedRoute);
         $this->assertContainsOnlyInstancesOf('Route4Me\AvoidanceZone', [$updatedRoute]);
-        $this->assertEquals('Test Territory Updated',$updatedRoute->territory_name, "The territory name not updated");
-        $this->assertEquals('ff5500',$updatedRoute->territory_color, "The territory color not updated");
+        $this->assertEquals('Test Territory Updated', $updatedRoute->territory_name, "The territory name not updated");
+        $this->assertEquals('ff5500', $updatedRoute->territory_color, "The territory color not updated");
     }
 
     public function tearDown()

Braces +1 added lines, -2 removed lines

@@ -199,8 +199,7 @@
 
             if ($result==true) {
                 echo "The test avoidance zone with territory_id=".$territoryId." removed. <br>";
-            }
-            else {
+            } else {
                 echo "Cannot remove the test avoidance zone with territory_id=".$territoryId."<br>";
             }
 

UnitTestFiles/Test/VehicleTests.php

Spacing +3 added lines, -3 removed lines

@@ -12,7 +12,7 @@ 
 
 class VehicleTests extends \PHPUnit\Framework\TestCase
 {
-    public static $createdVehicles=[];
+    public static $createdVehicles = [];
 
     public static function setUpBeforeClass()
     {
@@ -313,7 +313,7 @@ 
             VehicleCreateResponseV4::fromArray($response)
         );
 
-        $result =  VehicleCreateResponseV4::fromArray($response);
+        $result = VehicleCreateResponseV4::fromArray($response);
 
         $this->assertEquals('ISUZU FTR', $result->vehicle_alias);
         $this->assertEquals('1NP5DB9X93N507873', $result->vehicle_vin);
@@ -396,7 +396,7 @@ 
             VehicleCreateResponseV4::fromArray($response)
         );
 
-        $result =  VehicleCreateResponseV4::fromArray($response);
+        $result = VehicleCreateResponseV4::fromArray($response);
 
         $this->assertEquals('Peterbilt 579', $result->vehicle_alias);
         $this->assertEquals('1NP5DB9X93N507873', $result->vehicle_vin);