fix: return a tuple, not generator, from Table.bind()

ibis/expr/types/relations.py

@@ -671,7 +671,7 @@ def bind(self, *args: Any, **kwargs: Any) -> tuple[Value, ...]:
         dm = DerefMap.from_targets(self.op())
 
         bound = self._fast_bind(*args, **kwargs)
-        return (
+        return tuple(
             derefed.to_expr().name(name) if original is not derefed else original
             for name, original, derefed in zip(
                 (expr.get_name() for expr in bound),
@@ -5166,7 +5166,7 @@ def relocate(
     def window_by(self, time_col: str | ir.Value, /) -> WindowedTable:
         from ibis.expr.types.temporal_windows import WindowedTable
 
-        time_col = next(self.bind(time_col))
+        (time_col,) = self.bind(time_col)
 
         # validate time_col is a timestamp column
         if not isinstance(time_col, TimestampColumn):

ibis/tests/expr/test_table.py

@@ -2014,6 +2014,8 @@ def test_unbind_with_namespace():
 
 def test_table_bind():
     def eq(left, right):
+        assert isinstance(left, tuple)
+        assert isinstance(right, tuple)
         return all(a.equals(b) for a, b in zip(left, right))
 
     t = ibis.table({"a": "int", "b": "string"}, name="t")
@@ -2101,13 +2103,13 @@ def eq(left, right):
     assert eq(exprs, expected)
 
     # no args
-    assert tuple(t.bind()) == ()
+    assert t.bind() == ()
 
     def utter_failure(_):
         raise ValueError("¡moo!")
 
     with pytest.raises(ValueError, match="¡moo!"):
-        tuple(t.bind(foo=utter_failure))
+        t.bind(foo=utter_failure)
 
 
 # TODO: remove when dropna is fully deprecated