JPRO-3315 Cleanup Code Warnings

dump.go

@@ -16,11 +16,11 @@ import (
 /*
 Data struct to configure dump behavior
 
-    Out:              Stream to wite to
-    Connection:       Database connection to dump
-    IgnoreTables:     Mark sensitive tables to ignore
-    MaxAllowedPacket: Sets the largest packet size to use in backups
-    LockTables:       Lock all tables for the duration of the dump
+	Out:              Stream to wite to
+	Connection:       Database connection to dump
+	IgnoreTables:     Mark sensitive tables to ignore
+	MaxAllowedPacket: Sets the largest packet size to use in backups
+	LockTables:       Lock all tables for the duration of the dump
 */
 type Data struct {
 	Out              io.Writer
@@ -320,7 +320,7 @@ func (table *table) CreateSQL() (string, error) {
 	}
 
 	if info[0].String != table.Name {
-		return "", errors.New("Returned table is not the same as requested table")
+		return "", errors.New("returned table is not the same as requested table")
 	}
 
 	return info[1].String, nil

dump_test.go

@@ -8,7 +8,7 @@ import (
 	"strings"
 	"testing"
 
-	sqlmock "github.com/DATA-DOG/go-sqlmock"
+	"github.com/DATA-DOG/go-sqlmock"
 	"github.com/stretchr/testify/assert"
 )
 
@@ -186,7 +186,7 @@ func TestCreateSQLWrongTable(t *testing.T) {
 
 	result, err := table.CreateSQL()
 	assert.Error(t, err)
-	expectedError := errors.New("Returned table is not the same as requested table")
+	expectedError := errors.New("returned table is not the same as requested table")
 	assert.Equal(t, expectedError, err)
 
 	// we make sure that all expectations were met

mysqldump.go

@@ -54,15 +54,15 @@ func Dump(db *sql.DB, out io.Writer) error {
 // Will also close the database the dumper is connected to as well as the out stream if it has a Close method.
 //
 // Not required.
-func (d *Data) Close() error {
+func (data *Data) Close() error {
 	defer func() {
-		d.Connection = nil
-		d.Out = nil
+		data.Connection = nil
+		data.Out = nil
 	}()
-	if out, ok := d.Out.(io.Closer); ok {
+	if out, ok := data.Out.(io.Closer); ok {
 		out.Close()
 	}
-	return d.Connection.Close()
+	return data.Connection.Close()
 }
 
 func exists(p string) (bool, os.FileInfo) {