mirror of
https://gitee.com/johng/gf
synced 2026-06-06 02:25:47 +08:00
This pull request significantly improves PostgreSQL array type handling and conversion in the `pgsql` driver, providing more accurate type mapping and conversion logic, especially for array types. It introduces comprehensive documentation, refactors conversion logic to use the `pq` package for array types, and adds extensive unit tests to ensure correctness and error handling. Additionally, minor enhancements and clarifications are made to upsert formatting and table field queries. ### PostgreSQL Array Type Handling and Conversion * Refactored `CheckLocalTypeForField` and `ConvertValueForLocal` methods in `contrib/drivers/pgsql/pgsql_convert.go` to accurately map PostgreSQL array types (such as `_int2`, `_int4`, `_int8`, `_float4`, `_float8`, `_bool`, `_varchar`, `_text`, `_char`, `_bpchar`, `_numeric`, `_decimal`, `_money`, `_bytea`) to their corresponding Go types, using the `pq` package for conversion. Added detailed documentation and mapping tables for supported types. [[1]](diffhunk://#diff-a3b1e68bfa29fbcfda7c703bbe875fa82e958f6c3ad942ef82193a9dd8ad67e2R46-R63) [[2]](diffhunk://#diff-a3b1e68bfa29fbcfda7c703bbe875fa82e958f6c3ad942ef82193a9dd8ad67e2L56-R103) [[3]](diffhunk://#diff-a3b1e68bfa29fbcfda7c703bbe875fa82e958f6c3ad942ef82193a9dd8ad67e2R112-R209) * Added comprehensive unit tests in `contrib/drivers/pgsql/pgsql_z_unit_convert_test.go` to verify type mapping and conversion for all supported array types, including error cases for invalid input. ### Utility and API Improvements * Added a new `Bools()` method to the `gvar.Var` type in `container/gvar/gvar_slice.go` for converting values to `[]bool`, with corresponding unit tests in `container/gvar/gvar_z_unit_slice_test.go`. [[1]](diffhunk://#diff-32e887e540e0170f785508d105cb794e4d54d854b53b6950973c80022973c490R11-R15) [[2]](diffhunk://#diff-01453eca4d4b3e35d07ca105cb924c6441d0cd9df6cbcc337a89832c8d53057fR24-R41) ### SQL Formatting and Documentation * Improved documentation and formatting in the upsert logic of `contrib/drivers/pgsql/pgsql_format_upsert.go` to clarify the use of `EXCLUDED` in PostgreSQL's `ON CONFLICT DO UPDATE`. * Enhanced readability of the table field query in `contrib/drivers/pgsql/pgsql_table_fields.go` by reformatting SQL and clarifying field extraction. --------- Co-authored-by: hailaz <739476267@qq.com> Co-authored-by: houseme <housemecn@gmail.com>
97 lines
2.4 KiB
Go
97 lines
2.4 KiB
Go
// Copyright GoFrame Author(https://goframe.org). All Rights Reserved.
|
|
//
|
|
// This Source Code Form is subject to the terms of the MIT License.
|
|
// If a copy of the MIT was not distributed with this file,
|
|
// You can obtain one at https://github.com/gogf/gf.
|
|
|
|
package gconv_test
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"github.com/gogf/gf/v2/test/gtest"
|
|
"github.com/gogf/gf/v2/util/gconv"
|
|
)
|
|
|
|
var (
|
|
boolTestTrueValue = true
|
|
boolTestFalseValue = false
|
|
)
|
|
|
|
var boolTests = []struct {
|
|
value any
|
|
expect bool
|
|
}{
|
|
{true, true},
|
|
{false, false},
|
|
|
|
{0, false},
|
|
{1, true},
|
|
|
|
{[]byte(""), false},
|
|
|
|
{"", false},
|
|
{"0", false},
|
|
{"1", true},
|
|
{"123.456", true},
|
|
{"true", true},
|
|
{"false", false},
|
|
{"on", true},
|
|
{"off", false},
|
|
|
|
{complex(1, 2), true},
|
|
{complex(123.456, 789.123), true},
|
|
|
|
{[3]int{1, 2, 3}, true},
|
|
{[]int{1, 2, 3}, true},
|
|
|
|
{map[int]int{1: 1}, true},
|
|
{map[string]string{"Earth": "印度洋"}, true},
|
|
|
|
{struct{}{}, true},
|
|
{&struct{}{}, true},
|
|
{nil, false},
|
|
{(*bool)(nil), false},
|
|
|
|
{&boolTestTrueValue, true},
|
|
{&boolTestFalseValue, false},
|
|
|
|
{myBool(true), true},
|
|
{myBool(false), false},
|
|
{(*myBool)(&boolTestTrueValue), true},
|
|
{(*myBool)(&boolTestFalseValue), false},
|
|
|
|
{(*myBool)(nil), false},
|
|
}
|
|
|
|
func TestBool(t *testing.T) {
|
|
gtest.C(t, func(t *gtest.T) {
|
|
for _, test := range boolTests {
|
|
t.AssertEQ(gconv.Bool(test.value), test.expect)
|
|
}
|
|
})
|
|
}
|
|
|
|
func TestBools(t *testing.T) {
|
|
gtest.C(t, func(t *gtest.T) {
|
|
t.AssertEQ(gconv.Bools(nil), nil)
|
|
t.AssertEQ(gconv.Bools([]bool{true, false}), []bool{true, false})
|
|
t.AssertEQ(gconv.Bools([]int{1, 0, 2}), []bool{true, false, true})
|
|
t.AssertEQ(gconv.Bools([]string{"true", "false", "1", "0"}), []bool{true, false, true, false})
|
|
t.AssertEQ(gconv.Bools([]string{"t", "f", "T", "F"}), []bool{true, false, true, false})
|
|
t.AssertEQ(gconv.Bools([]string{"True", "False", "TRUE", "FALSE"}), []bool{true, false, true, false})
|
|
t.AssertEQ(gconv.Bools([]string{"yes", "no", "YES", "NO"}), []bool{true, false, true, false})
|
|
t.AssertEQ(gconv.Bools([]string{"on", "off", "ON", "OFF"}), []bool{true, false, true, false})
|
|
t.AssertEQ(gconv.Bools([]any{true, 0, "false", 1}), []bool{true, false, false, true})
|
|
t.AssertEQ(gconv.Bools(`[true, false, true]`), []bool{true, false, true})
|
|
t.AssertEQ(gconv.Bools(""), []bool{})
|
|
t.AssertEQ(gconv.Bools("true"), []bool{true})
|
|
})
|
|
}
|
|
|
|
func TestSliceBool(t *testing.T) {
|
|
gtest.C(t, func(t *gtest.T) {
|
|
t.AssertEQ(gconv.SliceBool([]bool{true, false}), []bool{true, false})
|
|
})
|
|
}
|