Skip to content

Conversation

@dverite
Copy link
Collaborator

@dverite dverite commented Jun 3, 2025

Use "go -cover" to build the executable for coverage tests
Add a new "coverage" target in the Makefile running "go test" and the e2e tests and merging the results
Add a new job in github workflow to generate and merge the coverage outputs

For the bats-test target, build and use a specific binary
with the -cover option
For the test target, generate coverage information in the
same directory as e2e with test.gocoverdir

Signed-off-by: Daniel Vérité <dverite@gmail.com>
@dverite dverite force-pushed the e2e-coverage branch 3 times, most recently from 582c191 to a4daf7e Compare June 3, 2025 15:09
Signed-off-by: Daniel Vérité <dverite@gmail.com>
@github-actions
Copy link

github-actions bot commented Jun 3, 2025

Code Coverage

Package Line Rate Health
github.com/qonto/postgresql-partition-manager/cmd 50%
github.com/qonto/postgresql-partition-manager/cmd/run 70%
github.com/qonto/postgresql-partition-manager/cmd/validate 65%
github.com/qonto/postgresql-partition-manager/internal/infra/config 71%
github.com/qonto/postgresql-partition-manager/internal/infra/logger 100%
github.com/qonto/postgresql-partition-manager/internal/infra/partition 78%
github.com/qonto/postgresql-partition-manager/internal/infra/postgresql 94%
github.com/qonto/postgresql-partition-manager/internal/infra/retry 100%
github.com/qonto/postgresql-partition-manager/internal/infra/uuid7 91%
github.com/qonto/postgresql-partition-manager 100%
github.com/qonto/postgresql-partition-manager/pkg/ppm 70%
Summary 75% (914 / 1222)

@dverite
Copy link
Collaborator Author

dverite commented Jun 10, 2025

#sre-storage

Copy link

@jbourdale jbourdale left a comment

Choose a reason for hiding this comment

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

LGTM 👌

@dverite dverite merged commit 2560486 into main Jun 12, 2025
14 checks passed
@dverite dverite deleted the e2e-coverage branch July 1, 2025 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants