Merge pull request #12 from thisissoon/fix/on-changes

fix(image loader): clear src and srcset on changes and pre load image

Author: edoparearyee

.travis.yml

@@ -24,3 +24,4 @@ script:
   - export CHROME_BIN=chromium-browser
   - xvfb-run -a npm run test -- --no-progress --single-run --code-coverage --browser=ChromeNoSandbox
   - xvfb-run -a npm run e2e -- --no-progress
+  - npm run coverage

src/app/image-loader/image-loader/image-loader.component.spec.ts

@@ -218,4 +218,10 @@ describe('ImageLoaderComponent', () => {
     component.ngOnChanges();
     expect(spy).toHaveBeenCalled();
   });
+
+  it('should preload image on changes', () => {
+    const spy = spyOn(component, 'preloadImage');
+    component.ngOnChanges();
+    expect(spy).toHaveBeenCalled();
+  });
 });

src/app/image-loader/image-loader/image-loader.component.ts

@@ -340,6 +340,9 @@ export class ImageLoaderComponent implements OnInit, AfterViewInit, OnDestroy, O
    * @memberof ImageLoaderComponent
    */
   public ngOnChanges(): void {
+    this.src = '';
+    this.srcset = '';
     this.setPlaceholder();
+    this.preloadImage();
   }
 }