diff options
| author | Mike Bayer <mike_mp@zzzcomputing.com> | 2023-02-02 17:22:22 -0500 |
|---|---|---|
| committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2023-02-03 09:45:22 -0500 |
| commit | 92e3c21ea9b8192ff3d6ad856389186dfe8b3d3d (patch) | |
| tree | 186813cb7473ff04001bb788857cb4fe514fb1ad /test/orm/inheritance/test_basic.py | |
| parent | 0635235090c85e2c1a18676ca49652d2c2094925 (diff) | |
| download | sqlalchemy-92e3c21ea9b8192ff3d6ad856389186dfe8b3d3d.tar.gz | |
dont add non-server-side cols to returning for versioning
Fixed regression where using the :paramref:`_orm.Mapper.version_id_col`
feature with a regular Python-side incrementing column would fail to work
for SQLite and other databases that don't support "rowcount" with
"RETURNING", as "RETURNING" would be assumed for such columns even though
that's not what actually takes place.
Fixes: #9228
Change-Id: I6a1a7fa4d63e183fe4ef0fbfd3cb5cac03b26d78
Diffstat (limited to 'test/orm/inheritance/test_basic.py')
| -rw-r--r-- | test/orm/inheritance/test_basic.py | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/test/orm/inheritance/test_basic.py b/test/orm/inheritance/test_basic.py index 37368f3ad..02f352786 100644 --- a/test/orm/inheritance/test_basic.py +++ b/test/orm/inheritance/test_basic.py @@ -2101,8 +2101,7 @@ class VersioningTest(fixtures.MappedTest): Column("parent", Integer, ForeignKey("base.id")), ) - @testing.emits_warning(r".*updated rowcount") - @testing.requires.sane_rowcount_w_returning + @testing.requires.sane_rowcount def test_save_update(self): subtable, base, stuff = ( self.tables.subtable, @@ -2170,8 +2169,7 @@ class VersioningTest(fixtures.MappedTest): s2.subdata = "sess2 subdata" sess2.flush() - @testing.emits_warning(r".*(update|delete)d rowcount") - @testing.requires.sane_rowcount_w_returning + @testing.requires.sane_rowcount def test_delete(self): subtable, base = self.tables.subtable, self.tables.base |