Images: Reject non-array images attribute

Author: mdtusz

src/components/images/defaults.js

@@ -15,11 +15,10 @@ var attributes = require('./attributes');
 
 module.exports = function supplyLayoutDefaults(layoutIn, layoutOut) {
 
-    if(!layoutIn.images) return;
+    if(!layoutIn.images || !Array.isArray(layoutIn.images)) return;
 
 
-    var containerIn = Array.isArray(layoutIn.images) ?
-            layoutIn.images : [layoutIn.images],
+    var containerIn = layoutIn.images,
         containerOut = layoutOut.images = [];
 
 

test/jasmine/tests/layout_images_test.js

@@ -24,6 +24,19 @@ describe('Layout images', function() {
             expect(layoutOut.images.length).toEqual(0);
         });
 
+        it('should reject when not an array', function() {
+            layoutIn.images = {
+                source: 'http://www.someimagesource.com',
+                opacity: 0.5,
+                width: 0.2,
+                height: 0.2
+            };
+
+            Images.supplyLayoutDefaults(layoutIn, layoutOut);
+
+            expect(layoutOut.images).not.toBeDefined();
+        });
+
         it('should coerce the correct defaults', function() {
             layoutIn.images[0] = { source: 'http://www.someimagesource.com' };