mirror of
https://github.com/hanzoai/tasks.git
synced 2026-10-04 02:04:03 +00:00
<!-- Describe what has changed in this PR --> **What changed?** `Driver` interface (with two implementations) was added to the postgresql persistence package in order to support other drivers than `lib/pq`. Fixes https://github.com/temporalio/temporal/issues/1775 Essentially, this PR does https://github.com/temporalio/temporal/pull/4914 in a backward-compatible and extendable way <!-- Tell your future self why have you made these changes --> **Why?** Since `lib/pq` does not support multi-node clusters <!-- How have you verified this change? Tested locally? Added a unit test? Checked in staging env? --> **How did you test it?** Existing tests <!-- Assuming the worst case, what can be broken when deploying this change to production? --> **Potential risks** No, since you should explicitly change the driver in your config file. However, `connectAttributes` depend on the driver (e.g., `binary_parameters` is supported only by `lib/pq` and is not a part of postresql standard), so one should be careful changing drivers in production. <!-- Is this PR a hotfix candidate or require that a notification be sent to the broader community? (Yes/No) --> **Is hotfix candidate?** No
186 lines
5.9 KiB
Go
186 lines
5.9 KiB
Go
// The MIT License
|
|
//
|
|
// Copyright (c) 2020 Temporal Technologies Inc. All rights reserved.
|
|
//
|
|
// Copyright (c) 2020 Uber Technologies, Inc.
|
|
//
|
|
// Permission is hereby granted, free of charge, to any person obtaining a copy
|
|
// of this software and associated documentation files (the "Software"), to deal
|
|
// in the Software without restriction, including without limitation the rights
|
|
// to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
|
|
// copies of the Software, and to permit persons to whom the Software is
|
|
// furnished to do so, subject to the following conditions:
|
|
//
|
|
// The above copyright notice and this permission notice shall be included in
|
|
// all copies or substantial portions of the Software.
|
|
//
|
|
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
|
|
// AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
|
// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
|
|
// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
|
|
// THE SOFTWARE.
|
|
|
|
package tests
|
|
|
|
import (
|
|
"strconv"
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/suite"
|
|
|
|
"go.temporal.io/server/common/persistence/sql/sqlplugin/postgresql"
|
|
"go.temporal.io/server/environment"
|
|
postgresqlversionV12 "go.temporal.io/server/schema/postgresql/v12"
|
|
postgresqlversionV96 "go.temporal.io/server/schema/postgresql/v96"
|
|
"go.temporal.io/server/tools/sql/clitest"
|
|
)
|
|
|
|
type PostgresqlSuite struct {
|
|
suite.Suite
|
|
pluginName string
|
|
}
|
|
|
|
func (p *PostgresqlSuite) TestPostgreSQLConnTestSuite() {
|
|
suite.Run(p.T(), clitest.NewSQLConnTestSuite(
|
|
environment.GetPostgreSQLAddress(),
|
|
strconv.Itoa(environment.GetPostgreSQLPort()),
|
|
p.pluginName, testPostgreSQLQuery,
|
|
))
|
|
}
|
|
|
|
func (p *PostgresqlSuite) TestPostgreSQLHandlerTestSuite() {
|
|
suite.Run(p.T(), clitest.NewHandlerTestSuite(
|
|
environment.GetPostgreSQLAddress(),
|
|
strconv.Itoa(environment.GetPostgreSQLPort()),
|
|
p.pluginName,
|
|
))
|
|
}
|
|
|
|
func (p *PostgresqlSuite) TestPostgreSQLSetupSchemaTestSuite() {
|
|
p.T().Setenv("SQL_HOST", environment.GetPostgreSQLAddress())
|
|
p.T().Setenv("SQL_PORT", strconv.Itoa(environment.GetPostgreSQLPort()))
|
|
p.T().Setenv("SQL_USER", testUser)
|
|
p.T().Setenv("SQL_PASSWORD", testPassword)
|
|
suite.Run(p.T(), clitest.NewSetupSchemaTestSuite(
|
|
environment.GetPostgreSQLAddress(),
|
|
strconv.Itoa(environment.GetPostgreSQLPort()),
|
|
p.pluginName,
|
|
testPostgreSQLQuery,
|
|
))
|
|
}
|
|
|
|
func (p *PostgresqlSuite) TestPostgreSQLUpdateSchemaTestSuite() {
|
|
p.T().Setenv("SQL_HOST", environment.GetPostgreSQLAddress())
|
|
p.T().Setenv("SQL_PORT", strconv.Itoa(environment.GetPostgreSQLPort()))
|
|
p.T().Setenv("SQL_USER", testUser)
|
|
p.T().Setenv("SQL_PASSWORD", testPassword)
|
|
suite.Run(p.T(), clitest.NewUpdateSchemaTestSuite(
|
|
environment.GetPostgreSQLAddress(),
|
|
strconv.Itoa(environment.GetPostgreSQLPort()),
|
|
p.pluginName,
|
|
testPostgreSQLQuery,
|
|
testPostgreSQLExecutionSchemaVersionDir,
|
|
postgresqlversionV96.Version,
|
|
testPostgreSQLVisibilitySchemaVersionDir,
|
|
postgresqlversionV96.VisibilityVersion,
|
|
))
|
|
}
|
|
|
|
func (p *PostgresqlSuite) TestPostgreSQLVersionTestSuite() {
|
|
p.T().Setenv("SQL_USER", testUser)
|
|
p.T().Setenv("SQL_PASSWORD", testPassword)
|
|
suite.Run(p.T(), clitest.NewVersionTestSuite(
|
|
environment.GetPostgreSQLAddress(),
|
|
strconv.Itoa(environment.GetPostgreSQLPort()),
|
|
p.pluginName,
|
|
testPostgreSQLExecutionSchemaFile,
|
|
testPostgreSQLVisibilitySchemaFile,
|
|
))
|
|
}
|
|
|
|
func TestPostgres(t *testing.T) {
|
|
s := &PostgresqlSuite{pluginName: postgresql.PluginName}
|
|
suite.Run(t, s)
|
|
}
|
|
|
|
func TestPostgrePGX(t *testing.T) {
|
|
s := &PostgresqlSuite{pluginName: postgresql.PluginNamePGX}
|
|
suite.Run(t, s)
|
|
}
|
|
|
|
type Postgresql12Suite struct {
|
|
suite.Suite
|
|
pluginName string
|
|
}
|
|
|
|
func (p *Postgresql12Suite) TestPostgreSQL12ConnTestSuite() {
|
|
suite.Run(p.T(), clitest.NewSQLConnTestSuite(
|
|
environment.GetPostgreSQLAddress(),
|
|
strconv.Itoa(environment.GetPostgreSQLPort()),
|
|
p.pluginName,
|
|
testPostgreSQLQuery,
|
|
))
|
|
}
|
|
|
|
func (p *Postgresql12Suite) TestPostgreSQL12HandlerTestSuite() {
|
|
suite.Run(p.T(), clitest.NewHandlerTestSuite(
|
|
environment.GetPostgreSQLAddress(),
|
|
strconv.Itoa(environment.GetPostgreSQLPort()),
|
|
p.pluginName,
|
|
))
|
|
}
|
|
|
|
func (p *Postgresql12Suite) TestPostgreSQL12SetupSchemaTestSuite() {
|
|
p.T().Setenv("SQL_HOST", environment.GetPostgreSQLAddress())
|
|
p.T().Setenv("SQL_PORT", strconv.Itoa(environment.GetPostgreSQLPort()))
|
|
p.T().Setenv("SQL_USER", testUser)
|
|
p.T().Setenv("SQL_PASSWORD", testPassword)
|
|
suite.Run(p.T(), clitest.NewSetupSchemaTestSuite(
|
|
environment.GetPostgreSQLAddress(),
|
|
strconv.Itoa(environment.GetPostgreSQLPort()),
|
|
p.pluginName,
|
|
testPostgreSQLQuery,
|
|
))
|
|
}
|
|
|
|
func (p *Postgresql12Suite) TestPostgreSQL12UpdateSchemaTestSuite() {
|
|
p.T().Setenv("SQL_HOST", environment.GetPostgreSQLAddress())
|
|
p.T().Setenv("SQL_PORT", strconv.Itoa(environment.GetPostgreSQLPort()))
|
|
p.T().Setenv("SQL_USER", testUser)
|
|
p.T().Setenv("SQL_PASSWORD", testPassword)
|
|
suite.Run(p.T(), clitest.NewUpdateSchemaTestSuite(
|
|
environment.GetPostgreSQLAddress(),
|
|
strconv.Itoa(environment.GetPostgreSQLPort()),
|
|
p.pluginName,
|
|
testPostgreSQLQuery,
|
|
testPostgreSQL12ExecutionSchemaVersionDir,
|
|
postgresqlversionV12.Version,
|
|
testPostgreSQL12VisibilitySchemaVersionDir,
|
|
postgresqlversionV12.VisibilityVersion,
|
|
))
|
|
}
|
|
|
|
func (p *Postgresql12Suite) TestPostgreSQL12VersionTestSuite() {
|
|
p.T().Setenv("SQL_USER", testUser)
|
|
p.T().Setenv("SQL_PASSWORD", testPassword)
|
|
suite.Run(p.T(), clitest.NewVersionTestSuite(
|
|
environment.GetPostgreSQLAddress(),
|
|
strconv.Itoa(environment.GetPostgreSQLPort()),
|
|
p.pluginName,
|
|
testPostgreSQL12ExecutionSchemaFile,
|
|
testPostgreSQL12VisibilitySchemaFile,
|
|
))
|
|
}
|
|
|
|
func TestPostgres12(t *testing.T) {
|
|
s := &Postgresql12Suite{pluginName: postgresql.PluginNameV12}
|
|
suite.Run(t, s)
|
|
}
|
|
|
|
func TestPostgre12PGX(t *testing.T) {
|
|
s := &Postgresql12Suite{pluginName: postgresql.PluginNameV12PGX}
|
|
suite.Run(t, s)
|
|
}
|