diff options
author | Abseil Team <absl-team@google.com> | 2019-11-15 11:20:18 -0800 |
---|---|---|
committer | Copybara-Service <copybara-worker@google.com> | 2019-11-15 11:20:42 -0800 |
commit | ea16e318d3a6f5081aecd4628152ea4254946d87 (patch) | |
tree | f204f63d7643927c3ca3c61b7449049b53f6495e /absl/testing/parameterized.py | |
parent | 454aa79fcec321e0a7203136fd5a0d0a3bce02ae (diff) | |
download | absl-py-ea16e318d3a6f5081aecd4628152ea4254946d87.tar.gz |
Add a helpful error message to let people know when they've specified named_parameters and forgotten to add names to their parameter tuples rather than crashing later with something obscure.
PiperOrigin-RevId: 280698220
Change-Id: I02cc1eecc3f3310acd0b5ad30f065d6aab240c9d
Diffstat (limited to 'absl/testing/parameterized.py')
-rwxr-xr-x | absl/testing/parameterized.py | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/absl/testing/parameterized.py b/absl/testing/parameterized.py index a7cc88b..b67aaeb 100755 --- a/absl/testing/parameterized.py +++ b/absl/testing/parameterized.py @@ -281,6 +281,10 @@ class _ParameterizedTestIter(object): testcase_params = {k: v for k, v in six.iteritems(testcase_params) if k != _NAMED_DICT_KEY} elif _non_string_or_bytes_iterable(testcase_params): + if not isinstance(testcase_params[0], six.string_types): + raise RuntimeError( + 'The first element of named test parameters is the test name ' + 'suffix and must be a string') testcase_name = testcase_params[0] testcase_params = testcase_params[1:] else: |