Skip to content

Commit ae7d64f

Browse files
committed
Add tests
1 parent 2f28bd5 commit ae7d64f

File tree

1 file changed

+5
-8
lines changed

1 file changed

+5
-8
lines changed

test/WorkflowCore.UnitTests/Services/WorkflowMiddlewareRunnerTests.cs

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -274,7 +274,7 @@ public async Task RunExecuteMiddleware_should_run_all_middleware_when_multiple_m
274274
Middleware.AddRange(new[] { middleware1, middleware2, middleware3 });
275275

276276
// Act
277-
await Runner.RunPostMiddleware(Workflow, Definition);
277+
await Runner.RunExecuteMiddleware(Workflow, Definition);
278278

279279
// Assert
280280
A
@@ -288,8 +288,8 @@ public async Task RunExecuteMiddleware_should_run_all_middleware_when_multiple_m
288288
.MustHaveHappenedOnceExactly());
289289
}
290290

291-
[Fact(DisplayName = "RunExecuteMiddleware should run middleware in ExecuteWorkflow and PostWorkflow phase")]
292-
public async Task RunExecuteMiddleware_should_run_middleware_in_ExecuteWorkflow_and_PostWorkflow_phase()
291+
[Fact(DisplayName = "RunExecuteMiddleware should only run middleware in ExecuteWorkflow phase")]
292+
public async Task RunExecuteMiddleware_should_only_run_middleware_in_ExecuteWorkflow_phase()
293293
{
294294
// Arrange
295295
var executeMiddleware = BuildWorkflowMiddleware(WorkflowMiddlewarePhase.ExecuteWorkflow, 1);
@@ -298,18 +298,15 @@ public async Task RunExecuteMiddleware_should_run_middleware_in_ExecuteWorkflow_
298298
Middleware.AddRange(new[] { preMiddleware, postMiddleware, executeMiddleware });
299299

300300
// Act
301-
// TODO: add same test when workflow not completed
302301
await Runner.RunExecuteMiddleware(Workflow, Definition);
303302

304303
// Assert
305304
A
306305
.CallTo(HandleMethodFor(executeMiddleware))
307-
.MustHaveHappenedOnceExactly()
308-
.Then(A
309-
.CallTo(HandleMethodFor(postMiddleware))
310-
.MustHaveHappenedOnceExactly());
306+
.MustHaveHappenedOnceExactly();
311307

312308
A.CallTo(HandleMethodFor(preMiddleware)).MustNotHaveHappened();
309+
A.CallTo(HandleMethodFor(postMiddleware)).MustNotHaveHappened();
313310
}
314311

315312
[Fact(DisplayName = "RunExecuteMiddleware should call top level error handler when middleware throws")]

0 commit comments

Comments
 (0)