Enhance error logging for virtual environment deletion in DeepnoteEnvironmentManager. Added detailed message output when deletion fails, improving debugging capabilities. Updated unit test comment for clarity.

Signed-off-by: Tomas Kislan <tomas@kislan.sk>

Author: tkislan

src/kernels/deepnote/environments/deepnoteEnvironmentManager.node.ts

@@ -208,6 +208,10 @@ export class DeepnoteEnvironmentManager implements IExtensionSyncActivationServi
         } catch (error) {
             // Log but don't fail - the directory might not exist or might already be deleted
             logger.warn(`Failed to delete virtual environment directory: ${config.venvPath.fsPath}`, error);
+            const msg = error instanceof Error ? error.message : String(error);
+            this.outputChannel.appendLine(
+                l10n.t('Failed to delete Deepnote virtual environment directory for "{0}": {1}', config.name, msg)
+            );
         }
 
         Cancellation.throwIfCanceled(token);

src/kernels/deepnote/environments/deepnoteEnvironmentManager.unit.test.ts

@@ -284,8 +284,7 @@ suite('DeepnoteEnvironmentManager', () => {
             // Delete the environment
             await manager.deleteEnvironment(config.id);
 
-            // Verify directory no longer exists (with a small delay to allow async operation)
-            await new Promise((resolve) => setTimeout(resolve, 100));
+            // Verify directory no longer exists
             assert.isFalse(fs.existsSync(venvDirPath), 'Directory should not exist after deletion');
         });
     });