Skip to content

Commit d82057a

Browse files
authored
codewhisperer: isTestFile should use normalize filePath (#3766)
1 parent 3a08fcb commit d82057a

File tree

2 files changed

+11
-2
lines changed

2 files changed

+11
-2
lines changed
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
{
2+
"type": "Bug Fix",
3+
"description": "CodeWhisperer: Fix test file supplemental context not working as expected on OS not using '/' as file path separator"
4+
}

src/codewhisperer/util/supplementalContext/codeParsingUtil.ts

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
import * as vscode from 'vscode'
77
import path = require('path')
88
import { DependencyGraph } from '../dependencyGraph/dependencyGraph'
9+
import { normalize } from '../../../shared/utilities/pathUtils'
910

1011
export interface utgLanguageConfig {
1112
extension: string
@@ -76,8 +77,12 @@ export async function isTestFile(
7677
fileContent?: string
7778
}
7879
): Promise<boolean> {
79-
const pathContainsTest = filePath.includes(`tests/`) || filePath.includes('test/') || filePath.includes('tst/')
80-
const fileNameMatchTestPatterns = isTestFileByName(filePath, languageConfig.languageId)
80+
const normalizedFilePath = normalize(filePath)
81+
const pathContainsTest =
82+
normalizedFilePath.includes('tests/') ||
83+
normalizedFilePath.includes('test/') ||
84+
normalizedFilePath.includes('tst/')
85+
const fileNameMatchTestPatterns = isTestFileByName(normalizedFilePath, languageConfig.languageId)
8186

8287
if (pathContainsTest || fileNameMatchTestPatterns) {
8388
return true

0 commit comments

Comments
 (0)