#1012: remove findX test (already covered by PersistenceBase tests)

puzzle · Oct 1, 2024 · 56b0017 · 56b0017
1 parent 274a02a
commit 56b0017
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 7 deletions.
diff --git a/backend/src/main/java/ch/puzzle/okr/service/persistence/ObjectivePersistenceService.java b/backend/src/main/java/ch/puzzle/okr/service/persistence/ObjectivePersistenceService.java
@@ -49,6 +49,7 @@ public Objective findObjectiveById(Long objectiveId, AuthorizationUser authoriza
         return findByAnyId(objectiveId, authorizationUser, SELECT_OBJECTIVE_BY_ID, noResultException);
     }
 
+    // TODO implement Test
     public List<Objective> findObjectiveByTeamId(Long teamId) {
         return getRepository().findObjectivesByTeamId(teamId);
     }

diff --git a/backend/src/test/java/ch/puzzle/okr/service/persistence/ObjectivePersistenceServiceIT.java b/backend/src/test/java/ch/puzzle/okr/service/persistence/ObjectivePersistenceServiceIT.java
@@ -72,13 +72,6 @@ void tearDown() {
         TenantContext.setCurrentTenant(null);
     }
 
-    @Test
-    void findAllShouldReturnListOfObjectives() {
-        List<Objective> objectives = objectivePersistenceService.findAll();
-
-        assertEquals(7, objectives.size());
-    }
-
     @Test
     void findObjectiveByIdShouldReturnObjectiveProperly() {
         Objective objective = objectivePersistenceService.findObjectiveById(3L, authorizationUser, exception);