refactor(logs): uniform Info and Infof removing \n from Info

Author: giusdp

internal/command/admin/deploy/docker/docker_up.go

@@ -38,7 +38,7 @@ type Up struct {
 }
 
 func (u *Up) Run(ctx context.Context, dk deploy.DockerShell, logger log.FLogger) error {
-	logger.Info("Deploying FunLess locally...\n")
+	logger.Info("Deploying FunLess locally...\n\n")
 
 	_ = logger.StartSpinner("Setting things up...")
 
@@ -62,7 +62,7 @@ func (u *Up) Run(ctx context.Context, dk deploy.DockerShell, logger log.FLogger)
 		return err
 	}
 
-	logger.Info("\nDeployment complete!")
+	logger.Info("\nDeployment complete!\n")
 	logger.Info("You can now start using FunLess! 🎉")
 
 	return nil

internal/command/admin/deploy/kubernetes/kubernetes_down.go

@@ -26,7 +26,7 @@ type Down struct {
 }
 
 func (k *Down) Run(ctx context.Context, remover deploy.KubernetesRemover, logger log.FLogger) error {
-	logger.Info("Removing Kubernetes FunLess deployment...\n")
+	logger.Info("Removing Kubernetes FunLess deployment...\n\n")
 
 	_ = logger.StartSpinner("Setting things up...")
 	if err := logger.StopSpinner(remover.WithConfig(k.KubeConfig)); err != nil {
@@ -38,7 +38,7 @@ func (k *Down) Run(ctx context.Context, remover deploy.KubernetesRemover, logger
 		return err
 	}
 
-	logger.Info("\nAll clear!")
+	logger.Info("\nAll clear!\n")
 
 	return nil
 }

internal/command/admin/deploy/kubernetes/kubernetes_up.go

@@ -27,7 +27,7 @@ type Up struct {
 }
 
 func (k *Up) Run(ctx context.Context, deployer deploy.KubernetesDeployer, logger log.FLogger) error {
-	logger.Info("Deploying FunLess on Kubernetes...\n")
+	logger.Info("Deploying FunLess on Kubernetes...\n\n")
 
 	_ = logger.StartSpinner("Setting things up...")
 	if err := logger.StopSpinner(deployer.WithConfig(k.KubeConfig)); err != nil {
@@ -84,8 +84,8 @@ func (k *Up) Run(ctx context.Context, deployer deploy.KubernetesDeployer, logger
 		return err
 	}
 
-	logger.Info("\nDeployment complete!")
-	logger.Info("You can now start using FunLess! 🎉")
+	logger.Info("\nDeployment complete!\n")
+	logger.Info("You can now start using FunLess! 🎉\n")
 
 	return nil
 }

internal/command/fn/build.go

@@ -36,7 +36,7 @@ type Build struct {
 }
 
 func (b *Build) Run(ctx context.Context, builder build.DockerBuilder, logger log.FLogger) error {
-	logger.Info(fmt.Sprintf("Building %s into a wasm binary...\n", b.Name))
+	logger.Info(fmt.Sprintf("Building %s into a wasm binary...\n\n", b.Name))
 
 	_ = logger.StartSpinner("Setting up...")
 	if err := logger.StopSpinner(setupBuilder(builder, b.Language, b.Destination)); err != nil {
@@ -60,7 +60,7 @@ func (b *Build) Run(ctx context.Context, builder build.DockerBuilder, logger log
 		return err
 	}
 
-	logger.Info(fmt.Sprintf("\nSuccessfully built %s.wasm 🥳🥳", b.Name))
+	logger.Info(fmt.Sprintf("\nSuccessfully built %s.wasm 🥳🥳\n", b.Name))
 	return nil
 }
 

internal/command/fn/create.go

@@ -68,6 +68,6 @@ func (c *Create) Run(ctx context.Context, builder build.DockerBuilder, fnHandler
 	}
 	_ = logger.StopSpinner(nil)
 
-	logger.Info(fmt.Sprintf("\nSuccessfully created function %s/%s.", c.Namespace, c.Name))
+	logger.Info(fmt.Sprintf("\nSuccessfully created function %s/%s.\n", c.Namespace, c.Name))
 	return nil
 }

internal/command/fn/delete.go

@@ -32,6 +32,6 @@ func (f *Delete) Run(ctx context.Context, fnHandler client.FnHandler, logger log
 		return extractError(err)
 	}
 
-	logger.Infof("\nSuccessfully deleted function %s/%s.", f.Namespace, f.Name)
+	logger.Infof("\nSuccessfully deleted function %s/%s.\n", f.Namespace, f.Name)
 	return nil
 }

internal/command/fn/delete_test.go

@@ -65,7 +65,7 @@ func TestFnDelete(t *testing.T) {
 		err := cmd.Run(testCtx, mockFnHandler, bufLogger)
 
 		require.NoError(t, err)
-		assert.Equal(t, fmt.Sprintf("\nSuccessfully deleted function %s/%s.", testNs, testFn), (&outbuf).String())
+		assert.Equal(t, fmt.Sprintf("\nSuccessfully deleted function %s/%s.\n", testNs, testFn), (&outbuf).String())
 		mockFnHandler.AssertExpectations(t)
 	})
 

internal/command/fn/invoke_test.go

@@ -73,7 +73,7 @@ func TestFnInvoke(t *testing.T) {
 		err := cmd.Run(testCtx, mockFnHandler, bufLogger)
 
 		require.NoError(t, err)
-		assert.Equal(t, string(testOutput)+"\n", (&outbuf).String())
+		assert.Equal(t, string(testOutput), (&outbuf).String())
 		mockFnHandler.AssertExpectations(t)
 	})
 

internal/command/fn/upload.go

@@ -49,7 +49,7 @@ func (u *Upload) Run(ctx context.Context, fnHandler client.FnHandler, logger log
 	}
 	_ = logger.StopSpinner(nil)
 
-	logger.Info(fmt.Sprintf("Successfully uploaded function %s/%s 👌", u.Namespace, u.Name))
+	logger.Info(fmt.Sprintf("Successfully uploaded function %s/%s 👌\n", u.Namespace, u.Name))
 	return nil
 }
 

internal/command/mod/get.go

@@ -40,10 +40,10 @@ func (g *Get) Run(ctx context.Context, modHandler client.ModHandler, logger log.
 		return extractError(err)
 	}
 	logger.Infof("Module: %s\n", *name)
-	logger.Info("Functions:")
+	logger.Info("Functions:\n")
 
 	for _, v := range functions {
-		logger.Info(*v.Name)
+		logger.Info(*v.Name + "\n")
 	}
 
 	if g.Count {