Function TestComputeMCPToolsDiff_ChangedTools

pkg/cli/audit_diff_test.go:430–466

(t *testing.T)

}

func TestComputeMCPToolsDiff_ChangedTools(t *testing.T) {
	run1 := &MCPToolUsageData{
		Summary: []MCPToolSummary{
			{ServerName: "github", ToolName: "issue_read", CallCount: 5, ErrorCount: 0},
			{ServerName: "github", ToolName: "create_pr", CallCount: 2, ErrorCount: 1},
		},
	}
	run2 := &MCPToolUsageData{
		Summary: []MCPToolSummary{
			{ServerName: "github", ToolName: "issue_read", CallCount: 10, ErrorCount: 0},
			{ServerName: "github", ToolName: "create_pr", CallCount: 2, ErrorCount: 3},
		},
	}

	diff := computeMCPToolsDiff(run1, run2)

	assert.Len(t, diff.ChangedTools, 2, "Should have 2 changed tools")

	issueRead := findMCPToolDiffEntry(diff.ChangedTools, "github", "issue_read")
	require.NotNil(t, issueRead, "Should find issue_read in changed tools")
	assert.Equal(t, "changed", issueRead.Status, "Status should be 'changed'")
	assert.Equal(t, 5, issueRead.Run1CallCount, "Run1 call count should be 5")
	assert.Equal(t, 10, issueRead.Run2CallCount, "Run2 call count should be 10")
	assert.Equal(t, "+5", issueRead.CallCountChange, "Call count change should be +5")
	assert.False(t, issueRead.IsAnomaly, "No error increase should not be anomaly")

	createPR := findMCPToolDiffEntry(diff.ChangedTools, "github", "create_pr")
	require.NotNil(t, createPR, "Should find create_pr in changed tools")
	assert.True(t, createPR.IsAnomaly, "Increased error count should be anomaly")
	assert.Equal(t, "error count increased", createPR.AnomalyNote, "Anomaly note should explain error increase")
	assert.Equal(t, 1, createPR.Run1ErrorCount, "Run1 error count should be 1")
	assert.Equal(t, 3, createPR.Run2ErrorCount, "Run2 error count should be 3")

	assert.Equal(t, 2, diff.Summary.ChangedToolCount, "Summary should show 2 changed tools")
	assert.True(t, diff.Summary.HasAnomalies, "Should have anomalies")
	assert.Equal(t, 1, diff.Summary.AnomalyCount, "Should have 1 anomaly")
}

func TestComputeMCPToolsDiff_BothNil(t *testing.T) {
	diff := computeMCPToolsDiff(nil, nil)