Add a test case that make the hot fix fail

We do not remove the hotfix in order to have a false positive
instead of a crash is something else is still buggy.

Author: Pierre-Sassoulas

pylint/extensions/check_elif.py

@@ -59,6 +59,7 @@ def visit_ifexp(self, node: nodes.IfExp) -> None:
 
     def visit_comprehension(self, node: nodes.Comprehension) -> None:
         self._if_counter += len(node.ifs)
+        self._elifs.insert(self._if_counter, True)
 
     @check_messages("else-if-used")
     def visit_if(self, node: nodes.If) -> None:
@@ -67,6 +68,8 @@ def visit_if(self, node: nodes.If) -> None:
             # current if node must directly follow an "else"
             if orelse and orelse == [node]:
                 if (
+                    # Safeguard if we missed an if in self._elifs
+                    # See https://github.com/PyCQA/pylint/pull/5369
                     self._if_counter < len(self._elifs)
                     and not self._elifs[self._if_counter]
                 ):

tests/functional/e/elif_checker.py

@@ -37,3 +37,17 @@ def _if_in_fstring_comprehension():
         )
     elif "i" in "true":
         raise TypeError("d")
+
+def _if_in_fstring_comprehension_with_elif():
+    order = {}
+    if "z" not in "false":
+        raise TypeError(
+            f" {', '.join(sorted(i for i in order or () if i not in vars))}"
+        )
+    elif "z" not in "true":
+        pass
+    else:
+        if "t" not in "false":  # [else-if-used]
+            raise TypeError("d")
+        else:
+            print("e")

tests/functional/e/elif_checker.txt

@@ -1,2 +1,3 @@
 else-if-used:12:8:my_function:"Consider using ""elif"" instead of ""else if""":HIGH
 else-if-used:24:20:my_function:"Consider using ""elif"" instead of ""else if""":HIGH
+else-if-used:50:8:_if_in_fstring_comprehension_with_elif:"Consider using ""elif"" instead of ""else if""":HIGH