Make more friendly error reporting on fixture files collision in config values. Thank to @BlueAcorn

9beb881a · Ivan Chepurnyi · e0b986c6 · 9beb881a
Commit 9beb881a authored Oct 07, 2014 by Ivan Chepurnyi
Hide whitespace changes
Inline Side-by-side

Showing with 10 additions and 0 deletions

app/code/community/EcomDev/PHPUnit/Model/Fixture/Processor/Config.php ...munity/EcomDev/PHPUnit/Model/Fixture/Processor/Config.php +10 -0

No files found.
--- a/app/code/community/EcomDev/PHPUnit/Model/Fixture/Processor/Config.php
+++ b/app/code/community/EcomDev/PHPUnit/Model/Fixture/Processor/Config.php
@@ -160,6 +160,16 @@ class EcomDev_PHPUnit_Model_Fixture_Processor_Config
            $value = $backend->getValue();
        }

+        if (is_array($value)) {
+            Mage::throwException(
+                sprintf(
+                    'There is a collision in configuration value %s. Got: %s',
+                    $path,
+                    print_r($value, true)
+                )
+            );
+        }
+        
        Mage::getConfig()->setNode($path, $value);
        return $this;
    }