Skip to content

feat: generate coverage for the integration tests#527

Merged
talos-bot merged 1 commit into
siderolabs:mainfrom
Unix4ever:generate-coverage-integration-tests
Jun 10, 2025
Merged

feat: generate coverage for the integration tests#527
talos-bot merged 1 commit into
siderolabs:mainfrom
Unix4ever:generate-coverage-integration-tests

Conversation

@Unix4ever

Copy link
Copy Markdown
Member

Use all canonical module paths to generate coverpkg.

@talos-bot talos-bot moved this from To Do to In Review in Planning Jun 9, 2025
@github-project-automation github-project-automation Bot moved this to To Do in Planning Jun 9, 2025
@Unix4ever Unix4ever force-pushed the generate-coverage-integration-tests branch from 700f5e5 to 37c4809 Compare June 9, 2025 12:30
for _, spec := range integrationTests.Tests {
build := golang.NewBuild(builder.meta, spec.Name, spec.Path, "go test -c -covermode=atomic")
build := golang.NewBuild(builder.meta, spec.Name, spec.Path,
fmt.Sprintf(

@smira smira Jun 9, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it'd be nice to merge the output coverage of this step, and unit-tests step, and push the combined to gocov codecov step for visualization

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it merged by putting everything into a single long coverage.txt?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know, but I would assume it can pick up multiple files and merge them itself - but you need to experiment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I've figured it out

@github-project-automation github-project-automation Bot moved this from In Review to Approved in Planning Jun 9, 2025
Use all canonical module paths to generate `coverpkg`.

Signed-off-by: Artem Chernyshev <artem.chernyshev@talos-systems.com>
@Unix4ever Unix4ever force-pushed the generate-coverage-integration-tests branch from 37c4809 to 5128bc1 Compare June 9, 2025 14:06
@Unix4ever

Copy link
Copy Markdown
Member Author

/m

@talos-bot talos-bot merged commit 5128bc1 into siderolabs:main Jun 10, 2025
14 checks passed
@github-project-automation github-project-automation Bot moved this from Approved to Done in Planning Jun 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants