[GROOVY-6403] SpreadMap is horribly broken - isEmpty, containsKey, keySet etc. all fail

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.8.9, 2.2.0-beta-1, 2.1.7
• Fix Version/s: 2.2.0-rc-3
• Assignee: Paul King
• Reporter: Tim Tiemens
• Created: 26/Oct/13 22:04
• Updated: 15/Oct/15 18:19
• Resolved: 03/Nov/13 03:44

Description

groovy.lang.SpreadMap implementation is incorrect.
Of the Map contract, it correctly implements:
size() get() equals() hashCode()

The rest of the Map methods have incorrect (missing) implementations:
isEmpty() keySet() containsKey() ..etc.

Also, List.toSpreadMap() is broken since it returns an instance of SpreadMap.