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

fix(cli): fix quiet mode tests by properly setting integrationTest option

Browse source
This commit is contained in:
cte 2026-01-18 09:25:59 -08:00
parent 7d429ce98e
commit c691ed05a3
1 changed files with 14 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
apps/cli/src/agent/__tests__/extension-host.test.ts
View file

@ -100,16 +100,17 @@ describe("ExtensionHost", () => {
ephemeral: false,
debug: false,
exitOnComplete: false,
integrationTest: true,
}
const host = new ExtensionHost(options)
// Options are stored but integrationTest is set to true
// Options are stored as provided
const storedOptions = getPrivate<ExtensionHostOptions>(host, "options")
expect(storedOptions.mode).toBe(options.mode)
expect(storedOptions.workspacePath).toBe(options.workspacePath)
expect(storedOptions.extensionPath).toBe(options.extensionPath)
expect(storedOptions.integrationTest).toBe(true) // Always set to true in constructor
expect(storedOptions.integrationTest).toBe(true)
})
it("should be an EventEmitter instance", () => {
@ -287,10 +288,11 @@ describe("ExtensionHost", () => {
describe("quiet mode", () => {
describe("setupQuietMode", () => {
it("should not modify console when integrationTest is true", () => {
// By default, constructor sets integrationTest = true
const host = createTestHost()
// Create host with integrationTest: true so constructor doesn't suppress console
const host = createTestHost({ integrationTest: true })
const originalLog = console.log
// Calling setupQuietMode again should be a no-op since integrationTest is true
callPrivate(host, "setupQuietMode")
// Console should not be modified since integrationTest is true
@ -298,10 +300,11 @@ describe("ExtensionHost", () => {
})
it("should suppress console when integrationTest is false", () => {
const host = createTestHost()
// Create host with integrationTest: true first, so constructor doesn't suppress
const host = createTestHost({ integrationTest: true })
const originalLog = console.log
// Override integrationTest to false
// Now override integrationTest to false
const options = getPrivate<ExtensionHostOptions>(host, "options")
options.integrationTest = false
@ -315,7 +318,8 @@ describe("ExtensionHost", () => {
})
it("should preserve console.error even when suppressing", () => {
const host = createTestHost()
// Create host with integrationTest: true first, so constructor doesn't suppress
const host = createTestHost({ integrationTest: true })
const originalError = console.error
// Override integrationTest to false
@ -332,7 +336,8 @@ describe("ExtensionHost", () => {
describe("restoreConsole", () => {
it("should restore original console methods when suppressed", () => {
const host = createTestHost()
// Create host with integrationTest: true first, so constructor doesn't suppress
const host = createTestHost({ integrationTest: true })
const originalLog = console.log
// Override integrationTest to false to actually suppress
@ -346,7 +351,7 @@ describe("ExtensionHost", () => {
})
it("should handle case where console was not suppressed", () => {
const host = createTestHost()
const host = createTestHost({ integrationTest: true })
expect(() => {
callPrivate(host, "restoreConsole")