groovy

ConfigSlurper Fails when ordering of Nested Elements changes

Details

  • Type: Bug Bug
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: 1.5.6, 1.6-rc-2
  • Fix Version/s: None
  • Component/s: groovy-jdk
  • Labels:
    None
  • Environment:
    Windows
  • Testcase included:
    yes
  • Number of attachments :
    1

Description

Config Slurper fails with nested elements depending on the order specified. For example:

This Fails
        def config = new ConfigSlurper().parse('''
        foo {
            fruit.citrus="orange"
            bar {
                fruit.citrus="lemon"
            }
        }
        ''')

        assertEquals "lemon", config.foo.bar.fruit.citrus
        assertEquals "orange", config.foo.fruit.citrus
This Passes
        def config = new ConfigSlurper().parse('''
        foo {
            bar {
                fruit.citrus="lemon"
            }
            fruit.citrus="orange"
        }
        ''')

        assertEquals "lemon", config.foo.bar.fruit.citrus
        assertEquals "orange", config.foo.fruit.citrus

Activity

Ascending order - Click to sort in descending order
Guillaume Laforge made changes -
Field Original Value New Value
Assignee Graeme Rocher [ graemerocher ]
Hide
Permalink
Paul King added a comment -

confirming that this still fails on master (currently 2.2.0-SNAPSHOT)

Show
Paul King added a comment - confirming that this still fails on master (currently 2.2.0-SNAPSHOT)
Hide
Permalink
Paul King added a comment -

The other workaround is to nest the different levels explicitly with closures - which is order independent:

def config = new ConfigSlurper().parse('''
foo {
  fruit {
    citrus="orange"
  }
  bar {
    fruit {
      citrus="lemon"
    }
  }
}
''')
assert config.foo.bar.fruit.citrus == 'lemon'
assert config.foo.fruit.citrus == 'orange'
Show
Paul King added a comment - The other workaround is to nest the different levels explicitly with closures - which is order independent: def config = new ConfigSlurper().parse(''' foo { fruit { citrus= "orange" } bar { fruit { citrus= "lemon" } } } ''') assert config.foo.bar.fruit.citrus == 'lemon' assert config.foo.fruit.citrus == 'orange'
Pascal Schumacher made changes -
Assignee Graeme Rocher [ graemerocher ]

People

  • Assignee:
    Unassigned
    Reporter:
    Paul Fairless
Vote (8)
Watch (4)

Dates

  • Created:
    Updated: