More Tests

src/main/java/at/ac/uibk/gitsearch/domain/enumeration/CheckFrequency.java

@@ -15,7 +15,7 @@ public enum CheckFrequency {
 		case NEVER: return now;
 		case DAILY: return now.minus(1, ChronoUnit.DAYS);
 		case WEEKLY: return now.minus(1, ChronoUnit.WEEKS);
-		case MONTHLY: return now.minus(1, ChronoUnit.MONTHS);
+		case MONTHLY: return now.minus(30, ChronoUnit.DAYS);
 		default: // should never happen
 			return now;
 		}

src/main/java/at/ac/uibk/gitsearch/service/UserWatchListQueryService.java

@@ -99,7 +99,7 @@ public class UserWatchListQueryService extends QueryService<UserWatchList> {
             }
             if (criteria.getUserIdId() != null) {
                 specification = specification.and(buildSpecification(criteria.getUserIdId(),
-                    root -> root.join(UserWatchList_.userId, JoinType.LEFT).get(User_.id)));
+                    root -> root.join(UserWatchList_.user, JoinType.LEFT).get(User_.id)));
             }
         }
         return specification;

src/test/java/at/ac/uibk/gitsearch/service/UserWatchListServiceIT.java

@@ -86,14 +86,24 @@ public class UserWatchListServiceIT {
         user.setActivated(false);
         userRepository.saveAndFlush(user);
         
-        UserWatchList wl = new UserWatchList("someTestWatchList", CheckFrequency.DAILY, user);
+        UserWatchList wlDAILY = new UserWatchList("someTestWatchList", CheckFrequency.DAILY, user);
 
         WatchListEntry wle1 = new WatchListEntry();
         wle1.setExerciseId("3");
         wle1.setExerciseName("Unused in this test");
-        wl.getWatchListEntries().add(wle1);
+        wlDAILY.getWatchListEntries().add(wle1);
         
-        userWatchListRepository.save(wl);
+        userWatchListRepository.save(wlDAILY);
+        
+        UserWatchList wlMONTHLY = new UserWatchList("someTestWatchList", CheckFrequency.MONTHLY, user);
+
+        WatchListEntry wle2 = new WatchListEntry();
+        wle2.setExerciseId("3");
+        wle2.setExerciseName("Unused in this test");
+        wlMONTHLY.getWatchListEntries().add(wle2);
+        
+        userWatchListRepository.save(wlMONTHLY);
+
     }
     
     @Test
@@ -119,4 +129,12 @@ public class UserWatchListServiceIT {
 		final Map<SearchResultDTO, UserWatchList> changesSinceAnHourBefore = userWatchListService.findChangesSince(justOneHourBeforeChange, user);
 		org.junit.Assert.assertTrue("At least one match, because changed an hour later", !changesSinceAnHourBefore.isEmpty());
     }
+    
+    @Test
+    @Transactional
+    public void testChangedBeforeCheckFrequency() {
+		final Map<SearchResultDTO, UserWatchList> allExercisesAreTooOld = userWatchListService.findChangesSince(CheckFrequency.MONTHLY, user);
+		org.junit.Assert.assertTrue("At least one match, because changed an hour later", allExercisesAreTooOld.isEmpty());
+    }
+
  }