Fairly extensive overhaul of the test code, and correcting some API-based gotchas found as a result of testing. First, test_basic.py code now uses api.py methods as opposed to lower level methods, ensuring we test what API consumers are most likely to use. Second, remote.py tests are (largely) complete and clean. In api.py and remote.py I've added a method to ensure that the API configurations are always up to date before API calls, preventing synchronization issues that previously had to be worked around. As a result of this the server.update() call is no more. The mechanism behind this remains rough and we may want to optimize things a bit later, though the API signatures around this new checking code are solid. Upgrading the internals should be possible to avoid additional reloading of the config. Ultimately we may want to teach cobbler's local API to ask cobbler's cobblerd for it's definitive copy though that is to be explored later.

1 files changed, 3 insertions, 0 deletions

diff --git a/cobbler/collection.py b/cobbler/collection.py
index 3df7234c..7f3a3aac 100644
--- a/cobbler/collection.py
+++ b/cobbler/collection.py
@@ -144,6 +144,9 @@ class Collection(serializable.Serializable):
             item = self.factory_produce(self.config,seed_data)
             self.add(item)
 
+    def copy(self,ref,newname):
+        ref.name = newname
+        return self.add(ref,save=True,with_copy=True,with_triggers=True,with_sync=True,check_for_duplicate_names=True,check_for_duplicate_netinfo=False)
 
     def rename(self,ref,newname,with_sync=True,with_triggers=True):
         """