diff options
author | github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> | 2023-04-17 07:25:14 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-04-17 07:25:14 +0000 |
commit | 1dba30b43ce7d2c43e6cc94120d8daca5d9ddabd (patch) | |
tree | bc8b8094f1b980173a6532ca6a1336d7654a7cf3 /tests | |
parent | ec96bdc206350e378137da27c2f4cd103a94dc63 (diff) | |
download | pylint-git-1dba30b43ce7d2c43e6cc94120d8daca5d9ddabd.tar.gz |
Improve output of `consider-using-generator` message for `min()` calls with `default` keyword (#8582) (#8583)
(cherry picked from commit 4a485e28f0a5118b37550123c79f1f6d0dec42a4)
Co-authored-by: Jacob Walls <jacobtylerwalls@gmail.com>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/functional/c/consider/consider_using_generator.py | 5 | ||||
-rw-r--r-- | tests/functional/c/consider/consider_using_generator.txt | 1 |
2 files changed, 6 insertions, 0 deletions
diff --git a/tests/functional/c/consider/consider_using_generator.py b/tests/functional/c/consider/consider_using_generator.py index e5ac31f32..ee561a7e9 100644 --- a/tests/functional/c/consider/consider_using_generator.py +++ b/tests/functional/c/consider/consider_using_generator.py @@ -18,3 +18,8 @@ tuple(0 for y in list(range(10))) sum(x*x for x in range(10)) min(x*x for x in range(10)) max(x*x for x in range(10)) + +# Keyword arguments +# https://github.com/pylint-dev/pylint/issues/8563 +min([x*x for x in range(10)], default=42) # [consider-using-generator] +min((x*x for x in range(10)), default=42) diff --git a/tests/functional/c/consider/consider_using_generator.txt b/tests/functional/c/consider/consider_using_generator.txt index 2c89a8d7e..8a2a4ec6d 100644 --- a/tests/functional/c/consider/consider_using_generator.txt +++ b/tests/functional/c/consider/consider_using_generator.txt @@ -3,3 +3,4 @@ consider-using-generator:11:0:11:35::Consider using a generator instead 'tuple(0 consider-using-generator:12:0:12:29::Consider using a generator instead 'sum(x * x for x in range(10))':UNDEFINED consider-using-generator:13:0:13:29::Consider using a generator instead 'min(x * x for x in range(10))':UNDEFINED consider-using-generator:14:0:14:29::Consider using a generator instead 'max(x * x for x in range(10))':UNDEFINED +consider-using-generator:24:0:24:41::Consider using a generator instead 'min((x * x for x in range(10)), default=42)':UNDEFINED |