Fixed bug that results in incorrect logging of the pythonVersion stri…

…ng. This addresses #8798.
microsoft · Aug 21, 2024 · a9bd905 · a9bd905
1 parent 2876bae
commit a9bd905
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/packages/pyright-internal/src/common/configOptions.ts b/packages/pyright-internal/src/common/configOptions.ts
@@ -1496,7 +1496,7 @@ export class ConfigOptions {
         const importFailureInfo: string[] = [];
         this.defaultPythonVersion = host.getPythonVersion(this.pythonPath, importFailureInfo);
         if (this.defaultPythonVersion !== undefined) {
-            console.info(`Assuming Python version ${this.defaultPythonVersion.toString()}`);
+            console.info(`Assuming Python version ${PythonVersion.toString(this.defaultPythonVersion)}`);
         }
 
         for (const log of importFailureInfo) {

diff --git a/packages/pyright-internal/src/tests/lsp/languageServerTestUtils.ts b/packages/pyright-internal/src/tests/lsp/languageServerTestUtils.ts
@@ -339,7 +339,7 @@ export async function runPyrightServer(
         testName: expect.getState().currentTestName ?? 'NoName',
         code,
         projectRoots: projectRootsArray.map((p) => (p.includes(':') ? UriEx.parse(p) : UriEx.file(p))),
-        pythonVersion: pythonVersion.toString(),
+        pythonVersion: PythonVersion.toString(pythonVersion),
         backgroundAnalysis,
         logFile: UriEx.file(path.join(__dirname, `log${process.pid}.txt`)),
         pid: process.pid.toString(),