backend: Check for shared storage also

[ganeti-local] / test / ganeti.masterd.instance_unittest.py

diff --git a/test/ganeti.masterd.instance_unittest.py b/test/ganeti.masterd.instance_unittest.py
index 0c74def..4dda147 100755 (executable)
--- a/test/ganeti.masterd.instance_unittest.py
+++ b/test/ganeti.masterd.instance_unittest.py
@@ -26,12 +26,15 @@ import sys
 import unittest
 
 from ganeti import constants
+from ganeti import errors
 from ganeti import utils
 from ganeti import masterd
 
 from ganeti.masterd.instance import \
-  ImportExportTimeouts, _TimeoutExpired, _DiskImportExportBase, \
-  ComputeRemoteExportHandshake, CheckRemoteExportHandshake
+  ImportExportTimeouts, _DiskImportExportBase, \
+  ComputeRemoteExportHandshake, CheckRemoteExportHandshake, \
+  ComputeRemoteImportDiskInfo, CheckRemoteExportDiskInfo, \
+  FormatProgress
 
 import testutils
 
@@ -43,15 +46,24 @@ class TestMisc(unittest.TestCase):
     self.assertEqual(tmo.listen, ImportExportTimeouts.DEFAULT_LISTEN_TIMEOUT)
     self.assertEqual(tmo.ready, ImportExportTimeouts.DEFAULT_READY_TIMEOUT)
     self.assertEqual(tmo.error, ImportExportTimeouts.DEFAULT_ERROR_TIMEOUT)
+    self.assertEqual(tmo.progress,
+                     ImportExportTimeouts.DEFAULT_PROGRESS_INTERVAL)
 
     tmo = ImportExportTimeouts(999)
     self.assertEqual(tmo.connect, 999)
 
+    tmo = ImportExportTimeouts(1, listen=2, error=3, ready=4, progress=5)
+    self.assertEqual(tmo.connect, 1)
+    self.assertEqual(tmo.listen, 2)
+    self.assertEqual(tmo.error, 3)
+    self.assertEqual(tmo.ready, 4)
+    self.assertEqual(tmo.progress, 5)
+
   def testTimeoutExpired(self):
-    self.assert_(_TimeoutExpired(100, 300, _time_fn=lambda: 500))
-    self.assertFalse(_TimeoutExpired(100, 300, _time_fn=lambda: 0))
-    self.assertFalse(_TimeoutExpired(100, 300, _time_fn=lambda: 100))
-    self.assertFalse(_TimeoutExpired(100, 300, _time_fn=lambda: 400))
+    self.assert_(utils.TimeoutExpired(100, 300, _time_fn=lambda: 500))
+    self.assertFalse(utils.TimeoutExpired(100, 300, _time_fn=lambda: 0))
+    self.assertFalse(utils.TimeoutExpired(100, 300, _time_fn=lambda: 100))
+    self.assertFalse(utils.TimeoutExpired(100, 300, _time_fn=lambda: 400))
 
   def testDiskImportExportBaseDirect(self):
     self.assertRaises(AssertionError, _DiskImportExportBase,
@@ -86,5 +98,64 @@ class TestRieHandshake(unittest.TestCase):
     self.assert_(CheckRemoteExportHandshake(cds, hs))
 
 
+class TestRieDiskInfo(unittest.TestCase):
+  def test(self):
+    cds = "bbf46ea9a"
+    salt = "ee5ad9"
+    di = ComputeRemoteImportDiskInfo(cds, salt, 0, "node1", 1234, "mag111")
+    self.assertEqual(CheckRemoteExportDiskInfo(cds, 0, di),
+                     ("node1", 1234, "mag111"))
+
+    for i in range(1, 100):
+      # Wrong disk index
+      self.assertRaises(errors.GenericError, CheckRemoteExportDiskInfo,
+                        cds, i, di)
+
+  def testInvalidHostPort(self):
+    cds = "3ZoJY8KtGJ"
+    salt = "drK5oYiHWD"
+
+    for host in [",", "...", "Hello World", "`", "!", "#", "\\"]:
+      di = ComputeRemoteImportDiskInfo(cds, salt, 0, host, 1234, "magic")
+      self.assertRaises(errors.OpPrereqError,
+                        CheckRemoteExportDiskInfo, cds, 0, di)
+
+    for port in [-1, 792825908, "HelloWorld!", "`#", "\\\"", "_?_"]:
+      di = ComputeRemoteImportDiskInfo(cds, salt, 0, "localhost", port, "magic")
+      self.assertRaises(errors.OpPrereqError,
+                        CheckRemoteExportDiskInfo, cds, 0, di)
+
+  def testCheckErrors(self):
+    cds = "0776450535a"
+    self.assertRaises(errors.GenericError, CheckRemoteExportDiskInfo,
+                      cds, 0, "")
+    self.assertRaises(errors.GenericError, CheckRemoteExportDiskInfo,
+                      cds, 0, ())
+    self.assertRaises(errors.GenericError, CheckRemoteExportDiskInfo,
+                      cds, 0, ("", 1, 2, 3, 4, 5))
+
+    # No host/port
+    self.assertRaises(errors.GenericError, CheckRemoteExportDiskInfo,
+                      cds, 0, ("", 1234, "magic", "", ""))
+    self.assertRaises(errors.GenericError, CheckRemoteExportDiskInfo,
+                      cds, 0, ("host", 0, "magic", "", ""))
+    self.assertRaises(errors.GenericError, CheckRemoteExportDiskInfo,
+                      cds, 0, ("host", 1234, "", "", ""))
+
+    # Wrong hash
+    self.assertRaises(errors.GenericError, CheckRemoteExportDiskInfo,
+                      cds, 0, ("nodeX", 123, "magic", "fakehash", "xyz"))
+
+
+class TestFormatProgress(unittest.TestCase):
+  def test(self):
+    FormatProgress((0, 0, None, None))
+    FormatProgress((100, 3.3, 30, None))
+    FormatProgress((100, 3.3, 30, 900))
+
+    self.assertEqual(FormatProgress((1500, 12, 30, None)),
+                     "1.5G, 12.0 MiB/s, 30%")
+
+
 if __name__ == "__main__":
   testutils.GanetiTestProgram()