Proper husky cfg;

Improved npmignore;
Imprtoved test;

Author: xobotyi

.npmignore

@@ -3,6 +3,8 @@ tests/
 examples/
 .idea/
 src/
+.prettierignore
+.prettierrc
 .travis.yml
 CODE_OF_CONDUCT.md
 karma.conf.js

package.json

@@ -67,6 +67,12 @@
     "webpack-cli": "^3.1.2",
     "webpack-dev-server": "^3.1.9"
   },
+  "husky": {
+    "hooks": {
+      "pre-commit": "npm run format && git add .",
+      "post-commit": "git update-index -g"
+    }
+  },
   "scripts": {
     "build": "rimraf ./dist && babel src -d dist --ignore \"node-modules/**\" --presets \"@babel/preset-react,@babel/preset-env\" --plugins \"@babel/plugin-proposal-class-properties\"",
     "push-codacy-coverage": "cat ./coverage/lcov.info | codacy-coverage -p .",
@@ -77,7 +83,6 @@
     "test:coverage": "cross-env NODE_ENV=test COVERAGE=true karma start --single-run",
     "format": "prettier --write src/**/*.js tests/**/*.js examples/**/*.js",
     "format:list": "prettier -l src/**/*.js tests/**/*.js examples/**/*.js",
-    "precommit": "npm run format && git add",
     "prepublishOnly": "npm run test && npm run build"
   }
 }

src/util/LoopController.js

@@ -48,6 +48,8 @@ function LoopControllerClass() {
 
         animationFrameId && cancelAnimationFrame(animationFrameId);
         requestAnimationFrame(animationFrameCallback);
+
+        return this;
     };
     /**
      * Stop the loop if it is active
@@ -62,6 +64,8 @@ function LoopControllerClass() {
 
         animationFrameId && cancelAnimationFrame(animationFrameId);
         animationFrameId = null;
+
+        return this;
     };
 
     /**

src/util/utilities.js

@@ -14,14 +14,14 @@ let scrollbarWidth = null;
  * @return {number}
  */
 export function getScrollbarWidth() {
-    if (!isset(document)) {
-        return 0;
-    }
-
     if (scrollbarWidth !== null) {
         return scrollbarWidth;
     }
 
+    if (!isset(document)) {
+        return (scrollbarWidth = 0);
+    }
+
     let el = document.createElement("div");
     el.setAttribute("style", "display:block;position:absolute;width:100px;height:100px;top:-9999px;overflow:scroll;");
 

tests/LoopController.spec.js

@@ -27,6 +27,8 @@ describe("LoopController", () => {
         let callCount = spy.callCount * 1;
         LoopController.stop();
 
+        LoopController.stop(); // 2nd call is just for coverage
+
         setTimeout(() => {
             expect(spy.callCount).toEqual(callCount);
             done();