store unit tests fixed after the refactor

Still to do:
1) store loader needs more unit tests

Author: UrszulaNeuman

dls_barcode/data_store/store.py

@@ -54,6 +54,7 @@ def backup_records(self, directory):
         file_name = time.strftime("%Y-%m-%d_%H-%M-%S", ts)
         backup_writer = StoreWriter(directory, file_name)
         backup = Backup(backup_writer)
+        self._sort_records()
         backup.backup_records(self.records)
 
     def delete_records(self, records_to_delete):

tests/system_tests/test_unipuck_scan.py

@@ -1,5 +1,7 @@
 import os, shutil
 
+from mock import MagicMock
+
 from dls_barcode.config.barcode_config import BarcodeConfig
 from dls_barcode.data_store import Store
 from dls_barcode.data_store.store_writer import StoreWriter
@@ -21,7 +23,7 @@
     shutil.rmtree(store_dir.value())
 
 comms_manger = StoreWriter(OPTIONS.get_store_directory(), "store")
-STORE = Store(comms_manger)
+STORE = Store(comms_manger, MagicMock())
 
 
 def test_generator():

tests/system_tests/test_unipuck_scan_locator.py

@@ -2,6 +2,7 @@
 
 import cv2
 import numpy as np
+from mock import MagicMock
 
 from dls_barcode.config.barcode_config import BarcodeConfig
 from dls_barcode.data_store import Store
@@ -27,7 +28,8 @@
 if os.path.isdir(store_dir.value()):
     shutil.rmtree(store_dir.value())
 comms_manger = StoreWriter(OPTIONS.get_store_directory(), "store")
-STORE = Store(comms_manger)
+STORE = Store(comms_manger, MagicMock)
+
 def test_generator():
     TEST_CASES = generate_test_cases()
     for params in TEST_CASES: