Sort imports and requires (#3056)

This will definitely save time, and ensure more uniformity.

It moves the `createServiceTester()` calls to a different place from where I'd like them, though I'm happy to have them checked by the linter.

Closes #2701

services/npm/npm-collaborators.service.js

@@ -1,7 +1,7 @@
 'use strict'
 
-const NpmBase = require('./npm-base')
 const { renderContributorBadge } = require('../../lib/contributor-count')
+const NpmBase = require('./npm-base')
 
 const keywords = ['node']
 

services/npm/npm-license.tester.js

@@ -1,7 +1,6 @@
 'use strict'
 
 const Joi = require('joi')
-
 const t = (module.exports = require('../tester').createServiceTester())
 
 t.create('gets the license of express')

services/npm/npm-type-definitions.tester.js

@@ -1,13 +1,12 @@
 'use strict'
 
 const Joi = require('joi')
+const t = (module.exports = require('../tester').createServiceTester())
 
 const isTypeDefinition = Joi.string().regex(
   /^((Flow|TypeScript)|(Flow \| TypeScript))$/
 )
 
-const t = (module.exports = require('../tester').createServiceTester())
-
 t.create('types (from dev dependencies + files)')
   .get('/chalk.json')
   .expectJSONTypes(

services/npm/npm-version.tester.js

@@ -2,7 +2,6 @@
 
 const Joi = require('joi')
 const { isSemver } = require('../test-validators')
-
 const t = (module.exports = require('../tester').createServiceTester())
 
 t.create('gets the package version of left-pad')