[Chandler-dev] don't eat tracebacks !

[Andi Vajda]

I noticed that several functional tests are eating tracebacks instead of 
reporting them, in code like this:

    try:
        collection = share.get()
    except:
        logger.Stop()
        logger.ReportFailure("Importing calendar: exception raised")
    else:
        App_ns.sidebarCollection.add(collection)
        User.idle()
        logger.Stop()
        logger.ReportPass("Importing calendar")

This is not so helpful with debugging. I added a new method to the QALogger, 
called ReportException which appends a traceback to the message before 
passing it on to ReportFailure.

    try:
        collection = share.get()
    except:
        logger.Stop()
        logger.ReportException("Importing calendar")
    else:
        App_ns.sidebarCollection.add(collection)
        User.idle()
        logger.Stop()
        logger.ReportPass("Importing calendar")

This is not yet checked in as it is part of another fix which remains to be 
approved for alpha 2 (or not).

Andi..
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

Open Source Applications Foundation "chandler-dev" mailing list
http://lists.osafoundation.org/mailman/listinfo/chandler-dev