[GROOVY-3959] TestNG script runner masks root cause of exception

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.6.4, 1.7.0
• Fix Version/s: 1.6.8, 1.7.1, 1.8-beta-1
• Assignee: Roshan Dawrani
• Reporter: Marcus Better
• Created: 29/Dec/09 11:05
• Updated: 07/Apr/10 23:46
• Resolved: 30/Dec/09 04:16

Description

The realRunTestNgTest() method of class org.codehaus.groovy.vmplugin.v5.TestNgUtils contains this code:

        } catch (ClassNotFoundException e) {
            throw new GroovyRuntimeException("Error running TestNG test.");
        }

This is very unhelpful because it hides the original exception e. It should be passed as root cause to the GroovyRuntimeException constructor.