Merge pull request #2620 from mcottret/fix/fix-default-image-placehol…

…der-src

Fix ComponentImage's default src not being base64 encoded. Fixes #2619

src/dom_components/model/ComponentImage.js

@@ -75,7 +75,7 @@ export default Component.extend(
      */
     getAttrToHTML(...args) {
       const attr = Component.prototype.getAttrToHTML.apply(this, args);
-      const src = this.get('src');
+      const src = this.getSrcResult();
       if (src) attr.src = src;
       return attr;
     },

test/specs/dom_components/model/ComponentImage.js

@@ -0,0 +1,34 @@
+import Component from '../../../../src/dom_components/model/Component';
+import ComponentImage from '../../../../src/dom_components/model/ComponentImage';
+
+describe('ComponentImage', () => {
+  let componentImage;
+
+  beforeEach(() => {
+    componentImage = new ComponentImage();
+  });
+
+  describe('.getAttrToHTML', () => {
+    let getSrcResultSpy;
+    const fakeAttributes = {};
+
+    beforeEach(() => {
+      spyOn(Component.prototype, 'getAttrToHTML').and.returnValue(
+        fakeAttributes
+      );
+      getSrcResultSpy = spyOn(componentImage, 'getSrcResult');
+    });
+
+    test('it should fill the `src` property with the result of `getSrcResult` if defined', () => {
+      let attributes = componentImage.getAttrToHTML();
+      expect(getSrcResultSpy).toHaveBeenCalled();
+      expect(attributes).toEqual(fakeAttributes);
+
+      let fakeSrcResult = 'fakeSrcResult';
+      getSrcResultSpy.and.returnValue(fakeSrcResult);
+      attributes = componentImage.getAttrToHTML();
+      expect(getSrcResultSpy).toHaveBeenCalledTimes(2);
+      expect(attributes).toEqual({ src: fakeSrcResult });
+    });
+  });
+});