Skip to content

This PR is to add comments to TestGetHCFSStatus in pkg/ddc/alluxio/hcfs_test.go. #4624

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
xkxkzzZ wants to merge 1 commit into from
+6 −0
Closed

This PR is to add comments to TestGetHCFSStatus in pkg/ddc/alluxio/hcfs_test.go. #4624

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions pkg/ddc/alluxio/hcfs_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,12 @@ func newAlluxioEngineHCFS(client client.Client, name string, namespace string) *
return engine
}

// TestGetHCFSStatus tests various scenarios of the GetHCFSStatus method.
// This test verifies the logic of retrieving HCFS status, covering the following cases:
// 1. In the normal case, it should correctly return the expected endpoint and filesystem version.
// 2. When the service is not registered, GetHCFSStatus should return an appropriate error.
// 3. When an error occurs during configuration retrieval, GetHCFSStatus should return the error.
// The test uses mocks of kubeclient.ExecCommandInContainerWithFullOutput to simulate different outputs.
func TestGetHCFSStatus(t *testing.T) {
mockExecCommon := func(ctx context.Context, podName string, containerName string, namespace string, cmd []string) (stdout string, stderr string, e error) {
return "conf", "", nil
Expand Down
Loading