chore: fix grammatical and spelling errors in comments and tests (#1490)

Author: mattkubej

lib/application.js

@@ -278,4 +278,5 @@ function respond(ctx) {
  * Make HttpError available to consumers of the library so that consumers don't
  * have a direct dependency upon `http-errors`
  */
+
 module.exports.HttpError = HttpError;

lib/request.js

@@ -146,7 +146,7 @@ module.exports = {
   },
 
   /**
-   * Set pathname, retaining the query-string when present.
+   * Set pathname, retaining the query string when present.
    *
    * @param {String} path
    * @api public
@@ -163,7 +163,7 @@ module.exports = {
   },
 
   /**
-   * Get parsed query-string.
+   * Get parsed query string.
    *
    * @return {Object}
    * @api public
@@ -176,7 +176,7 @@ module.exports = {
   },
 
   /**
-   * Set query-string as an object.
+   * Set query string as an object.
    *
    * @param {Object} obj
    * @api public
@@ -199,7 +199,7 @@ module.exports = {
   },
 
   /**
-   * Set querystring.
+   * Set query string.
    *
    * @param {String} str
    * @api public
@@ -216,7 +216,7 @@ module.exports = {
   },
 
   /**
-   * Get the search string. Same as the querystring
+   * Get the search string. Same as the query string
    * except it includes the leading ?.
    *
    * @return {String}
@@ -406,7 +406,7 @@ module.exports = {
   },
 
   /**
-   * Short-hand for:
+   * Shorthand for:
    *
    *    this.protocol == 'https'
    *
@@ -422,7 +422,7 @@ module.exports = {
    * When `app.proxy` is `true`, parse
    * the "X-Forwarded-For" ip address list.
    *
-   * For example if the value were "client, proxy1, proxy2"
+   * For example if the value was "client, proxy1, proxy2"
    * you would receive the array `["client", "proxy1", "proxy2"]`
    * where "proxy2" is the furthest down-stream.
    *
@@ -495,6 +495,7 @@ module.exports = {
    * @return {Object}
    * @api private
    */
+
   get accept() {
     return this._accept || (this._accept = accepts(this.req));
   },
@@ -505,6 +506,7 @@ module.exports = {
    * @param {Object}
    * @api private
    */
+
   set accept(obj) {
     this._accept = obj;
   },
@@ -607,7 +609,7 @@ module.exports = {
 
   /**
    * Check if the incoming request contains the "Content-Type"
-   * header field, and it contains any of the give mime `type`s.
+   * header field and if it contains any of the given mime `type`s.
    * If there is no request body, `null` is returned.
    * If there is no content type, `false` is returned.
    * Otherwise, it returns the first `type` that matches.

lib/response.js

@@ -433,6 +433,7 @@ module.exports = {
    * @return {boolean}
    * @api public
    */
+
   has(field) {
     return typeof this.res.hasHeader === 'function'
       ? this.res.hasHeader(field)
@@ -441,7 +442,7 @@ module.exports = {
   },
 
   /**
-   * Set header `field` to `val`, or pass
+   * Set header `field` to `val` or pass
    * an object of header fields.
    *
    * Examples:
@@ -564,8 +565,9 @@ module.exports = {
   },
 
   /**
-   * Flush any set headers, and begin the body
+   * Flush any set headers and begin the body
    */
+
   flushHeaders() {
     this.res.flushHeaders();
   }
@@ -577,6 +579,7 @@ module.exports = {
  * @return {Object}
  * @api public
  */
+
 /* istanbul ignore else */
 if (util.inspect.custom) {
   module.exports[util.inspect.custom] = module.exports.inspect;

test/application/onerror.js

@@ -66,7 +66,7 @@ describe('app.onerror(err)', () => {
     assert(msg === '\n  Foo\n');
   });
 
-  it('should use err.toString() instad of err.stack', () => {
+  it('should use err.toString() instead of err.stack', () => {
     const app = new Koa();
     app.env = 'dev';
 

test/application/respond.js

@@ -238,7 +238,7 @@ describe('app.respond', () => {
     });
   });
 
-  describe('when no middleware are present', () => {
+  describe('when no middleware is present', () => {
     it('should 404', () => {
       const app = new Koa();
 

test/application/use.js

@@ -98,7 +98,7 @@ describe('app.use(fn)', () => {
       .expect('generator');
   });
 
-  it('should throw error for non function', () => {
+  it('should throw error for non-function', () => {
     const app = new Koa();
 
     [null, undefined, 0, false, 'not a function'].forEach(v => {

test/context/onerror.js

@@ -239,7 +239,7 @@ describe('ctx.onerror(err)', () => {
   });
 
   describe('when non-error thrown', () => {
-    it('should response non-error thrown message', () => {
+    it('should respond with non-error thrown message', () => {
       const app = new Koa();
 
       app.use((ctx, next) => {

test/request/accepts.js

@@ -62,7 +62,7 @@ describe('ctx.accepts(types)', () => {
     });
   });
 
-  describe('when present in Accept as an exact match', () => {
+  describe('when value present in Accept is an exact match', () => {
     it('should return the type', () => {
       const ctx = context();
       ctx.req.headers.accept = 'text/plain, text/html';
@@ -71,7 +71,7 @@ describe('ctx.accepts(types)', () => {
     });
   });
 
-  describe('when present in Accept as a type match', () => {
+  describe('when value present in Accept is a type match', () => {
     it('should return the type', () => {
       const ctx = context();
       ctx.req.headers.accept = 'application/json, */*';
@@ -81,7 +81,7 @@ describe('ctx.accepts(types)', () => {
     });
   });
 
-  describe('when present in Accept as a subtype match', () => {
+  describe('when value present in Accept is a subtype match', () => {
     it('should return the type', () => {
       const ctx = context();
       ctx.req.headers.accept = 'application/json, text/*';

test/request/fresh.js

@@ -36,7 +36,7 @@ describe('ctx.fresh', () => {
       });
     });
 
-    describe('and etag do not match', () => {
+    describe('and etag does not match', () => {
       it('should return false', () => {
         const ctx = context();
         ctx.status = 200;

test/request/hostname.js

@@ -31,13 +31,13 @@ describe('req.hostname', () => {
       assert.equal(req.hostname, '[::1]');
     });
 
-    it('should parse localhost with non special schema port', () => {
+    it('should parse localhost with non-special schema port', () => {
       const req = request();
       req.header.host = '[::1]:1337';
       assert.equal(req.hostname, '[::1]');
     });
 
-    it('should reduce IPv6 with non special schema port, as hostname', () => {
+    it('should reduce IPv6 with non-special schema port as hostname', () => {
       const req = request();
       req.header.host = '[2001:cdba:0000:0000:0000:0000:3257:9652]:1337';
       assert.equal(req.hostname, '[2001:cdba::3257:9652]');