Skip to content

Commit e0ba757

Browse files
eduardoboucaseduardoboucas
committed
chore: fix tests
1 parent cea28d5 commit e0ba757

File tree

2 files changed

+32
-40
lines changed

2 files changed

+32
-40
lines changed

‎src/list.test.ts

Lines changed: 22 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ describe('list', () => {
5959
next_cursor: 'cursor_1',
6060
}),
6161
),
62-
url: `https://api.netlify.com/api/v1/blobs/${siteID}/${storeName}`,
62+
url: `https://api.netlify.com/api/v1/blobs/${siteID}/site:${storeName}`,
6363
})
6464
.get({
6565
headers: { authorization: `Bearer ${apiToken}` },
@@ -83,7 +83,7 @@ describe('list', () => {
8383
next_cursor: 'cursor_2',
8484
}),
8585
),
86-
url: `https://api.netlify.com/api/v1/blobs/${siteID}/${storeName}?cursor=cursor_1`,
86+
url: `https://api.netlify.com/api/v1/blobs/${siteID}/site:${storeName}?cursor=cursor_1`,
8787
})
8888
.get({
8989
headers: { authorization: `Bearer ${apiToken}` },
@@ -100,7 +100,7 @@ describe('list', () => {
100100
directories: [],
101101
}),
102102
),
103-
url: `https://api.netlify.com/api/v1/blobs/${siteID}/${storeName}?cursor=cursor_2`,
103+
url: `https://api.netlify.com/api/v1/blobs/${siteID}/site:${storeName}?cursor=cursor_2`,
104104
})
105105

106106
globalThis.fetch = mockStore.fetch
@@ -148,7 +148,7 @@ describe('list', () => {
148148
next_cursor: 'cursor_1',
149149
}),
150150
),
151-
url: `https://api.netlify.com/api/v1/blobs/${siteID}/${storeName}?directories=true`,
151+
url: `https://api.netlify.com/api/v1/blobs/${siteID}/site:${storeName}?directories=true`,
152152
})
153153
.get({
154154
headers: { authorization: `Bearer ${apiToken}` },
@@ -172,7 +172,7 @@ describe('list', () => {
172172
next_cursor: 'cursor_2',
173173
}),
174174
),
175-
url: `https://api.netlify.com/api/v1/blobs/${siteID}/${storeName}?directories=true&cursor=cursor_1`,
175+
url: `https://api.netlify.com/api/v1/blobs/${siteID}/site:${storeName}?directories=true&cursor=cursor_1`,
176176
})
177177
.get({
178178
headers: { authorization: `Bearer ${apiToken}` },
@@ -189,7 +189,7 @@ describe('list', () => {
189189
directories: ['dir3'],
190190
}),
191191
),
192-
url: `https://api.netlify.com/api/v1/blobs/${siteID}/${storeName}?directories=true&cursor=cursor_2`,
192+
url: `https://api.netlify.com/api/v1/blobs/${siteID}/site:${storeName}?directories=true&cursor=cursor_2`,
193193
})
194194
.get({
195195
headers: { authorization: `Bearer ${apiToken}` },
@@ -206,7 +206,7 @@ describe('list', () => {
206206
directories: [],
207207
}),
208208
),
209-
url: `https://api.netlify.com/api/v1/blobs/${siteID}/${storeName}?prefix=dir2%2F&directories=true`,
209+
url: `https://api.netlify.com/api/v1/blobs/${siteID}/site:${storeName}?prefix=dir2%2F&directories=true`,
210210
})
211211

212212
globalThis.fetch = mockStore.fetch
@@ -258,7 +258,7 @@ describe('list', () => {
258258
],
259259
}),
260260
),
261-
url: `https://api.netlify.com/api/v1/blobs/${siteID}/${storeName}?prefix=group%2F`,
261+
url: `https://api.netlify.com/api/v1/blobs/${siteID}/site:${storeName}?prefix=group%2F`,
262262
})
263263

264264
globalThis.fetch = mockStore.fetch
@@ -303,7 +303,7 @@ describe('list', () => {
303303
next_cursor: 'cursor_2',
304304
}),
305305
),
306-
url: `https://api.netlify.com/api/v1/blobs/${siteID}/${storeName}`,
306+
url: `https://api.netlify.com/api/v1/blobs/${siteID}/site:${storeName}`,
307307
})
308308
.get({
309309
headers: { authorization: `Bearer ${apiToken}` },
@@ -319,7 +319,7 @@ describe('list', () => {
319319
],
320320
}),
321321
),
322-
url: `https://api.netlify.com/api/v1/blobs/${siteID}/${storeName}?cursor=cursor_2`,
322+
url: `https://api.netlify.com/api/v1/blobs/${siteID}/site:${storeName}?cursor=cursor_2`,
323323
})
324324

