ci: improve coverage comment to update in place and collapse details by yiftach-armis · Pull Request #22 · ArmisSecurity/armis-cli

yiftach-armis · 2026-01-07T08:42:39Z

Use hidden marker to find and update existing coverage comment
Show only total coverage prominently
Collapse function-by-function details in
Details
block
Reduces PR comment noise from repeated coverage reports

- Use hidden marker to find and update existing coverage comment - Show only total coverage prominently - Collapse function-by-function details in <details> block - Reduces PR comment noise from repeated coverage reports

github-actions · 2026-01-07T08:43:48Z

Test Coverage Report

total: (statements) 56.5%

Coverage by function

github.com/ArmisSecurity/armis-cli/cmd/armis-cli/main.go:16:		main				0.0%
github.com/ArmisSecurity/armis-cli/internal/api/client.go:34:		WithHTTPClient			100.0%
github.com/ArmisSecurity/armis-cli/internal/api/client.go:42:		NewClient			100.0%
github.com/ArmisSecurity/armis-cli/internal/api/client.go:78:		IsDebug				100.0%
github.com/ArmisSecurity/armis-cli/internal/api/client.go:83:		StartIngest			78.4%
github.com/ArmisSecurity/armis-cli/internal/api/client.go:149:		GetIngestStatus			84.2%
github.com/ArmisSecurity/armis-cli/internal/api/client.go:182:		WaitForIngest			0.0%
github.com/ArmisSecurity/armis-cli/internal/api/client.go:224:		FetchNormalizedResults		78.6%
github.com/ArmisSecurity/armis-cli/internal/api/client.go:271:		FetchAllNormalizedResults	91.7%
github.com/ArmisSecurity/armis-cli/internal/api/client.go:296:		GetScanResult			66.7%
github.com/ArmisSecurity/armis-cli/internal/api/client.go:324:		WaitForScan			0.0%
github.com/ArmisSecurity/armis-cli/internal/api/client.go:345:		formatBytes			100.0%
github.com/ArmisSecurity/armis-cli/internal/cmd/root.go:41:		SetVersion			100.0%
github.com/ArmisSecurity/armis-cli/internal/cmd/root.go:49:		Execute				100.0%
github.com/ArmisSecurity/armis-cli/internal/cmd/root.go:53:		init				100.0%
github.com/ArmisSecurity/armis-cli/internal/cmd/root.go:65:		getEnvOrDefault			100.0%
github.com/ArmisSecurity/armis-cli/internal/cmd/root.go:72:		getEnvOrDefaultInt		100.0%
github.com/ArmisSecurity/armis-cli/internal/cmd/root.go:82:		getAPIBaseURL			100.0%
github.com/ArmisSecurity/armis-cli/internal/cmd/root.go:89:		getToken			100.0%
github.com/ArmisSecurity/armis-cli/internal/cmd/root.go:96:		getTenantID			100.0%
github.com/ArmisSecurity/armis-cli/internal/cmd/root.go:103:		getPageLimit			100.0%
github.com/ArmisSecurity/armis-cli/internal/cmd/root.go:110:		validatePageLimit		100.0%
github.com/ArmisSecurity/armis-cli/internal/cmd/scan.go:21:		init				100.0%
github.com/ArmisSecurity/armis-cli/internal/cmd/scan_image.go:94:	init				100.0%
github.com/ArmisSecurity/armis-cli/internal/cmd/scan_repo.go:72:	init				100.0%
github.com/ArmisSecurity/armis-cli/internal/httpclient/client.go:30:	NewClient			100.0%
github.com/ArmisSecurity/armis-cli/internal/httpclient/client.go:56:	Do				95.5%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:30:		write				66.7%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:61:		Write				90.0%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:92:		Format				100.0%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:97:		FormatWithOptions		96.0%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:135:	getSeverityIcon			100.0%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:152:	getSeverityColor		100.0%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:181:	init				50.0%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:187:	disableColors			0.0%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:190:	sortFindingsBySeverity		100.0%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:219:	loadSnippetFromFile		87.2%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:282:	formatCodeSnippet		0.0%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:319:	highlightColumns		0.0%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:355:	detectLanguage			100.0%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:653:	scanDuration			26.3%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:685:	renderSummaryDashboard		61.2%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:774:	renderFindings			100.0%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:789:	renderFinding			62.5%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:846:	renderGroupedFindings		100.0%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:867:	groupFindings			96.6%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:922:	severityRank			75.0%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:936:	isGitRepo			100.0%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:943:	getGitBlame			0.0%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:964:	parseGitBlame			85.7%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:1000:	maskEmail			100.0%
github.com/ArmisSecurity/armis-cli/internal/output/human.go:1023:	getTopLevelDomain		75.0%
github.com/ArmisSecurity/armis-cli/internal/output/json.go:14:		Format				100.0%
github.com/ArmisSecurity/armis-cli/internal/output/json.go:21:		FormatWithOptions		100.0%
github.com/ArmisSecurity/armis-cli/internal/output/junit.go:43:		Format				83.3%
github.com/ArmisSecurity/armis-cli/internal/output/junit.go:67:		convertToJUnitCases		91.7%
github.com/ArmisSecurity/armis-cli/internal/output/junit.go:99:		countFailures			100.0%
github.com/ArmisSecurity/armis-cli/internal/output/junit.go:110:	FormatWithOptions		100.0%
github.com/ArmisSecurity/armis-cli/internal/output/output.go:24:	GetFormatter			100.0%
github.com/ArmisSecurity/armis-cli/internal/output/output.go:40:	ShouldFail			100.0%
github.com/ArmisSecurity/armis-cli/internal/output/output.go:56:	ExitIfNeeded			0.0%
github.com/ArmisSecurity/armis-cli/internal/output/sarif.go:64:		Format				100.0%
github.com/ArmisSecurity/armis-cli/internal/output/sarif.go:87:		convertToSarifResults		100.0%
github.com/ArmisSecurity/armis-cli/internal/output/sarif.go:124:	severityToSarifLevel		100.0%
github.com/ArmisSecurity/armis-cli/internal/output/sarif.go:138:	FormatWithOptions		100.0%
github.com/ArmisSecurity/armis-cli/internal/progress/progress.go:14:	IsCI				100.0%
github.com/ArmisSecurity/armis-cli/internal/progress/progress.go:36:	NewReader			100.0%
github.com/ArmisSecurity/armis-cli/internal/progress/progress.go:51:	NewWriter			50.0%
github.com/ArmisSecurity/armis-cli/internal/progress/progress.go:75:	NewSpinner			100.0%
github.com/ArmisSecurity/armis-cli/internal/progress/progress.go:87:	Start				17.6%
github.com/ArmisSecurity/armis-cli/internal/progress/progress.go:117:	Stop				50.0%
github.com/ArmisSecurity/armis-cli/internal/progress/progress.go:126:	UpdateMessage			100.0%
github.com/ArmisSecurity/armis-cli/internal/progress/progress.go:131:	Update				0.0%
github.com/ArmisSecurity/armis-cli/internal/progress/progress.go:136:	GetElapsed			100.0%
github.com/ArmisSecurity/armis-cli/internal/progress/progress.go:140:	formatDuration			100.0%
github.com/ArmisSecurity/armis-cli/internal/scan/image/image.go:38:	NewScanner			100.0%
github.com/ArmisSecurity/armis-cli/internal/scan/image/image.go:51:	ScanImage			0.0%
github.com/ArmisSecurity/armis-cli/internal/scan/image/image.go:84:	ScanTarball			0.0%
github.com/ArmisSecurity/armis-cli/internal/scan/image/image.go:134:	exportImage			0.0%
github.com/ArmisSecurity/armis-cli/internal/scan/image/image.go:167:	isDockerAvailable		42.9%
github.com/ArmisSecurity/armis-cli/internal/scan/image/image.go:181:	getDockerCommand		75.0%
github.com/ArmisSecurity/armis-cli/internal/scan/image/image.go:190:	validateDockerCommand		100.0%
github.com/ArmisSecurity/armis-cli/internal/scan/image/image.go:197:	buildScanResult			0.0%
github.com/ArmisSecurity/armis-cli/internal/scan/image/image.go:224:	convertNormalizedFindings	0.0%
github.com/ArmisSecurity/armis-cli/internal/scan/image/image.go:314:	shouldFilterByExploitability	100.0%
github.com/ArmisSecurity/armis-cli/internal/scan/image/image.go:333:	cleanDescription		100.0%
github.com/ArmisSecurity/armis-cli/internal/scan/image/image.go:352:	isEmptyFinding			100.0%
github.com/ArmisSecurity/armis-cli/internal/scan/image/image.go:365:	mapSeverity			100.0%
github.com/ArmisSecurity/armis-cli/internal/scan/image/image.go:380:	formatElapsed			100.0%
github.com/ArmisSecurity/armis-cli/internal/scan/image/validate.go:11:	validateImageName		100.0%
github.com/ArmisSecurity/armis-cli/internal/scan/repo/ignore.go:18:	LoadIgnorePatterns		75.0%
github.com/ArmisSecurity/armis-cli/internal/scan/repo/ignore.go:52:	loadIgnoreFile			89.5%
github.com/ArmisSecurity/armis-cli/internal/scan/repo/ignore.go:86:	Match				100.0%
github.com/ArmisSecurity/armis-cli/internal/scan/repo/ignore.go:98:	shouldSkipDir			100.0%
github.com/ArmisSecurity/armis-cli/internal/scan/repo/repo.go:36:	NewScanner			100.0%
github.com/ArmisSecurity/armis-cli/internal/scan/repo/repo.go:49:	Scan				0.0%
github.com/ArmisSecurity/armis-cli/internal/scan/repo/repo.go:132:	tarGzDirectory			0.0%
github.com/ArmisSecurity/armis-cli/internal/scan/repo/repo.go:203:	calculateDirSize		0.0%
github.com/ArmisSecurity/armis-cli/internal/scan/repo/repo.go:236:	shouldSkip			0.0%
github.com/ArmisSecurity/armis-cli/internal/scan/repo/repo.go:267:	isTestFile			88.9%
github.com/ArmisSecurity/armis-cli/internal/scan/repo/repo.go:310:	buildScanResult			0.0%
github.com/ArmisSecurity/armis-cli/internal/scan/repo/repo.go:337:	convertNormalizedFindings	0.0%
github.com/ArmisSecurity/armis-cli/internal/scan/repo/repo.go:427:	shouldFilterByExploitability	100.0%
github.com/ArmisSecurity/armis-cli/internal/scan/repo/repo.go:446:	cleanDescription		100.0%
github.com/ArmisSecurity/armis-cli/internal/scan/repo/repo.go:465:	isEmptyFinding			100.0%
github.com/ArmisSecurity/armis-cli/internal/scan/repo/repo.go:478:	mapSeverity			100.0%
github.com/ArmisSecurity/armis-cli/internal/scan/repo/repo.go:493:	formatElapsed			0.0%
github.com/ArmisSecurity/armis-cli/internal/util/path.go:11:		SanitizePath			100.0%
github.com/ArmisSecurity/armis-cli/test/sample-repo/src/main.go:6:	main				0.0%
total:									(statements)			56.5%

ci: add comment describing coverage-comment job

0aac6c6

yiftach-armis merged commit e54ff8d into main Jan 7, 2026
5 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

ci: improve coverage comment to update in place and collapse details#22

ci: improve coverage comment to update in place and collapse details#22
yiftach-armis merged 2 commits intomainfrom
ci/improve-coverage-comment

yiftach-armis commented Jan 7, 2026

Uh oh!

github-actions bot commented Jan 7, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

yiftach-armis commented Jan 7, 2026

Uh oh!

github-actions bot commented Jan 7, 2026

Test Coverage Report

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant