fix issue in loosing internal ctx data for Transaction for package gdb; fix issue gogf#1732

Author: gqcn

database/gdb/gdb_core_transaction.go

@@ -69,11 +69,12 @@ func (c *Core) doBeginCtx(ctx context.Context) (*TX, error) {
 // Note that, you should not Commit or Rollback the transaction in function `f`
 // as it is automatically handled by this function.
 func (c *Core) Transaction(ctx context.Context, f func(ctx context.Context, tx *TX) error) (err error) {
-	var tx *TX
 	if ctx == nil {
 		ctx = c.db.GetCtx()
 	}
+	ctx = c.InjectInternalCtxData(ctx)
 	// Check transaction object from context.
+	var tx *TX
 	tx = TXFromCtx(ctx, c.db.GetGroup())
 	if tx != nil {
 		return tx.Transaction(ctx, f)

database/gdb/gdb_z_mysql_model_test.go

@@ -20,6 +20,7 @@ import (
 	"github.com/gogf/gf/v2/database/gdb"
 	"github.com/gogf/gf/v2/encoding/gjson"
 	"github.com/gogf/gf/v2/frame/g"
+	"github.com/gogf/gf/v2/os/gctx"
 	"github.com/gogf/gf/v2/os/gfile"
 	"github.com/gogf/gf/v2/os/gtime"
 	"github.com/gogf/gf/v2/test/gtest"

test/gtest/gtest_util.go

@@ -342,7 +342,7 @@ func AssertNil(value interface{}) {
 	if err, ok := value.(error); ok {
 		panic(fmt.Sprintf(`%+v`, err))
 	}
-	AssertNE(value, nil)
+	Assert(value, nil)
 }
 
 // DataPath retrieves and returns the testdata path of current package,