codecov[bot]
commented
2 days ago Codecov Report
Attention: Patch coverage is 69.35484% with 19 lines in your changes missing coverage. Please review.
Project coverage is 36.31%. Comparing base (
11e5b41) to head (add936b). Report is 1 commits behind head on main.
| Files with missing lines | Patch % | Lines |
|---|---|---|
| e2e/httplib.go | 69.35% | 16 Missing and 3 partials :warning: |
Additional details and impacted files
```diff @@ Coverage Diff @@ ## main #57 +/- ## ========================================== + Coverage 30.48% 36.31% +5.83% ========================================== Files 4 5 +1 Lines 351 413 +62 ========================================== + Hits 107 150 +43 - Misses 224 240 +16 - Partials 20 23 +3 ```:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This pull request includes significant changes to the
automq-operatorproject, focusing on workflow improvements, refactoring the main application, and enhancing end-to-end tests. The most important changes include renaming jobs in the GitHub Actions workflow, refactoring the main application to use a centralized API registry, and adding new end-to-end tests.Workflow Improvements:
.github/workflows/test.yml: Renamedjob2tojob1and vice versa, added a step to log disk space before freeing it up, and modified the job to install Kubernetes and applications instead of pruning the OS. [1] [2] [3] [4]Application Refactoring:
cmd/main.go: Removed direct Gin router setup in favor of using theAPIRegistryfunction from thecontrollerpackage. [1] [2]internal/controller/automq_apis.go: Added a new file to handle API registration using Gin.End-to-End Testing Enhancements:
e2e/automq_cluster_controller_test.go: Added a new end-to-end test for theautomq_controllerto verify custom resource creation and cleanup.e2e/automq_cluster_test.go: Refactored existing tests to include component status checks for Minio, Cert-Manager, Prometheus, and Kafka-UI. [1] [2] [3] [4]e2e/httplib.go: Added a new utility for making HTTP requests in end-to-end tests.e2e/utils.go: Minor import reorganization.