Skip to content

Commit

Permalink
added docstrings, and disconnect on connect error
Browse files Browse the repository at this point in the history
  • Loading branch information
CoYe committed Jun 26, 2017
1 parent ce75744 commit 9730864
Showing 1 changed file with 18 additions and 4 deletions.
22 changes: 18 additions & 4 deletions cloudshell/cli/session/expect_session.py
Original file line number Diff line number Diff line change
Expand Up @@ -62,10 +62,20 @@ def session_type(self):

@abstractmethod
def _connect_actions(self, prompt, logger):
"""Read out buffer and run on_session_start actions
:param prompt: expected string in output
:param logger: logger
"""

pass

@abstractmethod
def _initialize_session(self, prompt, logger):
"""Create handler and initialize session
:param prompt: expected string in output
:param logger: logger
"""

pass

def active(self):
Expand All @@ -91,14 +101,18 @@ def _clear_buffer(self, timeout, logger):
return out

def connect(self, prompt, logger):
"""Connect to device through ssh
"""Connect to device.
:param prompt: expected string in output
:param logger: logger
"""

self._initialize_session(prompt, logger)
self._connect_actions(prompt, logger)
self._active = True
try:
self._initialize_session(prompt, logger)
self._connect_actions(prompt, logger)
self._active = True
except:
self.disconnect()
raise

def send_line(self, command, logger):
"""
Expand Down

0 comments on commit 9730864

Please sign in to comment.