Side by Side Diff: flake8-eyeo/tests/A206.py
Issue 29602805:
Noissue - Remove bare exceptions from test cases (Closed)
Base URL: https://hg.adblockplus.org/codingtools/
Patch Set: Removed .orig files
Created Nov. 10, 2017, 12:07 a.m.
| Left: | ||
| Right: |
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« flake8-eyeo/tests/A205.py ('k')
|
no next file »
|
no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| OLD | NEW |
|---|---|
| 1 def noop(): | 1 def noop(): |
| 2 pass | 2 pass |
| 3 | 3 |
| 4 | 4 |
| 5 def else_after_return(): | 5 def else_after_return(): |
| 6 if True: | 6 if True: |
| 7 return | 7 return |
| 8 else: | 8 else: |
| 9 # A206 | 9 # A206 |
| 10 noop() | 10 noop() |
| 11 | 11 |
| 12 | 12 |
| 13 def else_after_raise(): | 13 def else_after_raise(): |
| 14 if True: | 14 if True: |
| 15 raise Exception() | 15 raise Exception() |
| 16 else: | 16 else: |
| 17 # A206 | 17 # A206 |
| 18 noop() | 18 noop() |
| 19 | 19 |
| 20 | 20 |
| 21 def else_after_return_in_except(): | 21 def else_after_return_in_except(): |
| 22 try: | 22 try: |
| 23 noop() | 23 noop() |
| 24 except: | 24 except Exception: |
| 25 return | 25 return |
| 26 else: | 26 else: |
| 27 # A206 | 27 # A206 |
| 28 noop() | 28 noop() |
| 29 | 29 |
| 30 | 30 |
| 31 for i in range(10): | 31 for i in range(10): |
| 32 if i == 0: | 32 if i == 0: |
| 33 continue | 33 continue |
| 34 else: | 34 else: |
| 35 # A206 | 35 # A206 |
| 36 noop() | 36 noop() |
| 37 | 37 |
| 38 if i == 1: | 38 if i == 1: |
| 39 break | 39 break |
| 40 else: | 40 else: |
| 41 # A206 | 41 # A206 |
| 42 noop() | 42 noop() |
| OLD | NEW |
« no previous file with comments
|
« flake8-eyeo/tests/A205.py ('k')
|
no next file »
|
no next file with comments »
