quick fix to make lv_test_screenshot read only

Author: Gauthier Provost

src/others/test/lv_test_screenshot_compare.c

@@ -43,7 +43,7 @@
 #endif
 
 #ifndef REF_IMG_TOLERANCE
-    #define REF_IMG_TOLERANCE 0
+    #define REF_IMG_TOLERANCE 5
 #endif
 
 #define ERR_FILE_NOT_FOUND  -1
@@ -103,7 +103,7 @@ static bool screenshot_compare(const char * fn_ref, uint8_t tolerance)
     char fn_ref_full[256];
     lv_snprintf(fn_ref_full, sizeof(fn_ref_full), "%s%s", REF_IMGS_PATH, fn_ref);
 
-    create_folders_if_needed(fn_ref_full);
+    //create_folders_if_needed(fn_ref_full);
 
     lv_draw_buf_t * draw_buf = lv_display_get_buf_active(NULL);
 
@@ -115,10 +115,10 @@ static bool screenshot_compare(const char * fn_ref, uint8_t tolerance)
     unsigned  ref_img_height = 0;
     unsigned  res = read_png_file(&ref_draw_buf, &ref_img_width, &ref_img_height, fn_ref_full);
     if(res) {
-        LV_LOG_WARN("%s%s", fn_ref_full, " was not found, creating it now from the rendered screen");
-        write_png_file(screen_buf_xrgb8888, draw_buf->header.w, draw_buf->header.h, fn_ref_full);
-        lv_free(screen_buf_xrgb8888);
-        return true;
+        LV_LOG_WARN("%s%s", fn_ref_full, " was not found");
+        //write_png_file(screen_buf_xrgb8888, draw_buf->header.w, draw_buf->header.h, fn_ref_full);
+        //lv_free(screen_buf_xrgb8888);
+        return false;
     }
 
     if(ref_img_width != draw_buf->header.w || ref_img_height != draw_buf->header.h) {
@@ -136,6 +136,11 @@ static bool screenshot_compare(const char * fn_ref, uint8_t tolerance)
             uint8_t * ptr_ref = &(ref_row[x * 4]);
             uint8_t * ptr_act = &screen_buf_tmp[x * 4];
 
+            // skip transparent pixels
+            if (ptr_ref[3] == 0) {
+                continue;
+            }
+
             if(LV_ABS((int32_t) ptr_act[0] - (int32_t) ptr_ref[0]) > tolerance ||
                LV_ABS((int32_t) ptr_act[1] - (int32_t) ptr_ref[1]) > tolerance ||
                LV_ABS((int32_t) ptr_act[2] - (int32_t) ptr_ref[2]) > tolerance) {
@@ -164,7 +169,7 @@ static bool screenshot_compare(const char * fn_ref, uint8_t tolerance)
         char fn_err_full[256];
         lv_snprintf(fn_err_full, sizeof(fn_err_full), "%s%s_err.png", REF_IMGS_PATH, fn_ref_no_ext);
 
-        write_png_file(screen_buf_xrgb8888, draw_buf->header.w, draw_buf->header.h, fn_err_full);
+        //write_png_file(screen_buf_xrgb8888, draw_buf->header.w, draw_buf->header.h, fn_err_full);
     }
 
     fflush(stdout);

src/others/xml/lv_xml_test.c

@@ -85,6 +85,7 @@ static lv_xml_test_t test;
 static lv_display_t * test_display;
 static lv_obj_t * cursor;
 static lv_tick_get_cb_t tick_cb_original;
+static bool test_initialized = false;
 
 /**********************
  *      MACROS
@@ -235,6 +236,7 @@ void lv_xml_test_run_init(void)
     }
     lv_refr_now(normal_display);
     test.step_act = 0;
+    test_initialized = true;
 }
 
 bool lv_xml_test_run_next(uint32_t slowdown)
@@ -251,10 +253,14 @@ bool lv_xml_test_run_next(uint32_t slowdown)
 
 void lv_xml_test_run_stop(void)
 {
+    if(!test_initialized) {
+        return;
+    }
     lv_obj_delete(cursor);
     lv_tick_set_cb(tick_cb_original);
     lv_display_delete(test_display);
     lv_test_indev_delete_all();
+    test_initialized = false;
 }