[GROOVY-7207] every() from DefaultGroovyMethods returns true for empty iterator/list

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Won't Fix
• Affects Version/s: 2.3.8, 2.4.0-beta-4
• Component/s: groovy-jdk
• Assignee: Unassigned
• Reporter: Dhiraj Mahapatro
• Created: 11/Dec/14 11:23
• Updated: 01/Dec/21 01:24
• Resolved: 19/Dec/14 19:47

Description

assert [].every()
assert [].every { it }
assert [].every { it > 1 }

Is this an expected behavior? In all the variants of every() in DefaultGroovyMethods the logic is simlar to

BooleanClosureWrapper bcw = new BooleanClosureWrapper(closure);
for (Iterator iter = InvokerHelper.asIterator(self); iter.hasNext();) {
    if (!bcw.call(iter.next())) {
        return false;
    }
}
return true;

So it defaults to true if !iter.hasNext(). Is this acceptable? Shouldn't it falsify based on empty List fails Groovy Truth?