Revert "Improve code style"

This reverts commit 9bb73d2.

Author: Naktibalda

.github/workflows/main.yml

@@ -3,25 +3,6 @@ name: CI
 on: [push, pull_request]
 
 jobs:
-  phpcs:
-    name: Code style
-    runs-on: 'ubuntu-latest'
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v3
-
-      - name: Install PHP
-        uses: shivammathur/setup-php@v2
-        with:
-          php-version: '8.1'
-          ini-values: memory_limit=-1, date.timezone='UTC'
-          tools: phpcs
-
-      - name: Check production code style
-        run: phpcs src/
-
-      - name: Check test code style
-        run: phpcs tests/ --standard=tests/phpcs.xml
   tests:
     runs-on: ubuntu-latest
 
@@ -30,7 +11,7 @@ jobs:
 
     strategy:
       matrix:
-        php: [8.0, 8.1, 8.2]
+        php: [8.0, 8.1]
 
     steps:
     - name: Checkout code

phpcs.xml

@@ -15,8 +15,8 @@ class JsonType extends Constraint
 {
     public function __construct(
         protected array $jsonType,
-        private bool $match = true
-    ) {
+        private bool $match = true)
+    {
     }
 
     /**

src/Codeception/Constraint/JsonType.php

@@ -92,6 +92,7 @@
  * ## Conflicts
  *
  * Conflicts with SOAP module
+ *
  */
 class REST extends Module implements DependsOnModule, PartedModule, API, ConflictsWithModule
 {
@@ -231,8 +232,8 @@ public function haveHttpHeader(string $name, string $value): void
      * ```
      *
      * @param string $name the name of the header to delete.
-     * @part  json
-     * @part  xml
+     * @part json
+     * @part xml
      */
     public function deleteHeader(string $name): void
     {
@@ -301,10 +302,10 @@ public function seeHttpHeaderOnce(string $name): void
     /**
      * Returns the value of the specified header name
      *
-     * @param  bool $first Whether to return the first value or all header values
+     * @param bool $first Whether to return the first value or all header values
      * @return string|array|null The first header value if $first is true, an array of values otherwise
-     * @part   json
-     * @part   xml
+     * @part json
+     * @part xml
      */
     public function grabHttpHeader(string $name, bool $first = true): string|array|null
     {
@@ -365,8 +366,8 @@ public function amBearerAuthenticated(string $accessToken): void
      * ```
      *
      * @throws ModuleException
-     * @part   json
-     * @part   xml
+     * @part json
+     * @part xml
      */
     public function amNTLMAuthenticated(string $username, string $password): void
     {
@@ -401,7 +402,6 @@ public function amNTLMAuthenticated(string $username, string $password): void
      * <?php
      * $I->amAWSAuthenticated();
      * ```
-     *
      * @throws ConfigurationException
      */
     public function amAWSAuthenticated(array $additionalAWSConfig = []): void
@@ -461,16 +461,16 @@ public function amAWSAuthenticated(array $additionalAWSConfig = []): void
      *                     keys: name, type, error, size, tmp_name (pointing to the real file path). Each key works
      *                     as the "name" attribute of a file input field.
      *
-     * @see  https://php.net/manual/en/features.file-upload.post-method.php
-     * @see  codecept_data_dir()
+     * @see https://php.net/manual/en/features.file-upload.post-method.php
+     * @see codecept_data_dir()
      * @part json
      * @part xml
      */
     public function sendPost(
         string $url,
         array|string|ArrayAccess|JsonSerializable $params = [],
-        array $files = []
-    ): ?string {
+        array $files = []): ?string
+    {
         return $this->execute('POST', $url, $params, $files);
     }
 
@@ -530,7 +530,8 @@ public function sendPut(
         string $url,
         array|string|ArrayAccess|JsonSerializable $params = [],
         array $files = []
-    ): ?string {
+    ): ?string
+    {
         return $this->execute('PUT', $url, $params, $files);
     }
 
@@ -549,7 +550,8 @@ public function sendPatch(
         string $url,
         array|string|ArrayAccess|JsonSerializable $params = [],
         array $files = []
-    ): ?string {
+    ): ?string
+    {
         return $this->execute('PATCH', $url, $params, $files);
     }
 
@@ -568,7 +570,8 @@ public function sendDelete(
         string $url,
         array|string|ArrayAccess|JsonSerializable $params = [],
         array $files = []
-    ): ?string {
+    ): ?string
+    {
         return $this->execute('DELETE', $url, $params, $files);
     }
 
@@ -582,8 +585,8 @@ public function send(
         string $method,
         string $url,
         array|string|ArrayAccess|JsonSerializable $params = [],
-        array $files = []
-    ): ?string {
+        array $files = []): ?string
+    {
         return $this->execute(strtoupper($method), $url, $params, $files);
     }
 
@@ -624,8 +627,8 @@ private function setHeaderLink(array $linkEntries): void
      * @link https://tools.ietf.org/html/rfc2068#section-19.6.2.4
      *
      * @author [email protected]
-     * @part   json
-     * @part   xml
+     * @part json
+     * @part xml
      */
     public function sendLink(string $url, array $linkEntries): void
     {
@@ -636,11 +639,11 @@ public function sendLink(string $url, array $linkEntries): void
     /**
      * Sends UNLINK request to given uri.
      *
-     * @param  array $linkEntries (entry is array with keys "uri" and "link-param")
-     * @link   https://tools.ietf.org/html/rfc2068#section-19.6.2.4
+     * @param array $linkEntries (entry is array with keys "uri" and "link-param")
+     * @link https://tools.ietf.org/html/rfc2068#section-19.6.2.4
      * @author [email protected]
-     * @part   json
-     * @part   xml
+     * @part json
+     * @part xml
      */
     public function sendUnlink(string $url, array $linkEntries): void
     {
@@ -655,8 +658,8 @@ protected function execute(
         string $method,
         string $url,
         array|string|ArrayAccess|JsonSerializable $parameters = [],
-        array $files = []
-    ): ?string {
+        array $files = []): ?string
+    {
         // allow full url to be requested
         if (!$url) {
             $url = $this->config['url'];
@@ -693,8 +696,7 @@ protected function execute(
                 $this->debugSection("Request", sprintf('%s %s', $method, $url));
                 $files = [];
             } else {
-                $this->debugSection(
-                    "Request",
+                $this->debugSection("Request",
                     sprintf('%s %s ', $method, $url) . json_encode($parameters, JSON_PRESERVE_ZERO_FRACTION | JSON_THROW_ON_ERROR)
                 );
                 $files = $this->formatFilesArray($files);
@@ -744,7 +746,7 @@ protected function isBinaryData(string $data): bool
     /**
      * Format a binary string for debug printing
      *
-     * @param  string $data the binary data string
+     * @param string $data the binary data string
      * @return string the debug string
      */
     protected function binaryToDebugString(string $data): string
@@ -755,11 +757,13 @@ protected function binaryToDebugString(string $data): string
     protected function encodeApplicationJson(
         string $method,
         array|string|ArrayAccess|JsonSerializable $parameters,
-    ): array|string {
+    ): array|string
+    {
         if (
             array_key_exists('Content-Type', $this->connectionModule->headers)
             && ($this->connectionModule->headers['Content-Type'] === 'application/json'
-            || preg_match('#^application/.+\+json$#', $this->connectionModule->headers['Content-Type']))
+                || preg_match('#^application/.+\+json$#', $this->connectionModule->headers['Content-Type'])
+            )
         ) {
             if ($parameters instanceof JsonSerializable) {
                 return json_encode($parameters, JSON_PRESERVE_ZERO_FRACTION | JSON_THROW_ON_ERROR);
@@ -821,7 +825,6 @@ private function formatFilesArray(array $files): array
             } elseif (is_object($value)) {
                 /**
                  * do nothing, probably the user knows what he is doing
-                 *
                  * @issue https://github.com/Codeception/Codeception/issues/3298
                  */
             } else {
@@ -858,6 +861,7 @@ private function checkFileBeforeUpload(string $file): void
 
     /**
      * Extends the function Module::validateConfig for shorten messages
+     *
      */
     protected function validateConfig(): void
     {
@@ -991,7 +995,7 @@ public function seeResponseIsValidOnJsonSchemaString(string $schema): void
      * Supply schema as relative file path in your project directory or an absolute path
      *
      * @part json
-     * @see  codecept_absolute_path()
+     * @see codecept_absolute_path()
      */
     public function seeResponseIsValidOnJsonSchema(string $schemaFilename): void
     {
@@ -1006,7 +1010,7 @@ public function seeResponseIsValidOnJsonSchema(string $schemaFilename): void
     /**
      * Converts string to json and asserts that no error occurred while decoding.
      *
-     * @param string $jsonString  the json encoded string
+     * @param string $jsonString the json encoded string
      * @param string $errorFormat optional string for custom sprintf format
      */
     protected function decodeAndValidateJson(string $jsonString, string $errorFormat = "Invalid json: %s. System message: %s.")
@@ -1059,7 +1063,7 @@ public function grabResponse(): string
      *
      * @return array Array of matching items
      * @throws Exception
-     * @part   json
+     * @part json
      */
     public function grabDataFromResponseByJsonPath(string $jsonPath): array
     {
@@ -1102,7 +1106,6 @@ public function grabDataFromResponseByJsonPath(string $jsonPath): array
      * // at least one item in store has price
      * $I->seeResponseJsonMatchesXpath('/store//price');
      * ```
-     *
      * @part json
      */
     public function seeResponseJsonMatchesXpath(string $xPath): void
@@ -1149,7 +1152,6 @@ public function seeResponseJsonMatchesXpath(string $xPath): void
      * // count the number of books written by given author is 5
      * $I->seeResponseJsonMatchesXpath("//author[text() = 'Nigel Rees']", 1.0);
      * ```
-     *
      * @part json
      */
     public function seeResponseJsonXpathEvaluatesTo(string $xPath, $expected): void
@@ -1161,7 +1163,7 @@ public function seeResponseJsonXpathEvaluatesTo(string $xPath, $expected): void
             "Received JSON did not evualated XPath `{$xPath}` as expected.\nJson Response: \n" . $response
         );
     }
-
+   
     /**
      * Opposite to seeResponseJsonXpathEvaluatesTo
      *
@@ -1345,7 +1347,7 @@ public function dontSeeResponseContainsJson(array $json = []): void
      * See [JsonType reference](https://codeception.com/docs/reference/JsonType).
      *
      * @part json
-     * @see  JsonType
+     * @see JsonType
      */
     public function seeResponseMatchesJsonType(array $jsonType, string $jsonPath = null): void
     {
@@ -1360,9 +1362,9 @@ public function seeResponseMatchesJsonType(array $jsonType, string $jsonPath = n
     /**
      * Opposite to `seeResponseMatchesJsonType`.
      *
-     * @part  json
+     * @part json
      * @param array $jsonType JsonType structure
-     * @see   seeResponseMatchesJsonType
+     * @see seeResponseMatchesJsonType
      */
     public function dontSeeResponseMatchesJsonType(array $jsonType, string $jsonPath = null): void
     {
@@ -1502,7 +1504,6 @@ public function seeResponseIsXml(): void
      * <?php
      * $I->seeXmlResponseMatchesXpath('//root/user[@id=1]');
      * ```
-     *
      * @part xml
      */
     public function seeXmlResponseMatchesXpath(string $xPath): void
@@ -1518,7 +1519,6 @@ public function seeXmlResponseMatchesXpath(string $xPath): void
      * <?php
      * $I->dontSeeXmlResponseMatchesXpath('//root/user[@id=1]');
      * ```
-     *
      * @part xml
      */
     public function dontSeeXmlResponseMatchesXpath(string $xPath): void
@@ -1572,7 +1572,7 @@ public function seeXmlResponseEquals(DOMDocument|string $xml): void
      * Comparison is done by canonicalizing both xml`s.
      *
      * @param mixed $xml
-     * @part  xml
+     * @part xml
      */
     public function dontSeeXmlResponseEquals(DOMDocument|string $xml): void
     {
@@ -1650,8 +1650,8 @@ public function dontSeeXmlResponseIncludes(DOMNode|XmlBuilder|array|string $xml)
      *
      * @param string $hash the hashed data response expected
      * @param string $algo the hash algorithm to use. Default sha1.
-     * @part  json
-     * @part  xml
+     * @part json
+     * @part xml
      */
     public function seeBinaryResponseEquals(string $hash, string $algo = 'sha1'): void
     {
@@ -1670,8 +1670,8 @@ public function seeBinaryResponseEquals(string $hash, string $algo = 'sha1'): vo
      *
      * @param string $hash the hashed data response expected
      * @param string $algo the hash algorithm to use. Default md5.
-     * @part  json
-     * @part  xml
+     * @part json
+     * @part xml
      */
     public function dontSeeBinaryResponseEquals(string $hash, string $algo = 'sha1'): void
     {

src/Codeception/Module/REST.php

@@ -25,9 +25,7 @@ public static function getTemplate(Template $template): ?Template
         $action = $template->getVar('action');
 
         // should only be applied to send* methods
-        if (!str_starts_with($action, 'send')) {
-            return null;
-        }
+        if (!str_starts_with($action, 'send')) return null;
 
         $conditionalDoc = "* JSON response will be automatically decoded \n     " . $template->getVar('doc');
 

src/Codeception/Step/AsJson.php

@@ -34,7 +34,7 @@ public function containsArray(array $needle): bool
     /**
      * @author [email protected]
      * @author [email protected]
-     * @link   https://www.php.net/manual/en/function.array-intersect-assoc.php#39822
+     * @link https://www.php.net/manual/en/function.array-intersect-assoc.php#39822
      */
     private function arrayIntersectRecursive(mixed $arr1, mixed $arr2): bool|array|null
     {