feat: add workflow; fix dbtest

Author: roman-vanesyan

.github/workflows/test.yaml

@@ -0,0 +1,46 @@
+name: Go
+
+on:
+  push:
+    branches: [main]
+  pull_request:
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        go-version: [ '1.22.x' ]
+
+    steps:
+      - uses: actions/checkout@v4
+      - name: Setup Go ${{ matrix.go-version }}
+        uses: actions/setup-go@v5
+        with:
+          go-version: ${{ matrix.go-version }}
+
+      - name: Restore Cache
+        uses: actions/cache@v4
+        with:
+          path: |
+            ~/.cache/go-build
+            ~/go/pkg/mod
+          key: ${{ runner.os }}-v1-go-${{ hashFiles('**/go.sum') }}
+          restore-keys: |
+            ${{ runner.os }}-go-
+
+      - name:  Run Tests
+        run: go test -v ./...
+
+  lint:
+    name: lint
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: stable
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@v6
+        with:
+          version: v1.58

authentication/password/handler_test.go

@@ -69,18 +69,16 @@ func TestUserRegistration(t *testing.T) {
 		)
 	})
 
-	// t.Run("user already exists", func(t *testing.T) {
-	// 	must.Must1(db.Reset(ctx))
-	//
-	// 	// TODO: implement
-	// })
+	t.Run("user already exists", func(t *testing.T) {
+		must.Must1(db.Reset(ctx))
+	})
 }
 
 func TestUserLogin(t *testing.T) {
 	ctx := context.Background()
 	db := dbtest.Must(ctx, t)
 
 	t.Run("user not found", func(t *testing.T) {
-		defer db.TruncateAllTables(ctx)
+		must.Must1(db.Reset(ctx))
 	})
 }

sql/db/dbtest/dbtest.go

@@ -1,4 +1,9 @@
 // Package dbtesting providers utilities for testing database related code.
+//
+// It provides a set of utility functions to initialize a database pool,
+// load DLL schema, clean it up, etc.
+//
+// Note the package is intended to be used only within testing files.
 package dbtest
 
 import (
@@ -8,7 +13,6 @@ import (
 	"io"
 	"os"
 	"path/filepath"
-	"runtime"
 	"strings"
 	"sync"
 	"testing"
@@ -22,10 +26,6 @@ import (
 	"go.inout.gg/common/must"
 )
 
-var (
-	MaxConfigLookupCallerDepth = 10
-)
-
 const (
 	queryFetchAllTables = `
 SELECT table_name
@@ -58,18 +58,18 @@ type Config struct {
 
 // MustLoadConfig loads the configuration from the environment.
 //
-// If no paths are provided, it defaults to ".test.env" in the current module path
+// If no paths are provided, it defaults to ".test.env" in the current
+// working directory (which for tests is the directory in which they are located at),
 // and in the root of the project.
 //
 // It panics if there is an error loading the configuration.
 func MustLoadConfig(paths ...string) *Config {
 	if len(paths) == 0 {
-		callingModulePath := lookupLastCallingModulePath(4)
-
-		rootPath := findModuleRoot(callingModulePath)
+		currentModulePath := must.Must(os.Getwd())
+		rootPath := findModuleRoot(currentModulePath)
 		paths = []string{
 			filepath.Join(rootPath, ".test.env"),
-			filepath.Join(callingModulePath, ".test.env"),
+			filepath.Join(currentModulePath, ".test.env"),
 		}
 	}
 
@@ -78,20 +78,6 @@ func MustLoadConfig(paths ...string) *Config {
 	return config
 }
 
-func lookupLastCallingModulePath(m int) string {
-	var p string
-	for i := 0; i < m; i++ {
-		_, basepath, _, ok := runtime.Caller(i)
-		if !ok {
-			break
-		}
-
-		p = filepath.Dir(basepath)
-	}
-
-	return p
-}
-
 // DB is a wrapper around pgxpool.Pool with useful utilities for DB management
 // in tests.
 type DB struct {