summaryrefslogtreecommitdiff
path: root/tests/unit/test_decision_engine.py
Commit message (Collapse)AuthorAgeFilesLines
* Fix bug for plugins using extend_default_ignorefix-extend-default-ignoreIan Stapleton Cordasco2021-04-151-133/+335
| | | | | | | | | | | | | | Since Flake8 3.0 we've had the ability for plugins to use `extend_default_ignore` to register codes they want disabled by default. This, however, was a permanent disabling unfortunately. Our code didn't have a way of understanding that this new set of `ignore` codes was actually the 'default' set for that run. Much like the extended_select_list, we now attach extended_ignore_list to be able to confidently determine if the ignore we get in the DecisionEngine is actually the Default Ignore list and what plugins what us to ignore by default. Refs https://github.com/PyCQA/pep8-naming/pull/157
* Add --extend-select command line argumentFrank Winklmeier2021-04-111-16/+25
| | | | | | | | | Implement `--extend-select` command line argument following what was done for `--extend-ignore` in !233. This option can be used to selectively add individual codes without overriding the default list entirely. Addresses the remaining item of issue #1061.
* move from optparse to argparseAnthony Sottile2019-08-171-3/+3
|
* mypy now passesAnthony Sottile2019-05-191-2/+1
|
* Add some extend_ignores into decision engine test casesVille Skyttä2018-05-201-44/+61
|
* Implement --extend-ignoreVille Skyttä2018-05-161-0/+1
| | | | Refs #365
* Remediate one wafer thing decision logic bugIan Stapleton Cordasco2017-07-281-4/+3
| | | | | | | | Previously we were hitting the catch-all return that decided the error was selected. Instead, let's add specific logic since we now know how we would hit that and we actually don't want it to be selected. Closes #354
* Add some better comments for decision logicIan Cordasco2017-06-041-1/+3
|
* Get *one* more line of test coverageIan Cordasco2017-06-041-0/+3
| | | | | | | This was mostly to prove to myself that we could possibly reach that return more than actually covering that return. This just shows how gnarly this logic actually is. I wish there were a better way to write it.
* Rename methods on the new DecisionEngineIan Cordasco2017-06-031-6/+7
| | | | | Rename `decision_for` to `more_specific_decision_for` and `should_report_error` to `decision_for`.
* Move unit tests for StyleGuide decision logicIan Cordasco2017-06-021-0/+178
Convert it to test the DecisionEngine class directly and put them in their own file.