Function TestDiscoverSkillFiles

internal/skills/discovery/discovery_test.go:1596–1669

(t *testing.T)

}

func TestDiscoverSkillFiles(t *testing.T) {
	tests := []struct {
		name      string
		stubs     func(*httpmock.Registry)
		wantPaths []string
		wantErr   string
	}{
		{
			name: "returns files with skill path prefix",
			stubs: func(reg *httpmock.Registry) {
				reg.Register(
					httpmock.REST("GET", "repos/monalisa/octocat-skills/git/trees/tree123"),
					httpmock.JSONResponse(map[string]interface{}{
						"sha": "tree123", "truncated": false,
						"tree": []map[string]interface{}{
							{"path": "SKILL.md", "type": "blob", "sha": "sha1", "size": 10},
							{"path": "scripts/setup.sh", "type": "blob", "sha": "sha2", "size": 50},
							{"path": "scripts", "type": "tree", "sha": "treesub"},
						},
					}))
			},
			wantPaths: []string{"skills/code-review/SKILL.md", "skills/code-review/scripts/setup.sh"},
		},
		{
			name: "truncated tree falls back to walkTree",
			stubs: func(reg *httpmock.Registry) {
				reg.Register(
					httpmock.REST("GET", "repos/monalisa/octocat-skills/git/trees/tree123"),
					httpmock.JSONResponse(map[string]interface{}{
						"sha": "tree123", "truncated": true, "tree": []map[string]interface{}{},
					}))
				reg.Register(
					httpmock.REST("GET", "repos/monalisa/octocat-skills/git/trees/tree123"),
					httpmock.JSONResponse(map[string]interface{}{
						"sha": "tree123",
						"tree": []map[string]interface{}{
							{"path": "SKILL.md", "type": "blob", "sha": "sha1", "size": 10},
						},
					}))
			},
			wantPaths: []string{"skills/code-review/SKILL.md"},
		},
		{
			name: "API error",
			stubs: func(reg *httpmock.Registry) {
				reg.Register(
					httpmock.REST("GET", "repos/monalisa/octocat-skills/git/trees/tree123"),
					httpmock.StatusStringResponse(500, "server error"))
			},
			wantErr: "could not fetch skill tree",
		},
	}
	for _, tt := range tests {
		t.Run(tt.name, func(t *testing.T) {
			reg := &httpmock.Registry{}
			defer reg.Verify(t)
			tt.stubs(reg)
			client := api.NewClientFromHTTP(&http.Client{Transport: reg})