handle review comments

Author: GokceGK

internal/cmd/beta/sfs/project-lock/describe/describe.go

@@ -66,7 +66,7 @@ func NewCmd(params *types.CmdParams) *cobra.Command {
 				var oApiErr *oapierror.GenericOpenAPIError
 				if sysErrors.As(err, &oApiErr) {
 					if oApiErr.StatusCode == http.StatusNotFound {
-						params.Printer.Info("No active lock found for project %s\n", projectLabel)
+						params.Printer.Outputf("No active lock found for project %s\n", projectLabel)
 						return err
 					}
 				}

internal/cmd/beta/sfs/project-lock/describe/describe_test.go

@@ -132,36 +132,35 @@ func TestBuildRequest(t *testing.T) {
 
 func TestOutputResult(t *testing.T) {
 	type args struct {
-		outputFormat   string
-		complianceLock *sfs.GetLockResponse
+		outputFormat string
+		projectLock  *sfs.GetLockResponse
 	}
 	tests := []struct {
 		name    string
 		args    args
 		wantErr bool
 	}{
 		{
-			name: "complianceLock is nil",
+			name: "projectLock is nil",
 			args: args{
 				outputFormat: print.PrettyOutputFormat,
-				complianceLock: nil,
-				complianceLock: nil,
+				projectLock:  nil,
 			},
 			wantErr: true,
 		},
 		{
 			name: "set empty project lock",
 			args: args{
-				outputFormat:   print.PrettyOutputFormat,
-				complianceLock: &sfs.GetLockResponse{},
+				outputFormat: print.PrettyOutputFormat,
+				projectLock:  &sfs.GetLockResponse{},
 			},
 			wantErr: false,
 		},
 		{
 			name: "set filled lock",
 			args: args{
 				outputFormat: print.PrettyOutputFormat,
-				complianceLock: &sfs.GetLockResponse{
+				projectLock: &sfs.GetLockResponse{
 					LockId: utils.Ptr(uuid.New().String()),
 				},
 			},
@@ -171,7 +170,7 @@ func TestOutputResult(t *testing.T) {
 	params := testparams.NewTestParams()
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			if err := outputResult(params.Printer, tt.args.outputFormat, tt.args.complianceLock); (err != nil) != tt.wantErr {
+			if err := outputResult(params.Printer, tt.args.outputFormat, tt.args.projectLock); (err != nil) != tt.wantErr {
 				t.Errorf("outputResult() error = %v, wantErr %v", err, tt.wantErr)
 			}
 		})

internal/cmd/beta/sfs/project-lock/lock/lock.go

@@ -100,7 +100,6 @@ func parseInput(p *print.Printer, cmd *cobra.Command, _ []string) (*inputModel,
 
 func buildRequest(ctx context.Context, model *inputModel, apiClient *sfs.APIClient) sfs.ApiEnableLockRequest {
 	return apiClient.DefaultAPI.EnableLock(ctx, model.Region, model.ProjectId)
-	return req
 }
 
 func outputResult(p *print.Printer, outputFormat, projectLabel string, resp *sfs.EnableLockResponse) error {