feat(patch): defaulting to snake case naming of the resource object provided if no table name option has been provided (#131)

* defaulting to snake case naming of the resource object provided if no table name option has been provided

* parallel tests

* reording linter flow

Author: Jacobbrewer1

.github/workflows/ci-code-approval.yml

@@ -29,15 +29,15 @@ jobs:
     name: Golangci-lint
     runs-on: ubuntu-latest
     steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
       # Sets up golang
       - uses: actions/setup-go@v5
         with:
-          go-version: ${{ vars.GO_VERSION }}
+          go-version-file: 'go.mod'
           cache-dependency-path: "**/*.sum"
 
-      - name: Checkout repository
-        uses: actions/checkout@v4
-
       - name: golangci-lint
         uses: golangci/golangci-lint-action@v6
         with:
@@ -54,7 +54,7 @@ jobs:
       - name: Set up Go
         uses: actions/setup-go@v5
         with:
-          go-version: ${{ vars.GO_VERSION }}
+          go-version-file: 'go.mod'
           cache-dependency-path: "**/*.sum"
 
       - name: Make
@@ -83,7 +83,7 @@ jobs:
       - name: Set up Go
         uses: actions/setup-go@v5
         with:
-          go-version: ${{ vars.GO_VERSION }}
+          go-version-file: 'go.mod'
           cache-dependency-path: "**/*.sum"
 
       - name: "Run Code Generation"

sql.go

@@ -31,6 +31,10 @@ func NewSQLPatch(resource any, opts ...PatchOpt) *SQLPatch {
 // It processes the fields of the struct, applying the necessary tags and options,
 // and prepares the SQL update statement components (fields and arguments).
 func (s *SQLPatch) patchGen(resource any) {
+	if s.table == "" {
+		s.table = getTableName(resource)
+	}
+
 	resource = dereferenceIfPointer(resource)
 	ensureStruct(resource)
 

sql_test.go

@@ -35,6 +35,24 @@ func (s *newSQLPatchSuite) TestNewSQLPatch_Success() {
 	s.Equal([]any{1, "test"}, patch.args)
 }
 
+func (s *newSQLPatchSuite) TestNewSQLPatch_Success_NoTableProvided() {
+	type testObj struct {
+		Id   *int    `db:"id_tag"`
+		Name *string `db:"name_tag"`
+	}
+
+	obj := testObj{
+		Id:   ptr(1),
+		Name: ptr("test"),
+	}
+
+	patch := NewSQLPatch(obj)
+
+	s.Equal([]string{"id_tag = ?", "name_tag = ?"}, patch.fields)
+	s.Equal([]any{1, "test"}, patch.args)
+	s.Equal("test_obj", patch.table)
+}
+
 func (s *newSQLPatchSuite) TestNewSQLPatch_Success_Filter_Where() {
 	type testObj struct {
 		Id   *int    `db:"id_tag"`
@@ -759,6 +777,30 @@ func (s *generateSQLSuite) TestGenerateSQL_Success() {
 	mw.AssertExpectations(s.T())
 }
 
+func (s *generateSQLSuite) TestGenerateSQL_Success_NoTableProvided() {
+	type testTable struct {
+		Id   *int    `db:"id"`
+		Name *string `db:"name"`
+	}
+
+	obj := testTable{
+		Id:   ptr(1),
+		Name: ptr("test"),
+	}
+
+	mw := NewMockWherer(s.T())
+	mw.On("Where").Return("age = ?", []any{18})
+
+	sqlStr, args, err := GenerateSQL(obj,
+		WithWhere(mw),
+	)
+	s.Require().NoError(err)
+	s.Equal("UPDATE test_table\nSET id = ?, name = ?\nWHERE (1=1)\nAND (\nage = ?\n)", sqlStr)
+	s.Equal([]any{1, "test", 18}, args)
+
+	mw.AssertExpectations(s.T())
+}
+
 type testFilter struct{}
 
 func (f *testFilter) Where() (sqlStr string, args []any) {
@@ -1612,6 +1654,35 @@ func (s *NewDiffSQLPatchSuite) TestNewDiffSQLPatch_Success_SqlGen() {
 	s.Equal([]any{"test2", 18}, args)
 }
 
+func (s *NewDiffSQLPatchSuite) TestNewDiffSQLPatch_Success_SqlGen_NoTableProvided() {
+	type testTable struct {
+		Id   *int    `db:"id"`
+		Name *string `db:"name"`
+	}
+
+	obj := testTable{
+		Id:   ptr(1),
+		Name: ptr("test"),
+	}
+
+	obj2 := testTable{
+		Id:   ptr(1),
+		Name: ptr("test2"),
+	}
+
+	mw := NewMockWherer(s.T())
+	mw.On("Where").Return("age = ?", []any{18})
+
+	patch, err := NewDiffSQLPatch(&obj, &obj2, WithWhere(mw))
+	s.Require().NoError(err)
+
+	sqlStr, args, err := patch.GenerateSQL()
+	s.Require().NoError(err)
+
+	s.Equal("UPDATE test_table\nSET name = ?\nWHERE (1=1)\nAND (\nage = ?\n)", sqlStr)
+	s.Equal([]any{"test2", 18}, args)
+}
+
 func (s *NewDiffSQLPatchSuite) TestNewDiffSQLPatch_Success_SqlGen_ValueField() {
 	type testObj struct {
 		Id   *int    `db:"id"`

utils.go

@@ -20,7 +20,7 @@ func isPointerToStruct[T any](t T) bool {
 }
 
 func dereferenceIfPointer(resource any) any {
-	if reflect.TypeOf(resource).Kind() == reflect.Ptr {
+	if reflect.TypeOf(resource).Kind() == reflect.Ptr && !reflect.ValueOf(resource).IsNil() {
 		return reflect.ValueOf(resource).Elem().Interface()
 	}
 	return resource
@@ -47,7 +47,9 @@ func getTag(fType *reflect.StructField, tagName string) string {
 }
 
 func getValue(fVal reflect.Value) any {
-	if fVal.Kind() == reflect.Ptr {
+	if fVal.Kind() == reflect.Ptr && fVal.IsNil() {
+		return nil
+	} else if fVal.Kind() == reflect.Ptr {
 		return fVal.Elem().Interface()
 	}
 	return fVal.Interface()
@@ -64,3 +66,22 @@ func IsValidType(val reflect.Value) bool {
 		return false
 	}
 }
+
+func getTableName(resource any) string {
+	typeOf := reflect.TypeOf(resource)
+	if typeOf.Kind() == reflect.Ptr {
+		typeOf = typeOf.Elem()
+	}
+	return toSnakeCase(typeOf.Name())
+}
+
+func toSnakeCase(s string) string {
+	result := make([]rune, 0, len(s)*2)
+	for i, r := range s {
+		if i > 0 && 'A' <= r && r <= 'Z' {
+			result = append(result, '_')
+		}
+		result = append(result, r)
+	}
+	return strings.ToLower(string(result))
+}