This is an automated email from the ASF dual-hosted git repository.
jimin pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-seata-go.git
The following commit(s) were added to refs/heads/master by this push:
new 4dd7de66 test: improve test coverage for pkg/datasource (#973)
4dd7de66 is described below
commit 4dd7de669b88bfb8487f24ca78a08638dd1ffdc0
Author: EVERFID <[email protected]>
AuthorDate: Sat Nov 8 00:16:19 2025 +0800
test: improve test coverage for pkg/datasource (#973)
---
pkg/datasource/init_test.go | 55 +++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 55 insertions(+)
diff --git a/pkg/datasource/init_test.go b/pkg/datasource/init_test.go
new file mode 100644
index 00000000..4a8e8f95
--- /dev/null
+++ b/pkg/datasource/init_test.go
@@ -0,0 +1,55 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements. See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package datasource
+
+import (
+ "database/sql"
+ "testing"
+
+ "github.com/stretchr/testify/assert"
+)
+
+// TestInit is the primary test that ensures Init() is called and provides
coverage
+func TestInit(t *testing.T) {
+ // Call Init() - this is the KEY to getting coverage
+ assert.NotPanics(t, func() {
+ Init()
+ }, "Init() should not panic")
+
+ // Verify that calling Init() has the expected side effect
+ // (registering database drivers)
+ drivers := sql.Drivers()
+ assert.NotNil(t, drivers, "Driver list should not be nil after Init()")
+
+ // Log what drivers are available for debugging
+ t.Logf("Registered drivers after Init(): %v", drivers)
+}
+
+// TestInitCalledMultipleTimes documents that Init() may panic if called
multiple times
+// This is expected behavior for driver registration
+func TestInitCalledMultipleTimes(t *testing.T) {
+ // Note: If Init() has already been called in TestInit, this may panic
+ // This documents the non-idempotent behavior
+ defer func() {
+ if r := recover(); r != nil {
+ t.Logf("Init() panicked when called again (expected):
%v", r)
+ }
+ }()
+
+ Init()
+}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
