fix tests

Author: eleanorjboyd

src/features/projectManager.ts

@@ -25,11 +25,20 @@ export class PythonProjectManagerImpl implements PythonProjectManager {
     private readonly _onDidChangeProjects = new EventEmitter<ProjectArray | undefined>();
     public readonly onDidChangeProjects = this._onDidChangeProjects.event;
 
+    /**
+     * Fires the onDidChangeProjects event if updateBool is true.
+     */
+    private fireDidChangeProjects(updateBool: boolean, projects: ProjectArray) {
+        if (updateBool) {
+            this._onDidChangeProjects.fire(projects);
+        }
+    }
+
     // Debounce the updateProjects method to avoid excessive update calls
     private readonly updateDebounce = createSimpleDebounce(100, () => this.updateProjects());
 
     initialize(): void {
-        this.add(this.getInitialProjects());
+        this.add(this.getInitialProjects(), true);
         this.disposables.push(
             this._onDidChangeProjects,
             new Disposable(() => this._projects.clear()),
@@ -95,7 +104,7 @@ export class PythonProjectManagerImpl implements PythonProjectManager {
      * then updates the internal _projects map to reflect the current state and
      * fires the onDidChangeProjects event if there are any changes.
      */
-    private updateProjects(): void {
+    private updateProjects(updateBool: boolean = true): void {
         const newProjects: ProjectArray = this.getInitialProjects();
         const existingProjects = Array.from(this._projects.values());
 
@@ -112,7 +121,7 @@ export class PythonProjectManagerImpl implements PythonProjectManager {
         projectsToAdd.forEach((w) => this._projects.set(w.uri.toString(), w));
 
         if (projectsToRemove.length > 0 || projectsToAdd.length > 0) {
-            this._onDidChangeProjects.fire(Array.from(this._projects.values()));
+            this.fireDidChangeProjects(updateBool, Array.from(this._projects.values()));
         }
     }
 
@@ -124,7 +133,7 @@ export class PythonProjectManagerImpl implements PythonProjectManager {
         return new PythonProjectsImpl(name, uri, options);
     }
 
-    async add(projects: PythonProject | ProjectArray): Promise<void> {
+    async add(projects: PythonProject | ProjectArray, updateBool: boolean = true): Promise<void> {
         const _projects = Array.isArray(projects) ? projects : [projects];
         if (_projects.length === 0) {
             return;
@@ -153,21 +162,21 @@ export class PythonProjectManagerImpl implements PythonProjectManager {
             // handles adding the project to this._projects map
             return this._projects.set(currProject.uri.toString(), currProject);
         });
-        this._onDidChangeProjects.fire(Array.from(this._projects.values()));
+        this.fireDidChangeProjects(updateBool, Array.from(this._projects.values()));
 
         if (edits.length > 0) {
             await addPythonProjectSetting(edits);
         }
     }
 
-    remove(projects: PythonProject | ProjectArray): void {
+    remove(projects: PythonProject | ProjectArray, updateBool: boolean = true): void {
         const _projects = Array.isArray(projects) ? projects : [projects];
         if (_projects.length === 0) {
             return;
         }
 
         _projects.forEach((w) => this._projects.delete(w.uri.toString()));
-        this._onDidChangeProjects.fire(Array.from(this._projects.values()));
+        this.fireDidChangeProjects(updateBool, Array.from(this._projects.values()));
     }
 
     /**
@@ -188,8 +197,8 @@ export class PythonProjectManagerImpl implements PythonProjectManager {
             return;
         }
 
-        // Remove the old project
-        this.remove(project);
+        // Remove the old project, do not fire event yet
+        this.remove(project, false);
 
         // Prepare new values
         const name = newName ?? project.name;
@@ -200,9 +209,12 @@ export class PythonProjectManagerImpl implements PythonProjectManager {
             iconPath: newOptions?.iconPath ?? (project as PythonProjectsImpl).iconPath,
         };
 
-        // Create and add the new project
+        // Create and add the new project, do not fire event yet
         const updatedProject = this.create(name, uri, options);
-        await this.add(updatedProject);
+        await this.add(updatedProject, false);
+
+        // Now trigger the update event only once
+        this.updateDebounce.trigger();
     }
 
     getProjects(uris?: Uri[]): ReadonlyArray<PythonProject> {

src/test/features/projectManager.updateUri.unit.test.ts

@@ -14,7 +14,7 @@ suite('Project Manager Update URI tests', () => {
         projectManager.dispose();
     });
 
-    test('updateProjectUri should update existing project URI', () => {
+    test('updateProjectUri should update existing project URI', async () => {
         const oldUri = Uri.file('/path/to/old/project');
         const newUri = Uri.file('/path/to/new/project');
 
@@ -36,7 +36,7 @@ suite('Project Manager Update URI tests', () => {
         assert.equal(oldProject.uri.fsPath, oldUri.fsPath, 'Old URI should match');
 
         // Update the project URI
-        projectManager.modifyProject(oldUri, 'project', newUri);
+        await projectManager.modifyProject(oldUri, 'project', newUri);
 
         // Verify project no longer exists with old URI
         const oldProjectAfterUpdate = projectManager.get(oldUri);
@@ -57,8 +57,8 @@ suite('Project Manager Update URI tests', () => {
 
         // Try to update a project that doesn't exist
         // This should not throw an error
-        assert.doesNotThrow(() => {
-            projectManager.modifyProject(oldUri, 'project', newUri);
+        assert.doesNotThrow(async () => {
+            await projectManager.modifyProject(oldUri, 'project', newUri);
         }, 'Should handle non-existent project gracefully');
 
         // Verify no project was created