ci: enable recent gci linter options and fix reported errors (#798)

.golangci.yml

@@ -9,7 +9,9 @@ linters-settings:
     statements: 50
   gci:
     sections:
-      - prefix(gofr.dev)
+      - standard
+      - default
+      - localmodule
   goconst:
     min-len: 2
     min-occurrences: 2

examples/using-file-bind/main_test.go

@@ -2,13 +2,14 @@ package main
 
 import (
 	"bytes"
-	"github.com/stretchr/testify/assert"
 	"io"
 	"mime/multipart"
 	"net/http"
 	"os"
 	"testing"
 	"time"
+
+	"github.com/stretchr/testify/assert"
 )
 
 func TestMain_BindError(t *testing.T) {

examples/using-migrations/main.go

@@ -5,7 +5,6 @@ import (
 	"fmt"
 
 	"gofr.dev/examples/using-migrations/migrations"
-
 	"gofr.dev/pkg/gofr"
 )
 

examples/using-migrations/migrations/1708322067_create_employee_table_test.go

@@ -6,6 +6,7 @@ import (
 
 	"github.com/DATA-DOG/go-sqlmock"
 	"github.com/stretchr/testify/assert"
+
 	"gofr.dev/pkg/gofr/migration"
 )
 

examples/using-migrations/migrations/1708322089_redis_add_employee_name.go

@@ -2,6 +2,7 @@ package migrations
 
 import (
 	"context"
+
 	"gofr.dev/pkg/gofr/migration"
 )
 

examples/using-migrations/migrations/1708322089_redis_add_employee_name_test.go

@@ -6,6 +6,7 @@ import (
 
 	"github.com/go-redis/redismock/v9"
 	"github.com/stretchr/testify/assert"
+
 	"gofr.dev/pkg/gofr/migration"
 )
 

examples/using-publisher/main.go

@@ -2,6 +2,7 @@ package main
 
 import (
 	"encoding/json"
+
 	"gofr.dev/pkg/gofr"
 )
 

pkg/gofr/container/container.go

@@ -6,6 +6,7 @@ import (
 	"time"
 
 	_ "github.com/go-sql-driver/mysql" // This is required to be blank import
+
 	"gofr.dev/pkg/gofr/config"
 	"gofr.dev/pkg/gofr/datasource"
 	"gofr.dev/pkg/gofr/datasource/file"

pkg/gofr/datasource/clickhouse/clickhouse_test.go

@@ -9,9 +9,8 @@ import (
 	"testing"
 	"time"
 
-	"go.uber.org/mock/gomock"
-
 	"github.com/stretchr/testify/assert"
+	"go.uber.org/mock/gomock"
 )
 
 func getClickHouseTestConnection(t *testing.T) (*MockConn, *MockMetrics, client) {

pkg/gofr/datasource/mongo/mongo_test.go

@@ -3,7 +3,6 @@ package mongo
 import (
 	"context"
 	"fmt"
-
 	"testing"
 
 	"github.com/stretchr/testify/assert"

pkg/gofr/datasource/pubsub/google/health.go

@@ -5,8 +5,9 @@ import (
 	"errors"
 	"time"
 
-	"gofr.dev/pkg/gofr/datasource"
 	"google.golang.org/api/iterator"
+
+	"gofr.dev/pkg/gofr/datasource"
 )
 
 func (g *googleClient) Health() (health datasource.Health) {

pkg/gofr/datasource/redis/hook.go

@@ -8,9 +8,9 @@ import (
 	"strings"
 	"time"
 
-	"gofr.dev/pkg/gofr/datasource"
-
 	"github.com/redis/go-redis/v9"
+
+	"gofr.dev/pkg/gofr/datasource"
 )
 
 // redisHook is a custom Redis hook for logging queries and their durations.

pkg/gofr/datasource/redis/redis_test.go

@@ -7,7 +7,6 @@ import (
 
 	"github.com/alicebob/miniredis/v2"
 	"github.com/stretchr/testify/assert"
-
 	"go.uber.org/mock/gomock"
 
 	"gofr.dev/pkg/gofr/config"

pkg/gofr/exporter.go

@@ -12,6 +12,7 @@ import (
 
 	"go.opentelemetry.io/otel/attribute"
 	sdktrace "go.opentelemetry.io/otel/sdk/trace"
+
 	"gofr.dev/pkg/gofr/logging"
 )
 

pkg/gofr/grpc.go

@@ -6,10 +6,10 @@ import (
 
 	grpc_middleware "github.com/grpc-ecosystem/go-grpc-middleware"
 	grpc_recovery "github.com/grpc-ecosystem/go-grpc-middleware/recovery"
-	grpc2 "gofr.dev/pkg/gofr/grpc"
 	"google.golang.org/grpc"
 
 	"gofr.dev/pkg/gofr/container"
+	grpc2 "gofr.dev/pkg/gofr/grpc"
 )
 
 type grpcServer struct {

pkg/gofr/http/middleware/basic_auth_test.go

@@ -6,6 +6,7 @@ import (
 	"testing"
 
 	"github.com/stretchr/testify/assert"
+
 	"gofr.dev/pkg/gofr/container"
 )
 

pkg/gofr/http/middleware/metrics_test.go

@@ -2,13 +2,12 @@ package middleware
 
 import (
 	"context"
-
-	"github.com/gorilla/mux"
-	"github.com/stretchr/testify/mock"
-
 	"net/http"
 	"net/http/httptest"
 	"testing"
+
+	"github.com/gorilla/mux"
+	"github.com/stretchr/testify/mock"
 )
 
 type mockMetrics struct {

pkg/gofr/http/middleware/web_socket.go

@@ -4,10 +4,10 @@ import (
 	"context"
 	"net/http"
 
+	gorillaWebsocket "github.com/gorilla/websocket"
+
 	"gofr.dev/pkg/gofr/container"
 	"gofr.dev/pkg/gofr/websocket"
-
-	gorillaWebsocket "github.com/gorilla/websocket"
 )
 
 // WSHandlerUpgrade middleware upgrades the incoming http request to a websocket connection using websocket upgrader.

pkg/gofr/http/middleware/web_socket_test.go

@@ -9,6 +9,7 @@ import (
 	"github.com/gorilla/websocket"
 	"github.com/stretchr/testify/assert"
 	"go.uber.org/mock/gomock"
+
 	"gofr.dev/pkg/gofr/container"
 	gofrWebSocket "gofr.dev/pkg/gofr/websocket"
 )

pkg/gofr/logging/remotelogger/dynamicLevelLogger_test.go

@@ -8,6 +8,7 @@ import (
 	"time"
 
 	"github.com/stretchr/testify/assert"
+
 	"gofr.dev/pkg/gofr/logging"
 	"gofr.dev/pkg/gofr/service"
 	"gofr.dev/pkg/gofr/testutil"

pkg/gofr/migration/interface.go

@@ -6,6 +6,7 @@ import (
 	"time"
 
 	goRedis "github.com/redis/go-redis/v9"
+
 	"gofr.dev/pkg/gofr/container"
 )
 

pkg/gofr/service/circuit_breaker_test.go

@@ -11,6 +11,7 @@ import (
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/mock"
 	"go.opentelemetry.io/otel"
+
 	"gofr.dev/pkg/gofr/logging"
 	"gofr.dev/pkg/gofr/testutil"
 )

pkg/gofr/service/retry_test.go

@@ -6,9 +6,9 @@ import (
 	"net/http/httptest"
 	"testing"
 
-	"gofr.dev/pkg/gofr/logging"
-
 	"github.com/stretchr/testify/assert"
+
+	"gofr.dev/pkg/gofr/logging"
 )
 
 type mockHTTP struct{}

pkg/gofr/websocket/websocket_test.go

@@ -7,10 +7,9 @@ import (
 	"testing"
 	"time"
 
-	"go.uber.org/mock/gomock"
-
 	"github.com/gorilla/websocket"
 	"github.com/stretchr/testify/assert"
+	"go.uber.org/mock/gomock"
 )
 
 func TestConnection_Bind_Success(t *testing.T) {