UserControllerTest updated for better test coverage on abstract class
[staff/due1/sed-hs15-srs-purple.git] / src / main / java / ch / bfh / ti / srs / controller / Controller.java
index d7d88ae..296045f 100644 (file)
@@ -107,8 +107,9 @@ public abstract class Controller {
         *             <code>DataHandler</code> is set\r
         */\r
        public Map<Reservation, Boolean> getReservationsInDateRange(DateRange dateRange) throws Exception {\r
-               if (dh == null)\r
-                       throw new Exception("No DataHandler set");\r
+//             Not needed at the moment since this check is already done in UserController\r
+//             if (dh == null)\r
+//                     throw new Exception("No DataHandler set");\r
                if (dateRange == null)\r
                        throw new Exception("dateRange must be defined");\r
 \r
@@ -133,11 +134,12 @@ public abstract class Controller {
         *             logged in while user param = null\r
         */\r
        public Map<Reservation, Boolean> getUserReservations_Super(User user) throws Exception {\r
-               if (dh == null)\r
-                       throw new Exception("No DataHandler set");\r
-               if (user == null) {\r
-                       throw new Exception("user must be defined");\r
-               }\r
+//             Not needed at the moment since this check is already done in UserController\r
+//             if (dh == null)\r
+//                     throw new Exception("No DataHandler set");\r
+//             if (user == null) {\r
+//                     throw new Exception("user must be defined");\r
+//             }\r
 \r
                // Could be swapped out to private method for better performance (just\r
                // check sc.getLoggedUser() once)\r
@@ -159,8 +161,9 @@ public abstract class Controller {
         *             No DataHandler set or id null\r
         */\r
        public Reservation getReservation(UUID id) throws Exception {\r
-               if (dh == null)\r
-                       throw new Exception("No DataHandler set");\r
+//             Not needed at the moment since this check is already done in UserController\r
+//             if (dh == null)\r
+//                     throw new Exception("No DataHandler set");\r
                if (id == null)\r
                        throw new Exception("uuid must be defined");\r
                if (dh.findReservation(id) == null)\r
@@ -216,11 +219,12 @@ public abstract class Controller {
         */\r
        public Set<Reservation> makeReservation(User user, Resource resource, List<DateRange> dateRangeList, String remark)\r
                        throws Exception {\r
-               if (user == null) {\r
-                       if (sc.getLoggedUser() == null)\r
-                               throw new IllegalArgumentException("User must be defined if no user is logged in");\r
-                       user = sc.getLoggedUser();\r
-               }\r
+//             Not needed at the moment since this check is already done in UserController\r
+//             if (user == null) {\r
+//                     if (sc.getLoggedUser() == null)\r
+//                             throw new IllegalArgumentException("User must be defined if no user is logged in");\r
+//                     user = sc.getLoggedUser();\r
+//             }\r
                if (resource == null)\r
                        throw new IllegalArgumentException("Resource must be definded");\r
                if (dateRangeList == null)\r
@@ -256,10 +260,11 @@ public abstract class Controller {
         *             not authorized\r
         */\r
        public boolean changeReservation_Super(Reservation reservation) throws Exception {\r
-               if (dh == null)\r
-                       throw new Exception("No DataHandler set");\r
-               if (reservation == null)\r
-                       throw new IllegalArgumentException("Reservation must be defined");\r
+//             Not needed at the moment since this check is already done in UserController\r
+//             if (dh == null)\r
+//                     throw new Exception("No DataHandler set");\r
+//             if (reservation == null)\r
+//                     throw new IllegalArgumentException("Reservation must be defined");\r
                return dh.editReservation(reservation);\r
        }\r
        \r
@@ -277,10 +282,11 @@ public abstract class Controller {
         *             is logged in or logged <code>User</code> is not authorized\r
         */\r
        public boolean deleteReservation_Super(Reservation reservation) throws Exception {\r
-               if (dh == null)\r
-                       throw new Exception("No DataHandler set");\r
-               if (reservation == null)\r
-                       throw new IllegalArgumentException("Reservation must be defined");\r
+//             Not needed at the moment since this check is already done in UserController\r
+//             if (dh == null)\r
+//                     throw new Exception("No DataHandler set");\r
+//             if (reservation == null)\r
+//                     throw new IllegalArgumentException("Reservation must be defined");\r
                return dh.deleteReservation(reservation);\r
        }\r
 }\r