Skip to content

Commit

Permalink
Better logging in error messages (#12253)
Browse files Browse the repository at this point in the history
* Better logging in error messages

* misc
  • Loading branch information
DonJayamanne authored Dec 1, 2022
1 parent b2ebc82 commit 320b783
Showing 1 changed file with 10 additions and 5 deletions.
15 changes: 10 additions & 5 deletions src/test/datascience/notebook/helper.ts
Original file line number Diff line number Diff line change
Expand Up @@ -605,7 +605,12 @@ async function getActiveInterpreterKernelConnection() {
areInterpreterPathsSame(item.interpreter.uri, interpreter.uri)
) as PythonKernelConnectionMetadata,
defaultNotebookTestTimeout,
`Kernel Connection pointing to active interpreter not found.0`
() =>
`Kernel Connection pointing to active interpreter not found.0, active interpreter
${interpreter?.id} (${getDisplayPath(interpreter?.uri)}) for kernels ${kernelFinder.kernels
.map((item) => `${item.id}=> ${item.kind} (${getDisplayPath(item.interpreter?.uri)})`)
.join(', ')}`,
500
);
}
async function getDefaultPythonRemoteKernelConnectionForActiveInterpreter() {
Expand All @@ -624,11 +629,11 @@ async function getDefaultPythonRemoteKernelConnectionForActiveInterpreter() {
) as RemoteKernelSpecConnectionMetadata,
defaultNotebookTestTimeout,
() =>
`Kernel Connection pointing to active interpreter not found.1, active interpreter ${getDisplayPath(
interpreter?.uri
)} for kernels ${kernelFinder.kernels
`Kernel Connection pointing to active interpreter not found.1, active interpreter
${interpreter?.id} (${getDisplayPath(interpreter?.uri)}) for kernels ${kernelFinder.kernels
.map((item) => `${item.id}=> ${item.kind} (${getDisplayPath(item.interpreter?.uri)})`)
.join(', ')}`
.join(', ')}`,
500
);
}
export async function getDefaultKernelConnection() {
Expand Down

0 comments on commit 320b783

Please sign in to comment.