Adds basic unit tests for scripts/stylesheets

Author: Brad Berger

.gitignore

@@ -1,2 +1,3 @@
 node_modules/
 bower_components/
+reports/

README.md

@@ -20,7 +20,6 @@ do not support those features.
 
 - `Array.isArray` [Browser support](http://kangax.github.io/compat-table/es5/#Array.isArray)
 - `Array.prototype.forEach` [Browser support](http://kangax.github.io/compat-table/es5/#Array.prototype.forEach)
-- `String.prototype.endsWith` [Browser support](https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_Objects/String/endsWith#Browser_compatibility)
 - `localStorage` [Browser support](http://caniuse.com/#search=localstorage)
 - `Array.prototype.map` [Browser support](https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_Objects/Array/map#Browser_compatibility)
 - `window.requestAnimationFrame` [Browser support](https://developer.mozilla.org/en-US/docs/Web/API/window/requestAnimationFrame)

dist/static.compat.js

@@ -11,6 +11,7 @@ module.exports = function(config) {
 
         // list of files / patterns to load in the browser
         files: [
+            "dist/static.js",
             "test/**/*.spec.js"
         ],
 
@@ -21,12 +22,18 @@ module.exports = function(config) {
         // preprocess matching files before serving them to the browser
         // available preprocessors: https://npmjs.org/browse/keyword/karma-preprocessor
         preprocessors: {
+            "src/static.js": ["coverage"]
+        },
+
+        coverageReporter: {
+            type: "html",
+            dir: "reports/"
         },
 
         // test results reporter to use
         // possible values: "dots", "progress"
         // available reporters: https://npmjs.org/browse/keyword/karma-reporter
-        reporters: ["progress"],
+        reporters: ["dots", "progress", "coverage"],
 
         // web server port
         port: 9876,

dist/static.js

@@ -27,6 +27,7 @@
     "gulp-uglify": "^1.2.0",
     "karma": "^0.13.10",
     "karma-chrome-launcher": "^0.2.0",
+    "karma-coverage": "^0.5.2",
     "karma-firefox-launcher": "^0.1.6",
     "karma-jasmine": "^0.3.6"
   }

karma.conf.js

@@ -1,171 +1,3 @@
-// Production steps of ECMA-262, Edition 5, 15.4.4.18
-// Reference: http://es5.github.io/#x15.4.4.18
-if (!Array.prototype.forEach) {
-
-    Array.prototype.forEach = function(callback, thisArg) {
-
-        var T, k;
-
-        if (this == null) {
-            throw new TypeError(" this is null or not defined");
-        }
-
-        // 1. Let O be the result of calling ToObject passing the |this| value as the argument.
-        var O = Object(this);
-
-        // 2. Let lenValue be the result of calling the Get internal method of O with the argument "length".
-        // 3. Let len be ToUint32(lenValue).
-        var len = O.length >>> 0;
-
-        // 4. If IsCallable(callback) is false, throw a TypeError exception.
-        // See: http://es5.github.com/#x9.11
-        if (typeof callback !== "function") {
-            throw new TypeError(callback + " is not a function");
-        }
-
-        // 5. If thisArg was supplied, let T be thisArg; else let T be undefined.
-        if (arguments.length > 1) {
-            T = thisArg;
-        }
-
-        // 6. Let k be 0
-        k = 0;
-
-        // 7. Repeat, while k < len
-        while (k < len) {
-
-            var kValue;
-
-            // a. Let Pk be ToString(k).
-            //   This is implicit for LHS operands of the in operator
-            // b. Let kPresent be the result of calling the HasProperty internal method of O with argument Pk.
-            //   This step can be combined with c
-            // c. If kPresent is true, then
-            if (k in O) {
-
-                // i. Let kValue be the result of calling the Get internal method of O with argument Pk.
-                kValue = O[k];
-
-                // ii. Call the Call internal method of callback with T as the this value and
-                // argument list containing kValue, k, and O.
-                callback.call(T, kValue, k, O);
-            }
-            // d. Increase k by 1.
-            k++;
-        }
-        // 8. return undefined
-    };
-}
-
-// Production steps of ECMA-262, Edition 5, 15.4.4.19
-// Reference: http://es5.github.io/#x15.4.4.19
-if (!Array.prototype.map) {
-
-    Array.prototype.map = function(callback, thisArg) {
-
-        var T, A, k;
-
-        if (this == null) {
-            throw new TypeError(" this is null or not defined");
-        }
-
-        // 1. Let O be the result of calling ToObject passing the |this|
-        //    value as the argument.
-        var O = Object(this);
-
-        // 2. Let lenValue be the result of calling the Get internal
-        //    method of O with the argument "length".
-        // 3. Let len be ToUint32(lenValue).
-        var len = O.length >>> 0;
-
-        // 4. If IsCallable(callback) is false, throw a TypeError exception.
-        // See: http://es5.github.com/#x9.11
-        if (typeof callback !== "function") {
-            throw new TypeError(callback + " is not a function");
-        }
-
-        // 5. If thisArg was supplied, let T be thisArg; else let T be undefined.
-        if (arguments.length > 1) {
-            T = thisArg;
-        }
-
-        // 6. Let A be a new array created as if by the expression new Array(len)
-        //    where Array is the standard built-in constructor with that name and
-        //    len is the value of len.
-        A = new Array(len);
-
-        // 7. Let k be 0
-        k = 0;
-
-        // 8. Repeat, while k < len
-        while (k < len) {
-
-            var kValue, mappedValue;
-
-            // a. Let Pk be ToString(k).
-            //   This is implicit for LHS operands of the in operator
-            // b. Let kPresent be the result of calling the HasProperty internal
-            //    method of O with argument Pk.
-            //   This step can be combined with c
-            // c. If kPresent is true, then
-            if (k in O) {
-
-                // i. Let kValue be the result of calling the Get internal
-                //    method of O with argument Pk.
-                kValue = O[k];
-
-                // ii. Let mappedValue be the result of calling the Call internal
-                //     method of callback with T as the this value and argument
-                //     list containing kValue, k, and O.
-                mappedValue = callback.call(T, kValue, k, O);
-
-                // iii. Call the DefineOwnProperty internal method of A with arguments
-                // Pk, Property Descriptor
-                // { Value: mappedValue,
-                //   Writable: true,
-                //   Enumerable: true,
-                //   Configurable: true },
-                // and false.
-
-                // In browsers that support Object.defineProperty, use the following:
-                // Object.defineProperty(A, k, {
-                //   value: mappedValue,
-                //   writable: true,
-                //   enumerable: true,
-                //   configurable: true
-                // });
-
-                // For best browser support, use the following:
-                A[k] = mappedValue;
-            }
-            // d. Increase k by 1.
-            k++;
-        }
-
-        // 9. return A
-        return A;
-    };
-}
-
-if (!Array.isArray) {
-    Array.isArray = function(arg) {
-        return Object.prototype.toString.call(arg) === "[object Array]";
-    };
-}
-
-
-if (!String.prototype.endsWith) {
-    String.prototype.endsWith = function(searchString, position) {
-        var subjectString = this.toString();
-        if (position === undefined || position > subjectString.length) {
-            position = subjectString.length;
-        }
-        position -= searchString.length;
-        var lastIndex = subjectString.indexOf(searchString, position);
-        return lastIndex !== -1 && lastIndex === position;
-    };
-}
-
 // http://paulirish.com/2011/requestanimationframe-for-smart-animating/
 // http://my.opera.com/emoller/blog/2011/12/20/requestanimationframe-for-smart-er-animating
 // requestAnimationFrame polyfill by Erik Möller. fixes from Paul Irish and Tino Zijdel

package.json

@@ -1,23 +1,11 @@
-(function(window) {
+(function() {
 
     var noop = function() { };
     var storage = window.localStorage || {
         getItem: noop,
         setItem: noop,
-        removeItem: noop
-    };
-
-    var isJavaScript = function(url) {
-        return url.toString().toLowerCase().endsWith(".js");  
-    };
-
-    var isCss = function(url) {
-        return url.toString().toLowerCase().endsWith(".css");
-    };
-
-    var isImage = function(url) {
-        var src = url.toLowerCase();
-        return src.endsWith(".png") || src.endsWith(".jpg") || src.endsWith(".jpeg") || src.endsWith(".webp");
+        removeItem: noop,
+        clear: noop
     };
 
     /**
@@ -31,6 +19,18 @@
         this.images = {};
     };
 
+    Static.prototype.isJS = function(url) {
+        return !!url.match(/.*\.js+$/i);
+    };
+
+    Static.prototype.isCSS = function(url) {
+        return !!url.match(/.*\.css+$/i);
+    };
+
+    Static.prototype.isImage = function(url) {
+        return !!url.match(/.*\.(jpe?g|png|webp|gif)+$/i);
+    };
+
     /**
      *
      * @param {string} bundle Name of the bundle.
@@ -91,13 +91,13 @@
 
         return this[bundle] = this.bundles[bundle] = Promise.all(resources.map(function(url) {
 
-            if (isJavaScript(url)) {
+            if (self.isJS(url)) {
                 return self.script(url);
             }
 
-            if (isCss(url)) {
+            if (self.isCSS(url)) {
                 return self.css(url);
-            }
+            } 
 
             return self.image(url);
 
@@ -389,15 +389,15 @@
 
         var promise = Promise.all(url.map(function(src) {
 
-            if (isJavaScript(src)) {
+            if (self.isJS(src)) {
                 return self.getScript(url);
             }
 
-            if (isCss(src)) {
+            if (self.isCSS(src)) {
                 return self.getStylesheet(url);
             }
 
-            if (isImage(url)) {
+            if (self.isImage(url)) {
                 return self.getImage(url);
             }
 
@@ -423,6 +423,13 @@
 
     };
 
+    Static.prototype._$$reset = function() {
+        this.bundles = [];
+        this.scripts = {};
+        this.stylesheets = {};
+        this.images = {};
+    };
+
     window.$static = new Static();
 
-})(window);
+})();