Update prettier to 2.0

Author: dgraham

.prettierrc

@@ -1,4 +1,5 @@
 {
+  "arrowParens": "avoid",
   "bracketSpacing": false,
   "semi": true,
   "singleQuote": true,

delegated-events.js

@@ -12,7 +12,7 @@ const currentTargetDesc = Object.getOwnPropertyDescriptor(
 
 function before(subject, verb, fn) {
   const source = subject[verb];
-  subject[verb] = function() {
+  subject[verb] = function () {
     fn.apply(subject, arguments);
     return source.apply(subject, arguments);
   };

package-lock.json

@@ -40,7 +40,7 @@
     "karma-chrome-launcher": "^3.1.0",
     "karma-mocha": "^1.3.0",
     "mocha": "^7.1.1",
-    "prettier": "^1.19.1",
+    "prettier": "^2.0.2",
     "rollup": "^1.32.1",
     "rollup-plugin-babel": "^4.4.0",
     "rollup-plugin-node-resolve": "^5.2.0",

package.json

@@ -1,6 +1,6 @@
 import {on, off} from '../delegated-events';
 
-(function() {
+(function () {
   const DEPTH = 25;
   const DISPATCHES = 1500;
 
@@ -36,12 +36,12 @@ import {on, off} from '../delegated-events';
     const handlers = new Map();
     return {
       name: 'native',
-      on: function(selector) {
+      on: function (selector) {
         const clone = matchHandler.bind(selector);
         handlers.set(selector, clone);
         document.addEventListener('test:bench', clone);
       },
-      off: function(selector) {
+      off: function (selector) {
         const handler = handlers.get(selector);
         handlers.delete(selector);
         document.removeEventListener('test:bench', handler);
@@ -52,10 +52,10 @@ import {on, off} from '../delegated-events';
   function delegated() {
     return {
       name: 'delegated',
-      on: function(selector) {
+      on: function (selector) {
         on('test:bench', selector, handler);
       },
-      off: function(selector) {
+      off: function (selector) {
         off('test:bench', selector, handler);
       }
     };
@@ -64,10 +64,10 @@ import {on, off} from '../delegated-events';
   function jquery() {
     return {
       name: 'jQuery',
-      on: function(selector) {
+      on: function (selector) {
         $(document).on('test:bench', selector, handler);
       },
-      off: function(selector) {
+      off: function (selector) {
         $(document).off('test:bench', selector, handler);
       }
     };
@@ -76,18 +76,18 @@ import {on, off} from '../delegated-events';
   function jqueryss() {
     return {
       name: 'jQuery + SelectorSet',
-      setup: function() {
-        return new Promise(function(resolve) {
+      setup: function () {
+        return new Promise(function (resolve) {
           const script = document.createElement('script');
           script.addEventListener('load', resolve);
           script.src = '../vendor/jquery-selector-set/jquery.selector-set.js';
           document.head.appendChild(script);
         });
       },
-      on: function(selector) {
+      on: function (selector) {
         $(document).on('test:bench', selector, handler);
       },
-      off: function(selector) {
+      off: function (selector) {
         $(document).off('test:bench', selector, handler);
       }
     };
@@ -96,10 +96,10 @@ import {on, off} from '../delegated-events';
   function zepto() {
     return {
       name: 'zepto',
-      on: function(selector) {
+      on: function (selector) {
         Zepto(document).on('test:bench', selector, handler);
       },
-      off: function(selector) {
+      off: function (selector) {
         Zepto(document).off('test:bench', selector, handler);
       }
     };
@@ -125,7 +125,7 @@ import {on, off} from '../delegated-events';
     const max = results.reduce((a, b) => (a.value > b.value ? a : b));
 
     results = results
-      .map(function(result, ix) {
+      .map(function (result, ix) {
         const percent = (100 * result.value) / max.value;
         return {
           name: result.name,
@@ -138,7 +138,7 @@ import {on, off} from '../delegated-events';
 
     const svg = document.querySelector('.js-results');
     const ns = 'http://www.w3.org/2000/svg';
-    results.forEach(function(result, ix) {
+    results.forEach(function (result, ix) {
       const row = document.createElementNS(ns, 'rect');
       row.setAttribute('fill', result.color);
       row.setAttribute('width', result.percent + '%');
@@ -170,12 +170,12 @@ import {on, off} from '../delegated-events';
   function run() {
     const selectors = build(DEPTH);
     const deepest = document.querySelector(selectors[selectors.length - 1]);
-    const results = [native, delegated, jquery, zepto, jqueryss].map(function(
+    const results = [native, delegated, jquery, zepto, jqueryss].map(function (
       test
     ) {
       const harness = test();
       const ready = harness.setup ? harness.setup() : Promise.resolve();
-      return ready.then(function() {
+      return ready.then(function () {
         selectors.forEach(harness.on);
         const duration = benchmark(dispatch, deepest);
         selectors.forEach(harness.off);

test/bench.js

@@ -10,7 +10,7 @@ function onButtonClick(event) {
 on('click', '.js-button', onButtonClick);
 off('click', 'js-button', onButtonClick);
 
-on('robot:singularity', '.js-robot-image', function(event) {
+on('robot:singularity', '.js-robot-image', function (event) {
   event.target;
   event.target.closest('.foo');
   // event.detail.name

test/test-flow.js

@@ -1,7 +1,7 @@
 import {on, off, fire} from '../delegated-events';
 
-describe('delegated event listeners', function() {
-  before(function() {
+describe('delegated event listeners', function () {
+  before(function () {
     const container = document.createElement('div');
     container.innerHTML = `
       <div class="js-test-parent">
@@ -10,9 +10,9 @@ describe('delegated event listeners', function() {
     document.body.appendChild(container);
   });
 
-  describe('custom event dispatch', function() {
-    it('fires custom events with detail', function() {
-      const observer = function(event) {
+  describe('custom event dispatch', function () {
+    it('fires custom events with detail', function () {
+      const observer = function (event) {
         assert(event.bubbles);
         assert(event.cancelable);
         assert.equal(event.type, 'test:detail');
@@ -25,8 +25,8 @@ describe('delegated event listeners', function() {
       document.removeEventListener('test:detail', observer);
     });
 
-    it('fires custom events without detail', function() {
-      const observer = function(event) {
+    it('fires custom events without detail', function () {
+      const observer = function (event) {
         assert(event.detail === undefined || event.detail === null);
         assert.instanceOf(event, CustomEvent);
       };
@@ -35,15 +35,15 @@ describe('delegated event listeners', function() {
       document.removeEventListener('test:fire', observer);
     });
 
-    it('returns canceled when default prevented', function() {
+    it('returns canceled when default prevented', function () {
       const observer = event => event.preventDefault();
       document.addEventListener('test:cancel', observer);
       const canceled = !fire(document.body, 'test:cancel');
       assert.equal(canceled, true);
       document.removeEventListener('test:cancel', observer);
     });
 
-    it('returns not canceled when default is not prevented', function() {
+    it('returns not canceled when default is not prevented', function () {
       const [observer, trace] = spy(event => assert.ok(event));
       document.addEventListener('test:event', observer);
       const canceled = !fire(document.body, 'test:event');
@@ -53,9 +53,9 @@ describe('delegated event listeners', function() {
     });
   });
 
-  describe('event observer registration', function() {
-    it('observes custom events', function() {
-      const observer = function(event) {
+  describe('event observer registration', function () {
+    it('observes custom events', function () {
+      const observer = function (event) {
         assert(event.bubbles);
         assert(event.cancelable);
         assert.equal(event.type, 'test:on');
@@ -71,21 +71,21 @@ describe('delegated event listeners', function() {
       off('test:on', 'body', observer);
     });
 
-    it('removes bubble event observers', function() {
+    it('removes bubble event observers', function () {
       const observer = event => assert.fail(event);
       on('test:off', '*', observer);
       off('test:off', '*', observer);
       fire(document.body, 'test:off');
     });
 
-    it('removes capture event observers', function() {
+    it('removes capture event observers', function () {
       const observer = event => assert.fail(event);
       on('test:off', '*', observer, {capture: true});
       off('test:off', '*', observer, {capture: true});
       fire(document.body, 'test:off');
     });
 
-    it('can reregister after removing', function() {
+    it('can reregister after removing', function () {
       const [observer, trace] = spy(event => assert.ok(event));
       on('test:register', 'body', observer);
       off('test:register', 'body', observer);
@@ -96,43 +96,43 @@ describe('delegated event listeners', function() {
     });
   });
 
-  describe('event propagation', function() {
-    before(function() {
+  describe('event propagation', function () {
+    before(function () {
       this.parent = document.querySelector('.js-test-parent');
       this.child = document.querySelector('.js-test-child');
     });
 
-    it('fires observers in tree order', function() {
+    it('fires observers in tree order', function () {
       const order = [];
 
       const parent = this.parent;
       const child = this.child;
 
-      const one = function(event) {
+      const one = function (event) {
         assert.strictEqual(child, event.target);
         assert.strictEqual(parent, event.currentTarget);
         assert.strictEqual(this, event.currentTarget);
         assert.strictEqual(this, parent);
         order.push(1);
       };
 
-      const two = function(event) {
+      const two = function (event) {
         assert.strictEqual(child, event.target);
         assert.strictEqual(child, event.currentTarget);
         assert.strictEqual(this, event.currentTarget);
         assert.strictEqual(this, child);
         order.push(2);
       };
 
-      const three = function(event) {
+      const three = function (event) {
         assert.strictEqual(child, event.target);
         assert.strictEqual(parent, event.currentTarget);
         assert.strictEqual(this, event.currentTarget);
         assert.strictEqual(this, parent);
         order.push(3);
       };
 
-      const four = function(event) {
+      const four = function (event) {
         assert.strictEqual(child, event.target);
         assert.strictEqual(child, event.currentTarget);
         assert.strictEqual(this, event.currentTarget);
@@ -153,7 +153,7 @@ describe('delegated event listeners', function() {
       assert.deepEqual([1, 2, 4, 3], order);
     });
 
-    it('clears currentTarget after propagation', function() {
+    it('clears currentTarget after propagation', function () {
       const [observer, trace] = spy(event => assert.ok(event.currentTarget));
       const event = new CustomEvent('test:clear', {bubbles: true});
 
@@ -164,7 +164,7 @@ describe('delegated event listeners', function() {
       off('test:clear', 'body', observer);
     });
 
-    it('does not interfere with currentTarget when selectors match', function() {
+    it('does not interfere with currentTarget when selectors match', function () {
       const [observer, trace] = spy(event =>
         assert.strictEqual(document.body, event.currentTarget)
       );
@@ -185,7 +185,7 @@ describe('delegated event listeners', function() {
       this.parent.removeEventListener('test:target:capture', observer2);
     });
 
-    it('does not interfere with currentTarget when no selectors match', function() {
+    it('does not interfere with currentTarget when no selectors match', function () {
       const [observer, trace] = spy(event => assert.ok(event.currentTarget));
       const event = new CustomEvent('test:currentTarget', {bubbles: true});
 
@@ -202,7 +202,7 @@ describe('delegated event listeners', function() {
       document.removeEventListener('test:currentTarget', observer);
     });
 
-    it('prevents redispatch after propagation is stopped', function() {
+    it('prevents redispatch after propagation is stopped', function () {
       const [observer, trace] = spy(event => event.stopPropagation());
       const event = new CustomEvent('test:redispatch', {bubbles: true});
 
@@ -217,7 +217,7 @@ describe('delegated event listeners', function() {
       off('test:redispatch', 'body', observer);
     });
 
-    it('stops propagation bubbling to parent', function() {
+    it('stops propagation bubbling to parent', function () {
       const one = event => assert.fail(event);
       const two = event => event.stopPropagation();
       on('test:bubble', '.js-test-parent', one);
@@ -227,7 +227,7 @@ describe('delegated event listeners', function() {
       off('test:bubble', '.js-test-child', two);
     });
 
-    it('stops immediate propagation', function() {
+    it('stops immediate propagation', function () {
       const one = event => event.stopImmediatePropagation();
       const two = event => assert.fail(event);
       on('test:immediate', '.js-test-child', one);
@@ -237,7 +237,7 @@ describe('delegated event listeners', function() {
       off('test:immediate', '.js-test-child', two);
     });
 
-    it('stops immediate propagation and bubbling', function() {
+    it('stops immediate propagation and bubbling', function () {
       const one = event => assert.fail(event);
       const two = event => event.stopImmediatePropagation();
       on('test:stop', '.js-test-parent', one);
@@ -247,10 +247,10 @@ describe('delegated event listeners', function() {
       off('test:stop', '.js-test-child', two);
     });
 
-    it('calculates selector matches before dispatching event', function() {
+    it('calculates selector matches before dispatching event', function () {
       this.child.classList.add('inactive');
 
-      const one = function(event) {
+      const one = function (event) {
         event.target.classList.remove('inactive');
         event.target.classList.add('active');
         assert.ok(event);
@@ -270,7 +270,7 @@ describe('delegated event listeners', function() {
 
   function spy(fn) {
     const tracker = {calls: 0};
-    const capture = function() {
+    const capture = function () {
       tracker.calls++;
       return fn.apply(this, arguments);
     };