[test] Add tests for mcpresult.ExtractTextContent

[github-actions]

Test Coverage Improvement: ExtractTextContent

Function Analyzed

• Package: internal/mcpresult
• Function: ExtractTextContent
• Previous Coverage: 0%
• New Coverage: 100%
• Complexity: Medium

Why This Function?

ExtractTextContent was selected because it had 0% coverage and meaningful complexity: it handles two distinct input slice types ([]interface{} vs []map[string]interface{}), filters content items by type (text/image/audio/resource/unknown), performs type assertions on the text field, and has several skip conditions. Covering all branches requires a rich test suite.

Tests Added

• ✅ Happy path – single text item via []interface{} and []map[string]interface{}
• ✅ Nil/missing content key and nil content value
• ✅ Unsupported content types (string, int, map, bool → returns "")
• ✅ Empty slices (both []interface{} and []map[string]interface{})
• ✅ Item type filtering: "text" and "" kept; "image", "audio", "resource" skipped; unknown types kept per backward-compat
• ✅ Non-map items in []interface{} are skipped
• ✅ Items with empty text, missing text key, or non-string text value are skipped
• ✅ Multi-item concatenation (including newlines and unicode)
• ✅ Mixed type scenarios (text interspersed with skipped types)
• ✅ Nil map input
• ✅ Complex realistic scenario combining all edge cases

Coverage Report

Before: 0.0% coverage
After:  100.0% coverage
Improvement: +100%

Test Execution

All 37 sub-tests pass:

ok  github.com/github/gh-aw-mcpg/internal/mcpresult  0.003s  coverage: 100.0% of statements
github.com/github/gh-aw-mcpg/internal/mcpresult/text_content.go:8: ExtractTextContent  100.0%

Note: Tests use only the standard testing package. testify is the project convention but its module zip is unavailable in the network-restricted environment; the test patterns (table-driven, t.Parallel, descriptive names) match the project style otherwise.

---

Generated by Test Coverage Improver
Next run will target the next most complex under-tested function

Warning

Firewall blocked 2 domains

The following domains were blocked by the firewall during workflow execution:

• proxy.golang.org
• sum.golang.org

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "proxy.golang.org"
    - "sum.golang.org"

See Network Configuration for more information.

Generated by Test Coverage Improver · sonnet46 3M · ◷

[lpcox]

@copilot fix the failing ci check https://github.com/github/gh-aw-mcpg/actions/runs/27029273303/job/79888506961?pr=7060

[Copilot]

Pull request overview

Adds a comprehensive table-driven test suite for internal/mcpresult.ExtractTextContent, improving confidence in the function’s branching behavior across supported content shapes and edge cases.

Changes:

• Added coverage for content being absent/nil, unsupported types, empty slices, and mixed item types.
• Added coverage for both []interface{} and []map[string]interface{} input forms (including non-map items and invalid text fields).
• Added a dedicated nil-map input test.

Show a summary per file

File	Description
internal/mcpresult/text_content_test.go	Adds comprehensive unit tests covering all ExtractTextContent branches and edge cases.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

• Files reviewed: 1/1 changed files
• Comments generated: 0

[Copilot]

@copilot fix the failing ci check https://github.com/github/gh-aw-mcpg/actions/runs/27029273303/job/79888506961?pr=7060

Fixed in commit 9ea340b. The failing CI check was from gofmt in internal/mcpresult/text_content_test.go; I applied formatting only and re-ran make lint plus go test ./internal/mcpresult successfully.

[lpcox]

@copilot fix the failing ci check https://github.com/github/gh-aw-mcpg/actions/runs/27066749712/job/79888856571?pr=7060