Merge pull request #7 from Idrinth/coverage

package.json

@@ -23,7 +23,6 @@
     "needle": "^2.5",
     "@types/needle": "^2.0.4",
     "form-urlencoded": "^4.1.4",
-    "@types/form-urlencoded": "^2.0.1",
     "app-root-path": "^3",
     "@types/app-root-path": "^1.2.4",
     "cli-table3": "^0.6",

src/middlewares/encoding.ts

@@ -4,7 +4,7 @@ import {
 import {
   Request,
 } from '../request';
-import * as formurlencoded from 'form-urlencoded';
+import formUrlEncoded from 'form-urlencoded';
 import {
   Result,
 } from '../result';
@@ -18,7 +18,7 @@ class Encoding {
       return request;
     }
     if (request.autohandle === 'form' && typeof request.body === 'object') {
-      request.body = formurlencoded(request.body,);
+      request.body = formUrlEncoded(request.body,);
     }
     return request;
   }

test/middlewares/cookie.ts

@@ -3,6 +3,8 @@ import {
   expect,
 } from 'chai';
 import 'mocha';
+import { Request } from '../../src/request';
+import { Result } from '../../src/result';
 
 describe('middlewares/cookie', () => {
   it('should be a class', () => {
@@ -14,4 +16,19 @@ describe('middlewares/cookie', () => {
   it('should have a static method process', () => {
     expect(Cookie.process,).to.be.a('function',);
   },);
+  it('should not set cookies by default', () => {
+    expect(Cookie.prepare(<Request>{}),).to.deep.equal({cookies: {}},);
+  },);
+  it('should get cookies by default', () => {
+    expect(() => Cookie.process(<Result><unknown>{response: {cookies: {abc: 'def'}}}),).to.not.throw();
+  },);
+  it('should set cookies it has', () => {
+    expect(Cookie.prepare(<Request>{}),).to.deep.equal({cookies: {abc: 'def'}},);
+  },);
+  it('should change no cookies if there are none', () => {
+    expect(() => Cookie.process(<Result><unknown>{response: {cookies: {}}}),).to.not.throw();
+  },);
+  it('should set cookies it has', () => {
+    expect(Cookie.prepare(<Request>{}),).to.deep.equal({cookies: {abc: 'def'}},);
+  },);
 },);

test/middlewares/csrf-header.ts

@@ -3,6 +3,8 @@ import {
   expect,
 } from 'chai';
 import 'mocha';
+import { Request } from '../../src/request';
+import { Result } from '../../src/result';
 
 describe('middlewares/csrf-header', () => {
   it('should be a class', () => {
@@ -14,4 +16,19 @@ describe('middlewares/csrf-header', () => {
   it('should have a static method process', () => {
     expect(CsrfHeader.process,).to.be.a('function',);
   },);
+  it('should not set token by default', () => {
+    expect(CsrfHeader.prepare(<Request>{}),).to.deep.equal({headers: {}},);
+  },);
+  it('should get token by default', () => {
+    expect(() => CsrfHeader.process(<Result><unknown>{response: {headers: {'x-csrf-token': 'def'}}}),).to.not.throw();
+  },);
+  it('should set token it has', () => {
+    expect(CsrfHeader.prepare(<Request>{}),).to.deep.equal({headers: {'x-csrf-token': 'def'}},);
+  },);
+  it('should change no token if there\'s none', () => {
+    expect(() => CsrfHeader.process(<Result><unknown>{response: {headers: {}}}),).to.not.throw();
+  },);
+  it('should set token it has', () => {
+    expect(CsrfHeader.prepare(<Request>{}),).to.deep.equal({headers: {'x-csrf-token': 'def'}},);
+  },);
 },);

test/middlewares/encoding.ts

@@ -3,6 +3,7 @@ import {
   expect,
 } from 'chai';
 import 'mocha';
+import { Request } from '../../src/request';
 
 describe('middlewares/encoding', () => {
   it('should be a class', () => {
@@ -14,4 +15,16 @@ describe('middlewares/encoding', () => {
   it('should have a static method process', () => {
     expect(Encoding.process,).to.be.a('function',);
   },);
+  it('prepare should json-transform object to string', () => {
+    expect(Encoding.prepare(<Request><unknown>{body: {a: 'b'}, autohandle: 'json'}).body,).to.equal('{"a":"b"}',);
+  },);
+  it('prepare should json-transform array to string', () => {
+    expect(Encoding.prepare(<Request><unknown>{body: ['a'], autohandle: 'json'}).body,).to.equal('["a"]',);
+  },);
+  it('prepare should form-transform object to string', () => {
+    expect(Encoding.prepare(<Request><unknown>{body: {a: 'b'}, autohandle: 'form'}).body,).to.equal('a=b',);
+  },);
+  it('prepare should not transform by default', () => {
+    expect(Encoding.prepare(<Request><unknown>{body: {}}).body,).to.deep.equal({},);
+  },);
 },);

test/middlewares/user-agent.ts

@@ -3,6 +3,7 @@ import {
   expect,
 } from 'chai';
 import 'mocha';
+import { Request } from '../../src/request';
 
 describe('middlewares/user-agent', () => {
   it('should be a class', () => {
@@ -14,4 +15,7 @@ describe('middlewares/user-agent', () => {
   it('should have a static method process', () => {
     expect(UserAgent.process,).to.be.a('function',);
   },);
+  it('prepare should add a user agent header', () => {
+    expect(UserAgent.prepare(<Request>{}).headers['user-agent'],).to.be.a('string',);
+  },);
 },);

tsconfig.json

@@ -1,5 +1,9 @@
 {
   "compilerOptions": {
     "experimentalDecorators": true
-  }
+  },
+  "typeRoots" : [
+    "./typings",
+    "@types"
+  ]
 }

typings/form-urlencoded.d.ts

@@ -0,0 +1,13 @@
+// Type definitions for form-urlencoded 4
+// Project: https://github.com/iambumblehead/form-urlencoded#readme
+declare module 'form-urlencoded' {
+  interface FormEncodedOptions {
+    sorted?: boolean;
+    skipIndex?: boolean;
+    ignorenull?: boolean;
+    skipBracket?: boolean;
+    useDot?: boolean;
+  }
+
+  export default function(data: any, opts?: FormEncodedOptions): string;
+}