[GROOVY-8416] Map-based constructor generated by @Immutable works only with HashMap

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.4.13
• Fix Version/s: 2.5.0-beta-3
• Assignee: Paul King
• Reporter: Andreas Mayer
• Created: 14/Dec/17 16:30
• Updated: 25/May/21 15:59
• Resolved: 06/Mar/18 23:38

Description

If you call the map-based constructor of an @Immutable class with a non-HashMap, it will try to set the properties via setters, which causes a ReadOnlyPropertyException. Somehow it works with HashMap or subclasses like LinkedHashMap. Why?

import groovy.transform.Immutable

@Immutable
class Point {
	int x
	int y
}

def coordinates = [x: 1, y: 2]
assert coordinates instanceof LinkedHashMap

def p1 = new Point(coordinates)
assert p1.x == 1

def p2 = new Point(new HashMap(coordinates))
assert p2.x == 1

try {
	new Point(new TreeMap(coordinates))
} catch (e) {
	assert e instanceof ReadOnlyPropertyException
}

try {
	new Point(coordinates.asImmutable())
} catch (e) {
	assert e instanceof ReadOnlyPropertyException
}

Issue Links

• is related to: GROOVY-10108 Tuple constructor fails for an @Immutable class having a single Map property