alirezazamani commented on a change in pull request #1520:
URL: https://github.com/apache/helix/pull/1520#discussion_r520144565
##########
File path:
helix-rest/src/test/java/org/apache/helix/rest/server/TestJobAccessor.java
##########
@@ -178,8 +178,8 @@ public void testGetAddJobContent() throws IOException {
@Test(dependsOnMethods = "testGetAddJobContent")
public void testInvalidGetAndUpdateJobContentStore() {
System.out.println("Start test :" + TestHelper.getTestMethodName());
- String validURI = "clusters/" + CLUSTER_NAME +
"/workflows/Workflow_0/jobs/Job_0/userContent";
- String invalidURI1 = "clusters/" + CLUSTER_NAME +
"/workflows/xxx/jobs/Job_0/userContent"; // workflow not exist
+ String validURI = "clusters/" + CLUSTER_NAME +
"/workflows/Workflow_0/jobs/JOB0/userContent";
+ String invalidURI1 = "clusters/" + CLUSTER_NAME +
"/workflows/xxx/jobs/JOB0/userContent"; // workflow not exist
Review comment:
See the comments above for an explanation of this change.
##########
File path:
helix-rest/src/test/java/org/apache/helix/rest/server/TestTaskAccessor.java
##########
@@ -38,8 +38,8 @@
@Test
public void testGetAddTaskUserContent() throws IOException {
System.out.println("Start test :" + TestHelper.getTestMethodName());
- String uri = "clusters/" + CLUSTER_NAME +
"/workflows/Workflow_0/jobs/Job_0/tasks/0/userContent";
- String uriTaskDoesNotExist = "clusters/" + CLUSTER_NAME +
"/workflows/Workflow_0/jobs/Job_0/tasks/xxx/userContent";
+ String uri = "clusters/" + CLUSTER_NAME +
"/workflows/Workflow_0/jobs/JOB0/tasks/0/userContent";
+ String uriTaskDoesNotExist = "clusters/" + CLUSTER_NAME +
"/workflows/Workflow_0/jobs/JOB0/tasks/xxx/userContent";
Review comment:
See the comments above for an explanation of this change.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
