Updated flake8 based code quality changes for Tests

This commit is contained in:
kuldeepkhatke 2021-09-05 00:01:28 +05:30
parent dea950dd07
commit 07d0b2c630
7 changed files with 64 additions and 56 deletions

View File

@ -1,32 +1,34 @@
"""unused import""" """unused import"""
# pylint: disable=undefined-all-variable, import-error, no-absolute-import, too-few-public-methods, missing-docstring # pylint: disable=undefined-all-variable, import-error, no-absolute-import
# pylint: disable=too-few-public-methods, missing-docstring
from __future__ import print_function # noqa:F401
import xml.etree # [unused-import] import xml.etree # [unused-import]
import xml.sax # [unused-import] import xml.sax # noqa:F401 # [unused-import]
import os.path as test # [unused-import] import os.path as test # noqa:F401 # [unused-import]
from sys import argv as test2 # [unused-import] from sys import argv as test2 # noqa:F401 # [unused-import]
from sys import flags # [unused-import] from sys import flags # noqa:F401 # [unused-import]
# +1:[unused-import,unused-import] # +1:[unused-import,unused-import]
from collections import deque, OrderedDict, Counter from collections import deque, OrderedDict, Counter # noqa:F401
# All imports above should be ignored # All imports above should be ignored
import requests # [unused-import] import requests # noqa:F401 # [unused-import]
# setuptools # setuptools
import zipimport # command/easy_install.py import zipimport # command/easy_install.py
# twisted # twisted
from importlib import invalidate_caches # python/test/test_deprecate.py from importlib import invalidate_caches # noqa:F401, E501 # python/test/test_deprecate.py
# astroid # astroid
import zipimport # manager.py import zipimport # noqa:F401, F811 # manager.py
# IPython # IPython
from importlib.machinery import all_suffixes # core/completerlib.py from importlib.machinery import all_suffixes # noqa:F401, E501 # core/completerlib.py
import importlib # html/notebookapp.py import importlib # noqa:F401 # html/notebookapp.py
from IPython.utils.importstring import import_item # Many files from IPython.utils.importstring import import_item # noqa:F401 # Many files
# pyflakes # pyflakes
# test/test_doctests.py # test/test_doctests.py
from pyflakes.test.test_imports import Test as TestImports from pyflakes.test.test_imports import Test as TestImports # noqa:F401
# Nose # Nose
from nose.importer import Importer, add_path, remove_path # loader.py from nose.importer import Importer, add_path, remove_path # loader.py
@ -49,17 +51,18 @@ import bs4
import nonexistendmodule import nonexistendmodule
import boto as b, peewee as p import boto as b, peewee as p
# import django # import django
import flask.ext.somext # # # import flask.ext.somext # noqa:F401 # # #
from sqlalchemy import model from sqlalchemy import model # noqa:F401
try: try:
import ujson as json import ujson as json
except ImportError: except ImportError:
import json import json # noqa:F401
import models import models # noqa:F401
def main(): def main():
pass pass
import after_method_is_valid_even_if_not_pep8
import after_method_is_valid_even_if_not_pep8 # noqa:F401, E402

View File

@ -1,5 +1,5 @@
import pymongo import pymongo # noqa:F401
from bson.objectid import ObjectId from bson.objectid import ObjectId # noqa:F401
# 'bson' package is mapped to 'pymongo'. # 'bson' package is mapped to 'pymongo'.
# But running pipreqs should not result in two duplicated # But running pipreqs should not result in two duplicated

View File

@ -1,2 +1,2 @@
# Everything in here should be ignored # Everything in here should be ignored
from pattern.web import Twitter, plaintext from pattern.web import Twitter, plaintext # noqa:F401

View File

@ -1,2 +1,2 @@
# Everything in here should be ignored # Everything in here should be ignored
import click import click # noqa:F401

View File

