[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

domdf_python_tools/dates.py

@@ -383,7 +383,7 @@ def is_bst(the_date: Union[time.struct_time, datetime.date]) -> bool:
 		'{word_join(_pytz_functions)}' require pytz (https://pypi.org/project/pytz/), but it could not be imported.
 
 		The error was: {e}.
-		"""
+		""",
 				)
 
 	else:
@@ -394,7 +394,7 @@ class SelfWrapper(ModuleType):
 			def __getattr__(self, name):
 				if name in _pytz_functions:
 					raise ImportError(
-							f"{name!r} requires pytz (https://pypi.org/project/pytz/), but it could not be imported."
+							f"{name!r} requires pytz (https://pypi.org/project/pytz/), but it could not be imported.",
 							)
 				else:
 					return getattr(_actual_module, name)

domdf_python_tools/delegators.py

@@ -83,7 +83,7 @@ def _f(f: _C) -> _C:
 					del to_params[param]
 
 			f.__signature__ = from_sig.replace(  # type: ignore[attr-defined]
-				parameters=[*from_params.values(), *to_params.values()]
+				parameters=[*from_params.values(), *to_params.values()],
 				)
 			f.__annotations__ = {**to_annotations, **from_annotations}
 
@@ -121,7 +121,7 @@ def _f(f: _C) -> _C:
 
 		elif tuple(from_params.keys()) == ("self", "args", "kwargs"):
 			f.__signature__ = from_sig.replace(  # type: ignore[attr-defined]
-				parameters=[from_params["self"], *to_sig.parameters.values()]
+				parameters=[from_params["self"], *to_sig.parameters.values()],
 				)
 
 			copy_annotations(f)

domdf_python_tools/import_tools.py

@@ -53,7 +53,7 @@
 		"discover_in_module",
 		"discover_entry_points",
 		"discover_entry_points_by_name",
-		"iter_submodules"
+		"iter_submodules",
 		]
 
 

domdf_python_tools/paths.py

@@ -517,7 +517,7 @@ def write_lines(
 			encoding: Optional[str] = "UTF-8",
 			errors: Optional[str] = None,
 			*,
-			trailing_whitespace: bool = False
+			trailing_whitespace: bool = False,
 			) -> None:
 		"""
 		Write the given list of lines to the file without trailing whitespace.
@@ -585,7 +585,7 @@ def open(  # type: ignore  # noqa: A003  # pylint: disable=redefined-builtin
 		encoding: Optional[str] = "UTF-8",
 		errors: Optional[str] = None,
 		newline: Optional[str] = NEWLINE_DEFAULT,
-		) -> IO[Any]:
+	) -> IO[Any]:
 		"""
 		Open the file pointed by this path and return a file object, as
 		the built-in :func:`open` function does.
@@ -1252,7 +1252,7 @@ def phase4(self) -> None:  # noqa: D102
 			"left_only": filecmp.dircmp.phase1,
 			"right_only": filecmp.dircmp.phase1,
 			"left_list": filecmp.dircmp.phase0,
-			"right_list": filecmp.dircmp.phase0
+			"right_list": filecmp.dircmp.phase0,
 			}
 
 	methodmap = _methodmap  # type: ignore[assignment]

domdf_python_tools/pretty_print.py

@@ -150,7 +150,7 @@ def _pprint_dict(
 				allowance + 1,
 				context,
 				level,
-				)
+			)
 
 		write(self._make_close('}', indent, obj))
 
@@ -250,7 +250,7 @@ def _format_attribute_items(self, items, stream, indent, allowance, context, lev
 				allowance if last else 1,
 				context,
 				level,
-				)
+			)
 
 			if not last:
 				write(delimnl)

domdf_python_tools/stringlist.py

@@ -252,7 +252,7 @@ def __setitem__(self, index: Union[SupportsIndex, slice], line: Union[String, It
 			for lline, idx in zip(
 				reversed(line),
 				reversed(range(index.start or 0, index.stop + 1, index.step or 1)),
-				):
+			):
 				self[idx] = lline
 		else:
 			line = cast(String, line)

domdf_python_tools/versions.py

@@ -303,7 +303,7 @@ def _iter_float(version_float: float) -> Generator[int, None, None]:
 	return _iter_string(str(version_float))
 
 
-def _prep_for_eq(other: Union[str, float, Version], ) -> Tuple[int, ...]:
+def _prep_for_eq(other: Union[str, float, Version]) -> Tuple[int, ...]:
 	"""
 	Prepare 'other' for use in ``__eq__``, ``__le__``, ``__ge__``, ``__gt__``, and ``__lt__``.
 	"""

tests/test_dates.py

@@ -246,7 +246,7 @@ def test_get_month_number_from_no(count: int):
 				('0', "The given month ('0') is not recognised."),
 				("-1", "The given month ('-1') is not recognised."),
 				("13", "The given month ('13') is not recognised."),
-				]
+				],
 		)
 def test_get_month_number_errors(value: Union[str, int], match: str):
 	with pytest.raises(ValueError, match=re.escape(match)):
@@ -321,7 +321,7 @@ def test_check_date():
 				date(2019, 4, 21),
 				date(2020, 4, 12),
 				date(2021, 4, 4),
-				]
+				],
 		)
 def test_calc_easter(date):
 	assert dates.calc_easter(date.year) == date
