Skip to content

Commit

Permalink
fix tests after renamings
Browse files Browse the repository at this point in the history
  • Loading branch information
Hendrik Muhs committed Feb 28, 2020
1 parent f957b34 commit 1a80758
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,10 @@ public void testNodeAttributes() {
Transform transform = createTransform(builder.build());
assertNotNull(transform.additionalSettings());
assertEquals(transformEnabled, Boolean.parseBoolean(transform.additionalSettings().get("node.attr.transform.node")));
assertEquals(remoteEnabled, Boolean.parseBoolean(transform.additionalSettings().get("node.attr.transform.remote_connect")));
assertEquals(
transformEnabled && remoteEnabled,
Boolean.parseBoolean(transform.additionalSettings().get("node.attr.transform.remote_connect"))
);
}

public void testNodeAttributesDirectlyGiven() {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ public void testNodeAssignmentProblems() {
assertNull(assignment.getExecutorNode());
assertThat(
assignment.getExplanation(),
equalTo("Not starting transform [new-task-id], reasons [current-data-node-with-transform-disabled:transform not enabled]")
equalTo("Not starting transform [new-task-id], reasons [current-data-node-with-transform-disabled:not a transform node]")
);

// dedicated transform node
Expand All @@ -108,7 +108,7 @@ public void testNodeAssignmentProblems() {
assignment.getExplanation(),
equalTo(
"Not starting transform [new-task-id], reasons ["
+ "current-data-node-with-transform-disabled:transform not enabled"
+ "current-data-node-with-transform-disabled:not a transform node"
+ "|"
+ "past-data-node-1:node has version: 7.5.0 but transform requires at least 7.7.0"
+ "]"
Expand Down Expand Up @@ -154,7 +154,7 @@ public void testNodeAssignmentProblems() {
+ "current-data-node-with-0-tasks-transform-remote-disabled:"
+ "transform requires a remote connection but remote is disabled"
+ "|"
+ "current-data-node-with-transform-disabled:transform not enabled"
+ "current-data-node-with-transform-disabled:not a transform node"
+ "]"
)
);
Expand Down

0 comments on commit 1a80758

Please sign in to comment.