Skip to content

Commit bc2aab8

Browse files
rajbosCopilot
andcommitted
fix: skip claudedesktop getEditorRoot assertion on non-Windows in test
ClaudeDesktopCoworkDataAccess.getCoworkBaseDir() intentionally returns '' on non-Windows platforms (the feature is Windows-only). The test 'getEditorRoot: all adapters return non-empty string' was failing on Linux CI because it called getEditorRoot() on every adapter without regard for platform availability. Skip the claudedesktop adapter check when not on win32. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 2cddffb commit bc2aab8

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

vscode-extension/test/unit/ecosystemAdapters.test.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -195,6 +195,8 @@ test('MistralVibeAdapter.getCandidatePaths: returns session log directory', () =
195195
test('getEditorRoot: all adapters return non-empty string', () => {
196196
const dummyFile = '/dummy/path/session.json';
197197
for (const adapter of allAdapters) {
198+
// claudedesktop is Windows-only; getCoworkBaseDir() returns '' on non-Windows platforms
199+
if (adapter.id === 'claudedesktop' && os.platform() !== 'win32') { continue; }
198200
const root = adapter.getEditorRoot(dummyFile);
199201
assert.ok(typeof root === 'string' && root.length > 0, `${adapter.id}: getEditorRoot should return non-empty string`);
200202
}

0 commit comments

Comments
 (0)