fix expected endpoint response

Signed-off-by: Meredith Lancaster <[email protected]>
This commit is contained in:
Meredith Lancaster
2025-12-09 08:02:38 -08:00
parent 97b7fa81c8
commit 0380590fdd
2 changed files with 7 additions and 7 deletions
@@ -48,13 +48,13 @@ describe('createStorageRecord', () => {
registry_url: 'https://my-registry.org' registry_url: 'https://my-registry.org'
}) })
}) })
.reply(201, {storage_records: [{id: '123'}, {id: '456'}]}) .reply(200, {storage_records: [{id: 123}, {id: 456}]})
}) })
it('persists the storage record', async () => { it('persists the storage record', async () => {
await expect(createStorageRecord(options)).resolves.toEqual([ await expect(createStorageRecord(options)).resolves.toEqual([
'123', 123,
'456' 456
]) ])
}) })
}) })
@@ -114,7 +114,7 @@ describe('createStorageRecord', () => {
registry_url: options.packageRegistryOptions.registryUrl registry_url: options.packageRegistryOptions.registryUrl
}) })
}) })
.reply(201, {storage_records: [{id: '123'}, {id: '456'}]}) .reply(200, {storage_records: [{id: 123}, {id: 456}]})
.times(1) .times(1)
}) })
@@ -124,7 +124,7 @@ describe('createStorageRecord', () => {
...options, ...options,
writeOptions: {} writeOptions: {}
}) })
).resolves.toEqual(['123', '456']) ).resolves.toEqual([123, 456])
}) })
}) })
}) })
+2 -2
View File
@@ -51,7 +51,7 @@ export type StorageRecordOptions = {
*/ */
export async function createStorageRecord( export async function createStorageRecord(
options: StorageRecordOptions options: StorageRecordOptions
): Promise<string[]> { ): Promise<number[]> {
const retries = options.writeOptions.retry ?? DEFAULT_RETRY_COUNT const retries = options.writeOptions.retry ?? DEFAULT_RETRY_COUNT
const octokit = github.getOctokit(options.token, {retry: {retries}}, retry) const octokit = github.getOctokit(options.token, {retry: {retries}}, retry)
@@ -67,7 +67,7 @@ export async function createStorageRecord(
? JSON.parse(response.data) ? JSON.parse(response.data)
: response.data : response.data
return data?.storage_records.map((r: {id: number}) => String(r.id)) return data?.storage_records.map((r: {id: number}) => r.id)
} catch (err) { } catch (err) {
const message = err instanceof Error ? err.message : err const message = err instanceof Error ? err.message : err
throw new Error(`Failed to persist storage record: ${message}`) throw new Error(`Failed to persist storage record: ${message}`)