fix session tests, remove RawQuery, fix docs

Author: agoncear-mwb

chdb-purego/chdb.go

@@ -141,22 +141,6 @@ func (c *connection) Ready() bool {
 	return false
 }
 
-// RawQuery will execute the given clickouse query without using any session.
-func RawQuery(argc int, argv []string) (result ChdbResult, err error) {
-	res := queryStableV2(argc, argv)
-	if res == nil {
-		// According to the C ABI of chDB v1.2.0, the C function query_stable_v2
-		// returns nil if the query returns no data. This is not an error. We
-		// will change this behavior in the future.
-		return newChdbResult(res), nil
-	}
-	if res.error_message != nil {
-		return nil, errors.New(ptrToGoString(res.error_message))
-	}
-
-	return newChdbResult(res), nil
-}
-
 // Session will keep the state of query.
 // If path is None, it will create a temporary directory and use it as the database path
 // and the temporary directory will be removed when the session is closed.

chdb.md

@@ -20,7 +20,7 @@ import "github.com/chdb-io/chdb-go/chdb"
 
 
 <a name="Query"></a>
-## func [Query](<https://github.com/agoncear-mwb/chdb-go/blob/main/chdb/wrapper.go#L8>)
+## func [Query](<https://github.com/chdb-io/chdb-go/blob/main/chdb/wrapper.go#L8>)
 
 ```go
 func Query(queryStr string, outputFormats ...string) (result chdbpurego.ChdbResult, err error)
@@ -29,7 +29,7 @@ func Query(queryStr string, outputFormats ...string) (result chdbpurego.ChdbResu
 Query calls query\_conn with a default in\-memory session and default output format of "CSV" if not provided.
 
 <a name="Session"></a>
-## type [Session](<https://github.com/agoncear-mwb/chdb-go/blob/main/chdb/session.go#L15-L20>)
+## type [Session](<https://github.com/chdb-io/chdb-go/blob/main/chdb/session.go#L15-L20>)
 
 
 
@@ -40,7 +40,7 @@ type Session struct {
 ```
 
 <a name="NewSession"></a>
-### func [NewSession](<https://github.com/agoncear-mwb/chdb-go/blob/main/chdb/session.go#L25>)
+### func [NewSession](<https://github.com/chdb-io/chdb-go/blob/main/chdb/session.go#L25>)
 
 ```go
 func NewSession(paths ...string) (*Session, error)
@@ -49,7 +49,7 @@ func NewSession(paths ...string) (*Session, error)
 NewSession creates a new session with the given path. If path is empty, a temporary directory is created. Note: The temporary directory is removed when Close is called.
 
 <a name="Session.Cleanup"></a>
-### func \(\*Session\) [Cleanup](<https://github.com/agoncear-mwb/chdb-go/blob/main/chdb/session.go#L77>)
+### func \(\*Session\) [Cleanup](<https://github.com/chdb-io/chdb-go/blob/main/chdb/session.go#L77>)
 
 ```go
 func (s *Session) Cleanup()
@@ -58,7 +58,7 @@ func (s *Session) Cleanup()
 Cleanup closes the session and removes the directory.
 
 <a name="Session.Close"></a>
-### func \(\*Session\) [Close](<https://github.com/agoncear-mwb/chdb-go/blob/main/chdb/session.go#L67>)
+### func \(\*Session\) [Close](<https://github.com/chdb-io/chdb-go/blob/main/chdb/session.go#L67>)
 
 ```go
 func (s *Session) Close()
@@ -71,7 +71,7 @@ temporary directory is created when NewSession was called with an empty path.
 ```
 
 <a name="Session.ConnStr"></a>
-### func \(\*Session\) [ConnStr](<https://github.com/agoncear-mwb/chdb-go/blob/main/chdb/session.go#L88>)
+### func \(\*Session\) [ConnStr](<https://github.com/chdb-io/chdb-go/blob/main/chdb/session.go#L88>)
 
 ```go
 func (s *Session) ConnStr() string
@@ -80,7 +80,7 @@ func (s *Session) ConnStr() string
 ConnStr returns the current connection string used for the underlying connection
 
 <a name="Session.IsTemp"></a>
-### func \(\*Session\) [IsTemp](<https://github.com/agoncear-mwb/chdb-go/blob/main/chdb/session.go#L93>)
+### func \(\*Session\) [IsTemp](<https://github.com/chdb-io/chdb-go/blob/main/chdb/session.go#L93>)
 
 ```go
 func (s *Session) IsTemp() bool
@@ -89,7 +89,7 @@ func (s *Session) IsTemp() bool
 IsTemp returns whether the session is temporary.
 
 <a name="Session.Path"></a>
-### func \(\*Session\) [Path](<https://github.com/agoncear-mwb/chdb-go/blob/main/chdb/session.go#L83>)
+### func \(\*Session\) [Path](<https://github.com/chdb-io/chdb-go/blob/main/chdb/session.go#L83>)
 
 ```go
 func (s *Session) Path() string
@@ -98,7 +98,7 @@ func (s *Session) Path() string
 Path returns the path of the session.
 
 <a name="Session.Query"></a>
-### func \(\*Session\) [Query](<https://github.com/agoncear-mwb/chdb-go/blob/main/chdb/session.go#L56>)
+### func \(\*Session\) [Query](<https://github.com/chdb-io/chdb-go/blob/main/chdb/session.go#L56>)
 
 ```go
 func (s *Session) Query(queryStr string, outputFormats ...string) (result chdbpurego.ChdbResult, err error)

chdb/session_test.go

@@ -2,6 +2,7 @@ package chdb
 
 import (
 	"os"
+	"path/filepath"
 	"testing"
 )
 
@@ -50,3 +51,49 @@ func TestSessionCleanup(t *testing.T) {
 		t.Errorf("Session directory should be removed after Cleanup: %s", session.Path())
 	}
 }
+
+func TestQuery(t *testing.T) {
+	session, _ := NewSession()
+	defer session.Cleanup()
+
+	session.Query("CREATE DATABASE IF NOT EXISTS testdb; " +
+		"CREATE TABLE IF NOT EXISTS testdb.testtable (id UInt32) ENGINE = MergeTree() ORDER BY id;")
+
+	session.Query("USE testdb; INSERT INTO testtable VALUES (1), (2), (3);")
+
+	ret, err := session.Query("SELECT * FROM testtable;")
+	if err != nil {
+		t.Errorf("Query failed: %s", err)
+	}
+	if string(ret.Buf()) != "1\n2\n3\n" {
+		t.Errorf("Query result should be 1\n2\n3\n, got %s", string(ret.Buf()))
+	}
+}
+
+func TestSessionPathAndIsTemp(t *testing.T) {
+	// Create a new session and check its Path and IsTemp
+	session, _ := NewSession()
+
+	if session.Path() == "" {
+		t.Errorf("Session path should not be empty")
+	}
+
+	if !session.IsTemp() {
+		t.Errorf("Session should be temporary")
+	}
+	session.Close()
+
+	// Create a new session with a specific path and check its Path and IsTemp
+	path := filepath.Join(os.TempDir(), "chdb_test2")
+	defer os.RemoveAll(path)
+	session, _ = NewSession(path)
+	defer session.Cleanup()
+
+	if session.Path() != path {
+		t.Errorf("Session path should be %s, got %s", path, session.Path())
+	}
+
+	if session.IsTemp() {
+		t.Errorf("Session should not be temporary")
+	}
+}