Add some linter to pre-commit configuration and format the code

Author: hramezani

.pre-commit-config.yaml

@@ -3,4 +3,41 @@ repos:
     rev: 22.3.0
     hooks:
     -   id: black
-        args: [--safe, --quiet]
+        args: [--safe, --quiet]
+
+-   repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v4.2.0
+    hooks:
+    -   id: trailing-whitespace
+    -   id: end-of-file-fixer
+    -   id: fix-encoding-pragma
+        args: [--remove]
+    -   id: check-yaml
+        language_version: python3
+
+-   repo: https://github.com/myint/autoflake
+    rev: v1.4
+    hooks:
+    -   id: autoflake
+        name: autoflake
+        args: ["--in-place", "--remove-unused-variables", "--remove-all-unused-imports"]
+        language: python
+        files: \.py$
+
+-   repo: https://github.com/PyCQA/flake8
+    rev: 4.0.1
+    hooks:
+    -   id: flake8
+        language_version: python3
+
+-   repo: https://github.com/asottile/reorder_python_imports
+    rev: v3.1.0
+    hooks:
+    -   id: reorder-python-imports
+        args: [--py3-plus]
+
+-   repo: https://github.com/asottile/pyupgrade
+    rev: v2.32.1
+    hooks:
+    -   id: pyupgrade
+        args: [--py3-plus]

README.rst

@@ -9,6 +9,10 @@
 .. image:: https://img.shields.io/badge/code%20style-black-000000.svg
     :target: https://github.com/psf/black
 
+.. image:: https://results.pre-commit.ci/badge/github/pytest-dev/pytest-localserver/master.svg
+   :target: https://results.pre-commit.ci/latest/github/pytest-dev/pytest-localserver/master
+   :alt: pre-commit.ci status
+
 ==================
 pytest-localserver
 ==================

pytest_localserver/__init__.py

@@ -1 +1 @@
-from pytest_localserver._version import version as VERSION
+from pytest_localserver._version import version as VERSION  # noqa

pytest_localserver/http.py

@@ -2,7 +2,6 @@
 #
 # This program is release under the MIT license. You can find the full text of
 # the license in the LICENSE file.
-
 import enum
 import itertools
 import json
@@ -11,7 +10,8 @@
 
 from werkzeug.datastructures import Headers
 from werkzeug.serving import make_server
-from werkzeug.wrappers import Response, Request
+from werkzeug.wrappers import Request
+from werkzeug.wrappers import Response
 
 
 class WSGIServer(threading.Thread):

pytest_localserver/https.py

@@ -2,7 +2,6 @@
 #
 # This program is release under the MIT license. You can find the full text of
 # the license in the LICENSE file.
-
 import os.path
 
 from pytest_localserver.http import ContentServer
@@ -108,7 +107,7 @@ class SecureContentServer(ContentServer):
 
     A more advanced tutorial can be found `here`_.
 
-    .. _pytest-localserver CA: https://raw.githubusercontent.com/pytest-dev/pytest-localserver/master/pytest_localserver/ca.crt
+    .. _pytest-localserver CA: https://raw.githubusercontent.com/pytest-dev/pytest-localserver/master/pytest_localserver/ca.crt  # noqa: E501
     .. _pyOpenSSH: https://launchpad.net/pyopenssl
     """
 

pytest_localserver/smtp.py

@@ -5,10 +5,9 @@
 # SmtpMailsink Copyright 2005 Aviarc Corporation
 # Written by Adam Feuer, Matt Branthwaite, and Troy Frever
 # which is Licensed under the PSF License
+import email
 
 import aiosmtpd.controller
-import email
-import sys
 
 
 class MessageDetails:

runtests.py

@@ -2658,10 +2658,9 @@
 p2dPWv10xmXLnHQLlQrVD1yYQwb9/yTK0Oq4oxv6j0jKNxLPbTMi6d3/e5X+FyKT7kc=
 """
 
-import sys
-import base64
-import zlib
-import imp
+import sys  # noqa: E402
+import base64  # noqa: E402
+import zlib  # noqa: E402
 
 
 class DictImporter:
@@ -2693,7 +2692,7 @@ def load_module(self, fullname):
         if is_pkg:
             module.__path__ = [fullname]
 
-        do_exec(co, module.__dict__)
+        do_exec(co, module.__dict__)  # noqa: F821
         return sys.modules[fullname]
 
     def get_source(self, name):
@@ -2714,4 +2713,4 @@ def get_source(self, name):
     sys.meta_path.insert(0, importer)
 
     entry = "import py; raise SystemExit(py.test.cmdline.main())"
-    do_exec(entry, locals())
+    do_exec(entry, locals())  # noqa: F821

setup.py

@@ -1,5 +1,5 @@
-from setuptools import setup, Command
-import sys
+from setuptools import Command
+from setuptools import setup
 
 
 def read(fname):
@@ -20,7 +20,8 @@ def finalize_options(self):
         pass
 
     def run(self):
-        import sys, subprocess
+        import subprocess
+        import sys
 
         errno = subprocess.call([sys.executable, "runtests.py"])
         raise SystemExit(errno)

tests/test_http.py

@@ -1,8 +1,10 @@
 import itertools
+
 import pytest
 import requests
 
-from pytest_localserver import http, plugin
+from pytest_localserver import http
+from pytest_localserver import plugin
 
 
 # define test fixture here again in order to run tests without having to
@@ -36,7 +38,7 @@ def test_some_content_retrieval(httpserver):
 def test_request_is_stored(httpserver):
     httpserver.serve_content("TEST!")
     assert len(httpserver.requests) == 0
-    resp = requests.get(httpserver.url)
+    requests.get(httpserver.url)
     assert len(httpserver.requests) == 1
 
 
@@ -160,7 +162,7 @@ def test_GET_request_not_chunked(httpserver, transfer_encoding_header):
         chunked=http.Chunked.NO,
     )
     with pytest.raises(requests.exceptions.ChunkedEncodingError):
-        resp = requests.get(httpserver.url, headers={"User-Agent": "Test method"})
+        requests.get(httpserver.url, headers={"User-Agent": "Test method"})
 
 
 @pytest.mark.parametrize("chunked_flag", [http.Chunked.NO, http.Chunked.AUTO])

tests/test_https.py

@@ -1,9 +1,7 @@
-import sys
-
-import pytest
 import requests
 
-from pytest_localserver import https, plugin
+from pytest_localserver import https
+from pytest_localserver import plugin
 
 
 # define test fixture here again in order to run tests without having to