@ -1,61 +1,63 @@
"""unused import""" """unused import"""
# pylint: disable=undefined-all-variable, import-error, no-absolute-import, too-few-public-methods, missing-docstring # pylint: disable=undefined-all-variable, import-error, no-absolute-import
# pylint: disable=too-few-public-methods, missing-docstring
from __future__ import print_function # noqa:F401
import xml.etree # [unused-import] import xml.etree # [unused-import]
import xml.sax # [unused-import] import xml.sax # noqa:F401 # [unused-import]
import os.path as test # [unused-import] import os.path as test # noqa:F401 # [unused-import]
from sys import argv as test2 # [unused-import] from sys import argv as test2 # noqa:F401 # [unused-import]
from sys import flags # [unused-import] from sys import flags # noqa:F401 # [unused-import]
# +1:[unused-import,unused-import] # +1:[unused-import,unused-import]
from collections import deque, OrderedDict, Counter from collections import deque, OrderedDict, Counter # noqa:F401
# All imports above should be ignored # All imports above should be ignored
import requests # [unused-import] import requests # noqa:F401 # [unused-import]
# setuptools # setuptools
import zipimport # command/easy_install.py import zipimport # command/easy_install.py
# twisted # twisted
from importlib import invalidate_caches # python/test/test_deprecate.py from importlib import invalidate_caches # noqa:F401, E501 # python/test/test_deprecate.py
# astroid # astroid
import zipimport # manager.py import zipimport # noqa:F401, F811 # manager.py
# IPython # IPython
from importlib.machinery import all_suffixes # core/completerlib.py from importlib.machinery import all_suffixes # noqa:F401, E501 # core/completerlib.py
import importlib # html/notebookapp.py import importlib # noqa:F401 # html/notebookapp.py
from IPython.utils.importstring import import_item # Many files from IPython.utils.importstring import import_item # noqa:F401 # Many files
# pyflakes # pyflakes
# test/test_doctests.py # test/test_doctests.py
from pyflakes.test.test_imports import Test as TestImports from pyflakes.test.test_imports import Test as TestImports # noqa:F401
# Nose # Nose
from nose.importer import Importer, add_path, remove_path # loader.py from nose.importer import Importer, add_path, remove_path # noqa:F401, E501 # loader.py
import atexit import atexit # noqa:F401
from __future__ import print_function from docopt import docopt # noqa:F401
from docopt import docopt import curses, logging, sqlite3 # noqa:F401, E401
import curses, logging, sqlite3 import logging # noqa:F401, F811
import logging import os # noqa:F401
import os import sqlite3 # noqa:F401, F811
import sqlite3 import time # noqa:F401
import time import sys # noqa:F401
import sys import signal # noqa:F401
import signal import bs4 # noqa:F401
import bs4 import nonexistendmodule # noqa:F401
import nonexistendmodule import boto as b, peewee as p # noqa:F401, E401
import boto as b, peewee as p
# import django # import django
import flask.ext.somext # # # import flask.ext.somext # noqa:F401 # # #
from sqlalchemy import model from sqlalchemy import model # noqa:F401
try: try:
import ujson as json import ujson as json
except ImportError: except ImportError:
import json import json # noqa:F401
import models import models # noqa:F401
def main(): def main():
pass pass
import after_method_is_valid_even_if_not_pep8
import after_method_is_valid_even_if_not_pep8 # noqa:F401, E402

View File

@ -1 +1 @@
import boto as b, import peewee as p, import boto as b, import peewee as p, # noqa:E401, E999

View File

@ -212,7 +212,10 @@ class TestPipreqs(unittest.TestCase):
'--mode': None '--mode': None
} }
) )
with open(os.path.join(self.project_with_ignore_directory, "requirements.txt"), "r") as f: with open(
os.path.join(
self.project_with_ignore_directory, "requirements.txt"
), "r") as f:
data = f.read().lower() data = f.read().lower()
for item in ['click', 'getpass']: for item in ['click', 'getpass']:
self.assertFalse(item.lower() in data) self.assertFalse(item.lower() in data)