karma - npm Package Compare versions

Comparing version 5.0.9 to 5.1.0

cucumber.js

@@ -12,3 +12,3 @@ // Shared configuration for Cucumber.js tests.
 module.exports = {
-  'default': options.join(' ')
+  default: options.join(' ')
 }

gruntfile.js

@@ -10,4 +10,3 @@ module.exports = function (grunt) {
       context: ['context/**/*.js'],
-      grunt: ['grunt.js', 'tasks/*.js'],
-      scripts: ['scripts/init-dev-env.js']
+      grunt: ['grunt.js', 'tasks/*.js']
     },
@@ -14,0 +13,0 @@ browserify: {

lib/browser.js

@@ -124,3 +124,3 @@ 'use strict'
     if (this.state === EXECUTING_DISCONNECTED) {
-      this.log.debug(`Lost socket connection, but browser continued to execute. Reconnected ` +
+      this.log.debug('Lost socket connection, but browser continued to execute. Reconnected ' +
         `on socket ${newSocket.id}.`)
@@ -127,0 +127,0 @@ this.setState(EXECUTING)

lib/config.js

@@ -165,3 +165,3 @@ 'use strict'
 
-  if (config.proxies && config.proxies.hasOwnProperty(config.urlRoot)) {
+  if (config.proxies && Object.prototype.hasOwnProperty.call(config.proxies, config.urlRoot)) {
     log.warn(`"${config.urlRoot}" is proxied, you should probably change urlRoot to avoid conflicts`)
@@ -168,0 +168,0 @@ }

lib/helper.js

@@ -75,6 +75,5 @@ 'use strict'
 exports.mmComparePatternWeights = (weight1, weight2) => {
-  let n1, n2, diff
-  n1 = weight1[0]
-  n2 = weight2[0]
-  diff = n1 - n2
+  const n1 = weight1[0]
+  const n2 = weight2[0]
+  const diff = n1 - n2
   if (diff !== 0) return diff / Math.abs(diff)
@@ -81,0 +80,0 @@ return weight1.length > 1 ? exports.mmComparePatternWeights(weight1.slice(1), weight2.slice(1)) : 0

lib/init.js

@@ -124,3 +124,3 @@ 'use strict'
   id: 'includedFiles',
-  question: `Which files do you want to include with <script> tag ?`,
+  question: 'Which files do you want to include with <script> tag ?',
   hint: 'This should be a script that bootstraps your test by configuring Require.js and ' +
@@ -127,0 +127,0 @@ 'kicking __karma__.start(), probably your test-main.js file.\n' +

lib/logger.js

@@ -29,3 +29,3 @@ // This is the **logger** module for *Karma*. It uses
     // Convert Array to Object for backwards compatibility.
-    if (appenders['map']) {
+    if (appenders.map) {
       if (appenders.length === 0) {
@@ -38,3 +38,3 @@ appenders = [constant.CONSOLE_APPENDER]
         if (appender.type === 'console') {
-          appenders['console'] = appender
+          appenders.console = appender
           if (helper.isDefined(appender.layout) && appender.layout.type === 'pattern') {
@@ -50,3 +50,3 @@ appender.layout.pattern = pattern
   } else {
-    appenders = { 'console': constant.CONSOLE_APPENDER }
+    appenders = { console: constant.CONSOLE_APPENDER }
   }
@@ -57,5 +57,5 @@
     categories: {
-      'default': {
-        'appenders': Object.keys(appenders),
-        'level': level
+      default: {
+        appenders: Object.keys(appenders),
+        level: level
       }
@@ -100,3 +100,3 @@ }
   let logger
-  if (loggerCache.hasOwnProperty(name)) {
+  if (Object.prototype.hasOwnProperty.call(loggerCache, name)) {
     logger = loggerCache[name]
@@ -103,0 +103,0 @@ } else {

lib/middleware/karma.js

@@ -22,5 +22,5 @@ /**
 const SCRIPT_TYPE = {
-  'js': 'text/javascript',
-  'dart': 'application/dart',
-  'module': 'module'
+  js: 'text/javascript',
+  dart: 'application/dart',
+  module: 'module'
 }
@@ -88,3 +88,3 @@ const FILE_TYPES = [
     let requestUrl = normalizedUrl.replace(/\?.*/, '')
-    const requestedRangeHeader = request.headers['range']
+    const requestedRangeHeader = request.headers.range
 
@@ -172,5 +172,5 @@ // redirect /__karma__ to /__karma__ (trailing slash)
                 log.warn(
-                  `Unable to determine file type from the file extension, defaulting to js.\n` +
+                  'Unable to determine file type from the file extension, defaulting to js.\n' +
                   `  To silence the warning specify a valid type for ${file.originalPath} in the configuration file.\n` +
-                  `  See http://karma-runner.github.io/latest/config/files.html`
+                  '  See http://karma-runner.github.io/latest/config/files.html'
                 )
@@ -177,0 +177,0 @@ } else {

lib/middleware/proxy.js

 const url = require('url')
+const { Agent: httpAgent } = require('http')
+const { Agent: httpsAgent } = require('https')
 const httpProxy = require('http-proxy')
@@ -45,2 +47,4 @@ const _ = require('lodash')
     const changeOrigin = proxyConfiguration.changeOrigin || false
+    const Agent = https ? httpsAgent : httpAgent
+    const agent = new Agent({ keepAlive: true })
     const proxy = httpProxy.createProxyServer({
@@ -50,3 +54,4 @@ target: { host: hostname, port, https, protocol },
       changeOrigin: changeOrigin,
-      secure: config.proxyValidateSSL
+      secure: config.proxyValidateSSL,
+      agent
     })
@@ -71,3 +76,3 @@
 
-    return { path: proxyPath, baseUrl: pathname, host: hostname, port, https, proxy }
+    return { path: proxyPath, baseUrl: pathname, host: hostname, port, https, proxy, agent }
   }), 'path').reverse()
@@ -118,4 +123,12 @@ }
 
-exports.create = function (/* config */config, /* config.proxies */proxies) {
-  return createProxyHandler(parseProxyConfig(proxies, config), config.urlRoot)
+exports.create = function (/* config */config, /* config.proxies */proxies, /* emitter */emitter) {
+  const proxyRecords = parseProxyConfig(proxies, config)
+  emitter.on('exit', (done) => {
+    log.debug('Destroying proxy agents')
+    proxyRecords.forEach((proxyRecord) => {
+      proxyRecord.agent.destroy()
+    })
+    done()
+  })
+  return createProxyHandler(proxyRecords, config.urlRoot)
 }

lib/middleware/source_files.js

@@ -35,3 +35,3 @@ 'use strict'
       const file = findByPath(files.served, requestedFilePath) || findByPath(files.served, requestedFilePathUnescaped)
-      const rangeHeader = request.headers['range']
+      const rangeHeader = request.headers.range
 
@@ -38,0 +38,0 @@ if (file) {

lib/preprocessor.js

@@ -59,3 +59,3 @@ 'use strict'
   try {
-    for (let process of preprocessors) {
+    for (const process of preprocessors) {
       content = await executeProcessor(process, file, content)
@@ -62,0 +62,0 @@ }

lib/runner.js

@@ -44,3 +44,3 @@ 'use strict'
   let exitCode = 1
-  let emitter = new EventEmitter()
+  const emitter = new EventEmitter()
   const options = {
@@ -47,0 +47,0 @@ hostname: config.hostname,

package.json

@@ -421,2 +421,3 @@ {
     "@commitlint/config-conventional": "^8.3.4",
+    "@commitlint/travis-cli": "^8.3.5",
     "@semantic-release/changelog": "^3.0.6",
@@ -429,8 +430,8 @@ "@semantic-release/git": "^7.0.18",
     "cucumber": "^6.0.5",
-    "eslint": "^5.16.0",
-    "eslint-config-standard": "^12.0.0",
-    "eslint-plugin-import": "^2.17.2",
-    "eslint-plugin-node": "^9.0.1",
-    "eslint-plugin-promise": "^4.1.1",
-    "eslint-plugin-standard": "^4.0.0",
+    "eslint": "^7.0.0",
+    "eslint-config-standard": "^14.1.1",
+    "eslint-plugin-import": "^2.20.2",
+    "eslint-plugin-node": "^11.1.0",
+    "eslint-plugin-promise": "^4.2.1",
+    "eslint-plugin-standard": "^4.0.1",
     "grunt": "^1.1.0",
@@ -471,3 +472,2 @@ "grunt-auto-release": "^0.0.7",
     "timer-shim": "^0.3.0",
-    "validate-commit-msg": "^2.11.1",
     "which": "^1.3.1"
@@ -482,3 +482,3 @@ },
   },
-  "version": "5.0.9",
+  "version": "5.1.0",
   "license": "MIT",
@@ -485,0 +485,0 @@ "husky": {

release.config.js

@@ -20,4 +20,5 @@ module.exports = {
   success: [
-    '@semantic-release/github'
+    '@semantic-release/github',
+    './tools/update-docs'
   ]
 }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Lines of code

10350

increased by0.12%

Worsened metrics

Total package byte prevSize

498040

decreased by-0.05%

Number of package files

91

decreased by-1.09%

Number of medium supply chain risk alerts

12

increased by20%

No dependency changes