A couple of things, the removal of the server.update() code wasn't good for performance, so we are (for now) putting it back. More importantly this set of codechanges goes through the unit tests we already had and makes sure they use top level API calls. While this may look trivial, it's not -- the top level calls ensure data is saved and read from disk as in the full application, so they are more exhaustive. As of this commit, all tests as committed pass.

1 files changed, 2 insertions, 0 deletions

diff --git a/cobbler/webui/CobblerWeb.py b/cobbler/webui/CobblerWeb.py
index f58142ea..14e2041e 100644
--- a/cobbler/webui/CobblerWeb.py
+++ b/cobbler/webui/CobblerWeb.py
@@ -62,6 +62,7 @@ class CobblerWeb(object):
                 self.remote.token_check(self.token)
                 self.username = self.remote.get_user_from_token(self.token)
                 # ensure config is up2date
+                self.remote.update(self.token)
                 return True
             except Exception, e:
                 if str(e).find("invalid token") != -1:
@@ -81,6 +82,7 @@ class CobblerWeb(object):
                 return False
             self.password = None # don't need it anymore, get rid of it
             # ensure configuration is up2date
+            self.remote.update(self.token)
             return True
         
         # login failed