test(goosefs): migrate node tests to ginkgo#5545
test(goosefs): migrate node tests to ginkgo#5545hxrshxz wants to merge 4 commits intofluid-cloudnative:masterfrom
Conversation
Signed-off-by: Harsh <harshmastic@gmail.com>
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
|
[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 migrates the unit tests in pkg/ddc/goosefs/node_test.go from the traditional Go testing framework to Ginkgo/Gomega as part of the broader initiative (#5407) to modernize Fluid's testing framework and improve code quality.
Changes:
- Replaced
testingimport with Ginkgo/Gomega imports - Converted table-driven test
TestSyncScheduleInfoToCacheNodesto use Ginkgo'sDescribeTablepattern - Renamed
runtimeimport toruntimeschemato avoid naming conflicts
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #5545 +/- ##
==========================================
+ Coverage 57.69% 61.29% +3.60%
==========================================
Files 444 444
Lines 30424 30540 +116
==========================================
+ Hits 17552 18720 +1168
+ Misses 11398 10277 -1121
- Partials 1474 1543 +69 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Signed-off-by: Harsh <harshmastic@gmail.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Harsh <harshmastic@gmail.com>
hxrshxz
force-pushed
the
ginkgo/goosefs-node
branch
2 times, most recently
from
c03a506 to
f7a9ae4
Compare
The test assertion should verify the runtime-specific label rather than the common label to accurately validate SyncScheduleInfoToCacheNodes behavior. This ensures nodes with worker pods get the correct runtime-specific label (e.g., fluid.io/s-goosefs-default-hbase) rather than just checking for the common label which might exist from other sources. Signed-off-by: Harsh <harshmastic@gmail.com>
|
1 participant
Ⅰ. Describe what this PR does
Migrate unit tests in pkg/ddc/goosefs/node_test.go to use Ginkgo/Gomega.
Ⅱ. Does this pull request fix one issue?
part of #5407
Ⅲ. List the added test cases
No new test cases. Migrated existing tests to Ginkgo/Gomega.
Ⅳ. Describe how to verify it
go test -v ./pkg/ddc/goosefs/... -count=1Ⅴ. Special notes for reviews
N/A