Attachment 883778 Details for Bug 1084579 – patch cleaning up error reporting

[patch] patch cleaning up error reporting

0001-1084579-certify-returns-Test-plan-is-not-frozen-Faul.patch (text/plain), 1.68 KB, created by Greg Nichols on 2014-04-07 22:59:16 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Greg Nichols

Created: 2014-04-07 22:59:16 UTC

Size: 1.68 KB

patch

obsolete

>From 6c398f5967dc271d11c082adf3e9ff9dfd115964 Mon Sep 17 00:00:00 2001
>From: Greg Nichols <gnichols@redhat.com>
>Date: Mon, 7 Apr 2014 18:58:00 -0400
>Subject: [PATCH] 1084579 - certify returns 'Test plan is not frozen' Fault
> code: -32000
>
>---
> hwcert/catalog.py | 8 +++++---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
>diff --git a/hwcert/catalog.py b/hwcert/catalog.py
>index c598ceb..e671859 100644
>--- a/hwcert/catalog.py
>+++ b/hwcert/catalog.py
>@@ -274,7 +274,7 @@ class CatalogText(Catalog):
>                 return openTests
> 
>             except xmlrpclib.Fault, error:
>-                print "Error: could not get test plan from hardware catalog:"
>+                print "\nError: could not get test plan from hardware catalog:"
>                 if not self.handleRPCException(error):
>                     return None
>             except Exception, e:
>@@ -283,8 +283,9 @@ class CatalogText(Catalog):
>                 return None
> 
>     def handleRPCException(self, rpcException):
>-        print "    Fault code: %s" % rpcException.faultCode
>-        print "    Fault string: %s" % rpcException.faultString
>+        print rpcException.faultString
>+        if self.debug != Constants.off:
>+            print "(Fault code: %s)" % rpcException.faultCode
>         if rpcException.faultCode == 300: # bad login/password
>             self.password = None
>             self.login = None
>@@ -349,6 +350,7 @@ class CatalogText(Catalog):
>                     return int(id)
> 
>             except xmlrpclib.Fault, error:
>+                print ""
>                 if self.debug != Constants.off:
>                     stop = datetime.datetime.now()
>                     print "Search took %s" % (stop-start)
>-- 
>1.9.0
>

Actions: View | Diff

Attachments on bug 1084579: 883778