From cc3ba7186f3915df15e3f44440aef5c1e525213a Mon Sep 17 00:00:00 2001
From: Roland Dobai <dobai.roland@gmail.com>
Date: Wed, 28 Aug 2019 16:17:28 +0530
Subject: [PATCH] esp_http_client: fix CI issues & return value

---
 .gitlab-ci.yml                                       | 12 ++++++++++++
 components/esp_http_client/include/esp_http_client.h |  1 +
 components/esp_http_client/test/test_http_client.c   |  8 ++++----
 3 files changed, 17 insertions(+), 4 deletions(-)

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 55e8ef1138..d6a78184d9 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -1181,6 +1181,18 @@ UT_029:
     - UT_T2_1
     - 8Mpsram
 
+UT_030:
+  <<: *unit_test_template
+  tags:
+    - ESP32_IDF
+    - UT_T1_1
+
+UT_031:
+  <<: *unit_test_template
+  tags:
+    - ESP32_IDF
+    - UT_T1_1
+
 IT_001:
   <<: *test_template
   parallel: 3
diff --git a/components/esp_http_client/include/esp_http_client.h b/components/esp_http_client/include/esp_http_client.h
index cce15407cb..102bf9e3ea 100644
--- a/components/esp_http_client/include/esp_http_client.h
+++ b/components/esp_http_client/include/esp_http_client.h
@@ -245,6 +245,7 @@ esp_err_t esp_http_client_get_header(esp_http_client_handle_t client, const char
  *
  * @return
  *     - ESP_OK
+ *     - ESP_ERR_INVALID_ARG
  */
 esp_err_t esp_http_client_get_username(esp_http_client_handle_t client, char **value);
 
diff --git a/components/esp_http_client/test/test_http_client.c b/components/esp_http_client/test/test_http_client.c
index 495ef09aaf..f85e6970fb 100644
--- a/components/esp_http_client/test/test_http_client.c
+++ b/components/esp_http_client/test/test_http_client.c
@@ -24,7 +24,7 @@
 #define USERNAME  "user"
 #define PASSWORD  "challenge"
 
-TEST_CASE("most_common_use", "Test in common case: Only URL and hostname are specified.")
+TEST_CASE("Test in common case: Only URL and hostname are specified.", "[ESP HTTP CLIENT]")
 {
     esp_http_client_config_t config_incorrect = {0};
 
@@ -50,7 +50,7 @@ TEST_CASE("most_common_use", "Test in common case: Only URL and hostname are spe
     TEST_ASSERT(esp_http_client_cleanup(client) == ESP_OK);
 }
 
-TEST_CASE("get_username_password", "Get username and password after initialization.")
+TEST_CASE("Get username and password after initialization.", "[ESP HTTP CLIENT]")
 {
     esp_http_client_config_t config_with_auth = {
         .host = HOST,
@@ -79,7 +79,7 @@ TEST_CASE("get_username_password", "Get username and password after initializati
  * Test case to test that, the esp_http_client_set_url won't drop username and password
  * when pass a path "/abc" for url.
  **/
-TEST_CASE("username_not_lost", "Username is unmodified when we change to new path")
+TEST_CASE("Username is unmodified when we change to new path", "[ESP HTTP CLIENT]")
 {
     esp_http_client_config_t config_with_auth = {
         .host = HOST,
@@ -106,7 +106,7 @@ TEST_CASE("username_not_lost", "Username is unmodified when we change to new pat
  * Test case to test that, the esp_http_client_set_url will reset username and password
  * when passing a full URL with username & password missing.
  **/
-TEST_CASE("username_is_reset", "Username is reset if new absolute URL doesnot specify username.")
+TEST_CASE("Username is reset if new absolute URL doesnot specify username.", "[ESP HTTP CLIENT]")
 {
     esp_http_client_config_t config_with_auth = {
         .host = HOST,