From eb83f3abe450c87cee4cdb32638c3885d7a1e4c8 Mon Sep 17 00:00:00 2001 From: laughingman7743 Date: Sun, 7 Sep 2025 13:44:44 +0900 Subject: [PATCH] Fix engine fixture usage in test_reserved_words MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The engine fixture returns a tuple (engine, conn), so we need to unpack it like other tests do. This fixes the CI error where engine.dialect was called on a tuple instead of the engine object. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- tests/pyathena/sqlalchemy/test_base.py | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/pyathena/sqlalchemy/test_base.py b/tests/pyathena/sqlalchemy/test_base.py index 907b8f19..6d31ec8c 100644 --- a/tests/pyathena/sqlalchemy/test_base.py +++ b/tests/pyathena/sqlalchemy/test_base.py @@ -361,6 +361,7 @@ def test_select_offset_limit(self, engine): def test_reserved_words(self, engine): """Presto uses double quotes, not backticks""" + engine, conn = engine fake_table = Table("bernoulli", MetaData(), Column("current_catalog", types.String())) query = ( fake_table.select()