Apply suggestions from code review

Co-authored-by: Piotr Halama <[email protected]>

Author: pPrecel

internal/cmd/alpha/module/enable.go

@@ -24,9 +24,9 @@ func newEnableCMD(kymaConfig *cmdcommon.KymaConfig) *cobra.Command {
 	}
 
 	cmd := &cobra.Command{
-		Use:   "enable",
+		Use:   "enable <module>",
 		Short: "Enable module.",
-		Long:  "Use this command to enabele module.",
+		Long:  "Use this command to enable module.",
 		Args:  cobra.ExactArgs(1),
 		Run: func(cmd *cobra.Command, args []string) {
 			cfg.module = args[0]

internal/kube/kyma/kyma_test.go

@@ -569,7 +569,7 @@ func fixModuleTemplate(moduleName string) *unstructured.Unstructured {
 }
 
 func Test_client_WaitForModuleState(t *testing.T) {
-	t.Run("match stete", func(t *testing.T) {
+	t.Run("match state", func(t *testing.T) {
 		scheme := runtime.NewScheme()
 		scheme.AddKnownTypes(GVRKyma.GroupVersion())
 		client := client{dynamic_fake.NewSimpleDynamicClient(scheme, fixDefaultKyma())}

internal/modules/enable.go

@@ -46,7 +46,7 @@ func applyCustomCR(writer io.Writer, ctx context.Context, client kube.Client, mo
 		return nil
 	}
 
-	fmt.Fprintf(writer, "waiting for module to be ready")
+	fmt.Fprintf(writer, "waiting for module to be ready\n")
 	err := client.Kyma().WaitForModuleState(ctx, module, "Ready", "Warning")
 	if err != nil {
 		return clierror.Wrap(err, clierror.New("failed to check module state"))

internal/modules/enable_test.go

@@ -67,7 +67,7 @@ func TestEnable(t *testing.T) {
 
 		err := enable(buffer, context.Background(), &client, "keda", "fast", false, testKedaCR)
 		require.Nil(t, err)
-		require.Equal(t, "adding keda module to the Kyma CR\nwaiting for module to be readyapplying kyma-system/default cr\nkeda module enabled\n", buffer.String())
+		require.Equal(t, "adding keda module to the Kyma CR\nwaiting for module to be ready\napplying kyma-system/default cr\nkeda module enabled\n", buffer.String())
 		require.Equal(t, []fake.FakeEnabledModule{expectedEnabledModule}, kymaClient.EnabledModules)
 		require.Equal(t, []unstructured.Unstructured{testKedaCR}, rootlessDynamicClient.ApplyObjs)
 	})
@@ -130,6 +130,6 @@ func TestEnable(t *testing.T) {
 
 		err := enable(buffer, context.Background(), &client, "keda", "fast", false, testKedaCR)
 		require.Equal(t, expectedCliErr, err)
-		require.Equal(t, "adding keda module to the Kyma CR\nwaiting for module to be readyapplying kyma-system/default cr\n", buffer.String())
+		require.Equal(t, "adding keda module to the Kyma CR\nwaiting for module to be ready\napplying kyma-system/default cr\n", buffer.String())
 	})
 }