Use %q to format-quote strings

Author: IsaacG

exercises/concept/gross-store/gross_store_test.go

@@ -28,12 +28,12 @@ func TestUnits(t *testing.T) {
 		qty, ok := units[tt.name]
 
 		if !ok {
-			t.Errorf(`Unit "%s" not found!`, tt.name)
+			t.Errorf("Unit %q not found!", tt.name)
 			continue
 		}
 
 		if qty != tt.qty {
-			t.Errorf(`Unit "%s" should have quantity %d, found %d`, tt.name, tt.qty, qty)
+			t.Errorf("Unit %q should have quantity %d, found %d", tt.name, tt.qty, qty)
 		}
 	}
 }

exercises/concept/weather-forecast/weather_forecast_test.go

@@ -111,7 +111,7 @@ func testComment(entityKind, entityName, comment, wantedPrefix string) (ok bool,
 
 	// Check if comment has wanted prefix
 	if !strings.HasPrefix(trimmedComment, wantedPrefix) {
-		errorString := fmt.Sprintf("%s comment for %s '%s' should start with '// %s ...': got '// %s'",
+		errorString := fmt.Sprintf("%s comment for %s %q should start with '// %s ...': got '// %s'",
 			entityKind, lowerEntity, entityName, wantedPrefix, trimmedComment)
 		return false, errorString
 	}
@@ -123,14 +123,14 @@ func testComment(entityKind, entityName, comment, wantedPrefix string) (ok bool,
 
 	if commentContent == "" {
 		lowerEntity := strings.ToLower(entityKind)
-		errorString := fmt.Sprintf("%s comment of '%s' should provide a description of the %s, e.g '// %s <%s_description>'",
+		errorString := fmt.Sprintf("%s comment of %q should provide a description of the %s, e.g '// %s <%s_description>'",
 			entityKind, entityName, lowerEntity, wantedPrefix, lowerEntity)
 		return false, errorString
 	}
 
 	// Check if comment ends in a period
 	if !strings.HasSuffix(trimmedComment, ".") {
-		return false, fmt.Sprintf("%s comment for %s '%s' should end with a period (.)",
+		return false, fmt.Sprintf("%s comment for %s %q should end with a period (.)",
 			entityKind, lowerEntity, entityName)
 	}
 

exercises/practice/nucleotide-count/.meta/gen.go

@@ -44,7 +44,7 @@ func (t testCase) SortedMapString() string {
 	for k, v := range t.Expected {
 		switch t := v.(type) {
 		case float64:
-			strs = append(strs, fmt.Sprintf("'%s':%d", k, int(t)))
+			strs = append(strs, fmt.Sprintf("%q:%d", k, int(t)))
 		default:
 			log.Fatalf("unexpected type %T for %v", t, v)
 		}

gomod-sync/gomod/version_test.go

@@ -96,10 +96,9 @@ go 1.18
 			}
 
 			if string(data) != test.Modified {
-				t.Fatalf("expected go.mod file with the following contents:\n"+
-					"%s\n"+
-					"but got:\n"+
-					"%s", test.Modified, string(data))
+				t.Fatalf(
+					"expected go.mod file with the following contents:\n%s\nbut got:\n%s", test.Modified, string(data),
+				)
 			}
 
 			err = os.Remove(filePath)