Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit ae412a5

Browse files
authoredJan 12, 2024
chore(ecosystem-tests): fix flaky vercel-edge, cloudflare-worker, and deno tests (#626)
1 parent 6456004 commit ae412a5

File tree

3 files changed

+14
-14
lines changed

3 files changed

+14
-14
lines changed
 

‎ecosystem-tests/cloudflare-worker/src/uploadWebApiTestCases.ts

+5-5
Original file line numberDiff line numberDiff line change
@@ -116,31 +116,31 @@ export function uploadWebApiTestCases({
116116
// @ts-expect-error we don't type support for `string` to avoid a footgun with passing the file path
117117
const file = await toFile(fineTune, 'finetune.jsonl');
118118
const result = await client.files.create({ file, purpose: 'fine-tune' });
119-
expectEqual(result.status, 'uploaded');
119+
expectEqual(result.filename, 'finetune.jsonl');
120120
});
121121
it('toFile handles Blob', async () => {
122122
const result = await client.files.create({ file: await toFile(new Blob([fineTune]), 'finetune.jsonl'), purpose: 'fine-tune' });
123-
expectEqual(result.status, 'uploaded');
123+
expectEqual(result.filename, 'finetune.jsonl');
124124
});
125125
it('toFile handles Uint8Array', async () => {
126126
const result = await client.files.create({
127127
file: await toFile(new TextEncoder().encode(fineTune), 'finetune.jsonl'),
128128
purpose: 'fine-tune',
129129
});
130-
expectEqual(result.status, 'uploaded');
130+
expectEqual(result.filename, 'finetune.jsonl');
131131
});
132132
it('toFile handles ArrayBuffer', async () => {
133133
const result = await client.files.create({
134134
file: await toFile(new TextEncoder().encode(fineTune).buffer, 'finetune.jsonl'),
135135
purpose: 'fine-tune',
136136
});
137-
expectEqual(result.status, 'uploaded');
137+
expectEqual(result.filename, 'finetune.jsonl');
138138
});
139139
it('toFile handles DataView', async () => {
140140
const result = await client.files.create({
141141
file: await toFile(new DataView(new TextEncoder().encode(fineTune).buffer), 'finetune.jsonl'),
142142
purpose: 'fine-tune',
143143
});
144-
expectEqual(result.status, 'uploaded');
144+
expectEqual(result.filename, 'finetune.jsonl');
145145
});
146146
}

‎ecosystem-tests/deno/main_test.ts

+4-4
Original file line numberDiff line numberDiff line change
@@ -105,26 +105,26 @@ Deno.test(async function toFileHandlesBlob() {
105105
file: await toFile(new Blob([fineTune]), 'finetune.jsonl'),
106106
purpose: 'fine-tune',
107107
});
108-
assertEquals(result.status, 'uploaded');
108+
assertEquals(result.filename, 'finetune.jsonl');
109109
});
110110
Deno.test(async function toFileHandlesUint8Array() {
111111
const result = await client.files.create({
112112
file: await toFile(new TextEncoder().encode(fineTune), 'finetune.jsonl'),
113113
purpose: 'fine-tune',
114114
});
115-
assertEquals(result.status, 'uploaded');
115+
assertEquals(result.filename, 'finetune.jsonl');
116116
});
117117
Deno.test(async function toFileHandlesArrayBuffer() {
118118
const result = await client.files.create({
119119
file: await toFile(new TextEncoder().encode(fineTune).buffer, 'finetune.jsonl'),
120120
purpose: 'fine-tune',
121121
});
122-
assertEquals(result.status, 'uploaded');
122+
assertEquals(result.filename, 'finetune.jsonl');
123123
});
124124
Deno.test(async function toFileHandlesDataView() {
125125
const result = await client.files.create({
126126
file: await toFile(new DataView(new TextEncoder().encode(fineTune).buffer), 'finetune.json'),
127127
purpose: 'fine-tune',
128128
});
129-
assertEquals(result.status, 'uploaded');
129+
assertEquals(result.filename, 'finetune.jsonl');
130130
});

‎ecosystem-tests/vercel-edge/src/uploadWebApiTestCases.ts

+5-5
Original file line numberDiff line numberDiff line change
@@ -147,28 +147,28 @@ export function uploadWebApiTestCases({
147147
// @ts-ignore this only doesn't error in vercel build...
148148
const file = await toFile(fineTune, 'finetune.jsonl');
149149
const result = await client.files.create({ file, purpose: 'fine-tune' });
150-
expectEqual(result.status, 'uploaded');
150+
expectEqual(result.filename, 'finetune.jsonl');
151151
});
152152
it('toFile handles Blob', async () => {
153153
const result = await client.files.create({
154154
file: await toFile(new Blob([fineTune]), 'finetune.jsonl'),
155155
purpose: 'fine-tune',
156156
});
157-
expectEqual(result.status, 'uploaded');
157+
expectEqual(result.filename, 'finetune.jsonl');
158158
});
159159
it('toFile handles Uint8Array', async () => {
160160
const result = await client.files.create({
161161
file: await toFile(new TextEncoder().encode(fineTune), 'finetune.jsonl'),
162162
purpose: 'fine-tune',
163163
});
164-
expectEqual(result.status, 'uploaded');
164+
expectEqual(result.filename, 'finetune.jsonl');
165165
});
166166
it('toFile handles ArrayBuffer', async () => {
167167
const result = await client.files.create({
168168
file: await toFile(new TextEncoder().encode(fineTune).buffer, 'finetune.jsonl'),
169169
purpose: 'fine-tune',
170170
});
171-
expectEqual(result.status, 'uploaded');
171+
expectEqual(result.filename, 'finetune.jsonl');
172172
});
173173
if (runtime !== 'edge') {
174174
// this fails in edge for some reason
@@ -177,7 +177,7 @@ export function uploadWebApiTestCases({
177177
file: await toFile(new DataView(new TextEncoder().encode(fineTune).buffer), 'finetune.jsonl'),
178178
purpose: 'fine-tune',
179179
});
180-
expectEqual(result.status, 'uploaded');
180+
expectEqual(result.filename, 'finetune.jsonl');
181181
});
182182
}
183183
}

0 commit comments

Comments
 (0)
Please sign in to comment.