Merge pull request #35 from puzzle-js/css-assets

Fix for loadAssetSeries

Author: cihadhoruzoglu

package-lock.json

@@ -1,7 +1,7 @@
 {
   "name": "@puzzle-js/client-lib",
   "main": "dist/index.js",
-  "version": "1.6.0",
+  "version": "1.6.1",
   "author": "<emre.kul@trendyol.com>",
   "license": "MIT",
   "repository": {

package.json

@@ -66,6 +66,10 @@ export class AssetHelper {
     }
 
     static loadAssetSeries(assets: IPageLibAsset[], callback?: Function) {
+        if (!assets.length && callback) {
+            callback();
+        }
+
         for (let i = 0, p: any = Promise.resolve(); i < assets.length; i++) {
             p = p.then(() => new Promise(resolve => {
                     const asset = assets[i];

src/assetHelper.ts

@@ -130,4 +130,18 @@ describe('Module - Asset Helper', () => {
         });
     });
 
+    it('should call given callback if assets length is 0', async (done) => {
+        // arrange
+        const spy = sinon.spy();
+        const assets: IPageLibAsset[] = [];
+
+        // act
+        await AssetHelper.loadAssetSeries(assets, spy);
+
+        // assert
+        setTimeout(() => {
+            expect(spy.calledOnce).to.eq(true);
+            done();
+        });
+    });
 });