@@ -382,7 +382,7 @@ def test_calc_easter(date):
 				(date(month=10, day=26, year=2024), True),
 				(date(month=10, day=27, year=2024), False),
 				(date(month=10, day=28, year=2024), False),
-				]
+				],
 		)
 def test_is_bst(the_date, result: bool):
 	assert dates.is_bst(the_date) is result

tests/test_dir_comparator.py

@@ -71,26 +71,26 @@ def test_cmpfiles(self, comparator_tmpdir):
 			comparator_tmpdir.dir,
 			comparator_tmpdir.dir,
 			["file"],
-			) == (["file"], [], []), "Comparing directory to itself fails"
+		) == (["file"], [], []), "Comparing directory to itself fails"
 		assert filecmp.cmpfiles(
 			comparator_tmpdir.dir,
 			comparator_tmpdir.dir_same,
 			["file"],
-			) == (["file"], [], []), "Comparing directory to same fails"
+		) == (["file"], [], []), "Comparing directory to same fails"
 
 		# Try it with shallow=False
 		assert filecmp.cmpfiles(
 			comparator_tmpdir.dir,
 			comparator_tmpdir.dir,
 			["file"],
 			shallow=False,
-			) == (["file"], [], []), "Comparing directory to itself fails"
+		) == (["file"], [], []), "Comparing directory to itself fails"
 		assert filecmp.cmpfiles(
 			comparator_tmpdir.dir,
 			comparator_tmpdir.dir_same,
 			["file"],
 			shallow=False,
-			), "Comparing directory to same fails"
+		), "Comparing directory to same fails"
 
 		# Add different file2
 		with open(os.path.join(comparator_tmpdir.dir, "file2"), 'w', encoding="UTF-8") as output:
@@ -100,7 +100,7 @@ def test_cmpfiles(self, comparator_tmpdir):
 			comparator_tmpdir.dir,
 			comparator_tmpdir.dir_same,
 			["file", "file2"],
-			) != (["file"], ["file2"], []), "Comparing mismatched directories fails"
+		) != (["file"], ["file2"], []), "Comparing mismatched directories fails"
 
 	def _assert_lists(self, actual, expected):
 		"""

tests/test_doctools.py

@@ -194,7 +194,7 @@ def function_in_class_with_same_args(self, a, b, c, d):
 				(False, ''),
 				(0, ''),
 				([], ''),
-				]
+				],
 		)
 def test_deindent_string(docstring, expects):
 	assert doctools.deindent_string(docstring) == expects
@@ -210,13 +210,15 @@ def test_decorators():
 
 	# set_opening_hours and ceil should have extra text at the beginning
 	assert (SpamCafe.set_opening_hours.__doc__ or '').startswith("I will not buy this record, it is scratched.")
-	assert (doctools.deindent_string(SpamCafe.set_opening_hours.__doc__
-										)).endswith(doctools.deindent_string(Cafe.set_opening_hours.__doc__))
+	assert (doctools.deindent_string(SpamCafe.set_opening_hours.__doc__, )).endswith(
+			doctools.deindent_string(Cafe.set_opening_hours.__doc__)
+			)
 	# Dedented both strings to be sure of equivalence
-	assert (SpamCafe.ceil.__doc__ or ''
-			).startswith("I don't know why the cafe has a ceil function, but we'd better document it properly.", )
-	assert doctools.deindent_string(SpamCafe.ceil.__doc__
-									).rstrip().endswith(doctools.deindent_string(math.ceil.__doc__).rstrip())
+	assert (SpamCafe.ceil.__doc__ or
+			'').startswith("I don't know why the cafe has a ceil function, but we'd better document it properly.")
+	assert doctools.deindent_string(SpamCafe.ceil.__doc__, ).rstrip().endswith(
+			doctools.deindent_string(math.ceil.__doc__).rstrip()
+			)
 	# Dedented both strings to be sure of equivalence
 
 	# Functions
@@ -225,18 +227,27 @@ def test_decorators():
 
 	if PEP_563:
 		assert undocumented_function.__annotations__ == {
-				'a': "float", 'b': "float", 'c': "float", 'd': "int", "return": "float"
+				'a': "float",
+				'b': "float",
+				'c': "float",
+				'd': "int",
+				"return": "float",
 				}
 	else:
 		assert undocumented_function.__annotations__ == {
-				'a': float, 'b': float, 'c': float, 'd': int, "return": float
+				'a': float,
+				'b': float,
+				'c': float,
+				'd': int,
+				"return": float,
 				}
 
 	assert (partially_documented_function.__doc__ or '').startswith(
-			"This function works like ``documented_function`` except it returns the result telepathically."
+			"This function works like ``documented_function`` except it returns the result telepathically.",
+			)
+	assert (doctools.deindent_string(partially_documented_function.__doc__, )).endswith(
+			doctools.deindent_string(documented_function.__doc__)
 			)
-	assert (doctools.deindent_string(partially_documented_function.__doc__
-										)).endswith(doctools.deindent_string(documented_function.__doc__))
 	# Dedented both strings to be sure of equivalence
 	assert DummyClass.function_in_class_with_same_args.__doc__ == documented_function.__doc__
 	assert DummyClass.function_in_class_with_same_args.__name__ == "function_in_class_with_same_args"