scriptoriumadmin/Roo-Code
1
0
Fork 0
mirror of https://github.com/RooVetGit/Roo-Code.git synced 2026-09-11 22:51:26 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

test: update environment details tests to match XML format

Browse source 
Updated the getEnvironmentDetails.spec.ts tests to expect XML format instead of markdown headings. The implementation was changed to use XML format with the fast-xml-parser library, but the tests were still expecting markdown format. This change updates the tests to match the new XML structure while preserving the same semantic checks.

Signed-off-by: Eric Wheeler <roo-code@z.ewheeler.org>
This commit is contained in:
Eric Wheeler 2025-07-17 15:54:06 -07:00
parent 8734faff54
commit 35d22e00ca
1 changed files with 33 additions and 30 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

63
src/core/environment/__tests__/getEnvironmentDetails.spec.ts
View file

@ -141,14 +141,13 @@ describe("getEnvironmentDetails", () => {
it("should return basic environment details", async () => {
const result = await getEnvironmentDetails(mockCline as Task)
expect(result).toContain("<environment_details>")
expect(result).toContain("</environment_details>")
expect(result).toContain("# VSCode Visible Files")
expect(result).toContain("# VSCode Open Tabs")
expect(result).toContain("# Current Time")
expect(result).toContain("# Current Cost")
expect(result).toContain("# Current Mode")
expect(result).toContain("<model>test-model</model>")
expect(result).toContain("<environment_details")
expect(result).toContain("<time")
expect(result).toContain("<cost")
expect(result).toContain("<mode")
expect(result).toContain('t="0.25"')
expect(result).toContain('c="USD"')
expect(result).toContain('model="test-model"')
expect(mockProvider.getState).toHaveBeenCalled()
@ -163,8 +162,8 @@ describe("getEnvironmentDetails", () => {
it("should include file details when includeFileDetails is true", async () => {
const result = await getEnvironmentDetails(mockCline as Task, true)
expect(result).toContain("# Current Workspace Directory")
expect(result).toContain("Files")
expect(result).toContain("<workspace")
expect(result).toContain("directory=")
expect(listFiles).toHaveBeenCalledWith(mockCwd, true, 50)
@ -211,9 +210,9 @@ describe("getEnvironmentDetails", () => {
const result = await getEnvironmentDetails(mockCline as Task)
expect(result).toContain("# Recently Modified Files")
expect(result).toContain("modified1.ts")
expect(result).toContain("modified2.ts")
expect(result).toContain("<recentlyModified>")
expect(result).toContain('path="modified1.ts"')
expect(result).toContain('path="modified2.ts"')
})
it("should include active terminal information", async () => {
@ -229,10 +228,11 @@ describe("getEnvironmentDetails", () => {
const result = await getEnvironmentDetails(mockCline as Task)
expect(result).toContain("# Actively Running Terminals")
expect(result).toContain("## Terminal terminal-1 (Active)")
expect(result).toContain("### Working Directory: `/test/path/src`")
expect(result).toContain("### Original command: `npm test`")
expect(result).toContain("<terminal")
expect(result).toContain('id="terminal-1"')
expect(result).toContain('status="Active"')
expect(result).toContain('cwd="/test/path/src"')
expect(result).toContain('command="npm test"')
expect(result).toContain("Test output")
mockCline.didEditFile = true
@ -262,10 +262,11 @@ describe("getEnvironmentDetails", () => {
const result = await getEnvironmentDetails(mockCline as Task)
expect(result).toContain("# Inactive Terminals with Completed Process Output")
expect(result).toContain("## Terminal terminal-2 (Inactive)")
expect(result).toContain("### Working Directory: `/test/path/build`")
expect(result).toContain("Command: `npm build`")
expect(result).toContain("<terminal")
expect(result).toContain('id="terminal-2"')
expect(result).toContain('status="Inactive"')
expect(result).toContain('cwd="/test/path/build"')
expect(result).toContain('command="npm build"')
expect(result).toContain("Build output")
expect(mockInactiveTerminal.cleanCompletedProcessQueue).toHaveBeenCalled()
@ -300,13 +301,15 @@ describe("getEnvironmentDetails", () => {
const result = await getEnvironmentDetails(mockCline as Task)
// Check active terminal working directory
expect(result).toContain("## Terminal terminal-1 (Active)")
expect(result).toContain("### Working Directory: `/some/path`")
expect(result).toContain("### Original command: `cd /some/path && npm start`")
expect(result).toContain('id="terminal-1"')
expect(result).toContain('status="Active"')
expect(result).toContain('cwd="/some/path"')
expect(result).toContain('command="cd /some/path &amp;&amp; npm start"')
// Check inactive terminal working directory
expect(result).toContain("## Terminal terminal-2 (Inactive)")
expect(result).toContain("### Working Directory: `/another/path`")
expect(result).toContain('id="terminal-2"')
expect(result).toContain('status="Inactive"')
expect(result).toContain('cwd="/another/path"')
// Verify the methods were called
expect(mockActiveTerminal.getCurrentWorkingDirectory).toHaveBeenCalled()
@ -319,8 +322,9 @@ describe("getEnvironmentDetails", () => {
const result = await getEnvironmentDetails(mockCline as Task)
expect(result).toContain("<role>You are a code assistant</role>")
expect(result).toContain("<custom_instructions>Custom instructions</custom_instructions>")
expect(result).toContain("<mode")
expect(result).toContain('role="You are a code assistant"')
expect(result).toContain('customInstructions="Custom instructions"')
})
it("should handle missing provider or state", async () => {
@ -341,8 +345,7 @@ describe("getEnvironmentDetails", () => {
const result2 = await getEnvironmentDetails(mockCline as Task)
// Verify the function still returns a result.
expect(result2).toContain("<environment_details>")
expect(result2).toContain("</environment_details>")
expect(result2).toContain("<environment_details")
})
it("should handle errors gracefully", async () => {