summaryrefslogtreecommitdiff
path: root/test/sql
diff options
context:
space:
mode:
authorFederico Caselli <cfederico87@gmail.com>2022-09-24 15:50:26 +0200
committerMike Bayer <mike_mp@zzzcomputing.com>2022-10-17 15:36:25 -0400
commit974b1bd0fc40e11fc2886b5a9fc333feeeebf546 (patch)
tree421f0545c13a203f40435c4646a0de664e0e9756 /test/sql
parent665c94cc2f0340735515c4f4477e11b556d2bcd8 (diff)
downloadsqlalchemy-974b1bd0fc40e11fc2886b5a9fc333feeeebf546.tar.gz
Revert automatic set of sequence start to 1
The :class:`.Sequence` construct restores itself to the DDL behavior it had prior to the 1.4 series, where creating a :class:`.Sequence` with no additional arguments will emit a simple ``CREATE SEQUENCE`` instruction **without** any additional parameters for "start value". For most backends, this is how things worked previously in any case; **however**, for MS SQL Server, the default value on this database is ``-2**63``; to prevent this generally impractical default from taking effect on SQL Server, the :paramref:`.Sequence.start` parameter should be provided. As usage of :class:`.Sequence` is unusual for SQL Server which for many years has standardized on ``IDENTITY``, it is hoped that this change has minimal impact. Fixes: #7211 Change-Id: I1207ea10c8cb1528a1519a0fb3581d9621c27b31
Diffstat (limited to 'test/sql')
-rw-r--r--test/sql/test_compiler.py6
-rw-r--r--test/sql/test_defaults.py10
-rw-r--r--test/sql/test_deprecations.py6
-rw-r--r--test/sql/test_functions.py6
-rw-r--r--test/sql/test_insert.py35
-rw-r--r--test/sql/test_insert_exec.py16
-rw-r--r--test/sql/test_returning.py2
-rw-r--r--test/sql/test_selectable.py6
-rw-r--r--test/sql/test_sequences.py234
9 files changed, 209 insertions, 112 deletions
diff --git a/test/sql/test_compiler.py b/test/sql/test_compiler.py
index 1d3d17326..97b1b9124 100644
--- a/test/sql/test_compiler.py
+++ b/test/sql/test_compiler.py
@@ -5452,7 +5452,7 @@ class DDLTest(fixtures.TestBase, AssertsCompiledSQL):
self.assert_compile(
schema.CreateSequence(s1),
- "CREATE SEQUENCE __[SCHEMA__none].s1 START WITH 1",
+ "CREATE SEQUENCE __[SCHEMA__none].s1",
schema_translate_map=schema_translate_map,
)
@@ -5465,7 +5465,7 @@ class DDLTest(fixtures.TestBase, AssertsCompiledSQL):
self.assert_compile(
schema.CreateSequence(s2),
- "CREATE SEQUENCE __[SCHEMA_foo].s2 START WITH 1",
+ "CREATE SEQUENCE __[SCHEMA_foo].s2",
schema_translate_map=schema_translate_map,
)
@@ -5478,7 +5478,7 @@ class DDLTest(fixtures.TestBase, AssertsCompiledSQL):
self.assert_compile(
schema.CreateSequence(s3),
- "CREATE SEQUENCE __[SCHEMA_bar].s3 START WITH 1",
+ "CREATE SEQUENCE __[SCHEMA_bar].s3",
schema_translate_map=schema_translate_map,
)
diff --git a/test/sql/test_defaults.py b/test/sql/test_defaults.py
index 08911a6c5..1249529f3 100644
--- a/test/sql/test_defaults.py
+++ b/test/sql/test_defaults.py
@@ -19,11 +19,13 @@ from sqlalchemy.sql import select
from sqlalchemy.sql import text
from sqlalchemy.testing import assert_raises_message
from sqlalchemy.testing import AssertsCompiledSQL
+from sqlalchemy.testing import config
from sqlalchemy.testing import eq_
from sqlalchemy.testing import expect_warnings
from sqlalchemy.testing import fixtures
from sqlalchemy.testing import mock
from sqlalchemy.testing.assertions import expect_deprecated
+from sqlalchemy.testing.provision import normalize_sequence
from sqlalchemy.testing.schema import Column
from sqlalchemy.testing.schema import Table
from sqlalchemy.types import TypeDecorator
@@ -1030,7 +1032,9 @@ class PKIncrementTest(fixtures.TablesTest):
Column(
"id",
Integer,
- Sequence("ai_id_seq", optional=True),
+ normalize_sequence(
+ config, Sequence("ai_id_seq", optional=True)
+ ),
primary_key=True,
),
Column("int1", Integer),
@@ -1171,7 +1175,7 @@ class AutoIncrementTest(fixtures.TestBase):
# and autoincrement=False. Using a ForeignKey
# would have the same effect
- some_seq = Sequence("some_seq")
+ some_seq = normalize_sequence(config, Sequence("some_seq"))
dataset_no_autoinc = Table(
"x",
@@ -1313,7 +1317,7 @@ class SpecialTypePKTest(fixtures.TestBase):
self._run_test(default=literal_column("1", type_=self.MyInteger))
def test_sequence(self):
- self._run_test(Sequence("foo_seq"))
+ self._run_test(normalize_sequence(config, Sequence("foo_seq")))
def test_text_clause_default_no_type(self):
self._run_test(default=text("1"))
diff --git a/test/sql/test_deprecations.py b/test/sql/test_deprecations.py
index 82a54d20a..ae34b0c0f 100644
--- a/test/sql/test_deprecations.py
+++ b/test/sql/test_deprecations.py
@@ -32,11 +32,13 @@ from sqlalchemy.sql import visitors
from sqlalchemy.sql.selectable import SelectStatementGrouping
from sqlalchemy.testing import assertions
from sqlalchemy.testing import AssertsCompiledSQL
+from sqlalchemy.testing import config
from sqlalchemy.testing import eq_
from sqlalchemy.testing import fixtures
from sqlalchemy.testing import is_
from sqlalchemy.testing import is_true
from sqlalchemy.testing import mock
+from sqlalchemy.testing.provision import normalize_sequence
from sqlalchemy.testing.schema import Column
from sqlalchemy.testing.schema import Table
@@ -603,7 +605,9 @@ class PKIncrementTest(fixtures.TablesTest):
Column(
"id",
Integer,
- Sequence("ai_id_seq", optional=True),
+ normalize_sequence(
+ config, Sequence("ai_id_seq", optional=True)
+ ),
primary_key=True,
),
Column("int1", Integer),
diff --git a/test/sql/test_functions.py b/test/sql/test_functions.py
index a48d42f50..f92dd8496 100644
--- a/test/sql/test_functions.py
+++ b/test/sql/test_functions.py
@@ -44,11 +44,13 @@ from sqlalchemy.sql.functions import GenericFunction
from sqlalchemy.testing import assert_raises
from sqlalchemy.testing import assert_raises_message
from sqlalchemy.testing import AssertsCompiledSQL
+from sqlalchemy.testing import config
from sqlalchemy.testing import eq_
from sqlalchemy.testing import fixtures
from sqlalchemy.testing import is_
from sqlalchemy.testing.assertions import expect_warnings
from sqlalchemy.testing.engines import all_dialects
+from sqlalchemy.testing.provision import normalize_sequence
table1 = table(
@@ -1070,7 +1072,7 @@ class ExecuteTest(fixtures.TestBase):
Column(
"id",
Integer,
- Sequence("t1idseq", optional=True),
+ normalize_sequence(config, Sequence("t1idseq", optional=True)),
primary_key=True,
),
Column("value", Integer),
@@ -1081,7 +1083,7 @@ class ExecuteTest(fixtures.TestBase):
Column(
"id",
Integer,
- Sequence("t2idseq", optional=True),
+ normalize_sequence(config, Sequence("t2idseq", optional=True)),
primary_key=True,
),
Column("value", Integer, default=7),
diff --git a/test/sql/test_insert.py b/test/sql/test_insert.py
index 23a850f08..395fe16d3 100644
--- a/test/sql/test_insert.py
+++ b/test/sql/test_insert.py
@@ -26,10 +26,12 @@ from sqlalchemy.sql import crud
from sqlalchemy.testing import assert_raises
from sqlalchemy.testing import assert_raises_message
from sqlalchemy.testing import AssertsCompiledSQL
+from sqlalchemy.testing import config
from sqlalchemy.testing import eq_
from sqlalchemy.testing import expect_raises_message
from sqlalchemy.testing import expect_warnings
from sqlalchemy.testing import fixtures
+from sqlalchemy.testing.provision import normalize_sequence
class ORMExpr:
@@ -143,7 +145,13 @@ class InsertTest(_InsertTestBase, fixtures.TablesTest, AssertsCompiledSQL):
)
def test_insert_literal_binds_sequence_notimplemented(self):
- table = Table("x", MetaData(), Column("y", Integer, Sequence("y_seq")))
+ table = Table(
+ "x",
+ MetaData(),
+ Column(
+ "y", Integer, normalize_sequence(config, Sequence("y_seq"))
+ ),
+ )
dialect = default.DefaultDialect()
dialect.supports_sequences = True
@@ -407,7 +415,12 @@ class InsertTest(_InsertTestBase, fixtures.TablesTest, AssertsCompiledSQL):
t1 = Table(
"t",
m,
- Column("id", Integer, Sequence("id_seq"), primary_key=True),
+ Column(
+ "id",
+ Integer,
+ normalize_sequence(config, Sequence("id_seq")),
+ primary_key=True,
+ ),
Column("data", String),
)
@@ -428,7 +441,12 @@ class InsertTest(_InsertTestBase, fixtures.TablesTest, AssertsCompiledSQL):
t1 = Table(
"t",
m,
- Column("id", Integer, Sequence("id_seq"), primary_key=True),
+ Column(
+ "id",
+ Integer,
+ normalize_sequence(config, Sequence("id_seq")),
+ primary_key=True,
+ ),
Column("data", String),
)
@@ -453,7 +471,9 @@ class InsertTest(_InsertTestBase, fixtures.TablesTest, AssertsCompiledSQL):
"t",
m,
Column("id", Integer, primary_key=True),
- Column("counter", Sequence("counter_seq")),
+ Column(
+ "counter", normalize_sequence(config, Sequence("counter_seq"))
+ ),
Column("data", String),
)
@@ -534,7 +554,12 @@ class InsertTest(_InsertTestBase, fixtures.TablesTest, AssertsCompiledSQL):
t1 = Table(
"t",
m,
- Column("id", Integer, Sequence("id_seq"), primary_key=True),
+ Column(
+ "id",
+ Integer,
+ normalize_sequence(config, Sequence("id_seq")),
+ primary_key=True,
+ ),
Column("data", String),
)
diff --git a/test/sql/test_insert_exec.py b/test/sql/test_insert_exec.py
index 429ebf163..d9dac75b3 100644
--- a/test/sql/test_insert_exec.py
+++ b/test/sql/test_insert_exec.py
@@ -17,11 +17,13 @@ from sqlalchemy import testing
from sqlalchemy import VARCHAR
from sqlalchemy.engine import cursor as _cursor
from sqlalchemy.testing import assert_raises_message
+from sqlalchemy.testing import config
from sqlalchemy.testing import eq_
from sqlalchemy.testing import expect_raises_message
from sqlalchemy.testing import fixtures
from sqlalchemy.testing import is_
from sqlalchemy.testing import mock
+from sqlalchemy.testing.provision import normalize_sequence
from sqlalchemy.testing.schema import Column
from sqlalchemy.testing.schema import Table
@@ -268,7 +270,9 @@ class InsertExecTest(fixtures.TablesTest):
Column(
"id",
Integer,
- Sequence("t4_id_seq", optional=True),
+ normalize_sequence(
+ config, Sequence("t4_id_seq", optional=True)
+ ),
primary_key=True,
),
Column("foo", String(30), primary_key=True),
@@ -296,7 +300,7 @@ class InsertExecTest(fixtures.TablesTest):
Column(
"id",
Integer,
- Sequence("t4_id_seq"),
+ normalize_sequence(config, Sequence("t4_id_seq")),
primary_key=True,
),
Column("foo", String(30)),
@@ -471,7 +475,7 @@ class TableInsertTest(fixtures.TablesTest):
Column(
"id",
Integer,
- Sequence("t_id_seq"),
+ normalize_sequence(config, Sequence("t_id_seq")),
primary_key=True,
),
Column("data", String(50)),
@@ -547,7 +551,11 @@ class TableInsertTest(fixtures.TablesTest):
self._test(
connection,
t.insert().values(
- id=func.next_value(Sequence("t_id_seq")), data="data", x=5
+ id=func.next_value(
+ normalize_sequence(config, Sequence("t_id_seq"))
+ ),
+ data="data",
+ x=5,
),
(testing.db.dialect.default_sequence_base, "data", 5),
)
diff --git a/test/sql/test_returning.py b/test/sql/test_returning.py
index c26f825c2..32d4c7740 100644
--- a/test/sql/test_returning.py
+++ b/test/sql/test_returning.py
@@ -591,7 +591,7 @@ class SequenceReturningTest(fixtures.TablesTest):
@classmethod
def define_tables(cls, metadata):
- seq = Sequence("tid_seq")
+ seq = provision.normalize_sequence(config, Sequence("tid_seq"))
Table(
"returning_tbl",
metadata,
diff --git a/test/sql/test_selectable.py b/test/sql/test_selectable.py
index 72c8dc28d..05847b1d0 100644
--- a/test/sql/test_selectable.py
+++ b/test/sql/test_selectable.py
@@ -50,6 +50,7 @@ from sqlalchemy.testing import assert_raises
from sqlalchemy.testing import assert_raises_message
from sqlalchemy.testing import AssertsCompiledSQL
from sqlalchemy.testing import AssertsExecutionResults
+from sqlalchemy.testing import config
from sqlalchemy.testing import eq_
from sqlalchemy.testing import fixtures
from sqlalchemy.testing import in_
@@ -57,6 +58,7 @@ from sqlalchemy.testing import is_
from sqlalchemy.testing import is_not
from sqlalchemy.testing import ne_
from sqlalchemy.testing.assertions import expect_raises_message
+from sqlalchemy.testing.provision import normalize_sequence
metadata = MetaData()
@@ -2705,7 +2707,9 @@ class ReduceTest(fixtures.TestBase, AssertsExecutionResults):
Column(
"person_id",
Integer,
- Sequence("person_id_seq", optional=True),
+ normalize_sequence(
+ config, Sequence("person_id_seq", optional=True)
+ ),
primary_key=True,
),
Column("name", String(50)),
diff --git a/test/sql/test_sequences.py b/test/sql/test_sequences.py
index 457aeb960..8acf030d0 100644
--- a/test/sql/test_sequences.py
+++ b/test/sql/test_sequences.py
@@ -10,6 +10,7 @@ from sqlalchemy.schema import CreateSequence
from sqlalchemy.schema import DropSequence
from sqlalchemy.sql import select
from sqlalchemy.testing import assert_raises_message
+from sqlalchemy.testing import config
from sqlalchemy.testing import eq_
from sqlalchemy.testing import fixtures
from sqlalchemy.testing import is_false
@@ -18,6 +19,7 @@ from sqlalchemy.testing.assertions import expect_deprecated
from sqlalchemy.testing.assertsql import AllOf
from sqlalchemy.testing.assertsql import CompiledSQL
from sqlalchemy.testing.assertsql import EachOf
+from sqlalchemy.testing.provision import normalize_sequence
from sqlalchemy.testing.schema import Column
from sqlalchemy.testing.schema import Table
@@ -26,75 +28,104 @@ class SequenceDDLTest(fixtures.TestBase, testing.AssertsCompiledSQL):
__dialect__ = "default"
__backend__ = True
- def test_create_drop_ddl(self):
- self.assert_compile(
- CreateSequence(Sequence("foo_seq")),
- "CREATE SEQUENCE foo_seq START WITH 1",
- )
-
- self.assert_compile(
- CreateSequence(Sequence("foo_seq", start=5)),
- "CREATE SEQUENCE foo_seq START WITH 5",
- )
-
- self.assert_compile(
- CreateSequence(Sequence("foo_seq", increment=2)),
- "CREATE SEQUENCE foo_seq INCREMENT BY 2 START WITH 1",
- )
-
- self.assert_compile(
- CreateSequence(Sequence("foo_seq", increment=2, start=5)),
- "CREATE SEQUENCE foo_seq INCREMENT BY 2 START WITH 5",
- )
-
- self.assert_compile(
- CreateSequence(
- Sequence("foo_seq", increment=2, start=0, minvalue=0)
- ),
- "CREATE SEQUENCE foo_seq INCREMENT BY 2 START WITH 0 MINVALUE 0",
- )
-
- self.assert_compile(
- CreateSequence(
- Sequence("foo_seq", increment=2, start=1, maxvalue=5)
- ),
- "CREATE SEQUENCE foo_seq INCREMENT BY 2 START WITH 1 MAXVALUE 5",
- )
-
- self.assert_compile(
- CreateSequence(
- Sequence("foo_seq", increment=2, start=1, nomaxvalue=True)
- ),
- "CREATE SEQUENCE foo_seq INCREMENT BY 2 START WITH 1 NO MAXVALUE",
- )
-
- self.assert_compile(
- CreateSequence(
- Sequence("foo_seq", increment=2, start=0, nominvalue=True)
- ),
- "CREATE SEQUENCE foo_seq INCREMENT BY 2 START WITH 0 NO MINVALUE",
- )
-
- self.assert_compile(
- CreateSequence(
- Sequence("foo_seq", start=1, maxvalue=10, cycle=True)
- ),
- "CREATE SEQUENCE foo_seq START WITH 1 MAXVALUE 10 CYCLE",
- )
-
- self.assert_compile(
- CreateSequence(Sequence("foo_seq", cache=1000, order=True)),
- "CREATE SEQUENCE foo_seq START WITH 1 CACHE 1000 ORDER",
- )
-
+ @testing.combinations(
+ (Sequence("foo_seq"), ""),
+ (Sequence("foo_seq", start=5), "START WITH 5"),
+ (Sequence("foo_seq", increment=2), "INCREMENT BY 2"),
+ (
+ Sequence("foo_seq", increment=2, start=5),
+ "INCREMENT BY 2 START WITH 5",
+ ),
+ (
+ Sequence("foo_seq", increment=2, start=0, minvalue=0),
+ "INCREMENT BY 2 START WITH 0 MINVALUE 0",
+ ),
+ (
+ Sequence("foo_seq", increment=2, start=1, maxvalue=5),
+ "INCREMENT BY 2 START WITH 1 MAXVALUE 5",
+ ),
+ (
+ Sequence("foo_seq", increment=2, start=1, nomaxvalue=True),
+ "INCREMENT BY 2 START WITH 1 NO MAXVALUE",
+ ),
+ (
+ Sequence("foo_seq", increment=2, start=0, nominvalue=True),
+ "INCREMENT BY 2 START WITH 0 NO MINVALUE",
+ ),
+ (
+ Sequence("foo_seq", start=1, maxvalue=10, cycle=True),
+ "START WITH 1 MAXVALUE 10 CYCLE",
+ ),
+ (
+ Sequence("foo_seq", cache=1000, order=True),
+ "CACHE 1000 ORDER",
+ ),
+ (Sequence("foo_seq", order=True), "ORDER"),
+ (Sequence("foo_seq", minvalue=42), "MINVALUE 42"),
+ (Sequence("foo_seq", minvalue=-42), "MINVALUE -42"),
+ (
+ Sequence("foo_seq", minvalue=42, increment=2),
+ "INCREMENT BY 2 MINVALUE 42",
+ ),
+ (
+ Sequence("foo_seq", minvalue=-42, increment=2),
+ "INCREMENT BY 2 MINVALUE -42",
+ ),
+ (
+ Sequence("foo_seq", minvalue=42, increment=-2),
+ "INCREMENT BY -2 MINVALUE 42",
+ ),
+ (
+ Sequence("foo_seq", minvalue=-42, increment=-2),
+ "INCREMENT BY -2 MINVALUE -42",
+ ),
+ (Sequence("foo_seq", maxvalue=99), "MAXVALUE 99"),
+ (Sequence("foo_seq", maxvalue=-99), "MAXVALUE -99"),
+ (
+ Sequence("foo_seq", maxvalue=99, increment=2),
+ "INCREMENT BY 2 MAXVALUE 99",
+ ),
+ (
+ Sequence("foo_seq", maxvalue=99, increment=-2),
+ "INCREMENT BY -2 MAXVALUE 99",
+ ),
+ (
+ Sequence("foo_seq", maxvalue=-99, increment=-2),
+ "INCREMENT BY -2 MAXVALUE -99",
+ ),
+ (
+ Sequence("foo_seq", minvalue=42, maxvalue=99),
+ "MINVALUE 42 MAXVALUE 99",
+ ),
+ (
+ Sequence("foo_seq", minvalue=42, maxvalue=99, increment=2),
+ "INCREMENT BY 2 MINVALUE 42 MAXVALUE 99",
+ ),
+ (
+ Sequence("foo_seq", minvalue=-42, maxvalue=-9, increment=2),
+ "INCREMENT BY 2 MINVALUE -42 MAXVALUE -9",
+ ),
+ (
+ Sequence("foo_seq", minvalue=42, maxvalue=99, increment=-2),
+ "INCREMENT BY -2 MINVALUE 42 MAXVALUE 99",
+ ),
+ (
+ Sequence("foo_seq", minvalue=-42, maxvalue=-9, increment=-2),
+ "INCREMENT BY -2 MINVALUE -42 MAXVALUE -9",
+ ),
+ )
+ def test_create_ddl(self, sequence, sql):
+ before = sequence.start
self.assert_compile(
- CreateSequence(Sequence("foo_seq", order=True)),
- "CREATE SEQUENCE foo_seq START WITH 1 ORDER",
+ CreateSequence(sequence),
+ ("CREATE SEQUENCE foo_seq " + sql).strip(),
)
+ eq_(sequence.start, before)
+ def test_drop_ddl(self):
self.assert_compile(
CreateSequence(Sequence("foo_seq"), if_not_exists=True),
- "CREATE SEQUENCE IF NOT EXISTS foo_seq START WITH 1",
+ "CREATE SEQUENCE IF NOT EXISTS foo_seq",
)
self.assert_compile(
@@ -113,7 +144,7 @@ class SequenceExecTest(fixtures.TestBase):
@classmethod
def setup_test_class(cls):
- cls.seq = Sequence("my_sequence")
+ cls.seq = normalize_sequence(config, Sequence("my_sequence"))
cls.seq.create(testing.db)
@classmethod
@@ -127,12 +158,12 @@ class SequenceExecTest(fixtures.TestBase):
assert ret >= testing.db.dialect.default_sequence_base
def test_execute(self, connection):
- s = Sequence("my_sequence")
+ s = normalize_sequence(config, Sequence("my_sequence"))
self._assert_seq_result(connection.scalar(s))
def test_execute_deprecated(self, connection):
- s = Sequence("my_sequence", optional=True)
+ s = normalize_sequence(config, Sequence("my_sequence", optional=True))
with expect_deprecated(
r"Using the .execute\(\) method to invoke a "
@@ -145,27 +176,27 @@ class SequenceExecTest(fixtures.TestBase):
"""test dialect executes a Sequence, returns nextval, whether
or not "optional" is set"""
- s = Sequence("my_sequence", optional=True)
+ s = normalize_sequence(config, Sequence("my_sequence", optional=True))
self._assert_seq_result(connection.scalar(s))
def test_execute_next_value(self, connection):
"""test func.next_value().execute()/.scalar() works
with connectionless execution."""
- s = Sequence("my_sequence")
+ s = normalize_sequence(config, Sequence("my_sequence"))
self._assert_seq_result(connection.scalar(s.next_value()))
def test_execute_optional_next_value(self, connection):
"""test func.next_value().execute()/.scalar() works
with connectionless execution."""
- s = Sequence("my_sequence", optional=True)
+ s = normalize_sequence(config, Sequence("my_sequence", optional=True))
self._assert_seq_result(connection.scalar(s.next_value()))
def test_func_embedded_select(self, connection):
"""test can use next_value() in select column expr"""
- s = Sequence("my_sequence")
+ s = normalize_sequence(config, Sequence("my_sequence"))
self._assert_seq_result(connection.scalar(select(s.next_value())))
@testing.requires.sequences_in_other_clauses
@@ -177,7 +208,7 @@ class SequenceExecTest(fixtures.TestBase):
t1 = Table("t", metadata, Column("x", Integer))
t1.create(testing.db)
connection.execute(t1.insert(), [{"x": 1}, {"x": 300}, {"x": 301}])
- s = Sequence("my_sequence")
+ s = normalize_sequence(config, Sequence("my_sequence"))
eq_(
list(
connection.execute(t1.select().where(t1.c.x > s.next_value()))
@@ -196,7 +227,7 @@ class SequenceExecTest(fixtures.TestBase):
Column("x", Integer),
)
t1.create(testing.db)
- s = Sequence("my_sequence")
+ s = normalize_sequence(config, Sequence("my_sequence"))
connection.execute(t1.insert().values(x=s.next_value()))
self._assert_seq_result(connection.scalar(t1.select()))
@@ -212,7 +243,9 @@ class SequenceExecTest(fixtures.TestBase):
Column("x", Integer, primary_key=True),
implicit_returning=False,
)
- s = Sequence("my_sequence_here", metadata=metadata)
+ s = normalize_sequence(
+ config, Sequence("my_sequence_here", metadata=metadata)
+ )
conn = connection
t1.create(conn)
@@ -245,7 +278,12 @@ class SequenceExecTest(fixtures.TestBase):
t1 = Table(
"t",
metadata,
- Column("x", Integer, Sequence("my_seq"), primary_key=True),
+ Column(
+ "x",
+ Integer,
+ normalize_sequence(config, Sequence("my_seq")),
+ primary_key=True,
+ ),
Column("data", String(50)),
implicit_returning=_implicit_returning,
)
@@ -302,7 +340,12 @@ class SequenceExecTest(fixtures.TestBase):
t1 = Table(
"t",
metadata,
- Column("x", Integer, Sequence("my_seq"), primary_key=True),
+ Column(
+ "x",
+ Integer,
+ normalize_sequence(config, Sequence("my_seq")),
+ primary_key=True,
+ ),
Column("data", String(50)),
implicit_returning=_implicit_returning,
)
@@ -339,7 +382,7 @@ class SequenceExecTest(fixtures.TestBase):
"""test inserted_primary_key contains the result when
pk_col=next_value(), when implicit returning is used."""
- s = Sequence("my_sequence")
+ s = normalize_sequence(config, Sequence("my_sequence"))
t1 = Table(
"t",
metadata,
@@ -366,6 +409,7 @@ class SequenceTest(fixtures.TestBase, testing.AssertsCompiledSQL):
(Sequence("foo_seq", increment=5),),
)
def test_start_increment(self, seq):
+ seq = normalize_sequence(config, seq)
seq.create(testing.db)
try:
with testing.db.connect() as conn:
@@ -384,7 +428,7 @@ class SequenceTest(fixtures.TestBase, testing.AssertsCompiledSQL):
"""test next_value() used on non-sequence platform
raises NotImplementedError."""
- s = Sequence("my_seq")
+ s = normalize_sequence(config, Sequence("my_seq"))
d = sqlite.dialect()
assert_raises_message(
NotImplementedError,
@@ -394,7 +438,7 @@ class SequenceTest(fixtures.TestBase, testing.AssertsCompiledSQL):
)
def test_checkfirst_sequence(self, connection):
- s = Sequence("my_sequence")
+ s = normalize_sequence(config, Sequence("my_sequence"))
s.create(connection, checkfirst=False)
assert self._has_sequence(connection, "my_sequence")
s.create(connection, checkfirst=True)
@@ -414,7 +458,7 @@ class SequenceTest(fixtures.TestBase, testing.AssertsCompiledSQL):
def test_checkfirst_table(self, connection):
m = MetaData()
- s = Sequence("my_sequence")
+ s = normalize_sequence(config, Sequence("my_sequence"))
t = Table("t", m, Column("c", Integer, s, primary_key=True))
t.create(connection, checkfirst=False)
assert self._has_sequence(connection, "my_sequence")
@@ -426,9 +470,9 @@ class SequenceTest(fixtures.TestBase, testing.AssertsCompiledSQL):
@testing.provide_metadata
def test_table_overrides_metadata_create(self, connection):
metadata = self.metadata
- Sequence("s1", metadata=metadata)
- s2 = Sequence("s2", metadata=metadata)
- s3 = Sequence("s3")
+ normalize_sequence(config, Sequence("s1", metadata=metadata))
+ s2 = normalize_sequence(config, Sequence("s2", metadata=metadata))
+ s3 = normalize_sequence(config, Sequence("s3"))
t = Table("t", metadata, Column("c", Integer, s3, primary_key=True))
assert s3.metadata is metadata
@@ -463,8 +507,8 @@ class SequenceTest(fixtures.TestBase, testing.AssertsCompiledSQL):
Integer,
autoincrement=True,
primary_key=True,
- default=Sequence(
- "my_sequence", metadata=self.metadata
+ default=normalize_sequence(
+ config, Sequence("my_sequence", metadata=self.metadata)
).next_value(),
),
)
@@ -477,7 +521,9 @@ class SequenceTest(fixtures.TestBase, testing.AssertsCompiledSQL):
@testing.provide_metadata
def test_shared_sequence(self, connection):
# test case for #6071
- common_seq = Sequence("common_sequence", metadata=self.metadata)
+ common_seq = normalize_sequence(
+ config, Sequence("common_sequence", metadata=self.metadata)
+ )
Table(
"table_1",
self.metadata,
@@ -511,7 +557,9 @@ class SequenceTest(fixtures.TestBase, testing.AssertsCompiledSQL):
is_false(testing.db.dialect.has_table(connection, "table_2"))
def test_next_value_type(self):
- seq = Sequence("my_sequence", data_type=BigInteger)
+ seq = normalize_sequence(
+ config, Sequence("my_sequence", data_type=BigInteger)
+ )
assert isinstance(seq.next_value().type, BigInteger)
@@ -528,7 +576,7 @@ class TableBoundSequenceTest(fixtures.TablesTest):
Column(
"cart_id",
Integer,
- Sequence("cart_id_seq"),
+ normalize_sequence(config, Sequence("cart_id_seq")),
primary_key=True,
autoincrement=False,
),
@@ -544,13 +592,15 @@ class TableBoundSequenceTest(fixtures.TablesTest):
Column(
"obj_id",
Integer,
- Sequence("obj_id_seq"),
+ normalize_sequence(config, Sequence("obj_id_seq")),
),
Column("name", String(128)),
Column(
"id",
Integer,
- Sequence("Manager_id_seq", optional=True),
+ normalize_sequence(
+ config, Sequence("Manager_id_seq", optional=True)
+ ),
primary_key=True,
),
implicit_returning=implicit_returning,
@@ -647,7 +697,7 @@ class SequenceAsServerDefaultTest(
def define_tables(cls, metadata):
m = metadata
- s = Sequence("t_seq", metadata=m)
+ s = normalize_sequence(config, Sequence("t_seq", metadata=m))
Table(
"t_seq_test",
m,
@@ -655,7 +705,7 @@ class SequenceAsServerDefaultTest(
Column("data", String(50)),
)
- s2 = Sequence("t_seq_2", metadata=m)
+ s2 = normalize_sequence(config, Sequence("t_seq_2", metadata=m))
Table(
"t_seq_test_2",
m,