Skip to content

Commit 61e23ef

Browse files
heyihongHyukjinKwon
authored andcommitted
[SPARK-51819][PYTHON] Update pyspark-errors test module to include missing tests
### What changes were proposed in this pull request? - Fix broken tests - Update pyspark-errors test module to include missing tests ### Why are the changes needed? - Improve test coverage ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Existing tests ### Was this patch authored or co-authored using generative AI tooling? No Closes #50608 from heyihong/SPARK-51819. Authored-by: Yihong He <[email protected]> Signed-off-by: Hyukjin Kwon <[email protected]>
1 parent cd47c3c commit 61e23ef

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

dev/sparktestsupport/modules.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1466,6 +1466,7 @@ def __hash__(self):
14661466
],
14671467
python_test_goals=[
14681468
# unittests
1469+
"pyspark.errors.tests.test_connect_errors_conversion",
14691470
"pyspark.errors.tests.test_errors",
14701471
"pyspark.errors.tests.test_traceback",
14711472
"pyspark.errors.tests.connect.test_parity_traceback",

python/pyspark/errors/tests/test_connect_errors_conversion.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,10 @@ def test_convert_exception_with_stacktrace(self):
145145
}
146146
truncated_message = "Root error message"
147147
exception = convert_exception(
148-
info=ErrorInfo(**info), truncated_message=truncated_message, resp=resp
148+
info=ErrorInfo(**info),
149+
truncated_message=truncated_message,
150+
resp=resp,
151+
display_server_stacktrace=True,
149152
)
150153

151154
self.assertIsInstance(exception, SparkConnectGrpcException)

0 commit comments

Comments
 (0)