Skip to content

Commit 5382075

Browse files
authored
Merge pull request #140 from victormlg/more_asserts
CFE-4499: Added asserts in aramid classes
2 parents 29e97cc + 5c081aa commit 5382075

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

cf_remote/aramid.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -122,6 +122,9 @@ def __init__(self, host, proc, action=None, retries=0): # TODO: timeout=60
122122
self.stderr = ""
123123
self.done = False
124124

125+
assert self.proc
126+
assert self.host
127+
125128
def communicate(self, timeout=1, ignore_failed=False):
126129
start = time.time()
127130
try:
@@ -216,6 +219,9 @@ def __init__(
216219

217220
self.tasks = []
218221
"""A helper attribute which may change in the future"""
222+
assert self.user
223+
assert self.port
224+
assert self.host_name
219225

220226
@property
221227
def login(self):

0 commit comments

Comments
 (0)