Fix pre-commit configuration and update files

* Fix flake8 URL
* Update hooks to latest versions

Author: nicoddemus

.pre-commit-config.yaml

@@ -1,6 +1,6 @@
 repos:
 -   repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v3.4.0
+    rev: v4.5.0
     hooks:
     -   id: check-byte-order-marker
     -   id: trailing-whitespace
@@ -9,19 +9,19 @@ repos:
         args: [--remove]
     -   id: check-yaml
 -   repo: https://github.com/asottile/reorder_python_imports
-    rev: v2.4.0
+    rev: v3.12.0
     hooks:
     -   id: reorder-python-imports
         args: ['--application-directories=.:src', --py3-plus]
 -   repo: https://github.com/python/black
-    rev: 20.8b1
+    rev: 23.10.0
     hooks:
       - id: black
--   repo: https://gitlab.com/pycqa/flake8
-    rev: 3.8.4
+-   repo: https://github.com/PyCQA/flake8
+    rev: 6.1.0
     hooks:
     -   id: flake8
-    additional_dependencies: [flake8-bugbear]
+        additional_dependencies: [flake8-bugbear]
 -   repo: local
     hooks:
     -   id: rst
@@ -31,6 +31,6 @@ repos:
         language: python
         additional_dependencies: [pygments, restructuredtext_lint]
 -   repo: https://github.com/myint/autoflake.git
-    rev: v1.4
+    rev: v2.2.1
     hooks:
     -   id: autoflake

docs/conf.py

@@ -12,7 +12,6 @@
 # serve to show the default.
 import datetime
 import os
-import sys
 
 # If extensions (or modules to document with autodoc) are in another directory,
 # add these directories to sys.path here. If the directory is relative to the
@@ -42,8 +41,8 @@
 master_doc = "index"
 
 # General information about the project.
-project = u"pytest-flask"
-copyright = u"%d, Vital Kudzelka and contributors" % datetime.date.today().year
+project = "pytest-flask"
+copyright = "%d, Vital Kudzelka and contributors" % datetime.date.today().year
 
 # The version info for the project you're documenting, acts as replacement for
 # |version| and |release|, also used in various other places throughout the
@@ -203,8 +202,8 @@
     (
         "index",
         "pytest-flask.tex",
-        u"pytest-flask Documentation",
-        u"Vital Kudzelka",
+        "pytest-flask Documentation",
+        "Vital Kudzelka",
         "manual",
     ),
 ]
@@ -235,7 +234,7 @@
 # One entry per manual page. List of tuples
 # (source start file, name, description, authors, manual section).
 man_pages = [
-    ("index", "pytest-flask", u"pytest-flask Documentation", [u"Vital Kudzelka"], 1)
+    ("index", "pytest-flask", "pytest-flask Documentation", ["Vital Kudzelka"], 1)
 ]
 
 # If true, show URL addresses after external links.
@@ -251,8 +250,8 @@
     (
         "index",
         "pytest-flask",
-        u"pytest-flask Documentation",
-        u"Vital Kudzelka",
+        "pytest-flask Documentation",
+        "Vital Kudzelka",
         "pytest-flask",
         "One line description of project.",
         "Miscellaneous",

src/pytest_flask/fixtures.py

@@ -1,6 +1,5 @@
 #!/usr/bin/env python
 import socket
-import warnings
 
 import pytest
 

src/pytest_flask/live_server.py

@@ -85,8 +85,8 @@ def stop(self):
                 self._process.terminate()
 
     def _stop_cleanly(self, timeout=5):
-        """Attempts to stop the server cleanly by sending a SIGINT signal and waiting for
-        ``timeout`` seconds.
+        """Attempts to stop the server cleanly by sending a SIGINT
+        signal and waiting for ``timeout`` seconds.
 
         :return: True if the server was cleanly stopped, False otherwise.
         """

tests/test_internal.py

@@ -11,5 +11,5 @@ def deprecated_fun():
 
         with pytest.warns(DeprecationWarning) as record:
             deprecated_fun()
-            assert len(record) == 1
-            assert record[0].message.args[0] == "testing decorator"
+        assert len(record) == 1
+        assert record[0].message.args[0] == "testing decorator"

tests/test_live_server.py

@@ -87,8 +87,8 @@ def test_stop_cleanly_join_exception(self, appdir, live_server, caplog):
     def test_clean_stop_live_server(self, appdir, monkeypatch, clean_stop):
         """Ensure the fixture is trying to cleanly stop the server.
 
-        Because this is tricky to test, we are checking that the _stop_cleanly() internal
-        function was called and reported success.
+        Because this is tricky to test, we are checking that the
+        _stop_cleanly() internal function was called and reported success.
         """
         from pytest_flask.fixtures import LiveServer