Extend the reflection canary with a TIME column

The canary test in test_reflect_column_types covered every standard
SQLAlchemy type except Time. TIME columns round-trip through the
storage path (covered in test_time_and_reflection), but the metadata
path — what inspect(engine).get_columns() reports — had no guardrail
against a future pysqlite change demoting the column to NullType.

Add a Time column to the fixture and a bounded isinstance assertion
that tolerates pysqlite's version-dependent TIME mapping (Time,
String, or DateTime). The existing no-NullType sweep at the end of
the test remains the primary contract; the union assertion just
documents the concrete shape we observe today.

Does not add a _DqliteTime colspec — reflection via the pysqlite
parent is the deliberate current design.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: antoineleclair

tests/integration/test_reflect_column_types.py

@@ -22,6 +22,7 @@
     String,
     Table,
     Text,
+    Time,
     create_engine,
     inspect,
     types,
@@ -48,6 +49,7 @@ def engine(self, engine_url: str) -> Generator[Engine]:
             Column("ratio", Float),
             Column("created", DateTime),
             Column("birthday", Date),
+            Column("start_time", Time),
             Column("flag", Boolean),
         )
         md.create_all(engine)
@@ -69,6 +71,11 @@ def test_reflects_standard_types(self, engine: Engine) -> None:
         assert isinstance(cols["ratio"]["type"], types.Float | types.Numeric)
         assert isinstance(cols["created"]["type"], types.DateTime)
         assert isinstance(cols["birthday"]["type"], types.Date)
+        # TIME may reflect to Time, String, or DateTime depending on the
+        # pysqlite version's ischema_names — the union accommodates the
+        # documented uncertainty. The no-NullType assertion below still
+        # applies, which is the main contract we care about.
+        assert isinstance(cols["start_time"]["type"], types.Time | types.String | types.DateTime)
         assert isinstance(cols["flag"]["type"], BOOLEAN_LIKE)
 
         # No column landed in NullType.