test: add Ginkgo/Gomega coverage for pkg/ddc/base #5803
test: add Ginkgo/Gomega coverage for pkg/ddc/base #5803hxrshxz wants to merge 3 commits intofluid-cloudnative:masterfrom
Conversation
Signed-off-by: Harsh <harshmastic@gmail.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Hi @hxrshxz. Thanks for your PR. I'm waiting for a fluid-cloudnative member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
There was a problem hiding this comment.
Pull request overview
This PR expands unit test coverage for the pkg/ddc/base dataset/setup/volume workflow by migrating tests to Ginkgo/Gomega, and it tightens TemplateEngine.Setup behavior so errors during runtime-status updates or dataset binding correctly report ready=false to the controller reconciliation loop.
Changes:
- Fix
TemplateEngine.Setupto returnready=falsewhenCheckAndUpdateRuntimeStatusorBindToDatasetfails. - Migrate
dataset_test.gofromtesting/testify-style to Ginkgo/Gomega table-driven specs. - Add new Ginkgo/Gomega specs covering setup error paths and volume operation delegation/error propagation.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| pkg/ddc/base/setup.go | Adjusts error returns in Setup so failures can’t be misinterpreted as “ready”. |
| pkg/ddc/base/dataset_test.go | Migrates dataset helper tests to Ginkgo/Gomega and strengthens assertions. |
| pkg/ddc/base/dataset_setup_volume_test.go | Adds new specs for setup error handling and Create/DeleteVolume delegation. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request refactors the dataset test suite to use the Ginkgo and Gomega frameworks and adds comprehensive tests for the TemplateEngine. It also fixes a bug in the Setup function where the return value was incorrectly handled during error states. Review feedback suggests cleaning up an unused constant and an empty test block, and adopting best practices for context keys by using custom types instead of raw strings.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #5803 +/- ##
==========================================
- Coverage 58.51% 58.27% -0.24%
==========================================
Files 473 478 +5
Lines 32283 32485 +202
==========================================
+ Hits 18890 18931 +41
- Misses 11849 12015 +166
+ Partials 1544 1539 -5 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Signed-off-by: Harsh <harshmastic@gmail.com>
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request introduces a new test suite for TemplateEngine setup and volume operations and refactors existing dataset tests to use the Ginkgo and Gomega frameworks. It also corrects the error return logic in the Setup function to ensure a false status is returned upon failure. Review feedback recommends removing an unused constant and an empty test block to improve code cleanliness.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
hxrshxz
changed the title
Signed-off-by: Harsh <harshmastic@gmail.com>
|
hxrshxz
changed the title
Ⅰ. Describe what this PR does
Add Ginkgo/Gomega test coverage for the file-scoped
pkg/ddc/basedataset, engine, setup, and volume paths.Ⅱ. Does this pull request fix one issue?
fixes #5676
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
pkg/ddc/base/dataset_test.goto Ginkgo/Gomegapkg/ddc/base/dataset_setup_volume_test.gopkg/ddc/base/dataset_setup_volume_test.goⅣ. Describe how to verify it
go test ./pkg/ddc/base/...Ⅴ. Special notes for reviews
N/A