Add comprehensive tests for PR overview sync functionality

Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>

Author: Copilot

src/test/github/pullRequestOverview.test.ts

@@ -137,4 +137,81 @@ describe('PullRequestOverview', function () {
 			assert.strictEqual(panel0!.getCurrentTitle(), 'Pull Request #2000');
 		});
 	});
+
+	describe('PR overview sync', function () {
+		it('emits event when PR overview becomes active', async function () {
+			// Set up PR model
+			repo.addGraphQLPullRequest(builder => {
+				builder.pullRequest(response => {
+					response.repository(r => {
+						r.pullRequest(pr => pr.number(1000));
+					});
+				});
+			});
+
+			const prItem = convertRESTPullRequestToRawPullRequest(new PullRequestBuilder().number(1000).build(), repo);
+			const prModel = new PullRequestModel(credentialStore, telemetry, repo, remote, prItem);
+
+			// Listen for the event
+			let eventFired = false;
+			let eventPR: PullRequestModel | undefined;
+			const disposable = PullRequestOverviewPanel.onDidChangeActivePullRequest(pr => {
+				eventFired = true;
+				eventPR = pr;
+			});
+
+			try {
+				// Create and show the panel - this should trigger the event
+				await PullRequestOverviewPanel.createOrShow(telemetry, EXTENSION_URI, pullRequestManager, prModel);
+
+				// Verify event was fired with correct PR
+				assert.strictEqual(eventFired, true, 'Event should have been fired when PR overview became active');
+				assert.strictEqual(eventPR?.number, 1000, 'Event should contain the correct PR model');
+			} finally {
+				disposable.dispose();
+			}
+		});
+
+		it('emits event when panel visibility changes', async function () {
+			// Set up PR model
+			repo.addGraphQLPullRequest(builder => {
+				builder.pullRequest(response => {
+					response.repository(r => {
+						r.pullRequest(pr => pr.number(2000));
+					});
+				});
+			});
+
+			const prItem = convertRESTPullRequestToRawPullRequest(new PullRequestBuilder().number(2000).build(), repo);
+			const prModel = new PullRequestModel(credentialStore, telemetry, repo, remote, prItem);
+
+			// Create panel first
+			await PullRequestOverviewPanel.createOrShow(telemetry, EXTENSION_URI, pullRequestManager, prModel);
+			const panel = PullRequestOverviewPanel.currentPanel;
+			assert.notStrictEqual(panel, undefined);
+
+			// Listen for the event
+			let eventCount = 0;
+			let lastEventPR: PullRequestModel | undefined;
+			const disposable = PullRequestOverviewPanel.onDidChangeActivePullRequest(pr => {
+				eventCount++;
+				lastEventPR = pr;
+			});
+
+			try {
+				// Reset event count to track only visibility changes
+				eventCount = 0;
+
+				// Simulate panel becoming visible - this should trigger the event
+				// We simulate this by calling the method directly since testing webview visibility is complex
+				(panel as any).onDidChangeViewState({ webviewPanel: { visible: true } });
+
+				// Verify event was fired
+				assert.strictEqual(eventCount, 1, 'Event should have been fired when panel became visible');
+				assert.strictEqual(lastEventPR?.number, 2000, 'Event should contain the correct PR model');
+			} finally {
+				disposable.dispose();
+			}
+		});
+	});
 });

src/test/view/prsTree.test.ts

@@ -225,4 +225,64 @@ describe('GitHub Pull Requests view', function () {
 			assert.deepStrictEqual(localItem1.iconPath!.toString(), 'https://avatars.com/you.jpg');
 		});
 	});
+
+	describe('PR overview sync', function () {
+		it('should sync tree view when PR overview becomes active', async function () {
+			// Set up repository and remotes
+			const remote = new GitHubRemote('origin', 'https://github.com/owner/repo', new Protocol('https://github.com/owner/repo'), GitHubServerType.GitHubDotCom);
+			const repository = new MockRepository();
+			repository.addRemote('origin', 'https://github.com/owner/repo');
+
+			// Set up github repository
+			const gitHubRepository = new MockGitHubRepository(remote, credentialStore, telemetry, sinon);
+
+			// Set up PR using the builder pattern
+			const pr = gitHubRepository.addGraphQLPullRequest(builder => {
+				builder.pullRequest(pr => {
+					pr.repository(r =>
+						r.pullRequest(p => {
+							p.databaseId(123);
+							p.number(123);
+							p.title('test-pr');
+							p.author(a => a.login('testuser').avatarUrl('https://avatars.com/testuser.jpg').url('https://github.com/testuser'));
+							p.baseRef!(b => b.repository(br => br.url('https://github.com/owner/repo')));
+							p.baseRepository(r => r.url('https://github.com/owner/repo'));
+						}),
+					);
+				});
+			}).pullRequest;
+
+			const prItem = await parseGraphQLPullRequest(pr.repository.pullRequest, gitHubRepository);
+			const pullRequestModel = new PullRequestModel(credentialStore, telemetry, gitHubRepository, remote, prItem);
+
+			// Set up folder manager  
+			const manager = new FolderRepositoryManager(0, context, repository, telemetry, new GitApiImpl(reposManager), credentialStore, createPrHelper);
+			reposManager.insertFolderManager(manager);
+			sinon.stub(manager, 'createGitHubRepository').returns(Promise.resolve(gitHubRepository));
+			sinon.stub(credentialStore, 'isAuthenticated').returns(true);
+			
+			await manager.updateRepositories();
+			provider.initialize([], credentialStore);
+
+			// Spy on expand and reveal methods to verify they get called
+			const expandSpy = sinon.spy(provider, 'expandPullRequest');
+			let revealCalled = false;
+			let revealElement: any = undefined;
+			sinon.stub(provider, 'reveal').callsFake(async (element, options) => {
+				revealCalled = true;
+				revealElement = element;
+				return Promise.resolve();
+			});
+
+			// Trigger the sync by calling the method directly
+			const syncMethod = (provider as any).syncWithActivePullRequest;
+			if (syncMethod) {
+				await syncMethod.call(provider, pullRequestModel);
+
+				// Verify that expandPullRequest was called
+				assert(expandSpy.calledWith(pullRequestModel), 'expandPullRequest should be called with the PR model');
+				// Note: reveal might not be called if PR node is not found in tree, which is expected in this test setup
+			}
+		});
+	});
 });