feat(esp_repl): Add integration tests

Author: SoucheSouche

esp_repl/.build-test-rules.yml

@@ -2,3 +2,6 @@ esp_repl/test_apps:
   enable:
     - if: IDF_TARGET == "linux"
       reason: "Sufficient to test on Linux target"
+  disable:
+    - if: IDF_VERSION_MAJOR <= 5 and IDF_VERSION_MINOR <= 4
+      reason: "those versions of esp-idf do not support eventfd for linux target"

esp_repl/idf_component.yml

@@ -1,4 +1,4 @@
-version: "1.0.0"
+version: "0.1.0"
 description: "esp_repl - Read Eval Print Loop component"
 url: https://github.com/espressif/idf-extra-components/tree/master/esp_repl
 dependencies:

esp_repl/include/esp_repl.h

@@ -19,6 +19,22 @@ extern "C" {
  */
 typedef struct esp_repl_instance *esp_repl_handle_t;
 
+/**
+ * @brief Function prototype called at the beginning of esp_repl().
+ *
+ * @param ctx User-defined context pointer.
+ * @param handle Handle to the REPL instance.
+ */
+typedef void (*esp_repl_on_enter_fn)(void *ctx, esp_repl_handle_t handle);
+
+/**
+ * @brief Enter callback configuration structure for the REPL.
+ */
+typedef struct esp_repl_on_enter {
+    esp_repl_on_enter_fn func; /**!< Function called at the beginning of esp_repl() */
+    void *ctx;                /**!< Context passed to the enter function */
+} esp_repl_on_enter_t;
+
 /**
  * @brief Function prototype called before executing a command.
  *
@@ -101,6 +117,7 @@ typedef struct esp_repl_config {
     esp_command_set_handle_t command_set_handle;   /**!< Handle to a set of commands */
     size_t max_cmd_line_size;                   /**!< Maximum allowed command line size */
     const char *history_save_path;              /**!< Path to file to save the history */
+    esp_repl_on_enter_t on_enter;               /**!< Enter callback and context */
     esp_repl_pre_executor_t pre_executor;       /**!< Pre-executor callback and context */
     esp_repl_post_executor_t post_executor;     /**!< Post-executor callback and context */
     esp_repl_on_stop_t on_stop;                 /**!< Stop callback and context */

esp_repl/src/esp_repl.c

@@ -184,6 +184,13 @@ void esp_repl(esp_repl_handle_t handle)
     esp_repl_config_t *config = &handle->config;
     esp_repl_state_t *state = &handle->state;
 
+    /* trigger a user defined callback before the function gets into the while loop
+     * if the user wants to perform some logic that needs to be done within the task
+     * running the REPL */
+    if (config->on_enter.func != NULL) {
+        config->on_enter.func(config->on_enter.ctx, handle);
+    }
+
     /* get the task handle of the task running this function.
      * It is necessary to gather this information in case esp_repl_stop()
      * is called from the same task as the one running esp_repl() (e.g.,
@@ -257,7 +264,7 @@ void esp_repl(esp_repl_handle_t handle)
             cmd_args.write_func = write;
         }
 
-        const esp_err_t exec_ret = esp_commands_execute(c_set, &cmd_args, cmd_line, &cmd_func_ret);
+        const esp_err_t exec_ret = esp_commands_execute(cmd_line, &cmd_func_ret, c_set, &cmd_args);
 
         /* forward the raw command line to the post executor callback (e.g., save in history).
         * this callback is not necessary for the user to register, continue if it isn't */