Merge pull request #61 from rgazelot/top-level-missing

Add a more detailed exception message when a top-level key is missing.
Incenteev · Aug 20, 2014 · ea15f3d · ea15f3d
2 parents 15b15d8 + 1363665
commit ea15f3d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Processor.php b/Processor.php
@@ -33,7 +33,7 @@ public function processFile(array $config)
         // Find the expected params
         $expectedValues = $yamlParser->parse(file_get_contents($config['dist-file']));
         if (!isset($expectedValues[$parameterKey])) {
-            throw new \InvalidArgumentException('The dist file seems invalid.');
+            throw new \InvalidArgumentException(sprintf('The top-level key %s is missing.', $parameterKey));
         }
         $expectedParams = (array) $expectedValues[$parameterKey];
 

diff --git a/Tests/ProcessorTest.php b/Tests/ProcessorTest.php
@@ -74,7 +74,7 @@ public function provideInvalidConfiguration()
                 array(
                     'file' => 'fixtures/invalid/missing_top_level.yml',
                 ),
-                'The dist file seems invalid.',
+                'The top-level key parameters is missing.',
             ),
             'invalid values in the existing file' => array(
                 array(