Move redaction into function and add test

libs/wallet/provider/uphold/uphold.go

@@ -237,12 +237,7 @@ func submit(
 	if err != nil {
 		return nil, resp, fmt.Errorf("%w: %s", errorutils.ErrFailedBodyRead, err.Error())
 	}
-	sbody := string(body)
-	for _, p := range []string{`"description":\s*"[^"]+"\s*,?`, `"\w+Country":\s*"[^"]+"\s*,?`} {
-		re := regexp.MustCompile(p)
-		sbody = re.ReplaceAllString(sbody, "")
-	}
-	sbody = regexp.MustCompile(`,}`).ReplaceAllString(sbody, "}")
+	sbody := redactUnneededContent(string(body))
 
 	if logger != nil {
 		logger.Debug().
@@ -1187,3 +1182,12 @@ func FundWallet(ctx context.Context, destWallet *Wallet, amount decimal.Decimal)
 
 	return balance.TotalProbi, nil
 }
+
+func redactUnneededContent(sbody string) string {
+	for _, p := range []string{`"description":\s*"[^"]+"\s*,?`, `"\w+Country":\s*"[^"]+"\s*,?`} {
+		re := regexp.MustCompile(p)
+		sbody = re.ReplaceAllString(sbody, "")
+	}
+	sbody = strings.ReplaceAll(sbody, ",}", "}")
+	return sbody
+}

libs/wallet/provider/uphold/uphold_test.go

@@ -3,6 +3,7 @@ package uphold
 import (
 	"context"
 	"encoding/hex"
+	"encoding/json"
 	"errors"
 	"net/http"
 	"net/url"
@@ -351,3 +352,14 @@ func requireDonorWallet(t *testing.T) *Wallet {
 
 	return &Wallet{Info: info, PrivKey: privateKey, PubKey: publicKey}
 }
+
+func TestRedactUnneeded(t *testing.T) {
+	response := `{"description":"some unneeded content","UKCountry":"foo","TestCountry":"bar","NoMatch": true}`
+	result := `{"NoMatch": true}`
+	testValue := redactUnneededContent(response)
+	assert.Equal(t, result, testValue)
+	var dat map[string]interface{}
+	if err := json.Unmarshal([]byte(testValue), &dat); err != nil {
+		t.Fatal(err)
+	}
+}