Skip to content

Comments

Add test coverage for FuncHostDebugViewProvider#4886

Draft
Copilot wants to merge 3 commits intonat/debuggerViewfrom
copilot/sub-pr-4883-again
Draft

Add test coverage for FuncHostDebugViewProvider#4886
Copilot wants to merge 3 commits intonat/debuggerViewfrom
copilot/sub-pr-4883-again

Conversation

Copy link
Contributor

Copilot AI commented Jan 15, 2026

Adds missing test coverage for the Function Host Debug View components introduced in #4883.

Test coverage:

  • Tree item creation for all node types (noHost, hostTask, hostError)
  • Type guard validation (isHostTaskNode, isHostErrorNode) with edge cases
  • Refresh event emission
  • getChildren behavior for different node types

Exports for testing:

  • FuncHostDebugViewProvider class
  • Type guard functions (isHostTaskNode, isHostErrorNode)

Tests follow existing repository patterns using Mocha.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copy link
Contributor Author

Copilot AI commented Jan 15, 2026

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • aka.ms
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/vscode-azurefunctions/vscode-azurefunctions/node_modules/.bin/gulp preTest (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Fix issues based on feedback for Function Host Debugger view Add test coverage for FuncHostDebugViewProvider Jan 15, 2026
Copilot AI requested a review from nturinski January 15, 2026 00:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants