Skip to content

Commit 59abedb

Browse files
committed
fix: resolve unused 'result' variable in lifecycle event test
- Added assertion to verify result.finalOutput in agent_end lifecycle test - Resolves ESLint error: 'result' is assigned a value but never used - Makes test more comprehensive by validating both events and return value
1 parent 95d3dfa commit 59abedb

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

packages/agents-core/test/run.test.ts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -161,10 +161,13 @@ describe('Runner.run', () => {
161161

162162
const result = await runner.run(agent, 'test input');
163163

164+
// Verify the result has the expected output
165+
expect(result.finalOutput).toBe('Hello World');
166+
164167
// Verify agent_end was called on both agent and runner
165168
expect(agentEndEvents).toHaveLength(1);
166169
expect(agentEndEvents[0].output).toBe('Hello World');
167-
170+
168171
expect(runnerEndEvents).toHaveLength(1);
169172
expect(runnerEndEvents[0].agent).toBe(agent);
170173
expect(runnerEndEvents[0].output).toBe('Hello World');
@@ -404,7 +407,7 @@ describe('Runner.run', () => {
404407
usage: new Usage(),
405408
};
406409
class SimpleStreamingModel implements Model {
407-
constructor(private resps: ModelResponse[]) {}
410+
constructor(private resps: ModelResponse[]) { }
408411
async getResponse(_req: ModelRequest): Promise<ModelResponse> {
409412
const r = this.resps.shift();
410413
if (!r) {

0 commit comments

Comments
 (0)