fix: WPS222 false positive for nested conditions

Author: vldmrdev

CHANGELOG.md

@@ -24,6 +24,7 @@ Semantic versioning in our case means:
 - Fixes false positive `WPS366` allowing the use
   of a single constant in `or`, #3610
 - Fixes the false positive `WPS330` when alternating unary operators, #3594
+- Fixes the false positive `WPS222` for nested conditions, #3630
 
 
 ## 1.6.0

tests/test_visitors/test_ast/test_complexity/test_counts/test_condition_counts.py

@@ -48,30 +48,39 @@
 # Real examples:
 
 complex_assignment = """
-some = zero and first or (second and last) or default()
+some = zero and first or (second and last) or default() or c or d
 """
 
 complex_condition = """
-if x == x1 and y == y1 and z == z1 or v == v1 or last():
+if x == x1 and y == y1 and z == z1 or v == v1 or last() or to_be() \
+    or not_to_be():
     ...
 """
 
+complex_list_comprehension = """
+def example(x, y):
+    return [i for i in range(x) if i % 2 == 0 or i == y or i > 10 or \
+        y < 4 or i != 0]
+"""
+
 complex_while = """
-while (x > x1 or y < y1) or (small(z) and v) or last():
+while (x > x1 or y < y1) or (small(z) and v) or first() or second() or last():
     ...
 """
 
 complex_match = """
 match some:
-    case 1 if (x > x1 or y < y1) or (small(z) and v) or last():
+    case 1 if (x > x1 or y < y1) or (small(z) and v) or first() or \
+        second() or last():
         ...
 """
 
 complex_gen_exp = """
 (
     ...
     for name in []
-    if (x > x1 or y < y1) or (small(z) and v) or last()
+    if (x > x1 or y < y1) or (small(z) and v) or (b() and g()) or \
+        second() or last()
 )
 """
 
@@ -111,6 +120,7 @@ def test_module_condition_counts_normal(
     [
         complex_assignment,
         complex_condition,
+        complex_list_comprehension,
         complex_while,
         complex_match,
         complex_gen_exp,
@@ -138,6 +148,7 @@ def test_module_condition_real_config(
     [
         complex_assignment,
         complex_condition,
+        complex_list_comprehension,
         complex_while,
         complex_match,
         complex_gen_exp,

wemake_python_styleguide/visitors/ast/complexity/counts.py

@@ -91,13 +91,7 @@ def visit_Compare(self, node: ast.Compare) -> None:
         self.generic_visit(node)
 
     def _count_conditions(self, node: ast.BoolOp) -> int:
-        counter = 0
-        for condition in node.values:
-            if isinstance(condition, ast.BoolOp):
-                counter += self._count_conditions(condition)
-            else:
-                counter += 1
-        return counter
+        return len(node.values)
 
     def _check_conditions(self, node: ast.BoolOp) -> None:
         conditions_count = self._count_conditions(node)