summaryrefslogtreecommitdiff
path: root/tests/functional/n/no/no_else_break.py
diff options
context:
space:
mode:
Diffstat (limited to 'tests/functional/n/no/no_else_break.py')
-rw-r--r--tests/functional/n/no/no_else_break.py137
1 files changed, 137 insertions, 0 deletions
diff --git a/tests/functional/n/no/no_else_break.py b/tests/functional/n/no/no_else_break.py
new file mode 100644
index 000000000..cf23f313a
--- /dev/null
+++ b/tests/functional/n/no/no_else_break.py
@@ -0,0 +1,137 @@
+""" Test that superfluous else return are detected. """
+
+# pylint:disable=invalid-name,missing-docstring,unused-variable
+
+
+def foo1(x, y, z):
+ for i in x:
+ if i > y: # [no-else-break]
+ break
+ else:
+ a = z
+
+
+def foo2(x, y, w, z):
+ for i in x:
+ if i > y: # [no-else-break]
+ break
+ elif i > w:
+ break
+ else:
+ a = z
+
+
+def foo3(x, y, z):
+ for i in x:
+ if i > y:
+ a = 1
+ if z: # [no-else-break]
+ b = 2
+ break
+ else:
+ c = 3
+ break
+ else:
+ d = 4
+ break
+
+
+def foo4(x, y):
+ for i in x:
+ if i > x: # [no-else-break]
+ if y:
+ a = 4
+ else:
+ b = 2
+ break
+ else:
+ c = 3
+ break
+
+
+def foo5(x, y, z):
+ for i in x:
+ if i > y: # [no-else-break]
+ if y:
+ a = 4
+ else:
+ b = 2
+ break
+ elif z:
+ c = 2
+ else:
+ c = 3
+ break
+
+
+def foo6(x, y):
+ for i in x:
+ if i > x:
+ if y: # [no-else-break]
+ a = 4
+ break
+ else:
+ b = 2
+ else:
+ c = 3
+ break
+
+
+def bar1(x, y, z):
+ for i in x:
+ if i > y:
+ break
+ return z
+
+
+def bar2(w, x, y, z):
+ for i in w:
+ if i > x:
+ break
+ if z < i:
+ a = y
+ else:
+ break
+ return
+
+
+def bar3(x, y, z):
+ for i in x:
+ if i > y:
+ if z:
+ break
+ else:
+ return z
+ return None
+
+
+def bar4(x):
+ for i in range(10):
+ if x: # [no-else-break]
+ break
+ else:
+ try:
+ return
+ except ValueError:
+ break
+
+
+def nested1(x, y, z):
+ for i in x:
+ if i > x:
+ for j in y:
+ if j < z:
+ break
+ else:
+ a = z
+
+
+def nested2(x, y, z):
+ for i in x:
+ if i > x:
+ for j in y:
+ if j > z:
+ break
+ break
+ else:
+ a = z