PR feedback changes.

Author: apathak18

.github/workflows/testing.yml

@@ -13,7 +13,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php-versions: ['8.1', '8.2']
+        php-versions: ['8.1', '8.2', '8.3']
         drupal-core: ['10.3.x']
         phpstan: ['0']
         include:

src/GraphQL/Utility/FileUpload.php

@@ -37,7 +37,7 @@ class FileUpload {
   use StringTranslationTrait;
 
   /**
-   * The entity storage for the 'file' entity type.
+   * The file storage where we will create new file entities from.
    *
    * @var \Drupal\file\FileStorageInterface
    */
@@ -294,10 +294,7 @@ public function saveFileUpload(UploadedFile $uploaded_file, array $settings): Fi
       $maxResolution = $settings['max_resolution'] ?? 0;
       $minResolution = $settings['min_resolution'] ?? 0;
       if (!empty($maxResolution) || !empty($minResolution)) {
-        $image_resolution_errors = $this->validateFileImageResolution($file, $maxResolution, $minResolution);
-        if (!empty($image_resolution_errors)) {
-          $errors = array_merge($errors, $image_resolution_errors);
-        }
+        $errors += $this->validateFileImageResolution($file, $maxResolution, $minResolution);
       }
 
       if (!empty($errors)) {

tests/src/Kernel/Framework/UploadFileServiceTest.php

@@ -15,6 +15,11 @@
  */
 class UploadFileServiceTest extends GraphQLTestBase {
 
+  /**
+   * {@inheritdoc}
+   */
+  protected static $modules = ['file_validator_test'];
+
   /**
    * The FileUpload object we want to test, gets prepared in setUp().
    *