gofumpt

acls.go

@@ -27,26 +27,26 @@ type ACL *api.ACL
 // GetVolumeACL returns the ACL for a volume.
 func (c *Client) GetVolumeACL(
 	ctx context.Context,
-	volumeName string) (ACL, error) {
-
+	volumeName string,
+) (ACL, error) {
 	return api.ACLInspect(ctx, c.API, volumeName)
 }
 
 // SetVolumeOwnerToCurrentUser sets the owner for a volume to the user that
 // was used to connect to the API.
 func (c *Client) SetVolumeOwnerToCurrentUser(
 	ctx context.Context,
-	volumeName string) error {
-
+	volumeName string,
+) error {
 	return c.SetVolumeOwner(ctx, volumeName, c.API.User())
 }
 
 // SetVolumeOwner sets the owner for a volume.
 func (c *Client) SetVolumeOwner(
 	ctx context.Context,
-	volumeName, userName string) error {
-
-	mode := api.FileMode(0777)
+	volumeName, userName string,
+) error {
+	mode := api.FileMode(0o777)
 
 	return api.ACLUpdate(
 		ctx,
@@ -68,8 +68,8 @@ func (c *Client) SetVolumeOwner(
 // SetVolumeMode sets the permissions to the specified mode (chmod)
 func (c *Client) SetVolumeMode(
 	ctx context.Context,
-	volumeName string, mode int) error {
-
+	volumeName string, mode int,
+) error {
 	filemode := api.FileMode(mode)
 
 	return api.ACLUpdate(

acls_test.go

@@ -16,8 +16,8 @@ limitations under the License.
 package goisilon
 
 import (
-	"testing"
 	"fmt"
+	"testing"
 
 	"github.com/stretchr/testify/assert"
 

api/api.go

@@ -60,7 +60,6 @@ var (
 
 // Client is an API client.
 type Client interface {
-
 	// Do sends an HTTP request to the OneFS API.
 	Do(
 		ctx context.Context,
@@ -214,8 +213,8 @@ func New(
 	ctx context.Context,
 	hostname, username, password, groupname string,
 	verboseLogging uint, authType uint8,
-	opts *ClientOptions) (Client, error) {
-
+	opts *ClientOptions,
+) (Client, error) {
 	if hostname == "" || username == "" || password == "" {
 		return nil, errNewClient
 	}
@@ -318,8 +317,8 @@ func (c *client) Get(
 	ctx context.Context,
 	path, id string,
 	params OrderedValues, headers map[string]string,
-	resp interface{}) error {
-
+	resp interface{},
+) error {
 	return c.executeWithRetryAuthenticate(
 		ctx, http.MethodGet, path, id, params, headers, nil, resp)
 }
@@ -328,8 +327,8 @@ func (c *client) Post(
 	ctx context.Context,
 	path, id string,
 	params OrderedValues, headers map[string]string,
-	body, resp interface{}) error {
-
+	body, resp interface{},
+) error {
 	return c.executeWithRetryAuthenticate(
 		ctx, http.MethodPost, path, id, params, headers, body, resp)
 }
@@ -338,8 +337,8 @@ func (c *client) Put(
 	ctx context.Context,
 	path, id string,
 	params OrderedValues, headers map[string]string,
-	body, resp interface{}) error {
-
+	body, resp interface{},
+) error {
 	return c.executeWithRetryAuthenticate(
 		ctx, http.MethodPut, path, id, params, headers, body, resp)
 }
@@ -348,8 +347,8 @@ func (c *client) Delete(
 	ctx context.Context,
 	path, id string,
 	params OrderedValues, headers map[string]string,
-	resp interface{}) error {
-
+	resp interface{},
+) error {
 	return c.executeWithRetryAuthenticate(
 		ctx, http.MethodDelete, path, id, params, headers, nil, resp)
 }
@@ -358,8 +357,8 @@ func (c *client) Do(
 	ctx context.Context,
 	method, path, id string,
 	params OrderedValues,
-	body, resp interface{}) error {
-
+	body, resp interface{},
+) error {
 	return c.executeWithRetryAuthenticate(ctx, method, path, id, params, nil, body, resp)
 }
 
@@ -375,8 +374,8 @@ func (c *client) DoWithHeaders(
 	ctx context.Context,
 	method, uri, id string,
 	params OrderedValues, headers map[string]string,
-	body, resp interface{}) error {
-
+	body, resp interface{},
+) error {
 	res, _, err := c.DoAndGetResponseBody(
 		ctx, method, uri, id, params, headers, body)
 	if err != nil {
@@ -412,8 +411,8 @@ func (c *client) DoAndGetResponseBody(
 	ctx context.Context,
 	method, uri, id string,
 	params OrderedValues, headers map[string]string,
-	body interface{}) (*http.Response, bool, error) {
-
+	body interface{},
+) (*http.Response, bool, error) {
 	var (
 		err                   error
 		req                   *http.Request
@@ -617,7 +616,7 @@ func parseJSONError(r *http.Response) error {
 func (c *client) authenticate(ctx context.Context, username string, password string, endpoint string) error {
 	headers := make(map[string]string, 1)
 	headers[headerKeyContentType] = headerValContentTypeJSON
-	var data = &setupConnection{Services: []string{"platform", "namespace"}, Username: username, Password: password}
+	data := &setupConnection{Services: []string{"platform", "namespace"}, Username: username, Password: password}
 	resp, _, err := c.DoAndGetResponseBody(ctx, http.MethodPost, "/session/1/session", "", nil, headers, data)
 	if err != nil {
 		return errors.New(fmt.Sprintf("Authentication error: %v", err))
@@ -699,8 +698,7 @@ func (c *client) executeWithRetryAuthenticate(ctx context.Context, method, uri s
 }
 
 func FetchValueIndexForKey(l string, match string, sep string) (int, int, int) {
-
-	var startIndex, endIndex = -1, -1
+	startIndex, endIndex := -1, -1
 	if strings.Contains(l, match) {
 		startIndex = strings.Index(l, match)
 		if startIndex != -1 && sep != "" {

api/api_logging.go

@@ -21,11 +21,12 @@ import (
 	"context"
 	"encoding/base64"
 	"fmt"
-	log "github.com/akutz/gournal"
 	"io"
 	"net/http"
 	"net/http/httputil"
 	"strings"
+
+	log "github.com/akutz/gournal"
 )
 
 func isBinOctetBody(h http.Header) bool {
@@ -40,10 +41,10 @@ func logRequest(ctx context.Context, w io.Writer, req *http.Request, verbose Ver
 
 	switch verbose {
 	case Verbose_Low:
-		//minimal logging, i.e. print request line only
+		// minimal logging, i.e. print request line only
 		fmt.Fprintf(w, "    %s %s %s\r\n", req.Method, req.URL.RequestURI(), req.Proto)
 	default:
-		//full logging, i.e. print full request message content
+		// full logging, i.e. print full request message content
 		buf, _ := httputil.DumpRequest(req, !isBinOctetBody(req.Header))
 		decodedBuf := encryptPassword(buf)
 		WriteIndented(w, decodedBuf)
@@ -63,17 +64,17 @@ func logResponse(ctx context.Context, res *http.Response, verbose VerboseType) {
 
 	switch verbose {
 	case Verbose_Low:
-		//minimal logging, i.e. pirnt status line only
+		// minimal logging, i.e. pirnt status line only
 		fmt.Fprintf(w, "    %s %s\r\n", res.Proto, res.Status)
 	case Verbose_Medium:
-		//print status line + headers
+		// print status line + headers
 		buf, _ = httputil.DumpResponse(res, false)
 	default:
-		//print full response message content
+		// print full response message content
 		buf, _ = httputil.DumpResponse(res, !isBinOctetBody(res.Header))
 	}
 
-	//when DumpResponse gets err, buf will be nil. No message content will be printed
+	// when DumpResponse gets err, buf will be nil. No message content will be printed
 	WriteIndented(w, buf)
 
 	log.Debug(ctx, w.String())

api/api_ordered_values.go

@@ -293,7 +293,6 @@ var hexChars = []byte{
 }
 
 func escapeTo(w io.Writer, s []byte) error {
-
 	var (
 		hexCount   = 0
 		spaceCount = 0

api/api_ordered_values_test.go

@@ -223,9 +223,9 @@ func TestParseQuery(t *testing.T) {
 
 func TestStructToOrderedValues(t *testing.T) {
 	type TestStruct struct {
-		TestString *string `json:"test_string,omitempty"`
-		TestInt    *int    `json:"test_int,omitempty"`
-		TestBool   *bool   `json:"test_bool,omitempty"`
+		TestString   *string `json:"test_string,omitempty"`
+		TestInt      *int    `json:"test_int,omitempty"`
+		TestBool     *bool   `json:"test_bool,omitempty"`
 		IgnoredField string
 	}
 	testString := "A"

api/common/utils/poll.go

@@ -25,8 +25,10 @@ import (
 
 var ErrWaitTimeout = errors.New("timed out waiting for the condition")
 
-type WaitWithContextFunc func(ctx context.Context) <-chan struct{}
-type ConditionWithContextFunc func(context.Context) (done bool, err error)
+type (
+	WaitWithContextFunc      func(ctx context.Context) <-chan struct{}
+	ConditionWithContextFunc func(context.Context) (done bool, err error)
+)
 
 func PollImmediateWithContext(ctx context.Context, interval, timeout time.Duration, condition ConditionWithContextFunc) error {
 	return poll(ctx, true, poller(interval, timeout), condition)

api/common/utils/utils.go

@@ -17,7 +17,6 @@ package utils
 
 // IsStringInSlice checks if a string is an element of a string slice
 func IsStringInSlice(str string, list []string) bool {
-
 	for _, b := range list {
 		if b == str {
 			return true
@@ -29,7 +28,6 @@ func IsStringInSlice(str string, list []string) bool {
 
 // IsStringInSlices checks if a string is an element of a any of the string slices
 func IsStringInSlices(str string, list ...[]string) bool {
-
 	for _, strs := range list {
 		if IsStringInSlice(str, strs) {
 			return true
@@ -41,7 +39,6 @@ func IsStringInSlices(str string, list ...[]string) bool {
 
 // RemoveStringFromSlice returns a slice that is a copy of the input "list" slice with the input "str" string removed
 func RemoveStringFromSlice(str string, list []string) []string {
-
 	result := make([]string, 0)
 
 	for _, v := range list {
@@ -55,11 +52,9 @@ func RemoveStringFromSlice(str string, list []string) []string {
 
 // RemoveStringsFromSlice generates a slice that is a copy of the input "list" slice with elements from the input "strs" slice removed
 func RemoveStringsFromSlice(filters []string, list []string) []string {
-
 	result := make([]string, 0)
 
 	for _, str := range list {
-
 		if !IsStringInSlice(str, filters) {
 			result = append(result, str)
 		}

api/common/utils/utils_test.go

@@ -27,19 +27,17 @@ func TestMain(m *testing.M) {
 }
 
 func TestIsStringInSlice(t *testing.T) {
-
-	var list = []string{"hello", "world", "jason"}
+	list := []string{"hello", "world", "jason"}
 
 	assert.True(t, IsStringInSlice("world", list))
 	assert.False(t, IsStringInSlice("mary", list))
 	assert.False(t, IsStringInSlice("harry", nil))
 }
 
 func TestRemoveStringFromSlice(t *testing.T) {
+	list := []string{"hello", "world", "jason"}
 
-	var list = []string{"hello", "world", "jason"}
-
-	var result = RemoveStringFromSlice("hello", list)
+	result := RemoveStringFromSlice("hello", list)
 
 	assert.Equal(t, 3, len(list))
 	assert.Equal(t, 2, len(result))
@@ -50,12 +48,11 @@ func TestRemoveStringFromSlice(t *testing.T) {
 }
 
 func TestRemoveStringsFromSlice(t *testing.T) {
+	list := []string{"hello", "world", "jason"}
 
-	var list = []string{"hello", "world", "jason"}
-
-	var filterList = []string{"hello", "there", "chap", "world"}
+	filterList := []string{"hello", "there", "chap", "world"}
 
-	var result = RemoveStringsFromSlice(filterList, list)
+	result := RemoveStringsFromSlice(filterList, list)
 
 	assert.Equal(t, 1, len(result))
 }

api/json/json_decode.go

@@ -548,8 +548,10 @@ func (d *decodeState) array(v reflect.Value) {
 	}
 }
 
-var nullLiteral = []byte("null")
-var textUnmarshalerType = reflect.TypeOf(new(encoding.TextUnmarshaler)).Elem()
+var (
+	nullLiteral         = []byte("null")
+	textUnmarshalerType = reflect.TypeOf(new(encoding.TextUnmarshaler)).Elem()
+)
 
 // object consumes an object from d.data[d.off-1:], decoding into the value v.
 // the first byte ('{') of the object has been read already.
@@ -784,7 +786,7 @@ var numberType = reflect.TypeOf(Number(""))
 func (d *decodeState) literalStore(item []byte, v reflect.Value, fromQuoted bool) {
 	// Check for unmarshaler.
 	if len(item) == 0 {
-		//Empty string given
+		// Empty string given
 		d.saveError(fmt.Errorf("json: invalid use of ,string struct tag, trying to unmarshal %q into %v", item, v.Type()))
 		return
 	}
@@ -966,7 +968,7 @@ func (d *decodeState) valueInterface() interface{} {
 
 // arrayInterface is like array but returns []interface{}.
 func (d *decodeState) arrayInterface() []interface{} {
-	var v = make([]interface{}, 0)
+	v := make([]interface{}, 0)
 	for {
 		// Look ahead for ] - can only happen on first iteration.
 		op := d.scanWhile(scanSkipSpace)

api/json/json_stream.go

@@ -260,8 +260,10 @@ func (m *RawMessage) UnmarshalJSON(data []byte) error {
 	return nil
 }
 
-var _ Marshaler = (*RawMessage)(nil)
-var _ Unmarshaler = (*RawMessage)(nil)
+var (
+	_ Marshaler   = (*RawMessage)(nil)
+	_ Unmarshaler = (*RawMessage)(nil)
+)
 
 // A Token holds a value of one of these types:
 //