incubator-ooo-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject [Bug 120603] [testuno]Test Create and Load Date and Time Field in Writer by UNO
Date Fri, 17 Aug 2012 06:45:02 GMT
https://issues.apache.org/ooo/show_bug.cgi?id=120603

aliuzhe@gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |aliuzhe@gmail.com

--- Comment #6 from aliuzhe@gmail.com ---
Thanks for your contribution. I has committed it at revision 1374159.
When I reviewed it, I found you catch some exceptions and ignore them. Think
about the situation. A bug leads the code in try block to throw an exception,
since you ignore the exception, your test will pass. That's bad.  Don't catch
the exceptions. Add throws Exception to your test method declaration.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

Mime
View raw message