This is an automated email from the ASF dual-hosted git repository.
zhengruifeng pushed a commit to branch branch-4.x
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/branch-4.x by this push:
new 2c4c5645a33f [MINOR][PYTHON][TESTS] Rename TVFParityTestsMixin to
TVFParityTests
2c4c5645a33f is described below
commit 2c4c5645a33f3d78f445a0b58c5b40b0be863b26
Author: Ruifeng Zheng <[email protected]>
AuthorDate: Fri May 22 09:45:40 2026 +0800
[MINOR][PYTHON][TESTS] Rename TVFParityTestsMixin to TVFParityTests
### What changes were proposed in this pull request?
Rename the class in `python/pyspark/sql/tests/connect/test_parity_tvf.py`
from `TVFParityTestsMixin` to `TVFParityTests`.
### Why are the changes needed?
`TVFParityTestsMixin` is actually the concrete Connect parity test case for
the file — it inherits from `ReusedConnectTestCase` (a `unittest.TestCase`) and
`TVFTestsMixin`, so the tests are picked up and run by the default `unittest`
loader. The `Mixin` suffix is misleading and inconsistent with every other
Connect parity file (`TypesParityTests`, `ConfParityTests`, etc.), all of which
use a plain `*ParityTests` name. This is a naming-convention fix; no test
behavior changes.
I scanned all 780 registered test modules in
`dev/sparktestsupport/modules.py` for the same pattern (file where every
top-level class is suffixed `Mixin`). `test_parity_tvf.py` is the only
occurrence.
### Does this PR introduce _any_ user-facing change?
No. Test-only change.
### How was this patch tested?
Existing tests. The class and its inherited test cases were already being
collected by `unittest`; only the class name changes.
### Was this patch authored or co-authored using generative AI tooling?
Generated-by: Claude Code claude-opus-4-7
Closes #56037 from zhengruifeng/tvf-parity-rename.
Authored-by: Ruifeng Zheng <[email protected]>
Signed-off-by: Ruifeng Zheng <[email protected]>
(cherry picked from commit 98cc24da0d9b157b073ac8a374182e33b7e0c56f)
Signed-off-by: Ruifeng Zheng <[email protected]>
---
python/pyspark/sql/tests/connect/test_parity_tvf.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/python/pyspark/sql/tests/connect/test_parity_tvf.py
b/python/pyspark/sql/tests/connect/test_parity_tvf.py
index 22676546d27c..c6c68933789d 100644
--- a/python/pyspark/sql/tests/connect/test_parity_tvf.py
+++ b/python/pyspark/sql/tests/connect/test_parity_tvf.py
@@ -19,7 +19,7 @@ from pyspark.sql.tests.test_tvf import TVFTestsMixin
from pyspark.testing.connectutils import ReusedConnectTestCase
-class TVFParityTestsMixin(TVFTestsMixin, ReusedConnectTestCase):
+class TVFParityTests(TVFTestsMixin, ReusedConnectTestCase):
pass
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
