From 636dfd62171f84b5664760394be1c9dadaba33ed Mon Sep 17 00:00:00 2001
From: Ian Southam <isoutham@schubergphilis.com>
Date: Mon, 15 Dec 2014 15:17:32 +0100
Subject: [PATCH] New unit test Some stupid typos pep8

---
 .../config/opt/cloud/bin/cs/CsDatabag.py      |  2 +-
 .../config/opt/cloud/bin/cs/CsRedundant.py    |  2 +-
 systemvm/test/python/TestCsCmdLine.py         | 33 +++++++++++++++++++
 3 files changed, 35 insertions(+), 2 deletions(-)
 create mode 100644 systemvm/test/python/TestCsCmdLine.py

diff --git a/systemvm/patches/debian/config/opt/cloud/bin/cs/CsDatabag.py b/systemvm/patches/debian/config/opt/cloud/bin/cs/CsDatabag.py
index 2b28430b584..0f19a7c0929 100644
--- a/systemvm/patches/debian/config/opt/cloud/bin/cs/CsDatabag.py
+++ b/systemvm/patches/debian/config/opt/cloud/bin/cs/CsDatabag.py
@@ -105,7 +105,7 @@ class CsCmdLine(CsDataBag):
             return "unknown"
 
     def get_domain(self):
-        if "domain" in self.idata();
+        if "domain" in self.idata():
             return self.idata()['domain']
         else:
             return "cloudnine.internal"
diff --git a/systemvm/patches/debian/config/opt/cloud/bin/cs/CsRedundant.py b/systemvm/patches/debian/config/opt/cloud/bin/cs/CsRedundant.py
index f12f8fab9b7..4a6140987db 100644
--- a/systemvm/patches/debian/config/opt/cloud/bin/cs/CsRedundant.py
+++ b/systemvm/patches/debian/config/opt/cloud/bin/cs/CsRedundant.py
@@ -146,7 +146,7 @@ class CsRedundant(object):
         if not self.cl.is_redundant():
             logging.error("Set backup called on non-redundant router")
             return
-        """ 
+        """
         if not self.cl.is_master():
             logging.error("Set backup called on node that is already backup")
             return
diff --git a/systemvm/test/python/TestCsCmdLine.py b/systemvm/test/python/TestCsCmdLine.py
new file mode 100644
index 00000000000..8b954d8bcbc
--- /dev/null
+++ b/systemvm/test/python/TestCsCmdLine.py
@@ -0,0 +1,33 @@
+import unittest
+from cs.CsDatabag import CsCmdLine
+import merge
+
+
+class TestCsCmdLine(unittest.TestCase):
+
+    def setUp(self):
+        merge.DataBag.DPATH = "."
+        self.cscmdline = CsCmdLine('cmdline', {})
+
+    def test_ini(self):
+        self.assertTrue(self.cscmdline is not None)
+
+    def test_idata(self):
+        self.assertTrue(self.cscmdline.idata() == {})
+
+    def test_get_priority(self):
+        self.assertTrue(self.cscmdline.get_priority() == 99)
+
+    def test_is_redundant(self):
+        self.assertTrue(self.cscmdline.is_redundant() is False)
+        self.cscmdline.set_redundant()
+        self.assertTrue(self.cscmdline.is_redundant() is True)
+
+    def test_get_guest_gw(self):
+        self.assertTrue(self.cscmdline.get_guest_gw() == '1.2.3.4')
+        tval = "192.168.1.4"
+        self.cscmdline.set_guest_gw(tval)
+        self.assertTrue(self.cscmdline.get_guest_gw() == tval)
+
+if __name__ == '__main__':
+    unittest.main()