diff --git a/pyproject.toml b/pyproject.toml index 055e6b8..360ffdc 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -229,7 +229,6 @@ ignore = [ "RUF015", # unnecessary-iterable-allocation-for-first-element "D417", # undocumented-param "RUF007", # zip-instead-of-pairwise - "D415", # missing-terminal-punctuation "RUF043", # pytest-raises-ambiguous-pattern "D205", # missing-blank-line-after-summary "D210", # surrounding-whitespace diff --git a/test/unit/domains/test_base.py b/test/unit/domains/test_base.py index 61c2ee9..bf07206 100644 --- a/test/unit/domains/test_base.py +++ b/test/unit/domains/test_base.py @@ -11,7 +11,7 @@ class NewDomain(Domain): - """New Domain class that inherits from Domain""" + """New Domain class that inherits from Domain.""" @property def carrier_type(self) -> type: diff --git a/test/unit/domains/test_spark_domains.py b/test/unit/domains/test_spark_domains.py index fdf568f..913031d 100644 --- a/test/unit/domains/test_spark_domains.py +++ b/test/unit/domains/test_spark_domains.py @@ -1233,7 +1233,7 @@ class TestSparkColumnDescriptors: @pytest.fixture def test_df(self) -> DataFrame: - """Get a base DataFrame""" + """Get a base DataFrame.""" return self.spark.createDataFrame( [ ( diff --git a/test/unit/measurements/test_interactive_measurements.py b/test/unit/measurements/test_interactive_measurements.py index a5de3e8..e4efa8a 100644 --- a/test/unit/measurements/test_interactive_measurements.py +++ b/test/unit/measurements/test_interactive_measurements.py @@ -1467,7 +1467,7 @@ def test_split(self): self.assertEqual(child.state, expected_state) def test_queue_transformation_on_active_accountant(self): - """queue_transformation runs immediately on active accountant""" + """queue_transformation runs immediately on active accountant.""" accountant = PrivacyAccountant.launch( measurement=self.measurement, data=self.data ) @@ -1495,7 +1495,7 @@ def test_queue_transformation_on_active_accountant(self): self.assertIsNone(accountant._pending_transformation) def test_queue_transformation_on_inactive_accountant(self): - """queue_transformation queues transformations on inactive account""" + """queue_transformation queues transformations on inactive account.""" accountant = PrivacyAccountant.launch( measurement=self.measurement, data=self.data ) diff --git a/test/unit/test_metrics.py b/test/unit/test_metrics.py index 5ec9ea4..c025d84 100644 --- a/test/unit/test_metrics.py +++ b/test/unit/test_metrics.py @@ -59,12 +59,12 @@ class TestNullMetric(TestCase): """TestCase for NullMetric.""" def test_valid(self): - """Validate is not implemented""" + """Validate is not implemented.""" with self.assertRaises(NotImplementedError): NullMetric().validate(3) def test_compare(self): - """Compare is not implemented""" + """Compare is not implemented.""" with self.assertRaises(NotImplementedError): NullMetric().compare(3, 2) @@ -1927,7 +1927,7 @@ def test_distance(self, metric: Metric, df1: Any, df2: Any, distance: Any): class TestDictMetric(TestCase): - """TestCase for DictMetric""" + """TestCase for DictMetric.""" def test_constructor_mutable_arguments(self): """Tests that mutable constructor arguments are copied.""" @@ -2125,7 +2125,7 @@ def test_distance_empty(self): class TestAddRemoveKeys(PySparkTest): - """TestCase for AddRemoveKeys""" + """TestCase for AddRemoveKeys.""" @parameterized.expand( [ diff --git a/test/unit/transformations/spark_transformations/test_join.py b/test/unit/transformations/spark_transformations/test_join.py index f7ae50b..cf56c71 100644 --- a/test/unit/transformations/spark_transformations/test_join.py +++ b/test/unit/transformations/spark_transformations/test_join.py @@ -1370,7 +1370,7 @@ def test_columns_ordering( def test_stability_relation_and_function( self, d_in: ExactNumberInput, d_out: ExactNumberInput, expected: bool ): - """Test that PrivateJoinOnKey's stability relation and function are correct""" + """Test that PrivateJoinOnKey's stability relation and function are correct.""" private_join = PrivateJoinOnKey( input_domain=DictDomain( {"left": self.left_domain, "right": self.right_domain} diff --git a/test/unit/utils/test_arb.py b/test/unit/utils/test_arb.py index 0ae6e33..7e02d07 100644 --- a/test/unit/utils/test_arb.py +++ b/test/unit/utils/test_arb.py @@ -1,4 +1,4 @@ -"""Test for :mod:`tmlt.core.utils.arb`""" +"""Test for :mod:`tmlt.core.utils.arb`.""" # SPDX-License-Identifier: Apache-2.0 diff --git a/test/unit/utils/test_cleanup.py b/test/unit/utils/test_cleanup.py index 579f7f2..187f791 100644 --- a/test/unit/utils/test_cleanup.py +++ b/test/unit/utils/test_cleanup.py @@ -48,7 +48,7 @@ def setUp(self): @staticmethod def _recursive_remove(p: Path): - """Recursively remove a path (just like `rm -r`)""" + """Recursively remove a path (just like `rm -r`).""" if not p.is_dir(): p.unlink() for f in p.iterdir(): diff --git a/test/unit/utils/test_misc.py b/test/unit/utils/test_misc.py index 2d39fea..0bc236c 100644 --- a/test/unit/utils/test_misc.py +++ b/test/unit/utils/test_misc.py @@ -1,4 +1,4 @@ -"""Test for :mod:`tmlt.core.utils.misc`""" +"""Test for :mod:`tmlt.core.utils.misc`.""" # SPDX-License-Identifier: Apache-2.0 # Copyright Tumult Labs 2025 diff --git a/test/unit/utils/test_testing.py b/test/unit/utils/test_testing.py index f558f34..ed77a12 100644 --- a/test/unit/utils/test_testing.py +++ b/test/unit/utils/test_testing.py @@ -1,4 +1,4 @@ -"""Test for :mod:`tmlt.core.utils.testing`""" +"""Test for :mod:`tmlt.core.utils.testing`.""" # SPDX-License-Identifier: Apache-2.0 # Copyright Tumult Labs 2025