Merge pull request #619 from orangemug/maintenance/update-deps-20200217-v2

Update all the deps

.circleci/config.yml

@@ -18,8 +18,8 @@ templates:
 
     - run: mkdir -p /tmp/artifacts/logs
     - run: npm run build
-    - run: npm run lint
+    - run: npm run lint-js
-    - run: npm run lint-styles
+    - run: npm run lint-css
     - store_artifacts:
         path: /tmp/artifacts
         destination: /artifacts
@@ -42,8 +42,8 @@ templates:
     - run: mkdir -p /tmp/artifacts/logs
     - run: npm run build
     - run: npm run profiling-build
-    - run: npm run lint
+    - run: npm run lint-js
-    - run: npm run lint-styles
+    - run: npm run lint-css
     - run: DOCKER_HOST=localhost npm test
     - run: ./node_modules/.bin/istanbul report --include /tmp/artifacts/coverage/coverage.json --dir /tmp/artifacts/coverage html lcov
     - store_artifacts:

appveyor.yml

@@ -22,5 +22,5 @@ install:
 build_script:
   - npm run build
 test_script:
-  - npm run lint
+  - npm run lint-js
-  - npm run lint-styles
+  - npm run lint-css

config/wdio.conf.js

@@ -97,7 +97,7 @@ exports.config = {
     //
     screenshotPath: SCREENSHOT_PATH,
     // Note: This is here because @orangemug currently runs Maputnik inside a docker container.
-    host: process.env.DOCKER_HOST || "0.0.0.0",
+    hostname: process.env.DOCKER_HOST || "0.0.0.0",
     // Set a base URL in order to shorten url command calls. If your `url` parameter starts
     // with `/`, the base url gets prepended, not including the path portion of your baseUrl.
     // If your `url` parameter starts without a scheme or `/` (like `some/path`), the base url
@@ -118,7 +118,6 @@ exports.config = {
     // Services take over a specific job you don't want to take care of. They enhance
     // your test setup with almost no effort. Unlike plugins, they don't add new
     // commands. Instead, they hook themselves up into the test process.
-    services: ['selenium-standalone'],
     //
     // Framework you want to run your specs with.
     // The following are supported: Mocha, Jasmine, and Cucumber
@@ -147,12 +146,16 @@ exports.config = {
     onPrepare: function (config, capabilities) {
       return new Promise(function(resolve, reject) {
         var compiler = webpack(webpackConfig);
+        const serverHost = isDocker() ? "0.0.0.0" : "localhost";
+
         server = new WebpackDevServer(compiler, {
+          host: serverHost,
           stats: {
             colors: true
           }
         });
-        server.listen(testConfig.port, (isDocker() ? "0.0.0.0" : "localhost"), function(err) {
+
+        server.listen(testConfig.port, serverHost, function(err) {
           if(err) {
             reject(err);
           }

package.json

@@ -10,8 +10,9 @@
     "test": "cross-env NODE_ENV=test wdio config/wdio.conf.js",
     "test-watch": "cross-env NODE_ENV=test wdio config/wdio.conf.js --watch",
     "start": "webpack-dev-server --progress --profile --colors --config config/webpack.config.js",
-    "lint": "eslint --ext js --ext jsx src test",
+    "lint-js": "eslint --ext js --ext jsx src test",
-    "lint-styles": "stylelint \"src/styles/*.scss\""
+    "lint-css": "stylelint \"src/styles/*.scss\"",
+    "lint": "npm run lint-js && npm run lint-css"
   },
   "repository": {
     "type": "git",
@@ -21,11 +22,11 @@
   "license": "MIT",
   "homepage": "https://github.com/maputnik/editor#readme",
   "dependencies": {
-    "@babel/runtime": "^7.8.3",
+    "@babel/runtime": "^7.8.4",
     "@mapbox/mapbox-gl-rtl-text": "^0.2.3",
-    "@mapbox/mapbox-gl-style-spec": "^13.10.2",
+    "@mapbox/mapbox-gl-style-spec": "^13.12.0",
     "classnames": "^2.2.6",
-    "codemirror": "^5.49.0",
+    "codemirror": "^5.52.0",
     "color": "^3.1.2",
     "detect-browser": "^4.8.0",
     "file-saver": "^2.0.2",
@@ -37,14 +38,14 @@
     "lodash.get": "^4.4.2",
     "lodash.isequal": "^4.5.0",
     "lodash.throttle": "^4.1.1",
-    "mapbox-gl": "^1.6.1",
+    "mapbox-gl": "^1.8.0",
     "mapbox-gl-inspect": "^1.3.1",
     "maputnik-design": "github:maputnik/design#f7a2b4d",
-    "ol": "^6.1.1",
+    "ol": "^6.2.1",
-    "ol-mapbox-style": "^5.0.2",
+    "ol-mapbox-style": "^6.0.1",
     "prop-types": "^15.7.2",
     "react": "^16.12.0",
-    "react-aria-menubutton": "^6.2.0",
+    "react-aria-menubutton": "^6.3.0",
     "react-aria-modal": "^4.0.0",
     "react-autobind": "^1.0.6",
     "react-autocomplete": "^1.8.1",
@@ -53,10 +54,10 @@
     "react-dom": "^16.12.0",
     "react-file-reader-input": "^2.0.0",
     "react-icon-base": "^2.1.2",
-    "react-icons": "^3.8.0",
+    "react-icons": "^3.9.0",
     "react-motion": "^0.5.2",
-    "react-sortable-hoc": "^1.10.1",
+    "react-sortable-hoc": "^1.11.0",
-    "reconnecting-websocket": "^4.2.0",
+    "reconnecting-websocket": "^4.4.0",
     "slugify": "^1.3.6",
     "url": "^0.11.0"
   },
@@ -101,7 +102,7 @@
     }
   },
   "devDependencies": {
-    "@babel/core": "^7.6.3",
+    "@babel/core": "^7.8.4",
     "@babel/plugin-proposal-class-properties": "^7.5.5",
     "@babel/plugin-transform-runtime": "^7.6.2",
     "@babel/preset-env": "^7.6.3",
@@ -115,38 +116,38 @@
     "@wdio/sync": "^5.14.4",
     "babel-eslint": "^10.0.3",
     "babel-loader": "8.0.6",
-    "babel-plugin-istanbul": "^5.2.0",
+    "babel-plugin-istanbul": "^6.0.0",
     "babel-plugin-static-fs": "^3.0.0",
-    "copy-webpack-plugin": "^5.0.4",
+    "copy-webpack-plugin": "^5.1.1",
     "cors": "^2.8.5",
-    "cross-env": "^6.0.3",
+    "cross-env": "^7.0.0",
-    "css-loader": "^3.2.0",
+    "css-loader": "^3.4.2",
-    "eslint": "^6.5.1",
+    "eslint": "^6.8.0",
-    "eslint-plugin-react": "^7.16.0",
+    "eslint-plugin-react": "^7.18.3",
     "express": "^4.17.1",
-    "file-loader": "^4.2.0",
+    "file-loader": "^5.1.0",
     "html-webpack-inline-svg-plugin": "^1.3.0",
     "html-webpack-plugin": "^3.2.0",
     "is-docker": "^2.0.0",
     "istanbul": "^0.4.5",
-    "istanbul-lib-coverage": "^2.0.5",
+    "istanbul-lib-coverage": "^3.0.0",
-    "mkdirp": "^0.5.1",
+    "mkdirp": "^1.0.3",
-    "mocha": "^6.2.1",
+    "mocha": "^7.0.1",
-    "node-sass": "^4.12.0",
+    "node-sass": "^4.13.1",
-    "react-hot-loader": "^4.12.15",
+    "react-hot-loader": "^4.12.19",
-    "sass-loader": "^8.0.0",
+    "sass-loader": "^8.0.2",
-    "selenium-standalone": "^6.16.0",
+    "selenium-standalone": "^6.17.0",
-    "style-loader": "^1.0.0",
+    "style-loader": "^1.1.3",
-    "stylelint": "^11.0.0",
+    "stylelint": "^13.2.0",
-    "stylelint-config-recommended-scss": "^4.0.0",
+    "stylelint-config-recommended-scss": "^4.2.0",
-    "stylelint-scss": "^3.11.1",
+    "stylelint-scss": "^3.14.2",
     "transform-loader": "^0.2.4",
-    "uuid": "^3.3.3",
+    "uuid": "^3.4.0",
-    "webdriverio": "^5.14.5",
+    "webdriverio": "^5.18.7",
-    "webpack": "^4.41.0",
+    "webpack": "^4.41.6",
-    "webpack-bundle-analyzer": "^3.5.2",
+    "webpack-bundle-analyzer": "^3.6.0",
     "webpack-cleanup-plugin": "^0.5.1",
-    "webpack-cli": "^3.3.9",
+    "webpack-cli": "^3.3.11",
-    "webpack-dev-server": "^3.8.2"
+    "webpack-dev-server": "^3.10.3"
   }
 }

test/functional/modals/index.js

@@ -6,8 +6,10 @@ var helper = require("../helper");
 
 
 function closeModal(wdKey) {
+  const selector = wd.$(wdKey);
+
   browser.waitUntil(function() {
-    const elem = $(wdKey);
+    const elem = $(selector);
     return elem.isDisplayedInViewport();
   });
 
@@ -15,8 +17,9 @@ function closeModal(wdKey) {
   closeBtnSelector.click();
 
   browser.waitUntil(function() {
-    const elem = $(wdKey);
+    return browser.execute((selector) => {
-    return !elem.isDisplayed();
+      return !document.querySelector(selector);
+    }, selector);
   });
 }