[2641] Add test for _is_session_valid()

This commit was actually to update the statement inside
_is_session_valid() to be a None check, but I've written a unittest for
it anyway.

src/bin/cmdctl/cmdctl.py.in

@@ -97,6 +97,9 @@ def check_file(file_name):
 class SecureHTTPRequestHandler(http.server.BaseHTTPRequestHandler):
     '''https connection request handler.
     Currently only GET and POST are supported.  '''
+    def __init__(self):
+        self.session_id = None
+
     def do_GET(self):
         '''The client should send its session id in header with
         the name 'cookie'
@@ -121,7 +124,7 @@ class SecureHTTPRequestHandler(http.server.BaseHTTPRequestHandler):
         return self.server.get_reply_data_for_GET(id, module)
 
     def _is_session_valid(self):
-        return self.session_id
+        return self.session_id is not None
 
     def _is_user_logged_in(self):
         login_time = self.server.user_sessions.get(self.session_id)

src/bin/cmdctl/tests/cmdctl_test.py

@@ -33,7 +33,7 @@ BUILD_FILE_PATH = os.environ['CMDCTL_BUILD_PATH'] + os.sep
 # Rewrite the class for unittest.
 class MySecureHTTPRequestHandler(SecureHTTPRequestHandler):
     def __init__(self):
-        pass
+        self.session_id = None
 
     def send_response(self, rcode):
         self.rcode = rcode
@@ -101,6 +101,12 @@ class TestSecureHTTPRequestHandler(unittest.TestCase):
         self.handler.rfile.close()
         os.remove('check.tmp')
 
+    def test_is_session_valid(self):
+        self.assertIsNone(self.handler.session_id)
+        self.assertFalse(self.handler._is_session_valid())
+        self.handler.session_id = 4234
+        self.assertTrue(self.handler._is_session_valid())
+
     def test_parse_request_path(self):
         self.handler.path = ''
         mod, cmd = self.handler._parse_request_path()