Skip to content

Commit ccf1d2b

Browse files
committed
Update tests to pass after refactor to Response
1 parent 8975891 commit ccf1d2b

File tree

2 files changed

+16
-19
lines changed

2 files changed

+16
-19
lines changed

packages/vertexai/src/methods/generate-content.test.ts

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -309,24 +309,23 @@ describe('generateContent()', () => {
309309
expect(mockFetch).to.be.called;
310310
});
311311
it('on-device', async () => {
312-
const expectedText = 'hi';
313312
const chromeAdapter = new ChromeAdapter();
314-
const mockIsAvailable = stub(chromeAdapter, 'isAvailable').resolves(true);
315-
const mockGenerateContent = stub(
313+
const isAvailableStub = stub(chromeAdapter, 'isAvailable').resolves(true);
314+
const mockResponse = getMockResponse(
315+
'unary-success-basic-reply-short.json'
316+
);
317+
const generateContentStub = stub(
316318
chromeAdapter,
317319
'generateContentOnDevice'
318-
).resolves({
319-
text: () => expectedText,
320-
functionCalls: () => undefined
321-
});
320+
).resolves(mockResponse as Response);
322321
const result = await generateContent(
323322
fakeApiSettings,
324323
'model',
325324
fakeRequestParams,
326325
chromeAdapter
327326
);
328-
expect(result.response.text()).to.equal(expectedText);
329-
expect(mockIsAvailable).to.be.called;
330-
expect(mockGenerateContent).to.be.calledWith(fakeRequestParams);
327+
expect(result.response.text()).to.include('Mountain View, California');
328+
expect(isAvailableStub).to.be.called;
329+
expect(generateContentStub).to.be.calledWith(fakeRequestParams);
331330
});
332331
});

packages/vertexai/src/methods/generate-content.ts

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@
1616
*/
1717

1818
import {
19-
EnhancedGenerateContentResponse,
2019
GenerateContentRequest,
2120
GenerateContentResponse,
2221
GenerateContentResult,
@@ -51,18 +50,15 @@ async function generateContentOnCloud(
5150
model: string,
5251
params: GenerateContentRequest,
5352
requestOptions?: RequestOptions
54-
): Promise<EnhancedGenerateContentResponse> {
55-
const response = await makeRequest(
53+
): Promise<Response> {
54+
return makeRequest(
5655
model,
5756
Task.GENERATE_CONTENT,
5857
apiSettings,
5958
/* stream */ false,
6059
JSON.stringify(params),
6160
requestOptions
6261
);
63-
const responseJson: GenerateContentResponse = await response.json();
64-
const enhancedResponse = createEnhancedContentResponse(responseJson);
65-
return enhancedResponse;
6662
}
6763

6864
export async function generateContent(
@@ -72,17 +68,19 @@ export async function generateContent(
7268
chromeAdapter: ChromeAdapter,
7369
requestOptions?: RequestOptions
7470
): Promise<GenerateContentResult> {
75-
let enhancedResponse;
71+
let response;
7672
if (await chromeAdapter.isAvailable(params)) {
77-
enhancedResponse = await chromeAdapter.generateContentOnDevice(params);
73+
response = await chromeAdapter.generateContentOnDevice(params);
7874
} else {
79-
enhancedResponse = await generateContentOnCloud(
75+
response = await generateContentOnCloud(
8076
apiSettings,
8177
model,
8278
params,
8379
requestOptions
8480
);
8581
}
82+
const responseJson: GenerateContentResponse = await response.json();
83+
const enhancedResponse = createEnhancedContentResponse(responseJson);
8684
return {
8785
response: enhancedResponse
8886
};

0 commit comments

Comments
 (0)