No worries, Brett. When you say you found it was broken, do you mean that you also saw the NPE during integration tests or that something else was wrong so you needed add the temporary "return null" ?
tc, -john. On Tue, Jul 5, 2011 at 6:21 PM, Brett Porter <[email protected]> wrote: > On 06/07/2011, at 8:40 AM, John Fallows wrote: > > > Out of curiousity, what was the rationale for removing it and has that > issue > > been addressed, allowing us to re-enable the feature? > > > When I started making integration tests, I found it was broken, and I > wasn't sure if it had ever worked. Didn't have time to fix it then, so the > tests stayed disabled. > > Sorry, probably not the answer you were hoping for :) > > - Brett > > -- > Brett Porter > [email protected] > http://brettporter.wordpress.com/ > http://au.linkedin.com/in/brettporter > > > > > -- >|< Kaazing Corporation >|< John Fallows | CTO | +1.650.960.8148 444 Castro St, Suite 1100 | Mountain View, CA 94041, USA
