michael/astro
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

[ci] format

Browse source
This commit is contained in:
matthewp 2022-05-04 18:56:21 +00:00 • committed by github-actions[bot]
parent 6643a3931d
commit e087f27a9b
2 changed files with 10 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
packages/astro/src/runtime/server/index.ts
View file

@ -470,7 +470,7 @@ export async function renderEndpoint(mod: EndpointHandler, request: Request, par
); );
} }
if(handler.length > 1) { if (handler.length > 1) {
// eslint-disable-next-line no-console // eslint-disable-next-line no-console
console.warn(` console.warn(`
API routes with 2 arguments have been deprecated. Instead they take a single argument in the form of: API routes with 2 arguments have been deprecated. Instead they take a single argument in the form of:
@ -479,19 +479,19 @@ export function get({ params, request }) {
//... //...
} }
Update your code to remove this warning.`) Update your code to remove this warning.`);
} }
const context = { const context = {
request, request,
params params,
}; };
const proxy = new Proxy(context, { const proxy = new Proxy(context, {
get(target, prop) { get(target, prop) {
if(prop in target) { if (prop in target) {
return Reflect.get(target, prop); return Reflect.get(target, prop);
} else if(prop in params) { } else if (prop in params) {
// eslint-disable-next-line no-console // eslint-disable-next-line no-console
console.warn(` console.warn(`
API routes no longer pass params as the first argument. Instead an object containing a params property is provided in the form of: API routes no longer pass params as the first argument. Instead an object containing a params property is provided in the form of:
@ -505,7 +505,7 @@ Update your code to remove this warning.`);
} else { } else {
return undefined; return undefined;
} }
} },
}) as APIContext & Params; }) as APIContext & Params;
return await handler.call(mod, proxy, request); return await handler.call(mod, proxy, request);

8
packages/astro/test/api-routes.test.js
View file

@ -15,19 +15,19 @@ describe('API routes', () => {
const one = JSON.parse(await fixture.readFile('/old-api/twoarg/one.json')); const one = JSON.parse(await fixture.readFile('/old-api/twoarg/one.json'));
expect(one).to.deep.equal({ expect(one).to.deep.equal({
param: 'one', param: 'one',
pathname: '/old-api/twoarg/one.json' pathname: '/old-api/twoarg/one.json',
}); });
const two = JSON.parse(await fixture.readFile('/old-api/twoarg/two.json')); const two = JSON.parse(await fixture.readFile('/old-api/twoarg/two.json'));
expect(two).to.deep.equal({ expect(two).to.deep.equal({
param: 'two', param: 'two',
pathname: '/old-api/twoarg/two.json' pathname: '/old-api/twoarg/two.json',
}); });
}); });
it('param first argument is supported', async () => { it('param first argument is supported', async () => {
const one = JSON.parse(await fixture.readFile('/old-api/onearg/one.json')); const one = JSON.parse(await fixture.readFile('/old-api/onearg/one.json'));
expect(one).to.deep.equal({ expect(one).to.deep.equal({
param: 'one' param: 'one',
}); });
}); });
}); });
@ -37,7 +37,7 @@ describe('API routes', () => {
const one = JSON.parse(await fixture.readFile('/context/data/one.json')); const one = JSON.parse(await fixture.readFile('/context/data/one.json'));
expect(one).to.deep.equal({ expect(one).to.deep.equal({
param: 'one', param: 'one',
pathname: '/context/data/one.json' pathname: '/context/data/one.json',
}); });
}); });
}); });