325325
globalThis.fetch = mockStore.fetch
@@ -373,7 +373,7 @@ describe('list', () => {
373373
next_cursor: 'cursor_1',
374374
}),
375375
),
376-
url: `${edgeURL}/${siteID}/${storeName}`,
376+
url: `${edgeURL}/${siteID}/site:${storeName}`,
377377
})
378378
.get({
379379
headers: { authorization: `Bearer ${edgeToken}` },
@@ -397,7 +397,7 @@ describe('list', () => {
397397
next_cursor: 'cursor_2',
398398
}),
399399
),
400-
url: `${edgeURL}/${siteID}/${storeName}?cursor=cursor_1`,
400+
url: `${edgeURL}/${siteID}/site:${storeName}?cursor=cursor_1`,
401401
})
402402
.get({
403403
headers: { authorization: `Bearer ${edgeToken}` },
@@ -414,7 +414,7 @@ describe('list', () => {
414414
directories: [],
415415
}),
416416
),
417-
url: `${edgeURL}/${siteID}/${storeName}?cursor=cursor_2`,
417+
url: `${edgeURL}/${siteID}/site:${storeName}?cursor=cursor_2`,
418418
})
419419
.get({
420420
headers: { authorization: `Bearer ${edgeToken}` },
@@ -431,7 +431,7 @@ describe('list', () => {
431431
directories: [],
432432
}),
433433
),
434-
url: `${edgeURL}/${siteID}/${storeName}?prefix=dir2%2F`,
434+
url: `${edgeURL}/${siteID}/site:${storeName}?prefix=dir2%2F`,
435435
})
436436

437437
globalThis.fetch = mockStore.fetch
@@ -487,7 +487,7 @@ describe('list', () => {
487487
next_cursor: 'cursor_1',
488488
}),
489489
),
490-
url: `${edgeURL}/${siteID}/${storeName}?directories=true`,
490+
url: `${edgeURL}/${siteID}/site:${storeName}?directories=true`,
491491
})
492492
.get({
493493
headers: { authorization: `Bearer ${edgeToken}` },
@@ -511,7 +511,7 @@ describe('list', () => {
511511
next_cursor: 'cursor_2',
512512
}),
513513
),
514-
url: `${edgeURL}/${siteID}/${storeName}?directories=true&cursor=cursor_1`,
514+
url: `${edgeURL}/${siteID}/site:${storeName}?directories=true&cursor=cursor_1`,
515515
})
516516
.get({
517517
headers: { authorization: `Bearer ${edgeToken}` },
@@ -528,7 +528,7 @@ describe('list', () => {
528528
directories: ['dir3'],
529529
}),
530530
),
531-
url: `${edgeURL}/${siteID}/${storeName}?directories=true&cursor=cursor_2`,
531+
url: `${edgeURL}/${siteID}/site:${storeName}?directories=true&cursor=cursor_2`,
532532
})
533533
.get({
534534
headers: { authorization: `Bearer ${edgeToken}` },
@@ -545,7 +545,7 @@ describe('list', () => {
545545
directories: [],
546546
}),
547547
),
548-
url: `${edgeURL}/${siteID}/${storeName}?prefix=dir2%2F&directories=true`,
548+
url: `${edgeURL}/${siteID}/site:${storeName}?prefix=dir2%2F&directories=true`,
549549
})
550550

551551
globalThis.fetch = mockStore.fetch
@@ -598,7 +598,7 @@ describe('list', () => {
598598
],
599599
}),
600600
),
601-
url: `${edgeURL}/${siteID}/${storeName}?prefix=group%2F`,
601+
url: `${edgeURL}/${siteID}/site:${storeName}?prefix=group%2F`,
602602
})
603603

604604
globalThis.fetch = mockStore.fetch
@@ -644,7 +644,7 @@ describe('list', () => {
644644
next_cursor: 'cursor_2',
645645
}),
646646
),
647-
url: `${edgeURL}/${siteID}/${storeName}`,
647+
url: `${edgeURL}/${siteID}/site:${storeName}`,
648648
})
649649
.get({
650650
headers: { authorization: `Bearer ${edgeToken}` },
@@ -667,7 +667,7 @@ describe('list', () => {
667667
next_cursor: 'cursor_3',
668668
}),
669669
),
670-
url: `${edgeURL}/${siteID}/${storeName}?cursor=cursor_2`,
670+
url: `${edgeURL}/${siteID}/site:${storeName}?cursor=cursor_2`,
671671
})
672672
.get({
673673
headers: { authorization: `Bearer ${edgeToken}` },
@@ -683,7 +683,7 @@ describe('list', () => {
683683
],
684684
}),
685685
),
686-
url: `${edgeURL}/${siteID}/${storeName}?cursor=cursor_3`,
686+
url: `${edgeURL}/${siteID}/site:${storeName}?cursor=cursor_3`,
687687
})
688688

