Quality Fixes

Author: MichaelJ2324

src/Endpoint/Abstracts/AbstractSugarBeanEndpoint.php

@@ -188,11 +188,13 @@ protected function configurePayload(): mixed
                 if (isset($data[AbstractSugarBeanCollectionEndpoint::SUGAR_FIELDS_DATA_PROPERTY])) {
                     $fields = $data[AbstractSugarBeanCollectionEndpoint::SUGAR_FIELDS_DATA_PROPERTY];
                 }
+
                 $data->reset();
                 $data->set($this->toArray());
                 if (!empty($fields)) {
                     $data[AbstractSugarBeanCollectionEndpoint::SUGAR_FIELDS_DATA_PROPERTY] = $fields;
                 }
+
                 $syncKeyField = $this->getSyncKeyField();
                 if (!empty($syncKeyField)) {
                     $data[Integrate::DATA_SYNC_KEY_FIELD] = $syncKeyField;
@@ -250,8 +252,10 @@ protected function parseResponse(Response $response): void
                         } else {
                             $model = $body[Integrate::INTEGRATE_RESPONSE_PROP];
                         }
+
                         $this->syncFromApi($model);
                     }
+
                     return;
             }
         }

src/Endpoint/Integrate.php

@@ -105,6 +105,7 @@ protected function configureAction(string $action, array $arguments = []): void
                 $action = self::INTEGRATE_ACTION_RETRIEVE;
                 break;
         }
+
         if ($this->_action !== $action) {
             $this->_action = $action;
         }
@@ -188,6 +189,7 @@ protected function parseResponse(Response $response): void
                             } else {
                                 $model = $body[Integrate::INTEGRATE_RESPONSE_PROP];
                             }
+
                             $this->syncFromApi($model);
                             if (isset($this->_sugarBean)) {
                                 $this->_sugarBean->set($this->toArray());

src/Endpoint/Traits/IntegrateSyncKeyTrait.php

@@ -42,6 +42,7 @@ public function getSyncKey(): string|int|null
         } elseif (property_exists($this, '_attributes')) {
             $key = $this->_attributes[$field];
         }
+
         //@codeCoverageIgnoreEnd
         return $key;
     }

tests/Endpoint/AbstractSugarBeanEndpointTest.php

@@ -55,6 +55,7 @@ public function testCompileRequest(): void
 
         $Bean->setUrlArgs(['Accounts','12345']);
         $Bean->setCurrentAction(AbstractSugarBeanEndpoint::BEAN_ACTION_UPSERT);
+
         $Bean->sync_key = '67890';
         $Request = $Bean->compileRequest();
         $this->assertEquals("PATCH", $Request->getMethod());
@@ -780,7 +781,7 @@ public function testUpsertAction(): void
         $Bean->set([
             'name' => 'Test Account',
             'account_type' => 'Prospect',
-            'sync_key' => '098765'
+            'sync_key' => '098765',
         ]);
         $Bean->upsert();
 
@@ -811,7 +812,7 @@ public function testUpsertAction(): void
             'account_type' => 'Prospect',
             'sync_key' => '098765',
             'sync_key_field_value' => '098765',
-            'fields' => ['test']
+            'fields' => ['test'],
         ], json_decode($payload, true));
         $this->assertEquals('12345', $Bean->id);
         $this->assertEquals('foobar', $Bean->test);

tests/Endpoint/IntegrateTest.php

@@ -197,6 +197,7 @@ public function testGetBySyncKey(): void
         $this->client->mockResponses->append(new Response('200', [], json_encode($this->responsePayload)));
         $endpoint->setCurrentAction(Integrate::MODEL_ACTION_RETRIEVE);
         $endpoint->execute();
+
         $request = $this->client->mockResponses->getLastRequest();
         $this->assertEquals('GET', $request->getMethod());
         $this->assertEquals('/rest/v11/integrate/Accounts', $request->getUri()->getPath());