quoting for CTE names

[sqlalchemy-bot]

Migrated issue, originally created by Michael Bayer (@zzzeek)

 def test_named_alias_no_quote(self):
        cte = select([literal(1).label("id")]).cte(name='CTE')

        s1 = select([cte.c.id]).alias(name="no_quotes")

        s = select([s1])
        self.assert_compile(
            s,
            'WITH "CTE" AS (SELECT :param_1 AS id) '
            'SELECT no_quotes.id FROM '
            '(SELECT "CTE".id AS id FROM "CTE") AS no_quotes'
        )

    def test_named_alias_quote(self):
        cte = select([literal(1).label("id")]).cte(name='CTE')

        s1 = select([cte.c.id]).alias(name="Quotes Required")

        s = select([s1])
        self.assert_compile(
            s,
            'WITH "CTE" AS (SELECT :param_1 AS id) '
            'SELECT "Quotes Required".id FROM '
            '(SELECT "CTE".id AS id FROM "CTE") AS "Quotes Required"'
        )

[sqlalchemy-bot]

Michael Bayer (@zzzeek) wrote:

Quote cte alias if needed

Fixed bug where CTE expressions would not have their name or alias name
quoted when the given name is case sensitive or otherwise requires quoting.
Pull request courtesy Eric Atkin.

Fixes: #4197
Change-Id: Ib8573e82b9a1ca94b50c7c5d73ee98b79465d689
Pull-request: zzzeek/sqlalchemy#426

→ b6e4a16

[sqlalchemy-bot]

Changes by Michael Bayer (@zzzeek):

• changed status to closed