You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by ch...@apache.org on 2018/07/19 08:27:13 UTC
[6/6] flink git commit: [hotfix] Harden
JarUploadHandlerTest#testUploadJar
[hotfix] Harden JarUploadHandlerTest#testUploadJar
Use lastIndexOf("_") instead of indexOf("_") in order to find the file name part
Project: http://git-wip-us.apache.org/repos/asf/flink/repo
Commit: http://git-wip-us.apache.org/repos/asf/flink/commit/78acb7b5
Tree: http://git-wip-us.apache.org/repos/asf/flink/tree/78acb7b5
Diff: http://git-wip-us.apache.org/repos/asf/flink/diff/78acb7b5
Branch: refs/heads/release-1.5
Commit: 78acb7b550e280f244a6322eac2f04187defdc52
Parents: c997829
Author: Till Rohrmann <tr...@apache.org>
Authored: Wed Jul 18 09:48:49 2018 +0200
Committer: zentol <ch...@apache.org>
Committed: Thu Jul 19 10:27:02 2018 +0200
----------------------------------------------------------------------
.../flink/runtime/webmonitor/handlers/JarUploadHandlerTest.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/flink/blob/78acb7b5/flink-runtime-web/src/test/java/org/apache/flink/runtime/webmonitor/handlers/JarUploadHandlerTest.java
----------------------------------------------------------------------
diff --git a/flink-runtime-web/src/test/java/org/apache/flink/runtime/webmonitor/handlers/JarUploadHandlerTest.java b/flink-runtime-web/src/test/java/org/apache/flink/runtime/webmonitor/handlers/JarUploadHandlerTest.java
index 164171d..648e7fc 100644
--- a/flink-runtime-web/src/test/java/org/apache/flink/runtime/webmonitor/handlers/JarUploadHandlerTest.java
+++ b/flink-runtime-web/src/test/java/org/apache/flink/runtime/webmonitor/handlers/JarUploadHandlerTest.java
@@ -106,7 +106,7 @@ public class JarUploadHandlerTest extends TestLogger {
assertThat(jarUploadResponseBody.getStatus(), equalTo(JarUploadResponseBody.UploadStatus.success));
final String returnedFileNameWithUUID = jarUploadResponseBody.getFilename();
assertThat(returnedFileNameWithUUID, containsString("_"));
- final String returnedFileName = returnedFileNameWithUUID.substring(returnedFileNameWithUUID.indexOf("_") + 1);
+ final String returnedFileName = returnedFileNameWithUUID.substring(returnedFileNameWithUUID.lastIndexOf("_") + 1);
assertThat(returnedFileName, equalTo(uploadedFile.getFileName().toString()));
}