Fix linter

Author: remyleone

cmd/vcr-compressor/main.go

@@ -2,11 +2,11 @@ package main
 
 import (
 	"encoding/json"
-	"fmt"
-	"github.com/scaleway/scaleway-sdk-go/api/k8s/v1"
-	"gopkg.in/dnaeon/go-vcr.v3/cassette"
 	"log"
 	"os"
+
+	"github.com/scaleway/scaleway-sdk-go/api/k8s/v1"
+	"gopkg.in/dnaeon/go-vcr.v3/cassette"
 )
 
 var transientStates = map[string]bool{
@@ -24,69 +24,82 @@ func main() {
 	}
 
 	chemin := os.Args[1]
+
 	inputCassette, err := cassette.Load(chemin)
 	if err != nil {
 		log.Fatalf("Erreur de lecture du fichier : %v\n", err)
 	}
 
 	outputCassette := cassette.New(chemin)
 	transitioning := false
-	for i := 0; i < len(inputCassette.Interactions); i++ {
+
+	for i := range len(inputCassette.Interactions) {
 		interaction := inputCassette.Interactions[i]
 		responseBody := interaction.Response.Body
 		requestMethod := interaction.Request.Method
 
 		if requestMethod != "GET" {
 			transitioning = false
+
 			log.Printf("Interaction %d is not a GET request. Recording it\n", i)
 			outputCassette.AddInteraction(interaction)
+
 			continue
 		}
 
 		if responseBody == "" {
 			log.Printf("Interaction %d got an empty response body. Recording it\n", i)
 			outputCassette.AddInteraction(interaction)
+
 			continue
 		}
 
 		var m map[string]interface{}
+
 		err := json.Unmarshal([]byte(responseBody), &m)
 		if err != nil {
-			fmt.Printf("Interaction %d have an error with unmarshalling reponse body: %v. Recording it\n", i, err)
+			log.Printf("Interaction %d have an error with unmarshalling response body: %v. Recording it\n", i, err)
 			outputCassette.AddInteraction(interaction)
+
 			continue
 		}
 
 		if m["status"] == nil {
 			log.Printf("Interaction %d does not contain a status field. Recording it\n", i)
 			outputCassette.AddInteraction(interaction)
+
 			continue
 		}
 
 		if m["status"] != nil {
 			// We test if the state is transient
 			if _, ok := transientStates[m["status"].(string)]; ok {
 				if transitioning {
-					fmt.Printf("Interaction %d is in a transient state while we are already in transitient state. No need to record it: %s\n", i, m["status"])
+					log.Printf("Interaction %d is in a transient state while we are already in transitient state. No need to record it: %s\n", i, m["status"])
+
 					continue
 				} else {
 					transitioning = true
-					fmt.Printf("Interaction %d is in a transient state: %s, Recording it\n", i, m["status"])
+
+					log.Printf("Interaction %d is in a transient state: %s, Recording it\n", i, m["status"])
 					outputCassette.AddInteraction(interaction)
 				}
 			} else {
 				if transitioning {
-					fmt.Printf("Interaction %d is not in a transient state anymore: %s, Recording it\n", i, m["status"])
+					log.Printf("Interaction %d is not in a transient state anymore: %s, Recording it\n", i, m["status"])
 					outputCassette.AddInteraction(interaction)
+
 					transitioning = false
 				} else {
-					fmt.Printf("Interaction %d is not in a transient state: %s, Recording it\n", i, m["status"])
+					log.Printf("Interaction %d is not in a transient state: %s, Recording it\n", i, m["status"])
 					outputCassette.AddInteraction(interaction)
 				}
-
 			}
 		}
 	}
 
-	outputCassette.Save()
+	err = outputCassette.Save()
+	if err != nil {
+		log.Fatalf("error while saving file: %v", err)
+	}
 }

cmd/vcr-viewer/main.go

@@ -2,10 +2,10 @@ package main
 
 import (
 	"encoding/json"
-	"fmt"
-	"gopkg.in/dnaeon/go-vcr.v3/cassette"
 	"log"
 	"os"
+
+	"gopkg.in/dnaeon/go-vcr.v3/cassette"
 )
 
 func main() {
@@ -14,35 +14,42 @@ func main() {
 	}
 
 	chemin := os.Args[1]
+
 	data, err := cassette.Load(chemin)
 	if err != nil {
 		log.Fatalf("Erreur de lecture du fichier : %v\n", err)
 	}
 
-	for i := 0; i < len(data.Interactions); i++ {
+	for i := range len(data.Interactions) {
 		interaction := data.Interactions[i]
-		fmt.Println("--------------")
-		fmt.Printf("Interaction %d:\n", i+1)
-		fmt.Printf("  Request:\n")
-		fmt.Printf("    Method: %s\n", interaction.Request.Method)
-		fmt.Printf("    URL: %s\n", interaction.Request.URL)
+
+		log.Println("--------------")
+		log.Printf("Interaction %d:\n", i+1)
+		log.Printf("  Request:\n")
+		log.Printf("    Method: %s\n", interaction.Request.Method)
+		log.Printf("    URL: %s\n", interaction.Request.URL)
+
 		if interaction.Request.Body != "" {
-			fmt.Printf("    Body: %s\n", interaction.Request.Body)
+			log.Printf("    Body: %s\n", interaction.Request.Body)
 		}
-		fmt.Printf("  Response:\n")
-		fmt.Printf("    Status: %s\n", interaction.Response.Status)
-		fmt.Printf("    Body: %s\n", interaction.Response.Body)
+
+		log.Printf("  Response:\n")
+		log.Printf("    Status: %s\n", interaction.Response.Status)
+		log.Printf("    Body: %s\n", interaction.Response.Body)
 
 		var m map[string]interface{}
+
 		err := json.Unmarshal([]byte(interaction.Response.Body), &m)
 		if err != nil {
 			continue
 		}
+
 		if m["status"] != nil {
-			fmt.Println("++++++++++++++++")
-			fmt.Printf("status: %s\n", m["status"]) // Modifie le champ "status" pour qu'il soit "ok"
-			fmt.Println("++++++++++++++++")
+			log.Println("++++++++++++++++")
+			log.Printf("status: %s\n", m["status"]) // Modifie le champ "status" pour qu'il soit "ok"
+			log.Println("++++++++++++++++")
 		}
-		fmt.Println("--------------")
+
+		log.Println("--------------")
 	}
 }