Skip to content

Commit d322178

Browse files
committed
style: lint
1 parent 52326fc commit d322178

File tree

2 files changed

+17
-22
lines changed

2 files changed

+17
-22
lines changed

packages/runtime/src/templates/edge-shared/utils.test.ts

Lines changed: 15 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -1,30 +1,25 @@
11
import { assertEquals } from 'https://deno.land/std@0.167.0/testing/asserts.ts'
2-
import {
3-
beforeEach,
4-
describe,
5-
it,
6-
} from "https://deno.land/std@0.167.0/testing/bdd.ts";
7-
import {updateModifiedHeaders, FetchEventResult} from './utils.ts'
8-
2+
import { beforeEach, describe, it } from 'https://deno.land/std@0.167.0/testing/bdd.ts'
3+
import { updateModifiedHeaders, FetchEventResult } from './utils.ts'
94

105
describe('updateModifiedHeaders', () => {
11-
it('does not modify the headers if \'x-middleware-override-headers\' is not found', () => {
6+
it("does not modify the headers if 'x-middleware-override-headers' is not found", () => {
127
const mockHeaders = new Headers()
138
// There shouldn't be a case where x-middleware-override-headers is not set and a header has
149
// been modified with 'x-middleware-request' added to it, this is more to confirm the test case
1510
mockHeaders.set('x-middleware-request-foo', 'bar')
1611

1712
const mockResult: FetchEventResult = {
18-
response: new Response('', {headers: mockHeaders}),
19-
waitUntil: Promise.resolve()
13+
response: new Response('', { headers: mockHeaders }),
14+
waitUntil: Promise.resolve(),
2015
}
2116

2217
mockResult.response = updateModifiedHeaders(mockResult.response)
2318

2419
assertEquals(mockResult.response.headers.get('x-middleware-request-foo'), 'bar')
2520
})
2621

27-
describe('when the \'x-middleware-override-headers\' headers is present', () => {
22+
describe("when the 'x-middleware-override-headers' headers is present", () => {
2823
let mockHeaders
2924
let mockResult: FetchEventResult
3025

@@ -34,29 +29,28 @@ describe('updateModifiedHeaders', () => {
3429
mockHeaders.set('x-middleware-request-hello', 'world')
3530
mockHeaders.set('x-middleware-request-test', '123')
3631
mockHeaders.set('x-middleware-override-headers', 'hello,test')
37-
32+
3833
mockResult = {
39-
response: new Response('', {headers: mockHeaders}),
40-
waitUntil: Promise.resolve()
34+
response: new Response('', { headers: mockHeaders }),
35+
waitUntil: Promise.resolve(),
4136
}
42-
37+
4338
mockResult.response = updateModifiedHeaders(mockResult.response)
44-
4539
})
4640

47-
it('does not modify headers that are missing \'x-middleware-request\' in the name', () => {
41+
it("does not modify headers that are missing 'x-middleware-request' in the name", () => {
4842
assertEquals(mockResult.response.headers.get('foo'), 'bar')
4943
})
50-
51-
it('removes \'x-middleware-request-\' from headers', () => {
44+
45+
it("removes 'x-middleware-request-' from headers", () => {
5246
assertEquals(mockResult.response.headers.get('x-middleware-request-hello'), null)
5347
assertEquals(mockResult.response.headers.get('x-middleware-request-test'), null)
5448

5549
assertEquals(mockResult.response.headers.get('hello'), 'world')
5650
assertEquals(mockResult.response.headers.get('test'), '123')
5751
})
58-
59-
it('removes \'x-middleware-override-headers\' after cleaning headers', () => {
52+
53+
it("removes 'x-middleware-override-headers' after cleaning headers", () => {
6054
assertEquals(mockResult.response.headers.get('x-middleware-override-headers'), null)
6155
})
6256
})

packages/runtime/src/templates/edge-shared/utils.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ export function updateModifiedHeaders(response: Response) {
7373
return response
7474
}
7575

76-
const headersToUpdate = overriddenHeaders.split(',').map(header => header.trim())
76+
const headersToUpdate = overriddenHeaders.split(',').map((header) => header.trim())
7777

7878
for (const header of headersToUpdate) {
7979
const oldHeaderKey = 'x-middleware-request-' + header
@@ -96,6 +96,7 @@ export const buildResponse = async ({
9696
request: Request
9797
context: Context
9898
}) => {
99+
debugger
99100
result.response = updateModifiedHeaders(result.response)
100101

101102
// They've returned the MiddlewareRequest directly, so we'll call `next()` for them.

0 commit comments

Comments
 (0)