[Bug] Credentials are required to be filled (#14)

* Credentials are required to be filled

* Up version

Author: ArtARTs36

bin/mr-linter

@@ -38,7 +38,7 @@ if ($loaded === false) {
     exit(1);
 }
 
-$application = new Application('Merge Request Linter', '0.3.0');
+$application = new Application('Merge Request Linter', '0.5.1');
 
 $filesystem = new LocalFileSystem();
 $environment = new LocalEnvironment();

src/Configuration/Loader/AbstractArrayConfigLoader.php

@@ -35,7 +35,13 @@ public function load(string $path): Config
             throw new ConfigInvalidException('HTTP Client unavailable');
         }
 
-        return new Config($rules, $this->credentialMapper->map($data['credentials']), new HttpClientConfig(
+        $credentials = $this->credentialMapper->map($data['credentials']);
+
+        if ($credentials->isEmpty()) {
+            throw new ConfigInvalidException('Credentials must be filled');
+        }
+
+        return new Config($rules, $credentials, new HttpClientConfig(
             $data['http_client']['type'] ?? HttpClientConfig::TYPE_DEFAULT,
             $data['http_client']['params'] ?? [],
         ));

src/Configuration/Loader/PhpConfigLoader.php

@@ -51,8 +51,8 @@ protected function createFromArray(array $config): Config
             throw ConfigInvalidException::fromKey('rules');
         }
 
-        if (! isset($config['credentials']) || ! is_array($config['credentials'])) {
-            throw ConfigInvalidException::fromKey('credentials');
+        if (empty($config['credentials']) || ! is_array($config['credentials'])) {
+            throw new ConfigInvalidException('Credentials must be filled');
         }
 
         return new Config(