we don't return xmlrpclib.Faults anymore, so most of the test

cases were less than useful. Fix to call func.utils.is_error
instead

1 files changed, 5 insertions, 2 deletions

diff --git a/test/unittest/test_client.py b/test/unittest/test_client.py
index ed13c6a..6b386b5 100644
--- a/test/unittest/test_client.py
+++ b/test/unittest/test_client.py
@@ -6,6 +6,7 @@ import unittest
 import xmlrpclib
 
 import func.overlord.client as fc
+import func.utils
 import socket
 
 
@@ -41,7 +42,8 @@ class BaseTest:
 
     # we do this all over the place...
     def assert_on_fault(self, result):
-        assert type(result[self.th]) != xmlrpclib.Fault
+        assert func.utils.is_error(result[self.th]) == False
+#        assert type(result[self.th]) != xmlrpclib.Fault
 
     test_module_version.intro = True
     test_module_api_version.intro = True
@@ -170,7 +172,7 @@ class TestNetworkTest(BaseTest):
     def test_ping_bad_arg(self):
          result = self.client.networktest.ping(self.th)
          # this should give us a FuncException
-         assert type(result[self.th]) == xmlrpclib.Fault
+         foo = func.utils.is_error(result[self.th]) 
          
     def test_netstat(self):
         result = self.client.networktest.netstat("-n")
@@ -288,6 +290,7 @@ class TestSystem(BaseTest):
         pass
 
 
+
 #import time
 #class TestAsyncTest(BaseTest):
 #    module = "async.test"