style: Make prettier happy

Formatting changes required to consider commit. In test specs, combined `return done(err)`.
mochajs · Nov 4, 2018 · 0fd5585 · 0fd5585
1 parent ce03b99
commit 0fd5585
Show file tree

Hide file tree

Showing 4 changed files with 31 additions and 23 deletions.
diff --git a/lib/interfaces/common.js b/lib/interfaces/common.js
@@ -93,6 +93,7 @@ module.exports = function(suites, context, mocha) {
 
       /**
        * Creates a suite.
+       *
        * @param {Object} opts Options
        * @param {string} opts.title Title of Suite
        * @param {Function} [opts.fn] Suite Function (not always applicable)
@@ -112,14 +113,20 @@ module.exports = function(suites, context, mocha) {
         if (typeof opts.fn === 'function') {
           var result = opts.fn.call(suite);
           if (typeof result !== 'undefined') {
-            utils.deprecate('Deprecation Warning: Suites do not support a return value;' + opts.title + ' returned :' + result);
+            utils.deprecate(
+              'Deprecation Warning: Suites do not support a return value;' +
+                opts.title +
+                ' returned :' +
+                result
+            );
           }
           suites.shift();
         } else if (typeof opts.fn === 'undefined' && !suite.pending) {
           throw new Error(
             'Suite "' +
               suite.fullTitle() +
-              '" was defined but no callback was supplied. Supply a callback or explicitly skip the suite.'
+              '" was defined but no callback was supplied. ' +
+              'Supply a callback or explicitly skip the suite.'
           );
         } else if (!opts.fn && suite.pending) {
           suites.shift();

diff --git a/lib/utils.js b/lib/utils.js
@@ -589,7 +589,7 @@ exports.getError = function(err) {
  *
  * @param {string} msg
  */
-exports.deprecate = function (msg) {
+exports.deprecate = function(msg) {
   msg = String(msg);
   if (seenDeprecationMsg.hasOwnProperty(msg)) {
     return;
@@ -601,10 +601,16 @@ exports.deprecate = function (msg) {
 var seenDeprecationMsg = {};
 
 var doDeprecationWarning = process.emitWarning
-  // Node.js v6+
-  ? function (msg) { process.emitWarning(msg, 'DeprecationWarning'); }
-  // Everything else
-  : function (msg) { process.nextTick(() => console.warn(msg)); };
+  ? function(msg) {
+      // Node.js v6+
+      process.emitWarning(msg, 'DeprecationWarning');
+    }
+  : function(msg) {
+      // Everything else
+      process.nextTick(function() {
+        console.warn(msg);
+      });
+    };
 
 /**
  * @summary

diff --git a/test/integration/deprecate.spec.js b/test/integration/deprecate.spec.js
@@ -4,12 +4,11 @@ var assert = require('assert');
 var run = require('./helpers').runMocha;
 var args = [];
 
-describe.only('utils.deprecate test', function () {
-  it('should print unique deprecation only once', function (done) {
-    run('deprecate.fixture.js', args, function (err, res) {
+describe.only('utils.deprecate test', function() {
+  it('should print unique deprecation only once', function(done) {
+    run('deprecate.fixture.js', args, function(err, res) {
       if (err) {
-        done(err);
-        return;
+        return done(err);
       }
       console.trace(res);
       var result = res.output.match(/deprecated thing/g) || [];

diff --git a/test/integration/suite.spec.js b/test/integration/suite.spec.js
@@ -9,8 +9,7 @@ describe('suite w/no callback', function() {
   it('should throw a helpful error message when a callback for suite is not supplied', function(done) {
     run('suite/suite-no-callback.fixture.js', args, function(err, res) {
       if (err) {
-        done(err);
-        return;
+        return done(err);
       }
       var result = res.output.match(/no callback was supplied/) || [];
       assert.equal(result.length, 1);
@@ -24,8 +23,7 @@ describe('skipped suite w/no callback', function() {
   it('should not throw an error when a callback for skipped suite is not supplied', function(done) {
     run('suite/suite-skipped-no-callback.fixture.js', args, function(err, res) {
       if (err) {
-        done(err);
-        return;
+        return done(err);
       }
       var pattern = new RegExp('Error', 'g');
       var result = res.output.match(pattern) || [];
@@ -40,8 +38,7 @@ describe('skipped suite w/ callback', function() {
   it('should not throw an error when a callback for skipped suite is supplied', function(done) {
     run('suite/suite-skipped-callback.fixture.js', args, function(err, res) {
       if (err) {
-        done(err);
-        return;
+        return done(err);
       }
       var pattern = new RegExp('Error', 'g');
       var result = res.output.match(pattern) || [];
@@ -51,13 +48,12 @@ describe('skipped suite w/ callback', function() {
   });
 });
 
-describe('suite returning a value', function () {
+describe('suite returning a value', function() {
   this.timeout(2000);
-  it('should give a deprecation warning for suite callback returning a value', function (done) {
-    run('suite/suite-returning-value.fixture.js', args, function (err, res) {
+  it('should give a deprecation warning for suite callback returning a value', function(done) {
+    run('suite/suite-returning-value.fixture.js', args, function(err, res) {
       if (err) {
-        done(err);
-        return;
+        return done(err);
       }
       var pattern = new RegExp('Deprecation Warning', 'g');
       var result = res.output.match(pattern) || [];