Split rsync initialization out of run so we can test

it better. (needed to support the previous changes to test/test_copier.py

diff --git a/htsworkflow/automation/copier.py b/htsworkflow/automation/copier.py
index 785691eae4e7afaded5e4b0e377ccb68156ed422..9ec1e2bdfc8597aaa42c0a016761b0a549af0310 100644 (file)
--- a/htsworkflow/automation/copier.py
+++ b/htsworkflow/automation/copier.py
@@ -170,7 +170,18 @@ class CopierBot(rpc.XmlRpcBot):
         self.register_function(self.startCopy)
         self.register_function(self.sequencingFinished)
         self.eventTasks.append(self.update)
-        
+       
+    def _init_rsync(self):
+        """
+        Initalize rsync class
+
+        This is only accessible for test purposes.
+        """
+        # we can't call any logging function until after start finishes.
+        # this got moved to a seperate function from run to help with test code
+        if self.rsync is None:
+            self.rsync = rsync(self.sources, self.destination, self.password)
+
     def read_config(self, section=None, configfile=None):
         """
         read the config file
@@ -195,9 +206,7 @@ class CopierBot(rpc.XmlRpcBot):
         """
         Start application
         """
-        # we can't call any logging function until after start finishes.
-        if self.rsync is None:
-            self.rsync = rsync(self.sources, self.destination, self.password)
+        self._init_rsync()
         super(CopierBot, self).run()
 
     def startCopy(self, *args):