add some refactor that makes __save_result testable

1 files changed, 16 insertions, 16 deletions

diff --git a/CeibalEncuesta/gtk3/CeibalEncuesta/CeibalEncuesta.py b/CeibalEncuesta/gtk3/CeibalEncuesta/CeibalEncuesta.py
index 3ceb360..c5bce11 100755
--- a/CeibalEncuesta/gtk3/CeibalEncuesta/CeibalEncuesta.py
+++ b/CeibalEncuesta/gtk3/CeibalEncuesta/CeibalEncuesta.py
@@ -75,7 +75,7 @@ Gdk.threads_init()
 class CeibalEncuesta(Gtk.Window):
 
     def __init__(self):
-        global BACKUP_PATH
+        self.backup_path = BACKUP_PATH
 
         Gtk.Window.__init__(self)
 
@@ -124,10 +124,10 @@ class CeibalEncuesta(Gtk.Window):
         self.__init()
 
         if getattr(self, "poll_id", None):
-            if str(self.poll_id) not in BACKUP_PATH:
-                BACKUP_PATH = BACKUP_PATH + "/%s.encuesta" % self.poll_id
-            if not os.path.exists(BACKUP_PATH):
-                self.__save_json(path=BACKUP_PATH)
+            if str(self.poll_id) not in self.backup_path:
+                self.backup_path = self.backup_path + "/%s.encuesta" % self.poll_id
+            if not os.path.exists(self.backup_path):
+                self.__save_json(path=self.backup_path)
 
         self.show_all()
 
@@ -516,23 +516,24 @@ class CeibalEncuesta(Gtk.Window):
             )
             archivo.close()
 
-            dialog = My_Alert_Dialog(
-                parent_window = self,
-                label = "Encuesta Exportada Correctamente.")
+            dialog = self._alert_dialog(
+                parent_window=self,
+                label="Encuesta Exportada Correctamente.")
+            dialog.run()
+            dialog.destroy()
 
-            response = dialog.run()
+            if os.path.exists(self.backup_path):
+                os.remove(self.backup_path)
 
-            dialog.destroy()
+    def _alert_dialog(self, parent_window, label):
+        return My_Alert_Dialog(parent_window=parent_window, label=label)
 
-            if os.path.exists(BACKUP_PATH):
-                os.remove(BACKUP_PATH)
 
     def __load_encuesta_respondida(self, widget, archivo):
         """
         Carga una encuesta parcial o totalmente
         respondida, desde un archivo json.
         """
-        global BACKUP_PATH
 
         if os.path.exists(TEMPPATH):
             os.remove(TEMPPATH)
@@ -562,7 +563,7 @@ class CeibalEncuesta(Gtk.Window):
                 "pollster_id": pollster_id,
                 "pollster_username": pollster_username}
 
-            BACKUP_PATH = BACKUP_PATH + "/%s.encuesta" % poll_id
+            self.backup_path = self.backup_path + "/%s.encuesta" % poll_id
 
             self.infowidget.set_info(dict)
 
@@ -599,7 +600,6 @@ class CeibalEncuesta(Gtk.Window):
         Carga una encuesta almacenada en
         un archivo json o shelve.
         """
-        global BACKUP_PATH
 
         if os.path.exists(TEMPPATH):
             os.remove(TEMPPATH)
@@ -658,7 +658,7 @@ class CeibalEncuesta(Gtk.Window):
 
         self.poll_id = poll_id
 
-        bck_path = BACKUP_PATH + "/%s.encuesta" % self.poll_id
+        bck_path = self.backup_path + "/%s.encuesta" % self.poll_id
         if os.path.exists(bck_path):
             msg = u"""
             Existe un archivo temporal para esta encuesta en %s,