Address review feedback: add comment to catch block, tighten test assertion

Agent-Logs-Url: https://github.com/microsoft/vscode-pull-request-github/sessions/42a1cce0-564c-42dd-afc9-909f2730d385

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

Author: Copilot

src/issues/stateManager.ts

@@ -356,7 +356,9 @@ export class StateManager {
 				issueItem.uri = folderManager.repository.rootUri;
 				return issueItem;
 			});
-		} catch (e) {
+		} catch {
+			// Errors from fetching issues are expected (e.g. network failures).
+			// Return undefined so the tree shows an empty state for this query.
 			return undefined;
 		} finally {
 			this._onDidChangeIssueData.fire();

src/test/issues/stateManager.test.ts

@@ -255,7 +255,7 @@ describe('StateManager branch behavior with useBranchForIssues setting', functio
 			await (sm as any).setIssueData(mockFolderManager);
 
 			// The event should fire at least twice: once from setIssues (per-query) and once from setIssueData (final)
-			assert.ok(changeEventCount >= 2, `onDidChangeIssueData should fire at least twice, but fired ${changeEventCount} times`);
+			assert.strictEqual(changeEventCount, 2, `onDidChangeIssueData should fire exactly twice for a single query, but fired ${changeEventCount} times`);
 		} finally {
 			vscode.workspace.getConfiguration = originalGetConfiguration;
 		}