windows are now required to be non-empty

src/users/broker.cpp

@@ -307,8 +307,8 @@ DMWindow_list* Broker::parse_dm_windows(const std::string attr_name,
     const Value &windows_param = (*user_description_file)[attr_name.c_str()];
     std::string error_message =  "Invalid user_description file: field ";
     error_message += attr_name;
-    error_message = "should be an array";
-    PPK_ASSERT_ERROR(windows_param.IsArray(),
+    error_message += " should be a non-empty array";
+    PPK_ASSERT_ERROR(windows_param.IsArray() && windows_param.GetArray().Size()!=0,
         error_message.c_str());
     error_message =  "Invalid user_description file: the field ";
     error_message += attr_name;

test/test_assertion_error.py

@@ -21,9 +21,9 @@ def make_error_file(seed=None,red_windows=None,yellow_windows=None) :
     }
     if seed :
         error_file["seed"] =seed
-    if red_windows :
+    if not(red_windows is None) :
         error_file["red_windows"] = red_windows
-    if yellow_windows :
+    if not(yellow_windows is None) :
         error_file["yellow_windows"] = yellow_windows
     with open(schedconf_file, 'w+') as error_description_file :
         json.dump(error_file,error_description_file)
@@ -69,7 +69,12 @@ def test_error_no_window(platform_multiC) :
     make_error_file(3)
     out_dir = error_user("dm_user_multi_behavior_no_windows", platform_multiC, "window")
 
-
+def test_empty_red_window(platform_multiC) :
+    make_error_file(3,red_windows=[])
+    out_dir = error_user("dm_user_multi_behavior_empty_red_windows", platform_multiC, "red_windows should be a non-empty array")
+def test_empty_yellow_window(platform_multiC) :
+    make_error_file(3,yellow_windows=[])
+    out_dir = error_user("dm_user_multi_behavior_empty_yellow_windows", platform_multiC, "yellow_windows should be a non-empty array")
 def test_error_invalid_window(platform_multiC) :
     invalid_windows_list = [[[0]], [[0.0, 1.0]], [[0, 1, 2]], [[0, 1], [1, 2], [3, 4], 5], [[1, 4.0]], [[1.0, 4]]]
     for i in range(len(invalid_windows_list)):