Add new message called duplicate-value

ChangeLog

@@ -19,6 +19,9 @@ Release date: TBA
 
 * Fix pyreverse diagrams type hinting for classmethods and staticmethods.
 
+* Add new message ``duplicate-value`` for identifying duplicate values inside sets.
+
+  Closes #5880
 * Fix pyreverse diagrams type hinting for methods returning None.
 
 * Fix matching ``--notes`` options that end in a non-word character.

doc/whatsnew/2.13.rst

@@ -66,6 +66,10 @@ New checkers
 
   Closes #5670
 
+* Added new message called ``duplicate-value`` which identifies duplicate values inside sets.
+
+  Closes #5880
+
 Removed checkers
 ================
 

pylint/checkers/base.py

@@ -1083,6 +1083,11 @@ class BasicChecker(_BasicChecker):
             "Used when an assert statement has a string literal as its first argument, which will "
             "cause the assert to always pass.",
         ),
+        "W0130": (
+            "Duplicate value %r in set",
+            "duplicate-value",
+            "Used when a set contains the same value multiple times.",
+        ),
     }
 
     reports = (("RP0101", "Statistics by type", report_by_type_stats),)
@@ -1474,6 +1479,19 @@ def visit_dict(self, node: nodes.Dict) -> None:
                 self.add_message("duplicate-key", node=node, args=key)
             keys.add(key)
 
+    @utils.check_messages("duplicate-value")
+    def visit_set(self, node: nodes.Set) -> None:
+        """Check duplicate value in set."""
+        values = set()
+        for v in node.elts:
+            if isinstance(v, nodes.Const):
+                value = v.value
+            else:
+                continue
+            if value in values:
+                self.add_message("duplicate-value", node=node, args=value)
+            values.add(value)
+
     def visit_tryfinally(self, node: nodes.TryFinally) -> None:
         """Update try...finally flag."""
         self._tryfinallys.append(node)

pylint/checkers/format.py

@@ -719,10 +719,8 @@ def is_line_length_check_activated(pylint_pattern_match_object) -> bool:
     def specific_splitlines(lines: str) -> List[str]:
         """Split lines according to universal newlines except those in a specific sets."""
         unsplit_ends = {
-            "\v",
-            "\x0b",
-            "\f",
-            "\x0c",
+            "\x0b",  # synonym of \v
+            "\x0c",  # synonym of \f
             "\x1c",
             "\x1d",
             "\x1e",

tests/functional/d/duplicate_value.py

@@ -0,0 +1,18 @@
+# pylint: disable = invalid-name, line-too-long
+"""Simple test sets for checking duplicate values"""
+
+set1 = {1, 2, 3, 4}
+set2 = {1, 1, 2} # [duplicate-value]
+set3 = {1, 2, 2} # [duplicate-value]
+
+set4 = {'one', 'two', 'three'}
+set5 = {'one', 'two', 'one'} # [duplicate-value]
+set6 = {'one', 'two', 'two'} # [duplicate-value]
+
+wrong_set = {12, 23, True, 6, True, 0, 12} # [duplicate-value, duplicate-value]
+correct_set = {12, 13, 23, 24, 89}
+
+wrong_set_mixed = {1, 2, 'value', 1} # [duplicate-value]
+wrong_set = {'arg1', 'arg2', False, 'arg1', True} # [duplicate-value]
+
+another_wrong_set = {2, 3, 'arg1', True, 'arg1', False, True} # [duplicate-value, duplicate-value]

tests/functional/d/duplicate_value.txt

@@ -0,0 +1,10 @@
+duplicate-value:5:7:5:16::Duplicate value 1 in set:UNDEFINED
+duplicate-value:6:7:6:16::Duplicate value 2 in set:UNDEFINED
+duplicate-value:9:7:9:28::Duplicate value 'one' in set:UNDEFINED
+duplicate-value:10:7:10:28::Duplicate value 'two' in set:UNDEFINED
+duplicate-value:12:12:12:42::Duplicate value 12 in set:UNDEFINED
+duplicate-value:12:12:12:42::Duplicate value True in set:UNDEFINED
+duplicate-value:15:18:15:36::Duplicate value 1 in set:UNDEFINED
+duplicate-value:16:12:16:49::Duplicate value 'arg1' in set:UNDEFINED
+duplicate-value:18:20:18:61::Duplicate value 'arg1' in set:UNDEFINED
+duplicate-value:18:20:18:61::Duplicate value True in set:UNDEFINED