Skip to content

Commit

Permalink
[hexagon][testing] filesystem-friendly test IDs
Browse files Browse the repository at this point in the history
- Change the formula used to compute pytest test-ID strings.
  The previous formula included '(' and ')' characters, which
  can cause the filename to require escaping / quoting on the
  Bash command line.
  • Loading branch information
Christian Convey committed Jul 26, 2022
1 parent 49e29b0 commit 9ca6be8
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/python/contrib/test_hexagon/pytest_util.py
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ def get_single_param_chunk(param_val, param_desc: Optional[str]):
need_prefix_separator = True

elif type(param_val) == TensorContentConstant:
val_str = f"const({param_val.elem_value})"
val_str = f"const[{param_val.elem_value}]"
need_prefix_separator = True

elif type(param_val) == TensorContentDtypeMin:
Expand All @@ -76,7 +76,7 @@ def get_single_param_chunk(param_val, param_desc: Optional[str]):
need_prefix_separator = True

elif type(param_val) == TensorContentSequentialCOrder:
val_str = f"seqC(start:{param_val.start_value},inc:{param_val.increment})"
val_str = f"seqC[start:{param_val.start_value},inc:{param_val.increment}]"
need_prefix_separator = True

else:
Expand Down

0 comments on commit 9ca6be8

Please sign in to comment.