wemake-services / wemake-python-styleguide / 2670

100%

push

travis-ci-com

Issue-1082: Forbid to use `continue` in `finally` (#1179)

* Issue-1082: added ContinueInFinallyBlockViolation and a visitor

* Issue-1082: tests added

* Issue-1082: Changelog updated

* Issue-1082: added case to noqa.py and 'WPS452': 1 => 2

* Issue-1082: integration tests moved to noqa38

* Issue-1082: test corrected according to code review and python 3.7, 3.6

* Issue-1082: changes according to code review

* Issue-1082: added mode fixture  to test and test for python<3.8

* Issue-1082: added pragma for ContinueInFinallyBlockViolation

* Issue-1082: removed unnecessary `else` statement from the test

* Issue-1082: added BreakInFinallyBlockViolation

* Issue-1082: tests for BreakInFinallyBlockViolation added

* Issue-1082: test cases with `break` in `finally` deleted

There is a special case and such tests for `break` in `finally`

* Issue-1082: `break` added; style changes

* Issue-1082: combined into test_finally_in_loops.py

* Issue-1082: tests changed according to merging continue and break violation

* Issue-1082: `continue` and `break` violation merged into  LoopControlFinallyViolation

* Issue-1082: refactoring

2311 of 2311 branches covered (100.0%)

Branch coverage included in aggregate %.

8 of 8 new or added lines in 2 files covered. (100.0%)

5226 of 5226 relevant lines covered (100.0%)

3.0 hits per line