chiselwright/catalyst-plugin-errorcatcher — pull request #1 by felliott (2012-03-07 20:11:16+00:00)
Title: Test fixes for 04.trace.t

Hello!

Attached are two patches for your consideration:

1.) Catayst::Plugin::ErrorCatcher HEAD fails a test on my system. The last test in 04.trace.t, "'evil' has value 'two\nlines' in QUERY section", fails because HTTP::Request::Common::POST converts the \n to \r\n. Included is a simple patch that fixes that test.

2.) This is not a failing test, but when _has_param_section() is called with a BODY type, it was printing the test name as "QUERY params block exists".

All tests pass with Catalyst 5.90007. Unfortunately, Catalyst >=5.90008 has a bug that causes all the BODY tests to fail, which I've reported to RT.

Thank you for writing this module. I'm looking forward to using this in my application; I think it will come in really handy!

Cheers,
Fitz Elliott