Merge pull request #6 from chilispa/retry-status

Retry request for specific HTTP status codes

Author: lucagiove

README.md

@@ -12,7 +12,7 @@ pip install -U requests
 pip install -U robotframework-requests
 ```
 
-Here is a sample test case.
+Here is a sample test case:
 
 |                    |                                 |                     |                       |               |
 | ----------------   | ------------------------------- | ------------------- | --------------------- | ------------- |

doc/RequestsLibrary.html

@@ -5,20 +5,22 @@
 
 
 class RequestsLibrary(RequestsKeywords):
-    """ RequestsLibrary is a HTTP client keyword library that uses
-    the requests module from Kenneth Reitz
-    https://github.com/kennethreitz/requests
+    """``RequestsLibrary`` is a HTTP client keyword library that uses
+    test library that uses the [https://github.com/kennethreitz/requests|Requests] HTTP client.
 
+    Here is a sample test case:
 
-        Examples:
-        | Create Session | google | http://www.google.com |
-        | Create Session | github  | http://github.com/api/v2/json |
-        | ${resp} | Get  google  |  / |
-        | Should Be Equal As Strings |  ${resp.status_code} | 200 |
-        | ${resp} | Get  github  | /user/search/bulkan |
-        | Should Be Equal As Strings  |  ${resp.status_code} | 200 |
-        | ${jsondata}  | To Json |  ${resp.content} |
-        | Dictionary Should Contain Value | ${jsondata['users'][0]} | Bulkan Savun Evcimen |
-
+    | ***** Settings *****   |                                 |                     |                       |               |
+    | Library                | Collections                     |                     |                       |               |
+    | Library                | RequestsLibrary                 |                     |                       |               |
+    | ***** Test Cases ***** |                                 |                     |                       |               |
+    | Get Requests           |                                 |                     |                       |               |
+    |                        | Create Session                  | github              | http://api.github.com |               |
+    |                        | Create Session                  | google              | http://www.google.com |               |
+    |                        | ${resp}=                        | Get Request         | google                | /             |
+    |                        | Should Be Equal As Strings      | ${resp.status_code} | 200                   |               |
+    |                        | ${resp}=                        | Get Request         | github                | /users/bulkan |
+    |                        | Should Be Equal As Strings      | ${resp.status_code} | 200                   |               |
+    |                        | Dictionary Should Contain Value | ${resp.json()}      | Bulkan Savun Evcimen  |               |
     """
     ROBOT_LIBRARY_SCOPE = 'GLOBAL'

src/RequestsLibrary/RequestsKeywords.py

@@ -1 +1 @@
-VERSION = '0.6.1'
+VERSION = '0.6.2'

src/RequestsLibrary/__init__.py

@@ -0,0 +1,49 @@
+*** Settings ***
+Library  String
+Library  Collections
+Library  ../src/RequestsLibrary/RequestsKeywords.py
+
+Suite Teardown  Delete All Sessions
+
+*** Test Cases ***
+Retry Session With Wrong Retry Status List
+    [Tags]  session  retry
+    ${retry_status_list}=   Create List  502baaad
+    Run Keyword And Expect Error  ValueError: *   Create Session  httpbin  http://httpbin.org  retry_status_list=${retry_status_list}
+
+Retry Session With Empty Retry Status List
+    [Tags]  session  retry
+    ${retry_status_list}=   Set Variable  ${Empty}
+    Create Session  httpbin  http://httpbin.org  retry_status_list=${retry_status_list}
+
+Retry Get Request Because Of 502 Error With Default Config
+    [Tags]  get  retry
+    ${retry_status_list}=   Create List  502  503
+    Create Session  httpbin  http://httpbin.org  retry_status_list=${retry_status_list}
+    Run Keyword And Expect Error  RetryError: *   Get Request  httpbin  /status/502
+
+Retry Get Request Because Of 502 Error With Max Retries 1
+    [Tags]  get  retry
+    ${retry_status_list}=   Create List  502
+    Create Session  httpbin  http://httpbin.org  max_retries=1  retry_status_list=${retry_status_list}
+    Run Keyword And Expect Error  RetryError: *   Get Request  httpbin  /status/502
+
+Retry Disabled Get Request
+    [Tags]  get  retry
+    ${retry_status_list}=   Create List  502  503
+    Create Session  httpbin  http://httpbin.org  max_retries=0  retry_status_list=${retry_status_list}
+    Get Request  httpbin  /status/502
+
+Retry Post Request Because Of 502 Error With Default Config
+    [Tags]  post  retry
+    ${retry_status_list}=   Create List  502
+    ${retry_method_list}=   Create List  GET  POST
+    Create Session  httpbin  http://httpbin.org  retry_status_list=${retry_status_list}  retry_method_list=${retry_method_list}
+    Run Keyword And Expect Error  RetryError: *   Post Request  httpbin  /status/502
+
+Retry Post Request Because Of 502 Error With Wrong Config
+    [Tags]  post  retry
+    ${retry_status_list}=   Create List  502
+    ${retry_method_list}=   Create List  WRONG
+    Create Session  httpbin  http://httpbin.org  retry_status_list=${retry_status_list}  retry_method_list=${retry_method_list}
+    Post Request  httpbin  /status/502

src/RequestsLibrary/version.py

@@ -252,14 +252,12 @@ Delete Request With URL Params
     ${resp}=  Delete Request  httpbin  /delete		${params}
     Should Be Equal As Strings  ${resp.status_code}  200
 
-
 Delete Request With No Data
     [Tags]  delete
     Create Session  httpbin  http://httpbin.org
     ${resp}=  Delete Request  httpbin  /delete
     Should Be Equal As Strings  ${resp.status_code}  200
 
-
 Delete Request With Data
     [Tags]  delete
     Create Session  httpbin  http://httpbin.org    debug=3