Skip to content

Commit fb61427

Browse files
Copilotvharseko
andauthored
fix: update RouterHandlerTest to use 4-argument buildRouteJson signature
Agent-Logs-Url: https://github.com/OpenIdentityPlatform/OpenIG/sessions/6022298b-9652-4845-9dd4-9272b2dc2fc3 Co-authored-by: vharseko <6818498+vharseko@users.noreply.github.com>
1 parent ce4ac19 commit fb61427

1 file changed

Lines changed: 8 additions & 8 deletions

File tree

openig-core/src/test/java/org/forgerock/openig/handler/router/RouterHandlerTest.java

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -321,14 +321,14 @@ public void onChanges_deploysRoute_whenOpenApiSpecFileIsAdded() throws Exception
321321

322322
when(mockSpecLoader.isOpenApiFile(specFile)).thenReturn(true);
323323
when(mockSpecLoader.tryLoad(specFile)).thenReturn(Optional.of(fakeSpec));
324-
when(mockOpenApiRouteBuilder.buildRouteJson(eq(fakeSpec), eq(specFile), anyBoolean())).thenReturn(routeJson);
324+
when(mockOpenApiRouteBuilder.buildRouteJson(eq(fakeSpec), eq(specFile), anyBoolean(), anyBoolean())).thenReturn(routeJson);
325325
when(mockRouteBuilder.build(any(), any(), any())).thenReturn(mockRoute);
326326

327327
RouterHandler handler = newHandler();
328328
handler.onChanges(addedChangeSet(specFile));
329329

330330
verify(mockSpecLoader).tryLoad(specFile);
331-
verify(mockOpenApiRouteBuilder).buildRouteJson(fakeSpec, specFile, false);
331+
verify(mockOpenApiRouteBuilder).buildRouteJson(fakeSpec, specFile, false, false);
332332
verify(mockRouteBuilder).build(any(), any(), any());
333333
}
334334

@@ -355,7 +355,7 @@ public void stop_destroysAllRoutes() throws Exception {
355355

356356
when(mockSpecLoader.isOpenApiFile(specFile)).thenReturn(true);
357357
when(mockSpecLoader.tryLoad(specFile)).thenReturn(Optional.of(fakeSpec));
358-
when(mockOpenApiRouteBuilder.buildRouteJson(eq(fakeSpec), eq(specFile), anyBoolean())).thenReturn(routeJson);
358+
when(mockOpenApiRouteBuilder.buildRouteJson(eq(fakeSpec), eq(specFile), anyBoolean(), anyBoolean())).thenReturn(routeJson);
359359
when(mockRouteBuilder.build(any(), any(), any())).thenReturn(mockRoute);
360360

361361
RouterHandler handler = newHandler();
@@ -380,7 +380,7 @@ public void onChanges_ignoresOpenApiSpecFile_whenEnabledIsFalse() throws Excepti
380380

381381
// Neither the loader nor the route builder should have been consulted
382382
verify(mockSpecLoader, never()).tryLoad(any());
383-
verify(mockOpenApiRouteBuilder, never()).buildRouteJson(any(), any(), any(Boolean.class));
383+
verify(mockOpenApiRouteBuilder, never()).buildRouteJson(any(), any(), any(Boolean.class), any(Boolean.class));
384384
verify(mockRouteBuilder, never()).build(any(), any(), any());
385385
}
386386

@@ -396,15 +396,15 @@ public void buildRouteJson_isCalledWithFalse_whenFailOnResponseViolationIsFalse(
396396

397397
when(mockSpecLoader.isOpenApiFile(specFile)).thenReturn(true);
398398
when(mockSpecLoader.tryLoad(specFile)).thenReturn(Optional.of(fakeSpec));
399-
when(mockOpenApiRouteBuilder.buildRouteJson(fakeSpec, specFile, false))
399+
when(mockOpenApiRouteBuilder.buildRouteJson(fakeSpec, specFile, false, false))
400400
.thenReturn(routeJson);
401401
when(mockRouteBuilder.build(any(), any(), any())).thenReturn(mockRoute);
402402

403403

404404
strictHandler.onChanges(addedChangeSet(specFile));
405405

406406
// Must be called with failOnResponseViolation=false
407-
verify(mockOpenApiRouteBuilder).buildRouteJson(fakeSpec, specFile, false);
407+
verify(mockOpenApiRouteBuilder).buildRouteJson(fakeSpec, specFile, false, false);
408408
}
409409

410410
@Test
@@ -419,14 +419,14 @@ public void buildRouteJson_isCalledWithTrue_whenFailOnResponseViolationIsTrue()
419419

420420
when(mockSpecLoader.isOpenApiFile(specFile)).thenReturn(true);
421421
when(mockSpecLoader.tryLoad(specFile)).thenReturn(Optional.of(fakeSpec));
422-
when(mockOpenApiRouteBuilder.buildRouteJson(fakeSpec, specFile, true))
422+
when(mockOpenApiRouteBuilder.buildRouteJson(fakeSpec, specFile, true, false))
423423
.thenReturn(routeJson);
424424
when(mockRouteBuilder.build(any(), any(), any())).thenReturn(mockRoute);
425425

426426
strictHandler.onChanges(addedChangeSet(specFile));
427427

428428
// Must be called with failOnResponseViolation=true
429-
verify(mockOpenApiRouteBuilder).buildRouteJson(fakeSpec, specFile, true);
429+
verify(mockOpenApiRouteBuilder).buildRouteJson(fakeSpec, specFile, true, false);
430430
}
431431

432432
@Test

0 commit comments

Comments
 (0)