Revert "Upgrade: packages (#813)" (#823)

This reverts commit d865b59.
box · Jul 18, 2018 · b3f1421 · b3f1421
1 parent b4fd3ac
commit b3f1421
Show file tree

Hide file tree

Showing 11 changed files with 1,441 additions and 2,129 deletions.
diff --git a/.eslintrc b/.eslintrc
@@ -124,18 +124,6 @@
                 "object": true,
                 "array": false
             }
-        ],
-        "react/button-has-type": [
-            "off"
-        ],
-        "no-else-return": [
-            "error",
-            {
-                "allowElseIf": true
-            }
-        ],
-        "import/no-cycle": [
-            "off"
         ]
     },
     "settings": {

diff --git a/package.json b/package.json
@@ -9,8 +9,8 @@
     "url": "[email protected]:box/box-content-preview.git"
   },
   "devDependencies": {
-    "@commitlint/cli": "^7.0.0",
-    "autoprefixer": "^8.6.3",
+    "@commitlint/cli": "^5.2.0",
+    "autoprefixer": "^7.2.1",
     "babel-core": "^6.26.0",
     "babel-eslint": "^8.0.3",
     "babel-loader": "^7.1.2",
@@ -25,20 +25,20 @@
     "babel-preset-react": "^6.23.0",
     "box-annotations": "^2.2.0",
     "box-locales": "^0.0.1",
-    "box-react-ui": "^23.5.0",
+    "box-react-ui": "^22.7.0",
     "chai": "^4.1.2",
     "chai-dom": "^1.5.0",
     "codeceptjs": "^1.1.8",
-    "conventional-changelog-cli": "^2.0.1",
-    "conventional-github-releaser": "^3.1.2",
+    "conventional-changelog-cli": "^1.3.5",
+    "conventional-github-releaser": "^2.0.0",
     "create-react-class": "^15.6.2",
     "css-loader": "^0.28.7",
     "cssnano-cli": "^1.0.5",
     "eslint": "^4.12.1",
-    "eslint-config-airbnb": "^17.0.0",
+    "eslint-config-airbnb": "^16.1.0",
     "eslint-config-prettier": "^2.9.0",
-    "eslint-import-resolver-webpack": "^0.10.0",
-    "eslint-plugin-babel": "^5.1.0",
+    "eslint-import-resolver-webpack": "^0.8.3",
+    "eslint-plugin-babel": "^4.1.1",
     "eslint-plugin-import": "^2.8.0",
     "eslint-plugin-jsx-a11y": "^6.0.2",
     "eslint-plugin-react": "^7.5.1",
@@ -50,7 +50,7 @@
     "husky": "^0.14.3",
     "i18n-webpack-plugin": "^1.0.0",
     "jsuri": "^1.3.1",
-    "karma": "^2.0.4",
+    "karma": "^1.5.0",
     "karma-chai": "^0.1.0",
     "karma-chai-dom": "^1.1.0",
     "karma-chrome-launcher": "^2.2.0",
@@ -64,10 +64,10 @@
     "karma-sinon": "^1.0.5",
     "karma-sinon-chai": "^1.3.3",
     "karma-sourcemap-loader": "^0.3.7",
-    "karma-webpack": "^3.0.0",
-    "lint-staged": "^7.2.0",
+    "karma-webpack": "^2.0.6",
+    "lint-staged": "^5.0.0",
     "lodash": "^4.17.5",
-    "mocha": "^5.2.0",
+    "mocha": "^4.0.1",
     "mock-local-storage": "^1.0.2",
     "mojito-rb-gen": "^0.0.1",
     "node-noop": "^1.0.0",
@@ -83,17 +83,17 @@
     "react": "^16.2.0",
     "react-dom": "^16.2.0",
     "react-virtualized": "^9.13.0",
-    "sass-loader": "^7.0.3",
+    "sass-loader": "^6.0.6",
     "saucelabs": "^1.4.0",
-    "selenium-webdriver": "^4.0.0-alpha.1",
-    "sinon": "^6.0.0",
-    "sinon-chai": "3.2.0",
-    "string-replace-loader": "^2.1.1",
-    "style-loader": "^0.21.0",
-    "stylelint": "^9.3.0",
+    "selenium-webdriver": "^3.6.0",
+    "sinon": "^4.1.2",
+    "sinon-chai": "2.14.0",
+    "string-replace-loader": "^1.3.0",
+    "style-loader": "^0.19.0",
+    "stylelint": "^8.3.1",
     "stylelint-config-standard": "^18.0.0",
-    "stylelint-order": "^0.8.1",
-    "stylelint-scss": "^3.1.3",
+    "stylelint-order": "^0.7.0",
+    "stylelint-scss": "^2.2.0",
     "webdriverio": "^4.12.0",
     "webpack": "^3.10.0",
     "webpack-bundle-analyzer": "^2.9.1",

diff --git a/src/lib/PageControls.js b/src/lib/PageControls.js
@@ -39,7 +39,6 @@ class PageControls extends EventEmitter {
 
     /** @property {HTMLElement} - Page number input element */
     pageNumInputEl;
-
     /**
      * [constructor]
      *

diff --git a/src/lib/__tests__/Controls-test.js b/src/lib/__tests__/Controls-test.js
@@ -1,6 +1,6 @@
 /* eslint-disable no-unused-expressions */
 import Controls from '../Controls';
-import { CLASS_HIDDEN } from '../constants';
+import { CLASS_HIDDEN } from './../constants';
 
 let controls;
 let clock;

diff --git a/src/lib/__tests__/DownloadReachability-test.js b/src/lib/__tests__/DownloadReachability-test.js
@@ -133,7 +133,6 @@ describe('lib/DownloadReachability', () => {
         afterEach(() => {
             fetchMock.restore();
         });
-
         it('should catch an errored response', () => {
             const setDownloadHostFallbackStub = sandbox.stub(DownloadReachability, 'setDownloadHostFallback');
             fetchMock.head('https://dl3.boxcloud.com', { throws: new Error() });

diff --git a/src/lib/__tests__/Preview-test.js b/src/lib/__tests__/Preview-test.js
@@ -1,4 +1,5 @@
 /* eslint-disable no-unused-expressions */
+import fetchMock from 'fetch-mock';
 import Preview from '../Preview';
 import loaders from '../loaders';
 import Logger from '../Logger';
@@ -39,6 +40,7 @@ describe('lib/Preview', () => {
     afterEach(() => {
         sandbox.verifyAndRestore();
         fixture.cleanup();
+        fetchMock.restore();
         preview.destroy();
         preview = null;
         stubs = null;

diff --git a/src/lib/viewers/doc/__tests__/DocPreloader-test.js b/src/lib/viewers/doc/__tests__/DocPreloader-test.js
@@ -162,16 +162,12 @@ describe('lib/viewers/doc/DocPreloader', () => {
 
     describe('cleanupPreload()', () => {
         it('should remove wrapper, clear out preload and image element, and revoke object URL', () => {
-            const removeChildStub = sandbox.stub();
-
-            docPreloader.wrapperEl = {
-                parentNode: {
-                    removeChild: removeChildStub
-                }
-            };
+            docPreloader.wrapperEl = document.createElement('div');
             docPreloader.preloadEl = document.createElement('div');
             docPreloader.imageEl = document.createElement('img');
             docPreloader.srcUrl = 'blah';
+            containerEl.appendChild(docPreloader.wrapperEl);
+
             sandbox
                 .mock(URL)
                 .expects('revokeObjectURL')
@@ -181,7 +177,7 @@ describe('lib/viewers/doc/DocPreloader', () => {
 
             expect(docPreloader.preloadEl).to.be.undefined;
             expect(docPreloader.imageEl).to.be.undefined;
-            expect(removeChildStub).to.be.called;
+            expect(containerEl).to.not.contain(docPreloader.wrapperEl);
         });
     });
 

diff --git a/src/lib/viewers/media/MediaBaseViewer.js b/src/lib/viewers/media/MediaBaseViewer.js
@@ -46,7 +46,6 @@ class MediaBaseViewer extends BaseViewer {
         this.handleAutoplay = this.handleAutoplay.bind(this);
         this.mediaendHandler = this.mediaendHandler.bind(this);
     }
-
     /**
      * @inheritdoc
      */

diff --git a/src/lib/viewers/text/BoxCSV.js b/src/lib/viewers/text/BoxCSV.js
@@ -43,7 +43,7 @@ class BoxCSV {
         return row % 2 === 0 ? 'bp-text-csv-even-row' : 'bp-text-csv-odd-row';
     }
 
-    /* eslint-disable */
+    /* eslint-disable react/prop-types */
     /**
      * Renders cell
      *
@@ -63,7 +63,7 @@ class BoxCSV {
             </div>
         );
     };
-    /* eslint-enable */
+    /* eslint-enable react/prop-types */
 
     /**
      * Renders CSV into an html table

diff --git a/src/lib/viewers/text/TextBaseViewer.js b/src/lib/viewers/text/TextBaseViewer.js
@@ -16,7 +16,6 @@ class TextBaseViewer extends BaseViewer {
         this.zoomOut = this.zoomOut.bind(this);
         this.zoomIn = this.zoomIn.bind(this);
     }
-
     /**
      * @inheritdoc
      */
-Original file line number
+Diff line change
@@ Expand Up / @@ -39,7 +39,6 @@ class PageControls extends EventEmitter { @@
         /** @property {HTMLElement} - Page number input element */
         pageNumInputEl;
         /**
          * [constructor]
          *
@@ Expand Down @@