Skip to content

test(evaluators): IsEmpty and IsNotEmpty on 0, False and whitespace #407

Closed

Description

@refeed

Problem

IsEmpty treats only None, "", [] and {} as empty, so 0 and False count as "not empty". That behavior is intended, but no test pins it, and " " (whitespace) is untested too.

What to change

Add 0, False, " " and {"a": None} to the parametrized cases in tests/core/evaluators/test_is_empty.py and test_is_not_empty.py.

Done when

Both files include those inputs with the expected verdicts.


Comment to claim this issue and wait to be assigned. Unclaimed or inactive for 3 days, it returns to the pool. Run pytest tests/ and black --line-length 120 before opening the PR.

Activity

  1. AK-Lmn commented on Oct 16, 2026

    @AK-Lmn
    Contributor

    Hi @refeed, please assign this issue to me. I have submitted PR #409 with the test coverage and all CI checks are passing.

  2. added a commit that references this issue on Oct 21, 2026
    b1c564c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions