Skip to content

Commit a2bd0a7

Browse files
authored
fix(e2e): skip Gemini invoke and logs tests (#1434)
* fix: disable gemini invoke tests temporarily * chore: fix formatting on e2e test files
1 parent b4f5daa commit a2bd0a7

6 files changed

Lines changed: 19 additions & 7 deletions

File tree

e2e-tests/e2e-helper.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ interface E2EConfig {
3232
language?: 'Python' | 'TypeScript';
3333
/** Skip logs and traces tests. */
3434
skipObservability?: boolean;
35+
skipInvoke?: boolean;
3536
/** Lifecycle configuration to pass via --idle-timeout / --max-lifetime flags. */
3637
lifecycleConfig?: {
3738
idleTimeout?: number;
@@ -138,7 +139,7 @@ export function createE2ESuite(cfg: E2EConfig) {
138139
deployTimeout
139140
);
140141

141-
it.skipIf(!canRun)(
142+
it.skipIf(!canRun || !!cfg.skipInvoke)(
142143
'invokes the deployed agent',
143144
async () => {
144145
expect(projectPath, 'Project should have been created').toBeTruthy();
@@ -226,7 +227,7 @@ export function createE2ESuite(cfg: E2EConfig) {
226227
120000
227228
);
228229

229-
it.skipIf(!canRun || !!cfg.skipObservability)(
230+
it.skipIf(!canRun || !!cfg.skipObservability || !!cfg.skipInvoke)(
230231
'logs returns entries from the invocation',
231232
async () => {
232233
await retry(

e2e-tests/googleadk-gemini.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
import { createE2ESuite } from './e2e-helper.js';
22

3-
createE2ESuite({ framework: 'GoogleADK', modelProvider: 'Gemini', requiredEnvVar: 'GEMINI_API_KEY' });
3+
createE2ESuite({ framework: 'GoogleADK', modelProvider: 'Gemini', requiredEnvVar: 'GEMINI_API_KEY', skipInvoke: true });

e2e-tests/harness-e2e-helper.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ interface HarnessE2EConfig {
2222
modelProvider: 'bedrock' | 'open_ai' | 'gemini';
2323
requiredEnvVar?: string;
2424
skipMemory?: boolean;
25+
skipInvoke?: boolean;
2526
}
2627

2728
export function createHarnessE2ESuite(cfg: HarnessE2EConfig) {
@@ -110,7 +111,7 @@ export function createHarnessE2ESuite(cfg: HarnessE2EConfig) {
110111
600000
111112
);
112113

113-
it.skipIf(!canRun)(
114+
it.skipIf(!canRun || !!cfg.skipInvoke)(
114115
'invokes the deployed harness',
115116
async () => {
116117
expect(projectPath, 'Project should have been created').toBeTruthy();

e2e-tests/harness-gemini.test.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,8 @@
11
import { createHarnessE2ESuite } from './harness-e2e-helper.js';
22

3-
createHarnessE2ESuite({ modelProvider: 'gemini', requiredEnvVar: 'GEMINI_API_KEY_ARN', skipMemory: true });
3+
createHarnessE2ESuite({
4+
modelProvider: 'gemini',
5+
requiredEnvVar: 'GEMINI_API_KEY_ARN',
6+
skipMemory: true,
7+
skipInvoke: true,
8+
});

e2e-tests/langgraph-gemini.test.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,8 @@
11
import { createE2ESuite } from './e2e-helper.js';
22

3-
createE2ESuite({ framework: 'LangChain_LangGraph', modelProvider: 'Gemini', requiredEnvVar: 'GEMINI_API_KEY' });
3+
createE2ESuite({
4+
framework: 'LangChain_LangGraph',
5+
modelProvider: 'Gemini',
6+
requiredEnvVar: 'GEMINI_API_KEY',
7+
skipInvoke: true,
8+
});

e2e-tests/strands-gemini.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
import { createE2ESuite } from './e2e-helper.js';
22

3-
createE2ESuite({ framework: 'Strands', modelProvider: 'Gemini', requiredEnvVar: 'GEMINI_API_KEY' });
3+
createE2ESuite({ framework: 'Strands', modelProvider: 'Gemini', requiredEnvVar: 'GEMINI_API_KEY', skipInvoke: true });

0 commit comments

Comments
 (0)