Skip to content

Commit

Permalink
fix(api): deferred values are not truthy
Browse files Browse the repository at this point in the history
  • Loading branch information
jcrist committed Jan 18, 2024
1 parent 1d06b87 commit 4533e4d
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 0 deletions.
5 changes: 5 additions & 0 deletions ibis/common/deferred.py
Original file line number Diff line number Diff line change
Expand Up @@ -97,6 +97,11 @@ def __getattr__(self, name):
def __iter__(self):
raise TypeError(f"{self.__class__.__name__!r} object is not iterable")

def __bool__(self):
raise TypeError(
f"The truth value of {self.__class__.__name__} objects is not defined"
)

def __getitem__(self, name):
return Deferred(Item(self, name))

Expand Down
8 changes: 8 additions & 0 deletions ibis/common/tests/test_deferred.py
Original file line number Diff line number Diff line change
Expand Up @@ -501,6 +501,14 @@ def test_deferred_is_not_iterable(obj):
next(obj)


@pytest.mark.parametrize("obj", [_, _.a, _.a.b[0]])
def test_deferred_is_not_truthy(obj):
with pytest.raises(
TypeError, match="The truth value of Deferred objects is not defined"
):
bool(obj)


def test_deferrable(table):
@deferrable
def f(a, b, c=3):
Expand Down

0 comments on commit 4533e4d

Please sign in to comment.