Remove write-only code
This commit is contained in:
@@ -299,7 +299,6 @@ class OaiCiTest():
|
|||||||
self.iperf_packetloss_threshold = ''
|
self.iperf_packetloss_threshold = ''
|
||||||
self.iperf_bitrate_threshold = ''
|
self.iperf_bitrate_threshold = ''
|
||||||
self.iperf_profile = ''
|
self.iperf_profile = ''
|
||||||
self.iperf_options = ''
|
|
||||||
self.iperf_tcp_rate_target = ''
|
self.iperf_tcp_rate_target = ''
|
||||||
self.finalStatus = False
|
self.finalStatus = False
|
||||||
self.air_interface=''
|
self.air_interface=''
|
||||||
@@ -520,7 +519,7 @@ class OaiCiTest():
|
|||||||
return (status, f'{ue_header}\n{msg}')
|
return (status, f'{ue_header}\n{msg}')
|
||||||
|
|
||||||
def Iperf(self, ctx, node, HTML, infra_file="ci_infra.yaml"):
|
def Iperf(self, ctx, node, HTML, infra_file="ci_infra.yaml"):
|
||||||
logging.debug(f'Iperf: iperf_args "{self.iperf_args}" iperf_packetloss_threshold "{self.iperf_packetloss_threshold}" iperf_bitrate_threshold "{self.iperf_bitrate_threshold}" iperf_profile "{self.iperf_profile}" iperf_options "{self.iperf_options}"')
|
logging.debug(f'Iperf: iperf_args "{self.iperf_args}" iperf_packetloss_threshold "{self.iperf_packetloss_threshold}" iperf_bitrate_threshold "{self.iperf_bitrate_threshold}" iperf_profile "{self.iperf_profile}"')
|
||||||
|
|
||||||
if self.ue_ids == [] or self.svr_id == None:
|
if self.ue_ids == [] or self.svr_id == None:
|
||||||
raise Exception("no module names in self.ue_ids or/and self.svr_id provided")
|
raise Exception("no module names in self.ue_ids or/and self.svr_id provided")
|
||||||
|
|||||||
@@ -185,10 +185,6 @@ def ExecuteActionWithParam(action, ctx, node):
|
|||||||
if CiTestObj.iperf_profile != 'balanced' and CiTestObj.iperf_profile != 'unbalanced' and CiTestObj.iperf_profile != 'single-ue':
|
if CiTestObj.iperf_profile != 'balanced' and CiTestObj.iperf_profile != 'unbalanced' and CiTestObj.iperf_profile != 'single-ue':
|
||||||
logging.error(f'test-case has wrong profile {CiTestObj.iperf_profile}, forcing balanced')
|
logging.error(f'test-case has wrong profile {CiTestObj.iperf_profile}, forcing balanced')
|
||||||
CiTestObj.iperf_profile = 'balanced'
|
CiTestObj.iperf_profile = 'balanced'
|
||||||
CiTestObj.iperf_options = test.findtext('iperf_options') or 'check'
|
|
||||||
if CiTestObj.iperf_options != 'check' and CiTestObj.iperf_options != 'sink':
|
|
||||||
logging.error('test-case has wrong option ' + CiTestObj.iperf_options)
|
|
||||||
CiTestObj.iperf_options = 'check'
|
|
||||||
if action == 'Iperf':
|
if action == 'Iperf':
|
||||||
success = CiTestObj.Iperf(ctx, node, HTML)
|
success = CiTestObj.Iperf(ctx, node, HTML)
|
||||||
elif action == 'Iperf2_Unidir':
|
elif action == 'Iperf2_Unidir':
|
||||||
|
|||||||
Reference in New Issue
Block a user