689689
globalThis.fetch = mockStore.fetch

‎src/main.test.ts

Lines changed: 10 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -247,22 +247,22 @@ describe('get', () => {
247247
.get({
248248
headers: { authorization: `Bearer ${tokens[0]}` },
249249
response: new Response(value),
250-
url: `${edgeURL}/${siteID}/images/${key}`,
250+
url: `${edgeURL}/${siteID}/site:images/${key}`,
251251
})
252252
.get({
253253
headers: { authorization: `Bearer ${tokens[0]}` },
254254
response: new Response(value),
255-
url: `${edgeURL}/${siteID}/images/${key}`,
255+
url: `${edgeURL}/${siteID}/site:images/${key}`,
256256
})
257257
.get({
258258
headers: { authorization: `Bearer ${tokens[1]}` },
259259
response: new Response(value),
260-
url: `${edgeURL}/${siteID}/images/${key}`,
260+
url: `${edgeURL}/${siteID}/site:images/${key}`,
261261
})
262262
.get({
263263
headers: { authorization: `Bearer ${tokens[1]}` },
264264
response: new Response(value),
265-
url: `${edgeURL}/${siteID}/images/${key}`,
265+
url: `${edgeURL}/${siteID}/site:images/${key}`,
266266
})
267267

268268
globalThis.fetch = mockStore.fetch
@@ -294,22 +294,22 @@ describe('get', () => {
294294
.get({
295295
headers: { authorization: `Bearer ${tokens[0]}` },
296296
response: new Response(value),
297-
url: `${edgeURL}/${siteID}/images/${key}`,
297+
url: `${edgeURL}/${siteID}/site:images/${key}`,
298298
})
299299
.get({
300300
headers: { authorization: `Bearer ${tokens[0]}` },
301301
response: new Response(value),
302-
url: `${edgeURL}/${siteID}/images/${key}`,
302+
url: `${edgeURL}/${siteID}/site:images/${key}`,
303303
})
304304
.get({
305305
headers: { authorization: `Bearer ${tokens[1]}` },
306306
response: new Response(value),
307-
url: `${edgeURL}/${siteID}/images/${key}`,
307+
url: `${edgeURL}/${siteID}/site:images/${key}`,
308308
})
309309
.get({
310310
headers: { authorization: `Bearer ${tokens[1]}` },
311311
response: new Response(value),
312-
url: `${edgeURL}/${siteID}/images/${key}`,
312+
url: `${edgeURL}/${siteID}/site:images/${key}`,
313313
})
314314

315315
globalThis.fetch = mockStore.fetch
@@ -1383,7 +1383,7 @@ describe('Custom `fetch`', () => {
13831383
const mockStore = new MockFetch().get({
13841384
headers: { authorization: `Bearer ${mockToken}` },
13851385
response: new Response(value),
1386-
url: `${edgeURL}/${siteID}/images/${key}`,
1386+
url: `${edgeURL}/${siteID}/site:images/${key}`,
13871387
})
13881388
const context = {
13891389
edgeURL,
@@ -1459,22 +1459,14 @@ describe(`getStore`, () => {
14591459
}),
14601460
).toThrowError(`Store name must be a sequence of Unicode characters whose UTF-8 encoding is at most 64 bytes long.`)
14611461

1462-
expect(() =>
1463-
getStore({
1464-
name: 'deploy:foo',
1465-
token: apiToken,
1466-
siteID,
1467-
}),
1468-
).toThrowError('Store name must not start with the `deploy:` reserved keyword.')
1469-
14701462
const context = {
14711463
siteID,
14721464
token: apiToken,
14731465
}
14741466

14751467
env.NETLIFY_BLOBS_CONTEXT = Buffer.from(JSON.stringify(context)).toString('base64')
14761468

1477-
expect(() => getStore('deploy:foo')).toThrowError('Store name must not start with the `deploy:` reserved keyword.')
1469+
expect(() => getStore('some/store')).toThrowError('Store name must not contain forward slashes (/).')
14781470
})
14791471

14801472
test('Throws when there is no `fetch` implementation available', async () => {

0 commit comments

Comments
 (0)