aboutsummaryrefslogtreecommitdiffstats
path: root/resources/libraries/python
diff options
context:
space:
mode:
authorJuraj Linkeš <juraj.linkes@pantheon.tech>2020-05-22 14:25:24 +0200
committerVratko Polak <vrpolak@cisco.com>2020-06-09 15:12:51 +0000
commitbe5265bc9612f5aedf4f25720c4bd8941e359410 (patch)
treeb4d5a10d57743c4ada06a611e37a64061b5c84b0 /resources/libraries/python
parente84eec8494a5f13fce73e13dfdb38af90608edb7 (diff)
Remove leading tc[nn] from test names
The test names are unique without it and the information doesn't add anything extra. Change-Id: Idc7d6d1d21c8c05691e1757227a0a3787406d370 Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
Diffstat (limited to 'resources/libraries/python')
-rw-r--r--resources/libraries/python/autogen/Regenerator.py10
-rw-r--r--resources/libraries/python/autogen/Testcase.py12
2 files changed, 8 insertions, 14 deletions
diff --git a/resources/libraries/python/autogen/Regenerator.py b/resources/libraries/python/autogen/Regenerator.py
index 3ef589d351..0d291e231e 100644
--- a/resources/libraries/python/autogen/Regenerator.py
+++ b/resources/libraries/python/autogen/Regenerator.py
@@ -130,9 +130,7 @@ def add_default_testcases(testcase, iface, suite_id, file_out, tc_kwargs_list):
:type file_out: file
:type tc_kwargs_list: dict
"""
- # We bump tc number in any case, so that future enables/disables
- # do not affect the numbering of other test cases.
- for num, kwargs in enumerate(tc_kwargs_list, start=1):
+ for kwargs in tc_kwargs_list:
# TODO: Is there a better way to disable some combinations?
emit = True
if kwargs[u"frame_size"] == 9000:
@@ -163,7 +161,7 @@ def add_default_testcases(testcase, iface, suite_id, file_out, tc_kwargs_list):
if kwargs[u"frame_size"] not in MIN_FRAME_SIZE_VALUES:
emit = False
if emit:
- file_out.write(testcase.generate(num=num, **kwargs))
+ file_out.write(testcase.generate(**kwargs))
def add_tcp_testcases(testcase, file_out, tc_kwargs_list):
@@ -176,8 +174,8 @@ def add_tcp_testcases(testcase, file_out, tc_kwargs_list):
:type file_out: file
:type tc_kwargs_list: dict
"""
- for num, kwargs in enumerate(tc_kwargs_list, start=1):
- file_out.write(testcase.generate(num=num, **kwargs))
+ for kwargs in tc_kwargs_list:
+ file_out.write(testcase.generate(**kwargs))
def write_default_files(in_filename, in_prolog, kwargs_list):
diff --git a/resources/libraries/python/autogen/Testcase.py b/resources/libraries/python/autogen/Testcase.py
index c4ed60685b..3ffbc4594a 100644
--- a/resources/libraries/python/autogen/Testcase.py
+++ b/resources/libraries/python/autogen/Testcase.py
@@ -29,21 +29,18 @@ class Testcase:
- cores_str - Number of physical cores to use, example: "2".
- frame_num - Framesize as a number, example: "${74}".
- frame_str - Framesize in upper case, example: "74B".
- - tc_num - Start of testcase name, example: "tc04".
:type template_string: str
"""
self.template = Template(template_string)
- def generate(self, num, frame_size, phy_cores):
+ def generate(self, frame_size, phy_cores):
"""Return string of test case code with placeholders filled.
Fail if there are placeholders left unfilled.
It is not required for all placeholders to be present in template.
- :param num: Test case number. Example value: 4.
:param frame_size: Imix string or numeric frame size. Example: 74.
:param phy_cores: Number of physical cores to use. Example: 2.
- :type num: int
:type frame_size: str or int
:type phy_cores: int or str
:returns: Filled template, usable as test case code.
@@ -66,7 +63,6 @@ class Testcase:
{
u"cores_num": f"${{{cores_num:d}}}",
u"cores_str": phy_cores,
- u"tc_num": f"tc{num:02d}",
}
)
return self.template.substitute(subst_dict)
@@ -84,7 +80,7 @@ class Testcase:
:rtype: Testcase
"""
template_string = f'''
-| ${{tc_num}}-${{frame_str}}-${{cores_str}}c-{suite_id}
+| ${{frame_str}}-${{cores_str}}c-{suite_id}
| | [Tags] | ${{frame_str}} | ${{cores_str}}C
| | frame_size=${{frame_num}} | phy_cores=${{cores_num}}
'''
@@ -105,13 +101,13 @@ class Testcase:
# (TCP, QUIC, SCTP, ...) where DUT (not TG) decides frame size.
if u"tcphttp" in suite_id:
template_string = f'''
-| ${{tc_num}}-IMIX-${{cores_str}}c-{suite_id}
+| IMIX-${{cores_str}}c-{suite_id}
| | [Tags] | ${{cores_str}}C
| | phy_cores=${{cores_num}}
'''
else:
template_string = f'''
-| ${{tc_num}}-${{frame_str}}-${{cores_str}}c-{suite_id[:-4]}-{suite_id[-3:]}
+| ${{frame_str}}-${{cores_str}}c-{suite_id[:-4]}-{suite_id[-3:]}
| | [Tags] | ${{cores_str}}C\n| | phy_cores=${{cores_num}}
'''
return cls(template_string)