Skip to content

Commit bd5db52

Browse files
authored
Merge pull request #51 from ericcrosson-bitgo/remove-unused-comments
chore(express-wrapper): remove extraneous comments
2 parents bff318d + b6d6bb0 commit bd5db52

File tree

1 file changed

+0
-8
lines changed

1 file changed

+0
-8
lines changed

packages/express-wrapper/test/test-server.ts

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ import { buildApiClient, supertestRequestFactory } from '@api-ts/superagent-wrap
1010

1111
import { createServer } from '../src';
1212

13-
// TODO: wallet-platform throws errors instead of returning values
1413
const PutHello = httpRoute({
1514
path: '/hello',
1615
method: 'PUT',
@@ -30,12 +29,10 @@ const PutHello = httpRoute({
3029
appMiddlewareRan: t.boolean,
3130
routeMiddlewareRan: t.boolean,
3231
}),
33-
// In wallet-platform, sometimes we return a 202 (by throwing)
3432
invalidRequest: t.type({
3533
errors: t.string,
3634
}),
3735
notFound: t.unknown,
38-
// 500s are also used on prime team
3936
// DISCUSS: what if a response isn't listed here but shows up?
4037
internalError: t.unknown,
4138
},
@@ -84,7 +81,6 @@ test('should offer a delightful developer experience', async (t) => {
8481
// Configure app-level middleware
8582
app.use(express.json());
8683
app.use(appMiddleware);
87-
// Configure route-level middleware
8884
return {
8985
'hello.world': {
9086
put: [routeMiddleware, CreateHelloWorld],
@@ -114,7 +110,6 @@ test('should invoke app-level middleware', async (t) => {
114110
// Configure app-level middleware
115111
app.use(express.json());
116112
app.use(appMiddleware);
117-
// Configure route-level middleware
118113
return {
119114
'hello.world': {
120115
put: [CreateHelloWorld],
@@ -137,7 +132,6 @@ test('should invoke route-level middleware', async (t) => {
137132
const app = createServer(ApiSpec, (app: express.Application) => {
138133
// Configure app-level middleware
139134
app.use(express.json());
140-
// Configure route-level middleware
141135
return {
142136
'hello.world': {
143137
put: [routeMiddleware, CreateHelloWorld],
@@ -160,7 +154,6 @@ test('should infer status code from response type', async (t) => {
160154
const app = createServer(ApiSpec, (app: express.Application) => {
161155
// Configure app-level middleware
162156
app.use(express.json());
163-
// Configure route-level middleware
164157
return {
165158
'hello.world': {
166159
put: [CreateHelloWorld],
@@ -183,7 +176,6 @@ test('should return a 400 when request fails to decode', async (t) => {
183176
const app = createServer(ApiSpec, (app: express.Application) => {
184177
// Configure app-level middleware
185178
app.use(express.json());
186-
// Configure route-level middleware
187179
return {
188180
'hello.world': {
189181
put: [CreateHelloWorld],

0 commit comments

Comments
 (0)