[GROOVY-9896] Switch return statement transformation misses last case without break

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.5.14, 3.0.7, 4.0.0-alpha-2
• Fix Version/s: 2.5.15, 3.0.8, 4.0.0-alpha-3
• Assignee: Eric Milles
• Reporter: Eric Milles
• Created: 14/Jan/21 16:28
• Updated: 27/Aug/21 10:22
• Resolved: 02/Mar/21 15:19

Description

Consider the following:

def test(str) {
  switch(str) {
   case 'foo':
   case 'bar':
    'value'
  //break
  }
}
println test('foo')
println test('bar')
println test('baz')

When this script is executed, it prints "null" three times. If the break statement is added, it prints "value", "value", "null" as expected. ReturnAdder only transforms case blocks that end with break. This is possibly the reason GROOVY-4727 fails.