Merge changes from solution

Author: hugo-valle

tests/test_module1.py

@@ -328,12 +328,63 @@ def test_load_data_load_recs_module1(parse):
 
 @pytest.mark.test_sensor_app_load_data_return_module1
 def test_sensor_app_load_data_return_module1(parse):
+    # First, let's verify the user did not accidentally deleted
+    # the two lines provided for them. 
+
+    # data = []                   # list to store data read from files
+    # print("Sensor Data App")
+
+    sensor = parse("sensor_app")
+    assert sensor.success, sensor.message
+    
+    original_data = (
+        sensor.assign_().match(
+            {
+                "0_type": "Assign",
+                "0_targets_0_type": "Name",
+                "0_targets_0_id": "data",
+                "0_value_type": "List",
+            }
+        )
+    .exists()
+    )
+    assert (
+        original_data
+    ), """Do you have a `data` variable set to an empty list on top of the file? 
+        You  need to have these two lines of code before you being testing Module1
+        data = []        
+        print("Sensor Data App")
+        """
+    
+    print_app = (
+        sensor.calls().match(
+            {
+                "type": "Expr",
+                "value_type": "Call",
+                "value_func_type": "Name",
+                "value_func_id": "print",
+                "value_args_0_type": "Constant",
+                "value_args_0_value": "Sensor Data App"
+            }
+        )
+        .exists()
+    )
+    assert (
+        print_app
+    ), """Do you have a `print("Sensor Data App")` statement? 
+        You  need to have these two lines of code before you being testing Module1
+        data = []            
+        print("Sensor Data App")"""
+
+
+    ######################################################
+    # Now we can test the actual module
+    ######################################################
+    
     # from load_data import load_sensor_data
     # data = load_sensor_data()
     # print("Loaded records {}".format(len(data)))
 
-    sensor = parse("sensor_app")
-    assert sensor.success, sensor.message
 
     load_sensor_data_import = sensor.from_imports(
         "load_data", "load_sensor_data")
@@ -348,7 +399,7 @@ def test_sensor_app_load_data_return_module1(parse):
             {
                 "1_type": "Assign",
                 "1_targets_0_type": "Name",
-                "1_targets_0_id": "recs",
+                "1_targets_0_id": "data",
                 "1_value_type": "Call",
                 "1_value_func_type": "Name",
                 "1_value_func_id": "load_sensor_data",
@@ -370,13 +421,13 @@ def test_sensor_app_load_data_return_module1(parse):
                 "value_args_0_type": "Call",
                 "value_args_0_func_type": "Attribute",
                 "value_args_0_func_value_type": "Constant",
-                "value_args_0_func_value_value": "Loaded records: [{}]",
+                "value_args_0_func_value_value": "Loaded records: {}",
                 "value_args_0_func_attr": "format",
                 "value_args_0_args_0_type": "Call",
                 "value_args_0_args_0_func_type": "Name",
                 "value_args_0_args_0_func_id": "len",
                 "value_args_0_args_0_args_0_type": "Name",
-                "value_args_0_args_0_args_0_id": "recs"
+                "value_args_0_args_0_args_0_id": "data"
             }
         )
         .exists()