Skip to content

Commit

Permalink
feat(core): lowercase mocked flowId / namespace for it to comply with…
Browse files Browse the repository at this point in the history
… flow validation
  • Loading branch information
brian-mulier-p committed Dec 14, 2023
1 parent 546bbc7 commit 3755bb8
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions core/src/main/java/io/kestra/core/utils/TestsUtils.java
Original file line number Diff line number Diff line change
Expand Up @@ -94,8 +94,8 @@ public static Flow mockFlow() {

private static Flow mockFlow(StackTraceElement caller) {
return Flow.builder()
.namespace(caller.getClassName())
.id(caller.getMethodName())
.namespace(caller.getClassName().toLowerCase())
.id(caller.getMethodName().toLowerCase())
.revision(1)
.build();
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ void exception() {

conditionService.valid(flow, conditions, conditionContext);

LogEntry matchingLog = TestsUtils.awaitLog(logs, logEntry -> logEntry.getNamespace().equals("io.kestra.core.services.ConditionServiceTest") && logEntry.getFlowId().equals("exception"));
LogEntry matchingLog = TestsUtils.awaitLog(logs, logEntry -> logEntry.getNamespace().equals("io.kestra.core.services.conditionservicetest") && logEntry.getFlowId().equals("exception"));
assertThat(matchingLog, notNullValue());
}
}

0 comments on commit 3755bb8

Please sign in to comment.