update dependencies (#10)

* update dependencies

* fix lint errors

.npmignore

@@ -7,3 +7,5 @@ coverage
 .github
 schibstedspain-openads-appnexus-prebid-*.tgz
 versiona.js
+.nyc_output
+dist/test

.travis.yml

@@ -1,3 +1,4 @@
+dist: trusty
 language: node_js
 node_js:
   - "8"

package.json

@@ -8,10 +8,10 @@
     "cleanDistFolder": "rm -Rf ./dist",
     "prepublishOnly": "npm run cleanDistFolder && npm run build",
     "build": "npm run cleanDistFolder && babel src --ignore test --out-dir dist ",
-    "test": "mocha --recursive --require babel-polyfill --require babel-core/register \"src/test/**/*.js\"",
+    "test": "mocha --recursive --require @babel/register \"src/test/**/*.js\"",
     "lint": "sui-lint js",
-    "coverage": "istanbul cover --report html _mocha -- ./src/*test --recursive --compilers js:babel-register",
-    "coverage:ci": "istanbul cover _mocha -- ./src/*test --recursive --compilers js:babel-register && codecov",
+    "coverage": "nyc --reporter=html --exclude=\"src/test\" npm run test",
+    "coverage:ci": "nyc --reporter=cobertura --exclude=\"src/test\" npm run test && codecov",
     "check": "npm run lint && npm run test",
     "versiona": "node versiona.js"
   },
@@ -29,19 +29,20 @@
   "author": "One Punch Team",
   "license": "ISC",
   "devDependencies": {
-    "@s-ui/lint": "^2.18.0",
-    "babel-cli": "^6.24.0",
-    "babel-loader": "^7.1.1",
-    "babel-polyfill": "^6.23.0",
-    "babel-preset-sui": "1",
-    "babel-register": "^6.24.0",
-    "chai": "^4.0.2",
+    "@babel/cli": "^7.8.0",
+    "@babel/core": "^7.8.0",
+    "@babel/plugin-transform-modules-commonjs": "^7.8.0",
+    "@babel/register": "^7.8.0",
+    "@babel/runtime": "^7.8.0",
+    "@s-ui/lint": "^3.14.0",
+    "babel-loader": "^8.0.6",
+    "babel-preset-sui": "^2.14.0",
+    "chai": "^4.2.0",
     "codecov": "^3.0.1",
-    "istanbul": "^1.1.0-alpha.1",
     "jsdom": "^11.10.0",
-    "mocha": "^5.2.0",
-    "onchange": "^3.3.0",
-    "sinon": "^2.3.4",
+    "mocha": "^7.0.0",
+    "nyc": "^15.0.0",
+    "sinon": "^8.0.4",
     "versiona": "^4.2.1"
   },
   "pre-commit": [
@@ -51,6 +52,9 @@
   "babel": {
     "presets": [
       "sui"
+    ],
+    "plugins": [
+      "@babel/plugin-transform-modules-commonjs"
     ]
   },
   "eslintConfig": {
@@ -59,7 +63,11 @@
     ]
   },
   "dependencies": {
-    "@schibstedspain/openads-connector-api": "^1.2.0",
+    "@schibstedspain/openads-connector-api": "^2.0.1",
     "loglevel": "^1.6.1"
+  },
+  "prettier": "./node_modules/@s-ui/lint/.prettierrc.js",
+  "stylelint": {
+    "extends": "./node_modules/@s-ui/lint/stylelint.config.js"
   }
-}
+}

src/openads-appnexus/repository/AdListenerRepository.js

@@ -12,11 +12,8 @@ export default class AdListenerRepository {
   }
 
   find({id}) {
-    return Promise.resolve().then(
-      () =>
-        this._ads.has(id)
-          ? this._ads.get(id)
-          : this._registerAdSubscription({id})
+    return Promise.resolve().then(() =>
+      this._ads.has(id) ? this._ads.get(id) : this._registerAdSubscription({id})
     )
   }
 

src/openads-appnexus/service/generateUUID.js

@@ -1,7 +1,7 @@
 export const generateUUID = () => {
   let dt = new Date().getTime()
   return 'xxxxxxxx-xxxx-4xxx-yxxx-xxxxxxxxxxxx'.replace(/[xy]/g, c => {
-    let r = (dt + Math.random() * 16) % 16 | 0
+    const r = (dt + Math.random() * 16) % 16 | 0
     dt = Math.floor(dt / 16)
     return (c === 'x' ? r : (r & 0x3) | 0x8).toString(16)
   })

src/test/openads-appnexus/service/DebouncerTest.js

@@ -223,7 +223,7 @@ describe('Debouncer', () => {
         ]
       ]
 
-      let result = []
+      const result = []
 
       const debounceCallback = input => {
         result.push(input)

src/test/openads-appnexus/service/ReplayEventBusTest.js

@@ -45,9 +45,9 @@ describe('ReplayEventBus', () => {
     })
   })
   describe('Given a registered ReplayEventBus', () => {
-    let observerSpy = sinon.spy()
+    const observerSpy = sinon.spy()
     beforeEach(function() {
-      observerSpy.reset()
+      observerSpy.resetHistory()
     })
     it('Should execute observer callback using the raised payload', done => {
       const givenEventName = 'givenEventName'