[jira] [Created] (TAP5-2267) Contributing ExceptionHandlerAssistant instances to RequestExceptionHandler is broken

2013-12-24 Thread Dragan Sahpaski (JIRA)
Dragan Sahpaski created TAP5-2267:
-

 Summary: Contributing ExceptionHandlerAssistant instances to 
RequestExceptionHandler is broken
 Key: TAP5-2267
 URL: https://issues.apache.org/jira/browse/TAP5-2267
 Project: Tapestry 5
  Issue Type: Bug
  Components: tapestry-core
Affects Versions: 5.4
Reporter: Dragan Sahpaski
Priority: Minor


DefaultRequestExceptionHandler fails with ClassCastException when an instance 
of ExceptionHandlerAssistant is contributed to RequestExceptionHandler.

This had been broken accidentally in commit 
cb3d4c853f47cccf9193c33cfd085d6ca27c8706 when implementing contributions of 
assistant classes that are autobuilt.

Although DefaultRequestExceptionHandlerTest covers contributions of 
ExceptionHandlerAssistant instances, it didn't fail because of an oversight in 
the test.

The patch resolves the bug (trivially) and improves existing tests in 
DefaultRequestExceptionHandlerTest which now fail without patching 
DefaultRequestExceptionHandler.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Updated] (TAP5-2267) Contributing ExceptionHandlerAssistant instances to RequestExceptionHandler is broken

2013-12-24 Thread Dragan Sahpaski (JIRA)

 [ 
https://issues.apache.org/jira/browse/TAP5-2267?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Dragan Sahpaski updated TAP5-2267:
--

Attachment: TAP5-2267.patch

 Contributing ExceptionHandlerAssistant instances to RequestExceptionHandler 
 is broken
 -

 Key: TAP5-2267
 URL: https://issues.apache.org/jira/browse/TAP5-2267
 Project: Tapestry 5
  Issue Type: Bug
  Components: tapestry-core
Affects Versions: 5.4
Reporter: Dragan Sahpaski
Priority: Minor
 Attachments: TAP5-2267.patch


 DefaultRequestExceptionHandler fails with ClassCastException when an instance 
 of ExceptionHandlerAssistant is contributed to RequestExceptionHandler.
 This had been broken accidentally in commit 
 cb3d4c853f47cccf9193c33cfd085d6ca27c8706 when implementing contributions of 
 assistant classes that are autobuilt.
 Although DefaultRequestExceptionHandlerTest covers contributions of 
 ExceptionHandlerAssistant instances, it didn't fail because of an oversight 
 in the test.  
 The patch resolves the bug (trivially) and improves existing tests in 
 DefaultRequestExceptionHandlerTest which now fail without patching 
 DefaultRequestExceptionHandler.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)