SQLAlchemy deprecation in SqlAlchemyDictTypeTest

[hammerhead]

Summary of the changes / Why this is an improvement

@amotl, this is an attempt at fixing the deprecation warning in SqlAlchemyDictTypeTest. I'm not 100% sure about the test_update_with_dict_column, this would need manual verification.

Checklist

• CLA is signed

[hammerhead]

This looks to be valid:

from crate.client.sqlalchemy import types
from sqlalchemy.orm import Session
import sqlalchemy as sa

engine = sa.create_engine("crate://")
metadata_obj = sa.MetaData()

mytable = sa.Table(
    "characters",
    metadata_obj,
    sa.Column("id", sa.String, primary_key=True),
    sa.Column("details", types.Object),
    autoload_with=engine,
)

session = Session(bind=engine)
session.execute(
    mytable.update()
    .where(mytable.c.id == "1")
    .values({mytable.c.details["x"]: "Trillian"})
)

Resulting query as per sys.jobs_log: UPDATE characters SET details['x'] = ? WHERE characters.id = ?

[hammerhead]

The previous test might have been wrong, using values({"data['x']": "Trillian"}) throws an exception in a real-world example without any mocking: TypeError: Object of type BindParameter is not JSON serializable

[amotl]

Oh. Thanks for spotting and investigating diligently.

[amotl]

throws an exception in a real-world example without any mocking

So, if that did not work before anyway, we will not have to announce anything as "breaking change", right? When looking at this again, I bet this was the spot which may have tripped me in the past already. Thanks again!

[amotl]

I've added 2c6db67 to make the tests succeed on my end.