add unit test and fix lint errors

Author: mbirinci

src/core.ts

@@ -2,7 +2,7 @@ import {Module} from "./module";
 import {EVENT, RESOURCE_LOADING_TYPE} from "./enums";
 import {IPageFragmentConfig, IPageLibAsset, IPageLibConfiguration} from "./types";
 import {on} from "./decorators";
-import {AssetHelper} from "./assetHelper";
+import { AssetHelper } from "./assetHelper";
 
 export class Core extends Module {
   private static observer: IntersectionObserver | undefined;
@@ -56,11 +56,10 @@ export class Core extends Module {
   @on(EVENT.ON_PAGE_LOAD)
   static pageLoaded() {
     const onFragmentRenderAssets = Core.__pageConfiguration.assets.filter(asset => {
-      const fragment = Core.__pageConfiguration.fragments.find(fragment => fragment.name === asset.fragment);
-      if(fragment){
-        return asset.loadMethod === RESOURCE_LOADING_TYPE.ON_PAGE_RENDER && fragment.attributes.if !== "true" && !asset.preLoaded;
+      if(asset.loadMethod === RESOURCE_LOADING_TYPE.ON_PAGE_RENDER && !asset.preLoaded) {
+        const fragment = Core.__pageConfiguration.fragments.find(fragment => fragment.name === asset.fragment);
+        return fragment && fragment.attributes.if !== "true";
       }
-      return asset.loadMethod === RESOURCE_LOADING_TYPE.ON_PAGE_RENDER && !asset.preLoaded;
     });
 
     const scripts = Core.createLoadQueue(onFragmentRenderAssets);

src/modules/storage.ts

@@ -20,7 +20,7 @@ export class Storage extends Module {
 
     static async printApplicationCacheInfo() {
         const cacheNames = await caches.keys();
-        let storageList: { [key: string]: any } = {
+        const storageList: { [key: string]: any } = {
             total: 0
         };
 

src/types.ts

@@ -32,7 +32,7 @@ export interface IPageFragmentConfig {
   name: string;
   chunked: boolean;
   clientAsync: boolean;
-  attributes: { [name: string]: string };
+  attributes: { [name: string]: string, if: string };
   source: string | undefined;
 }
 

test/core.spec.ts

@@ -3,10 +3,14 @@ import {JSDOM} from "jsdom";
 import {PuzzleJs} from "../src/puzzle";
 import {Core} from "../src/core";
 import {createPageLibConfiguration} from "./mock";
+import sinon from "sinon";
+import {AssetHelper} from "../src/assetHelper";
 import * as faker from "faker";
 import {IPageLibAsset, IPageLibConfiguration, IPageLibDependency} from "../src/types";
 import {RESOURCE_LOADING_TYPE, RESOURCE_TYPE} from "../src/enums";
 
+const sandbox = sinon.createSandbox();
+
 declare global {
   interface Window {
     [key: string]: any;
@@ -25,6 +29,7 @@ declare var global: Global;
 describe('Module - Core', () => {
   beforeEach(() => {
     global.window = (new JSDOM(``, {runScripts: "outside-only"})).window;
+    sandbox.verifyAndRestore();
   });
 
   afterEach(() => {
@@ -164,4 +169,43 @@ describe('Module - Core', () => {
         expect(queue).to.deep.eq(
           []);
     });
+
+    it('should create true load queue for js assets excluding conditional fragments', function () {
+        const assets = [
+            {
+                name: 'bundle1',
+                dependent: ['vendor1'],
+                preLoaded: false,
+                link: 'bundle1.js',
+                fragment: 'test',
+                loadMethod: RESOURCE_LOADING_TYPE.ON_PAGE_RENDER,
+                type: RESOURCE_TYPE.JS
+            }
+        ] as IPageLibAsset[];
+        const dependencies = [
+            {
+                name: 'vendor1',
+                link: 'vendor1.js',
+                preLoaded: false
+            }
+        ] as IPageLibDependency[];
+        const config = {
+            dependencies,
+            assets,
+            fragments: [{
+                name: 'test',
+                attributes: {
+                    if: "true"
+                }
+            }],
+            page: 'page'
+        } as IPageLibConfiguration;
+
+        const mockLoadJsSeries = sandbox.mock(AssetHelper);
+
+        Core.config(JSON.stringify(config));
+        Core.pageLoaded();
+
+        mockLoadJsSeries.expects("loadJsSeries").calledWith([]);
+    });
 });