itholic commented on code in PR #38422:
URL: https://github.com/apache/spark/pull/38422#discussion_r1009604533
##########
R/pkg/tests/fulltests/test_sparkSQL.R:
##########
@@ -3990,12 +3990,21 @@ test_that("Call DataFrameWriter.load() API in Java
without path and check argume
expect_error(read.df(source = "json"),
paste("Error in load : analysis error - Unable to infer schema
for JSON.",
"It must be specified manually"))
- expect_error(read.df("arbitrary_path"), "Error in load : analysis error -
Path does not exist")
- expect_error(read.json("arbitrary_path"), "Error in json : analysis error -
Path does not exist")
- expect_error(read.text("arbitrary_path"), "Error in text : analysis error -
Path does not exist")
- expect_error(read.orc("arbitrary_path"), "Error in orc : analysis error -
Path does not exist")
+ expect_error(read.df("arbitrary_path"),
Review Comment:
Yeah, I totally agree that the current approach too easy to break so we'd
better to find nicer way, but I have no idea to test SparkR in my test
environment for now.
Let me manually fix the text first, while figuring out a better way to test
it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
