Skip to content

Commit 1bde086

Browse files
authored
fix: update asset snapshots and fix useDevDeploy skip test (#921)
- Update 3 asset snapshots (file listing, cdk/bin/cdk.ts, cdk/lib/cdk-stack.ts) to match current harness-implementation branch content - Fix useDevDeploy test: when skip=true, isComplete is correctly true (skipped means done), not false as the test previously asserted
1 parent 35ec0c1 commit 1bde086

2 files changed

Lines changed: 15 additions & 2 deletions

File tree

src/assets/__tests__/__snapshots__/assets.snapshot.test.ts.snap

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,6 +110,8 @@ async function main() {
110110
memoryName?: string;
111111
containerUri?: string;
112112
hasDockerfile?: boolean;
113+
tools?: { type: string; name: string }[];
114+
apiKeyArn?: string;
113115
}[] = [];
114116
for (const entry of specAny.harnesses ?? []) {
115117
const harnessPath = path.resolve(projectRoot, entry.path, 'harness.json');
@@ -121,6 +123,8 @@ async function main() {
121123
memoryName: harnessSpec.memory?.name,
122124
containerUri: harnessSpec.containerUri,
123125
hasDockerfile: !!harnessSpec.dockerfile,
126+
tools: harnessSpec.tools,
127+
apiKeyArn: harnessSpec.model?.apiKeyArn,
124128
});
125129
} catch (err) {
126130
throw new Error(
@@ -312,7 +316,15 @@ export interface AgentCoreStackProps extends StackProps {
312316
/**
313317
* Harness role configurations. Each entry creates an IAM execution role for a harness.
314318
*/
315-
harnesses?: { name: string; executionRoleArn?: string; memoryName?: string }[];
319+
harnesses?: {
320+
name: string;
321+
executionRoleArn?: string;
322+
memoryName?: string;
323+
containerUri?: string;
324+
hasDockerfile?: boolean;
325+
tools?: { type: string; name: string }[];
326+
apiKeyArn?: string;
327+
}[];
316328
}
317329
318330
/**
@@ -485,6 +497,7 @@ exports[`Assets Directory Snapshots > File listing > should match the expected f
485497
"evaluators/python-lambda/execution-role-policy.json",
486498
"evaluators/python-lambda/lambda_function.py",
487499
"evaluators/python-lambda/pyproject.toml",
500+
"harness/invoke.py.template",
488501
"mcp/python-lambda/README.md",
489502
"mcp/python-lambda/handler.py",
490503
"mcp/python-lambda/pyproject.toml",

src/cli/tui/hooks/__tests__/useDevDeploy.test.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ describe('useDevDeploy', () => {
4747
const { lastFrame } = render(<Harness skip={true} />);
4848

4949
await vi.waitFor(() => {
50-
expect(lastFrame()).toContain('isComplete:false');
50+
expect(lastFrame()).toContain('isComplete:true');
5151
});
5252

5353
expect(mockHandleDeploy).not.toHaveBeenCalled();

0 commit comments

Comments
 (0)