GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/19332
[SPARK-22093][TESTS] Fixes `assume` in the tests for `Utils.resolveURIs`
and `Utils.resolveURI`
## What changes were proposed in this pull request?
This PR proposes to remove `assume` in `Utils.resolveURIs` and replace
`assume` to `assert` in `Utils.resolveURI` in the test cases in `UtilsSuite`.
It looks `Utils.resolveURIs` supports multiple but also single paths as
input. So, it looks not meaningful to check if the input has `,`.
For the test for `Utils.resolveURI`, I replaced it to `assert` because it
looks taking single path and in order to prevent future mistakes when adding
more tests here.
## How was this patch tested?
Fixed unit tests.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/HyukjinKwon/spark SPARK-22093
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/19332.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #19332
----
commit beaedb09d7b8456f82763b21756f0cd46a9b33fd
Author: hyukjinkwon <gurwls...@gmail.com>
Date: 2017-09-23T10:51:22Z
Fixes `assume` in the tests for Utils.resolveURIs and Utils.resolveURI
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
