Skip to content

Commit 792b9d6

Browse files
committed
refactor(types): changed type-names from HandlerXXX to MockXXX
BREAKING CHANGE: type-names changed
1 parent cfcdbde commit 792b9d6

File tree

6 files changed

+40
-43
lines changed

6 files changed

+40
-43
lines changed

src/handler-context.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { HandlerRequest, HandlerResponseElement, ResponseData } from './types';
1+
import { MockRequest, HandlerResponseElement, ResponseData } from './types';
22

33
async function delay(ms: number): Promise<any> {
44
return new Promise<ResponseData>(resolve => {
@@ -7,11 +7,11 @@ async function delay(ms: number): Promise<any> {
77
}
88

99
class HandlerContext {
10-
private req: HandlerRequest;
10+
private req: MockRequest;
1111
private realFetch: (input: RequestInfo, init?: RequestInit) => Promise<Response>;
1212

1313
constructor(
14-
req: HandlerRequest,
14+
req: MockRequest,
1515
realFetch: (input: RequestInfo, init?: RequestInit) => Promise<Response>
1616
) {
1717
this.req = req;

src/middleware-utils.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { HandlerRequest, Middleware, ResponseData } from './types';
1+
import { MockRequest, Middleware, ResponseData } from './types';
22

33
const defaultFailure: ResponseData = {
44
status: 500,
@@ -7,7 +7,7 @@ const defaultFailure: ResponseData = {
77

88
export default class MiddlewareUtils {
99
static combine(...middlewares: Middleware[]): Middleware {
10-
return (request: HandlerRequest, response: ResponseData) => {
10+
return (request: MockRequest, response: ResponseData) => {
1111
return middlewares.reduce(
1212
(currentResponse, middleware) => currentResponse.then(resp => middleware(request, resp)),
1313
Promise.resolve(response)
@@ -16,7 +16,7 @@ export default class MiddlewareUtils {
1616
}
1717

1818
static delayMiddleware(delayMs: number): Middleware {
19-
return (request: HandlerRequest, response: ResponseData) => {
19+
return (request: MockRequest, response: ResponseData) => {
2020
return new Promise<ResponseData>(resolve => {
2121
setTimeout(() => resolve(response), delayMs);
2222
});
@@ -27,7 +27,7 @@ export default class MiddlewareUtils {
2727
probabilityOfFailure: number,
2828
failure: ResponseData = defaultFailure
2929
): Middleware {
30-
return (request: HandlerRequest, response: ResponseData) => {
30+
return (request: MockRequest, response: ResponseData) => {
3131
return new Promise<ResponseData>(resolve => {
3232
const rnd = Math.random();
3333

src/spy-middleware.ts

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { HandlerRequest, ResponseData, RouteMatcher } from './types';
1+
import { MockRequest, ResponseData, RouteMatcher } from './types';
22

33
const allMatcher: RouteMatcher = {
44
test(input: RequestInfo, init?: RequestInit) {
@@ -7,7 +7,7 @@ const allMatcher: RouteMatcher = {
77
};
88

99
export interface Entry {
10-
request: HandlerRequest;
10+
request: MockRequest;
1111
response: ResponseData;
1212
}
1313

@@ -19,10 +19,7 @@ export default class SpyMiddleware {
1919
this.entries = [];
2020
}
2121

22-
middleware(
23-
request: HandlerRequest,
24-
response: ResponseData
25-
): ResponseData | Promise<ResponseData> {
22+
middleware(request: MockRequest, response: ResponseData): ResponseData | Promise<ResponseData> {
2623
const entry = { request, response };
2724
this.entries.unshift(entry);
2825
return response;

src/types.ts

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import HandlerContext from './handler-context';
1+
import MockContext from './handler-context';
22

33
export type Opaque<K, T> = T & { __TYPE__: K };
44

@@ -24,7 +24,7 @@ export interface ResponseData {
2424
statusText?: string;
2525
}
2626

27-
export interface HandlerRequest {
27+
export interface MockRequest {
2828
input: RequestInfo;
2929
init?: RequestInit;
3030
body?: any;
@@ -34,23 +34,23 @@ export interface HandlerRequest {
3434
method: HttpMethod;
3535
}
3636
export type HandlerResponseElement = (data: ResponseData) => Promise<ResponseData>;
37-
export type HandlerResponse = (...elements: Array<HandlerResponseElement>) => Promise<ResponseData>;
38-
export type Handler = (
39-
req: HandlerRequest,
40-
res: HandlerResponse,
41-
ctx: HandlerContext
37+
export type MockResponse = (...elements: Array<HandlerResponseElement>) => Promise<ResponseData>;
38+
export type MockHandler = (
39+
req: MockRequest,
40+
res: MockResponse,
41+
ctx: MockContext
4242
) => Promise<ResponseData>;
4343

4444
export type RequestUrl = Opaque<'RequestUrl', string>;
4545
export type MatcherUrl = Opaque<'MatcherUrl', string>;
4646

4747
export interface Route {
4848
matcher: RouteMatcher;
49-
handler: Handler;
49+
handler: MockHandler;
5050
}
5151

5252
export type Middleware = (
53-
request: HandlerRequest,
53+
request: MockRequest,
5454
response: ResponseData
5555
) => ResponseData | Promise<ResponseData>;
5656

src/yet-another-fetch-mock.ts

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
import {
22
Configuration,
3-
Handler,
4-
HandlerRequest,
5-
HandlerResponse,
3+
MockHandler,
4+
MockRequest,
5+
MockResponse,
66
HandlerResponseElement,
77
HttpMethod,
88
MatcherUrl,
@@ -49,39 +49,39 @@ class FetchMock {
4949
this.scope.fetch = this.realFetch;
5050
}
5151

52-
get(url: string, handler: Handler): void {
52+
get(url: string, handler: MockHandler): void {
5353
this.mock(MatcherUtils.get(url as MatcherUrl), handler);
5454
}
5555

56-
head(url: string, handler: Handler): void {
56+
head(url: string, handler: MockHandler): void {
5757
this.mock(MatcherUtils.head(url as MatcherUrl), handler);
5858
}
5959

60-
post(url: string, handler: Handler): void {
60+
post(url: string, handler: MockHandler): void {
6161
this.mock(MatcherUtils.post(url as MatcherUrl), handler);
6262
}
6363

64-
put(url: string, handler: Handler): void {
64+
put(url: string, handler: MockHandler): void {
6565
this.mock(MatcherUtils.put(url as MatcherUrl), handler);
6666
}
6767

68-
delete(url: string, handler: Handler): void {
68+
delete(url: string, handler: MockHandler): void {
6969
this.mock(MatcherUtils.del(url as MatcherUrl), handler);
7070
}
7171

72-
connect(url: string, handler: Handler): void {
72+
connect(url: string, handler: MockHandler): void {
7373
this.mock(MatcherUtils.connect(url as MatcherUrl), handler);
7474
}
7575

76-
options(url: string, handler: Handler): void {
76+
options(url: string, handler: MockHandler): void {
7777
this.mock(MatcherUtils.options(url as MatcherUrl), handler);
7878
}
7979

80-
patch(url: string, handler: Handler): void {
80+
patch(url: string, handler: MockHandler): void {
8181
this.mock(MatcherUtils.patch(url as MatcherUrl), handler);
8282
}
8383

84-
mock(matcher: RouteMatcher, handler: Handler) {
84+
mock(matcher: RouteMatcher, handler: MockHandler) {
8585
this.routes.push({ matcher, handler });
8686
}
8787

@@ -112,8 +112,8 @@ class FetchMock {
112112
}
113113
} else {
114114
pathParams = findPathParams(url, matchingRoute.matcher.matcherUrl);
115-
const req: HandlerRequest = { input, init, url, method, pathParams, queryParams, body };
116-
const res: HandlerResponse = async (...args: Array<HandlerResponseElement>) => {
115+
const req: MockRequest = { input, init, url, method, pathParams, queryParams, body };
116+
const res: MockResponse = async (...args: Array<HandlerResponseElement>) => {
117117
let responseData: ResponseData = {};
118118
for (const responseElement of args) {
119119
responseData = await responseElement(responseData);

test/middleware-utils.test.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { HandlerRequest, ResponseData } from '../src/types';
1+
import { MockRequest, ResponseData } from '../src/types';
22
import MathMock from './math-mock';
33
import MiddlewareUtils from '../src/middleware-utils';
44
import ConsoleMock from './console-mock';
@@ -21,7 +21,7 @@ describe('middleware-utils', () => {
2121
const startTime = new Date().getTime();
2222

2323
const combined = MiddlewareUtils.combine(delay, failure);
24-
const result = combined({} as HandlerRequest, 'data' as ResponseData);
24+
const result = combined({} as MockRequest, 'data' as ResponseData);
2525

2626
(result as Promise<ResponseData>).then(res => {
2727
const endTime = new Date().getTime();
@@ -36,7 +36,7 @@ describe('middleware-utils', () => {
3636

3737
it('should delay the response', done => {
3838
const delay = MiddlewareUtils.delayMiddleware(100);
39-
const result = delay({} as HandlerRequest, 'delayed' as ResponseData);
39+
const result = delay({} as MockRequest, 'delayed' as ResponseData);
4040
const startTime = new Date().getTime();
4141

4242
(result as Promise<String>).then(res => {
@@ -51,7 +51,7 @@ describe('middleware-utils', () => {
5151
it('should have a random failure rate', done => {
5252
MathMock.fixRandom(0.2);
5353
const delay = MiddlewareUtils.failurerateMiddleware(0.3);
54-
const result = delay({} as HandlerRequest, 'normal-response' as ResponseData);
54+
const result = delay({} as MockRequest, 'normal-response' as ResponseData);
5555

5656
(result as Promise<ResponseData>).then(res => {
5757
expect(res.status).toBe(500);
@@ -62,7 +62,7 @@ describe('middleware-utils', () => {
6262
it('should have a random failure rate2', done => {
6363
MathMock.fixRandom(0.4);
6464
const delay = MiddlewareUtils.failurerateMiddleware(0.3);
65-
const result = delay({} as HandlerRequest, 'normal-response' as ResponseData);
65+
const result = delay({} as MockRequest, 'normal-response' as ResponseData);
6666

6767
(result as Promise<String>).then(res => {
6868
expect(res).toBe('normal-response');
@@ -73,7 +73,7 @@ describe('middleware-utils', () => {
7373
it('should support custom error response', done => {
7474
MathMock.fixRandom(0.2);
7575
const delay = MiddlewareUtils.failurerateMiddleware(0.3, { status: 1337 });
76-
const result = delay({} as HandlerRequest, 'normal-response' as ResponseData);
76+
const result = delay({} as MockRequest, 'normal-response' as ResponseData);
7777

7878
(result as Promise<ResponseData>).then(res => {
7979
expect(res.status).toBe(1337);
@@ -83,7 +83,7 @@ describe('middleware-utils', () => {
8383

8484
it('should should log the request', () => {
8585
const logging = MiddlewareUtils.loggingMiddleware();
86-
logging({ init: { headers: {} } } as HandlerRequest, 'resp' as ResponseData);
86+
logging({ init: { headers: {} } } as MockRequest, 'resp' as ResponseData);
8787

8888
expect(console.log).toBeCalledTimes(5);
8989
});

0 commit comments

Comments
 (0)