[PHPBB3-9701] Enable notices in unit tests Created: 02/Jul/10 Updated: 12/Jul/10 Resolved: 08/Jul/10 |
|
| Status: | Unverified Fix |
| Project: | phpBB3 |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 3.0.8-RC1 |
| Type: | Task | Priority: | Blocker |
| Reporter: | bantu | Assignee: | naderman |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||
| Fix URL: | http://github.com/naderman/phpbb3/tree/ticket/9701 | ||||||||
| Description |
|
The unit tests in develop-olympus do not display notices. Notices should be enabled to make sure we don't miss anything. |
| Comments |
| Comment by naderman [ 08/Jul/10 ] |
|
They are displayed just fine for me. Can you please run a test with $x[ABC] for example and see if it really doesn't show the notice? |
| Comment by naderman [ 08/Jul/10 ] |
|
Well I do get the problem in that particular test. Something else must be at fault here. |
| Comment by naderman [ 08/Jul/10 ] |
|
Alright finally figured out what this was caused by: When the template's display() method triggered an error / threw an exception the error level would not be reset afterwards. Added a catch clause now where it gets reset. That should work. Not an issue in 3.1 since that should have NOTICE-free templates. |