Skip to content

Commit e580196

Browse files
committed
Merge branch '3.1.x' into 3.2.x
2 parents def66cc + 00d9eac commit e580196

File tree

1 file changed

+26
-0
lines changed

1 file changed

+26
-0
lines changed

grails-core/src/test/groovy/org/grails/config/PropertySourcesConfigSpec.groovy

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@ package org.grails.config
22

33
import org.springframework.core.env.MapPropertySource
44
import org.springframework.core.env.MutablePropertySources
5+
import spock.lang.Ignore
6+
import spock.lang.Issue
57
import spock.lang.Specification
68

79
/*
@@ -48,4 +50,28 @@ class PropertySourcesConfigSpec extends Specification {
4850

4951

5052
}
53+
54+
/*
55+
56+
We need to settle on whether the following is a bug or not.
57+
There are some tests in ConfigMapSpec that indirectly assert some
58+
behavior that I think would be inconsistent with making the following
59+
test pass.
60+
61+
*/
62+
@Ignore
63+
@Issue('grails/grails-core#10188')
64+
void 'test replacing nested property values'() {
65+
given: 'a PropertySourcesConfig'
66+
def cfg = new PropertySourcesConfig()
67+
68+
when: 'a nested property is assigned a value'
69+
cfg.foo.bar = ['one': '1']
70+
71+
and: 'later is assigned a different value'
72+
cfg.foo.bar = ['two': '2']
73+
74+
then: 'the second value is not merged with the first value'
75+
cfg.foo.bar == ['two': '2']
76+
}
5177
}

0 commit comments

Comments
 (0)