Skip to content

Commit 9db77d2

Browse files
advancedresearcharraycursoragent
andcommitted
test: verify security advisory write tools are registered
Add regression coverage ensuring create, update, and CVE request tools are registered in the security_advisories toolset with write hints. Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent de6b7d6 commit 9db77d2

1 file changed

Lines changed: 19 additions & 0 deletions

File tree

pkg/github/security_advisories_write_test.go

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -512,3 +512,22 @@ func Test_validateSeverityOrCVSS(t *testing.T) {
512512
assert.Error(t, validateSeverityOrCVSS("high", "CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:H/A:H", false))
513513
})
514514
}
515+
516+
func TestSecurityAdvisoryWriteToolsRegistered(t *testing.T) {
517+
expected := map[string]bool{
518+
"create_repository_security_advisory": false,
519+
"update_repository_security_advisory": false,
520+
"request_cve_for_repository_security_advisory": false,
521+
}
522+
523+
for _, tool := range AllTools(translations.NullTranslationHelper) {
524+
if _, ok := expected[tool.Tool.Name]; ok {
525+
assert.Equal(t, ToolsetMetadataSecurityAdvisories.ID, tool.Toolset.ID)
526+
require.NotNil(t, tool.Tool.Annotations)
527+
assert.Equal(t, expected[tool.Tool.Name], tool.Tool.Annotations.ReadOnlyHint)
528+
delete(expected, tool.Tool.Name)
529+
}
530+
}
531+
532+
assert.Empty(t, expected, "missing security advisory write tools: %v", expected)
533+
}

0 commit comments

Comments
 (0)