Removed old jshint/jscs rules

refs bcf5a1bc34

- leftovers

core/test/functional/routes/api/posts_spec.js

@@ -1011,7 +1011,6 @@ describe('Post API', function () {
                     .expect('Content-Type', /json/)
                     .expect('Cache-Control', testUtils.cacheRules.private)
                     .end(function (err, res) {
-                        /*jshint unused:false*/
                         if (err) {
                             return done(err);
                         }
@@ -1023,8 +1022,7 @@ describe('Post API', function () {
                             .expect('Content-Type', /json/)
                             .expect('Cache-Control', testUtils.cacheRules.private)
                             .expect(401)
-                            .end(function (err, res) {
-                                /*jshint unused:false*/
+                            .end(function (err) {
                                 if (err) {
                                     return done(err);
                                 }
@@ -1054,7 +1052,6 @@ describe('Post API', function () {
                             .expect('Cache-Control', testUtils.cacheRules.private)
                             .expect(400)
                             .end(function (err, res) {
-                                /*jshint unused:false*/
                                 if (err) {
                                     return done(err);
                                 }
@@ -1300,8 +1297,7 @@ describe('Post API', function () {
                             .send(jsonResponse)
                             .expect('Content-Type', /json/)
                             .expect('Cache-Control', testUtils.cacheRules.private)
-                            .end(function (err, res) {
-                                /*jshint unused:false*/
+                            .end(function (err) {
                                 if (err) {
                                     return done(err);
                                 }
@@ -1329,7 +1325,6 @@ describe('Post API', function () {
                             .expect('Cache-Control', testUtils.cacheRules.private)
                             .send(jsonResponse)
                             .end(function (err, res) {
-                                /*jshint unused:false*/
                                 if (err) {
                                     return done(err);
                                 }

core/test/functional/routes/api/public_api_spec.js

@@ -501,7 +501,7 @@ describe('Public API', function () {
         function createFilter(publishedAt, op) {
             // This line deliberately uses double quotes because GQL cannot handle either double quotes
             // or escaped singles, see TryGhost/GQL#34
-            return encodeURIComponent("published_at:" + op + "'" + publishedAt + "'");  // jscs:ignore
+            return encodeURIComponent("published_at:" + op + "'" + publishedAt + "'");
         }
 
         request

core/test/functional/routes/api/settings_spec.js

@@ -155,8 +155,7 @@ describe('Settings API', function () {
                     .set('Authorization', 'Bearer ' + 'invalidtoken')
                     .send(jsonResponse)
                     .expect(401)
-                    .end(function (err, res) {
-                        /*jshint unused:false*/
+                    .end(function (err) {
                         if (err) {
                             return done(err);
                         }

core/test/functional/routes/api/upload_icon_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     supertest = require('supertest'),
     testUtils = require('../../../utils'),
     path = require('path'),

core/test/functional/routes/api/upload_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     supertest = require('supertest'),
     testUtils = require('../../../utils'),
     path = require('path'),

core/test/functional/routes/api/users_spec.js

@@ -466,8 +466,7 @@ describe('User API', function () {
                             .set('Authorization', 'Bearer ' + 'invalidtoken')
                             .send(jsonResponse)
                             .expect(401)
-                            .end(function (err, res) {
-                                /*jshint unused:false*/
+                            .end(function (err) {
                                 if (err) {
                                     return done(err);
                                 }
@@ -509,7 +508,6 @@ describe('User API', function () {
                             .send(jsonResponse)
                             .expect(200)
                             .end(function (err, res) {
-                                /*jshint unused:false*/
                                 if (err) {
                                     return done(err);
                                 }

core/test/integration/model/base/listeners_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     Promise = require('bluebird'),
     moment = require('moment-timezone'),

core/test/integration/web/site_spec.js

@@ -1,4 +1,4 @@
-const should = require('should'), // jshint ignore:line
+const should = require('should'),
     sinon = require('sinon'),
     testUtils = require('../../utils'),
     configUtils = require('../../utils/configUtils'),

core/test/unit/adapters/scheduling/index_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     rewire = require('rewire'),
     Promise = require('bluebird'),

core/test/unit/adapters/storage/LocalFileStorage_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     fs = require('fs-extra'),
     moment = require('moment'),

core/test/unit/adapters/storage/index_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     fs = require('fs-extra'),
     StorageBase = require('ghost-storage-base'),
     configUtils = require('../../../utils/configUtils'),

core/test/unit/apps/private-blogging/input_password_spec.js

@@ -1,5 +1,4 @@
 // We use the name input_password to match the helper for consistency:
-// jscs:disable requireCamelCaseOrUpperCaseIdentifiers
 var should = require('should'),
 
 // Stuff we are testing

core/test/unit/apps/private-blogging/middleware_spec.js

@@ -1,5 +1,5 @@
 /*globals describe, beforeEach, afterEach, it*/
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     crypto = require('crypto'),
     fs = require('fs-extra'),

core/test/unit/config/index_spec.js

@@ -1,11 +1,9 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     path = require('path'),
     rewire = require('rewire'),
     _ = require('lodash'),
     configUtils = require('../../utils/configUtils');
 
-// jscs:disable requireDotNotation
-
 describe('Config', function () {
     before(function () {
         configUtils.restore();

core/test/unit/config/utils_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     configUtils = require('../../../server/config/utils');
 
 describe('UNIT: Config utils', function () {

core/test/unit/controllers/frontend/error_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     common = require('../../../../server/lib/common'),
 

core/test/unit/controllers/frontend/format-response_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
 
     // Stuff we are testing
     formatResponse = require('../../../../server/controllers/frontend/format-response');

core/test/unit/data/db/backup_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     rewire = require('rewire'),
     _ = require('lodash'),

core/test/unit/data/meta/asset_url_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     getAssetUrl = require('../../../../server/data/meta/asset_url'),
     settingsCache = require('../../../../server/services/settings/cache'),

core/test/unit/data/meta/canonical_url_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     getCanonicalUrl = require('../../../../server/data/meta/canonical_url'),
     markdownToMobiledoc = require('../../../utils/fixtures/data-generator').markdownToMobiledoc;
 

core/test/unit/data/meta/description_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     getMetaDescription = require('../../../../server/data/meta/description');
 
 describe('getMetaDescription', function () {

core/test/unit/data/meta/excerpt_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     getExcerpt = require('../../../../server/data/meta/excerpt');
 
 describe('getExcerpt', function () {

core/test/unit/data/meta/title_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     getTitle = require('../../../../server/data/meta/title'),
     settingsCache = require('../../../../server/services/settings/cache'),

core/test/unit/data/meta/url_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     getUrl = require('../../../../server/data/meta/url'),
     markdownToMobiledoc = require('../../../utils/fixtures/data-generator').markdownToMobiledoc;
 

core/test/unit/data/schema/integrity_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     _ = require('lodash'),
     crypto = require('crypto'),
     schema = require('../../../../server/data/schema'),

core/test/unit/data/validation/index_spec.js

@@ -126,11 +126,9 @@ describe('Validation', function () {
         var validator = validation.validator;
 
         it('isEmptyOrUrl filters javascript urls', function () {
-            /*jshint scripturl:true */
             validator.isEmptyOrURL('javascript:alert(0)').should.be.false();
             validator.isEmptyOrURL('http://example.com/lol/<script>lalala</script>/').should.be.false();
             validator.isEmptyOrURL('http://example.com/lol?somequery=<script>lalala</script>').should.be.false();
-            /*jshint scripturl:false */
             validator.isEmptyOrURL('').should.be.true();
             validator.isEmptyOrURL('http://localhost:2368').should.be.true();
             validator.isEmptyOrURL('http://example.com/test/').should.be.true();

core/test/unit/helpers/asset_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     configUtils = require('../../utils/configUtils'),
     helpers = require('../../../server/helpers'),

core/test/unit/helpers/author_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
 
 // Stuff we are testing
     helpers = require('../../../server/helpers');

core/test/unit/helpers/authors_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     helpers = require('../../../server/helpers'),
     models = require('../../../server/models'),

core/test/unit/helpers/body_class_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     themeList = require('../../../server/services/themes').list,
 
 // Stuff we are testing

core/test/unit/helpers/content_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
 
 // Stuff we are testing
     helpers = require('../../../server/helpers');

core/test/unit/helpers/date_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
 
 // Stuff we are testing
     helpers = require('../../../server/helpers'),

core/test/unit/helpers/encode_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
 
 // Stuff we are testing
     helpers = require('../../../server/helpers');

core/test/unit/helpers/excerpt_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
 
 // Stuff we are testing
     helpers = require('../../../server/helpers');

core/test/unit/helpers/facebook_url_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
 
 // Stuff we are testing
     helpers = require('../../../server/helpers');

core/test/unit/helpers/foreach_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     _ = require('lodash'),
     helpers = require.main.require('core/server/helpers'),

core/test/unit/helpers/ghost_foot_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
 
 // Stuff we are testing

core/test/unit/helpers/ghost_head_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     _ = require('lodash'),
     Promise = require('bluebird'),

core/test/unit/helpers/has_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
 
 // Stuff we are testing

core/test/unit/helpers/img_url_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     configUtils = require('../../utils/configUtils'),
 

core/test/unit/helpers/index_spec.js

@@ -1,5 +1,4 @@
-// jscs:disable requireCamelCaseOrUpperCaseIdentifiers
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     _ = require('lodash'),
     hbs = require.main.require('core/server/services/themes/engine'),
 

core/test/unit/helpers/is_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     helpers = require('../../../server/helpers'),
     common = require('../../../server/lib/common'),

core/test/unit/helpers/next_post_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     Promise = require('bluebird'),
     markdownToMobiledoc = require('../../utils/fixtures/data-generator').markdownToMobiledoc,

core/test/unit/helpers/page_url_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
 
 // Stuff we are testing
     helpers = require('../../../server/helpers');

core/test/unit/helpers/pagination_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     hbs = require('../../../server/services/themes/engine'),
     configUtils = require('../../utils/configUtils'),
     path = require('path'),

core/test/unit/helpers/plural_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
 
 // Stuff we are testing
     helpers = require('../../../server/helpers');

core/test/unit/helpers/post_class_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
 
 // Stuff we are testing
     helpers = require('../../../server/helpers');

core/test/unit/helpers/prev_post_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     Promise = require('bluebird'),
     markdownToMobiledoc = require('../../utils/fixtures/data-generator').markdownToMobiledoc,

core/test/unit/helpers/reading_time_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
 
 // Stuff we are testing
     helpers = require('../../../server/helpers');

core/test/unit/helpers/tags_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
 
 // Stuff we are testing

core/test/unit/helpers/title_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
 
 // Stuff we are testing
     helpers = require('../../../server/helpers');

core/test/unit/helpers/twitter_url_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
 
 // Stuff we are testing
     helpers = require('../../../server/helpers');

core/test/unit/helpers/url_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     Promise = require('bluebird'),
     configUtils = require('../../utils/configUtils'),

core/test/unit/helpers/utils_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     helperUtils = require('../../../server/helpers/utils');
 
 describe('Helpers Utils', function () {

core/test/unit/lib/common/errors_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     common = require('../../../../server/lib/common');
 
 describe('Errors', function () {

core/test/unit/lib/fs/package-json/filter_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     packageJSON = require('../../../../../server/lib/fs/package-json');
 
 describe('lib/fs/package-json', function () {

core/test/unit/lib/fs/package-json/parse_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     tmp = require('tmp'),
     fs = require('fs-extra'),
     packageJSON = require('../../../../../server/lib/fs/package-json');

core/test/unit/lib/fs/package-json/read_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     tmp = require('tmp'),
     join = require('path').join,
     fs = require('fs-extra'),

core/test/unit/lib/fs/zip-folder_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     path = require('path'),
     fs = require('fs-extra'),
     extract = require('extract-zip'),

core/test/unit/lib/ghost-version_spec.js

@@ -1,4 +1,3 @@
-// jshint unused: false
 const should = require('should'),
     rewire = require('rewire'),
     testUtils = require('../../utils');

core/test/unit/lib/image/blog-icon_spec.js

@@ -1,4 +1,3 @@
-// jshint unused: false
 var should = require('should'),
     sinon = require('sinon'),
     _ = require('lodash'),

core/test/unit/lib/image/gravatar_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     nock = require('nock'),
     configUtils = require('../../../utils/configUtils'),
     gravatar = require('../../../../server/lib/image/gravatar');

core/test/unit/lib/mobiledoc/atoms/soft-return_spec.js

@@ -1,4 +1,4 @@
-const should = require('should'); // jshint ignore:line
+const should = require('should');
 const atom = require('../../../../../server/lib/mobiledoc/atoms/soft-return');
 const SimpleDom = require('simple-dom');
 const serializer = new SimpleDom.HTMLSerializer(SimpleDom.voidMap);

core/test/unit/lib/mobiledoc/cards/code_spec.js

@@ -1,4 +1,4 @@
-const should = require('should'); // jshint ignore:line
+const should = require('should');
 const card = require('../../../../../server/lib/mobiledoc/cards/code');
 const SimpleDom = require('simple-dom');
 const serializer = new SimpleDom.HTMLSerializer(SimpleDom.voidMap);

core/test/unit/lib/mobiledoc/cards/hr_spec.js

@@ -1,4 +1,4 @@
-const should = require('should'); // jshint ignore:line
+const should = require('should');
 const card = require('../../../../../server/lib/mobiledoc/cards/hr');
 const SimpleDom = require('simple-dom');
 const serializer = new SimpleDom.HTMLSerializer(SimpleDom.voidMap);

core/test/unit/lib/mobiledoc/cards/html_spec.js

@@ -1,4 +1,4 @@
-const should = require('should'); // jshint ignore:line
+const should = require('should');
 const card = require('../../../../../server/lib/mobiledoc/cards/html');
 const SimpleDom = require('simple-dom');
 const serializer = new SimpleDom.HTMLSerializer(SimpleDom.voidMap);

core/test/unit/lib/mobiledoc/cards/image_spec.js

@@ -1,4 +1,4 @@
-const should = require('should');  // jshint ignore:line
+const should = require('should');
 const card = require('../../../../../server/lib/mobiledoc/cards/image');
 const SimpleDom = require('simple-dom');
 const serializer = new SimpleDom.HTMLSerializer(SimpleDom.voidMap);

core/test/unit/lib/mobiledoc/cards/markdown_spec.js

@@ -1,4 +1,4 @@
-const should = require('should'); // jshint ignore:line
+const should = require('should');
 const card = require('../../../../../server/lib/mobiledoc/cards/markdown');
 const SimpleDom = require('simple-dom');
 const serializer = new SimpleDom.HTMLSerializer(SimpleDom.voidMap);

core/test/unit/lib/mobiledoc/converters/mobiledoc-converter_spec.js

@@ -1,4 +1,4 @@
-const should = require('should'); // jshint ignore:line
+const should = require('should');
 const converter = require('../../../../../server/lib/mobiledoc/converters/mobiledoc-converter');
 
 describe('Mobiledoc converter', function () {

core/test/unit/lib/promise/pipeline_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     Promise = require('bluebird'),
 

core/test/unit/lib/security/password_spec.js

@@ -1,4 +1,4 @@
-const should = require('should'), // jshint ignore:line
+const should = require('should'),
     security = require('../../../../server/lib/security');
 
 describe('Lib: Security - Password', function () {

core/test/unit/lib/security/string_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     security = require('../../../../server/lib/security');
 
 describe('Lib: Security - String', function () {

core/test/unit/lib/security/tokens_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     uuid = require('uuid'),
     security = require('../../../../server/lib/security');
 

core/test/unit/models/base/listeners_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     rewire = require('rewire'),
     common = require('../../../../server/lib/common'),

core/test/unit/models/plugins/pagination_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     Promise = require('bluebird'),
     rewire = require('rewire'),

core/test/unit/models/post_spec.js

@@ -1,6 +1,6 @@
 /* eslint no-invalid-this:0 */
 
-const should = require('should'), // jshint ignore:line
+const should = require('should'),
     sinon = require('sinon'),
     _ = require('lodash'),
     testUtils = require('../../utils'),

core/test/unit/models/tag_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     models = require('../../../server/models'),
     testUtils = require('../../utils'),

core/test/unit/public/ghost_sdk_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     ghostSdk = require('../../../server/public/ghost-sdk'),
     configUtils = require('../../utils/configUtils'),
     urlService = require('../../../server/services/url');

core/test/unit/services/apps/dependencies_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     EventEmitter = require('events').EventEmitter,
     _ = require('lodash'),

core/test/unit/services/apps/permissions_spec.js

@@ -101,7 +101,6 @@ describe('Apps', function () {
             sandbox.stub(perms, 'getPackageContents').returns(Promise.reject(new Error('package.json file is malformed')));
 
             perms.read().then(function (readPerms) {
-                /*jshint unused:false*/
                 done(new Error('should not resolve'));
             }).catch(function (err) {
                 err.message.should.equal('package.json file is malformed');

core/test/unit/services/auth/authenticate_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     passport = require('passport'),
     rewire = require('rewire'),

core/test/unit/services/auth/passport_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     passport = require('passport'),
     ghostPassport = require('../../../../server/services/auth/passport'),

core/test/unit/services/channels/custom_channels_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'),  // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     _ = require('lodash'),
 

core/test/unit/services/channels/loader_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'),  // jshint ignore:line
+var should = require('should'),
     _ = require('lodash'),
     rewire = require('rewire'),
     channelUtils = require('../../../utils/channelUtils'),

core/test/unit/services/mail/utils_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     mail = require('../../../../server/services/mail'),
     sandbox = sinon.sandbox.create();

core/test/unit/services/permissions/can-this_spec.js

@@ -1,4 +1,4 @@
-const should = require('should'), // jshint ignore:line
+const should = require('should'),
     sinon = require('sinon'),
     testUtils = require('../../../utils'),
     Promise = require('bluebird'),

core/test/unit/services/permissions/index_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     testUtils = require('../../../utils'),
     Promise = require('bluebird'),

core/test/unit/services/permissions/parse-context_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     parseContext = require('../../../../server/services/permissions/parse-context');
 
 describe('Permissions', function () {

core/test/unit/services/permissions/public_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     _ = require('lodash'),
     common = require('../../../../server/lib/common'),
     applyPublicRules = require('../../../../server/services/permissions/public');

core/test/unit/services/slack_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line,
+var should = require('should'),
     sinon = require('sinon'),
     _ = require('lodash'),
     nock = require('nock'),

core/test/unit/services/themes/active_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
 
     config = require('../../../../server/config'),

core/test/unit/services/themes/config_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
 
     themeConfig = require('../../../../server/services/themes/config'),

core/test/unit/services/themes/list_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     _ = require('lodash'),
 

core/test/unit/services/themes/loader_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     fs = require('fs-extra'),
     tmp = require('tmp'),

core/test/unit/services/themes/middleware_spec.js

@@ -1,4 +1,4 @@
-var should = require('should'), // jshint ignore:line
+var should = require('should'),
     sinon = require('sinon'),
     hbs = require('../../../../server/services/themes/engine'),
 

core/test/unit/services/url/Queue_spec.js

@@ -1,4 +1,4 @@
-// jshint unused: false
+
 const _ = require('lodash');
 const Promise = require('bluebird');
 const should = require('should');

core/test/unit/services/url/Resources_spec.js

@@ -1,4 +1,4 @@
-// jshint unused: false
+
 const should = require('should');
 const _ = require('lodash');
 const sinon = require('sinon');

core/test/unit/services/url/UrlGenerator_spec.js

@@ -1,4 +1,4 @@
-// jshint unused: false
+
 const _ = require('lodash');
 const Promise = require('bluebird');
 const should = require('should');

core/test/unit/services/url/UrlService_spec.js

@@ -1,4 +1,4 @@
-// jshint unused: false
+
 const _ = require('lodash');
 const Promise = require('bluebird');
 const should = require('should');

core/test/unit/services/url/Urls_spec.js

@@ -1,4 +1,4 @@
-// jshint unused: false
+
 const _ = require('lodash');
 const Promise = require('bluebird');
 const should = require('should');