From 4baa63b276a807ad3f17932115824e9814b45340 Mon Sep 17 00:00:00 2001 From: dkayiwa Date: Fri, 29 Dec 2023 15:25:26 +0300 Subject: [PATCH] Fix failing tests --- .../org/openmrs/module/queue/api/QueueEntryServiceTest.java | 3 ++- .../org/openmrs/module/queue/api/QueueRoomServiceTest.java | 2 +- .../java/org/openmrs/module/queue/api/QueueServiceTest.java | 2 +- .../openmrs/module/queue/api/RoomProviderMapServiceTest.java | 2 +- 4 files changed, 5 insertions(+), 4 deletions(-) diff --git a/api/src/test/java/org/openmrs/module/queue/api/QueueEntryServiceTest.java b/api/src/test/java/org/openmrs/module/queue/api/QueueEntryServiceTest.java index 351d6f1..8be08a2 100644 --- a/api/src/test/java/org/openmrs/module/queue/api/QueueEntryServiceTest.java +++ b/api/src/test/java/org/openmrs/module/queue/api/QueueEntryServiceTest.java @@ -153,7 +153,7 @@ public void shouldNotCreateDuplicateOverlappingQueueEntryRecords() { @Test public void shouldVoidQueueEntry() { - User user = new User(); + User user = new User(1); UserContext userContext = mock(UserContext.class); when(userContext.getAuthenticatedUser()).thenReturn(user); Context.setUserContext(userContext); @@ -168,6 +168,7 @@ public void shouldVoidQueueEntry() { assertThat(queueEntry.getDateVoided(), notNullValue()); assertThat(queueEntry.getVoidedBy(), equalTo(user)); assertThat(queueEntry.getVoidReason(), equalTo("voidReason")); + Context.logout(); } @Test diff --git a/api/src/test/java/org/openmrs/module/queue/api/QueueRoomServiceTest.java b/api/src/test/java/org/openmrs/module/queue/api/QueueRoomServiceTest.java index 87765a9..667d873 100644 --- a/api/src/test/java/org/openmrs/module/queue/api/QueueRoomServiceTest.java +++ b/api/src/test/java/org/openmrs/module/queue/api/QueueRoomServiceTest.java @@ -88,7 +88,7 @@ public void shouldCreateNewQueueRoom() { @Test public void shouldRetireQueueRoom() { - User user = new User(); + User user = new User(1); UserContext userContext = mock(UserContext.class); when(userContext.getAuthenticatedUser()).thenReturn(user); Context.setUserContext(userContext); diff --git a/api/src/test/java/org/openmrs/module/queue/api/QueueServiceTest.java b/api/src/test/java/org/openmrs/module/queue/api/QueueServiceTest.java index ded4247..f38ebda 100644 --- a/api/src/test/java/org/openmrs/module/queue/api/QueueServiceTest.java +++ b/api/src/test/java/org/openmrs/module/queue/api/QueueServiceTest.java @@ -102,7 +102,7 @@ public void shouldCreateNewQueue() { @Test public void shouldRetireQueue() { - User user = new User(); + User user = new User(1); UserContext userContext = mock(UserContext.class); when(userContext.getAuthenticatedUser()).thenReturn(user); Context.setUserContext(userContext); diff --git a/api/src/test/java/org/openmrs/module/queue/api/RoomProviderMapServiceTest.java b/api/src/test/java/org/openmrs/module/queue/api/RoomProviderMapServiceTest.java index 1851824..741c98b 100644 --- a/api/src/test/java/org/openmrs/module/queue/api/RoomProviderMapServiceTest.java +++ b/api/src/test/java/org/openmrs/module/queue/api/RoomProviderMapServiceTest.java @@ -84,7 +84,7 @@ public void shouldCreateNewRoomProviderMap() { @Test public void shouldVoidRoomProviderMap() { - User user = new User(); + User user = new User(1); UserContext userContext = mock(UserContext.class); when(userContext.getAuthenticatedUser()).thenReturn(user); Context.setUserContext(userContext);