Update Robot Framework format - step 1
[integration/test.git] / csit / libraries / SxpLib.robot
index 55d7166529f9db2bea2550fd7f41f5e46a24024f..e2f7c61a1c5d9428e61da15ed8069a970ec98ae8 100644 (file)
@@ -1,22 +1,29 @@
 *** Settings ***
-Documentation     Library containing Keywords used for SXP testing
-Library           Collections
-Library           RequestsLibrary
-Library           ./Sxp.py
-Resource          CompareStream.robot
-Resource          KarafKeywords.robot
-Resource          TemplatedRequests.robot
-Resource          Utils.robot
-Resource          ../variables/Variables.robot
+Documentation       Library containing Keywords used for SXP testing
+
+Library             Collections
+Library             RequestsLibrary
+Library             ./Sxp.py
+Resource            CompareStream.robot
+Resource            KarafKeywords.robot
+Resource            TemplatedRequests.robot
+Resource            Utils.robot
+Resource            ../variables/Variables.robot
+
 
 *** Variables ***
-${REST_CONTEXT}    /restconf/operations/sxp-controller
+${REST_CONTEXT}     /restconf/operations/sxp-controller
+
 
 *** Keywords ***
 Post To Controller
-    [Arguments]    ${session}    ${path}    ${data}    ${rest_context}=${REST_CONTEXT}
     [Documentation]    Post request to Controller and checks response
-    ${resp} =    RequestsLibrary.Post Request    ${session}    ${rest_context}:${path}    data=${data}    headers=${HEADERS_XML}
+    [Arguments]    ${session}    ${path}    ${data}    ${rest_context}=${REST_CONTEXT}
+    ${resp} =    RequestsLibrary.Post Request
+    ...    ${session}
+    ...    ${rest_context}:${path}
+    ...    data=${data}
+    ...    headers=${HEADERS_XML}
     Log    ${resp.text}
     Log    ${session}
     Log    ${path}
@@ -28,86 +35,112 @@ Post To Controller
     BuiltIn.Should Be True    ${result}    RPC result is False
 
 Add Node
+    [Documentation]    Add node via RPC to ODL
     [Arguments]    ${node}    ${password}=${EMPTY}    ${version}=version4    ${port}=64999    ${session}=session    ${ip}=${node}
     ...    ${ssl_stores}=${EMPTY}    ${retry_open_timer}=1
-    [Documentation]    Add node via RPC to ODL
     ${data} =    Sxp.Add Node Xml    ${node}    ${port}    ${password}    ${version}    ${ip}
     ...    keystores=${ssl_stores}    retry_open_timer=${retry_open_timer}
     Post To Controller    ${session}    add-node    ${data}
 
 Delete Node
-    [Arguments]    ${node}    ${session}=session
     [Documentation]    Delete connection via RPC from node
+    [Arguments]    ${node}    ${session}=session
     ${data} =    Sxp.Delete Node Xml    ${node}
     Post To Controller    ${session}    delete-node    ${data}
 
 Add Connection
+    [Documentation]    Add connection via RPC to node
     [Arguments]    ${version}    ${mode}    ${ip}    ${port}    ${node}=127.0.0.1    ${password}=${EMPTY}
     ...    ${session}=session    ${domain}=global    ${security_mode}=${EMPTY}
-    [Documentation]    Add connection via RPC to node
     ${data} =    Sxp.Add Connection Xml    ${version}    ${mode}    ${ip}    ${port}    ${node}
     ...    ${password}    ${domain}    security_mode=${security_mode}
     Post To Controller    ${session}    add-connection    ${data}
 
 Get Connections
-    [Arguments]    ${node}=127.0.0.1    ${session}=session    ${domain}=global
     [Documentation]    Gets all connections via RPC from node
+    [Arguments]    ${node}=127.0.0.1    ${session}=session    ${domain}=global
     ${data} =    Sxp.Get Connections From Node Xml    ${node}    ${domain}
-    ${resp} =    RequestsLibrary.Post Request    ${session}    ${REST_CONTEXT}:get-connections    data=${data}    headers=${HEADERS_XML}
+    ${resp} =    RequestsLibrary.Post Request
+    ...    ${session}
+    ...    ${REST_CONTEXT}:get-connections
+    ...    data=${data}
+    ...    headers=${HEADERS_XML}
     BuiltIn.Should be Equal As Strings    ${resp.status_code}    200
-    [Return]    ${resp.text}
+    RETURN    ${resp.text}
 
 Delete Connections
-    [Arguments]    ${ip}    ${port}    ${node}=127.0.0.1    ${session}=session    ${domain}=global
     [Documentation]    Delete connection via RPC from node
+    [Arguments]    ${ip}    ${port}    ${node}=127.0.0.1    ${session}=session    ${domain}=global
     ${data} =    Sxp.Delete Connections Xml    ${ip}    ${port}    ${node}    ${domain}
     Post To Controller    ${session}    delete-connection    ${data}
 
 Clean Connections
-    [Arguments]    ${node}=127.0.0.1    ${session}=session    ${domain}=global
     [Documentation]    Delete all connections via RPC from node
+    [Arguments]    ${node}=127.0.0.1    ${session}=session    ${domain}=global
     ${resp} =    Get Connections    ${node}    ${session}    ${domain}
     @{connections} =    Sxp.Parse Connections    ${resp}
     FOR    ${connection}    IN    @{connections}
-        Delete Connections    ${connection['peer-address']}    ${connection['tcp-port']}    ${node}    ${session}    ${domain}
+        Delete Connections
+        ...    ${connection['peer-address']}
+        ...    ${connection['tcp-port']}
+        ...    ${node}
+        ...    ${session}
+        ...    ${domain}
     END
 
 Verify Connection
+    [Documentation]    Verify that connection is ON
     [Arguments]    ${version}    ${mode}    ${ip}    ${port}=64999    ${node}=127.0.0.1    ${state}=on
     ...    ${session}=session    ${domain}=global
-    [Documentation]    Verify that connection is ON
     ${resp} =    Get Connections    ${node}    ${session}    ${domain}
     Should Contain Connection    ${resp}    ${ip}    ${port}    ${mode}    ${version}    ${state}
 
 Add Bindings
-    [Arguments]    ${sgt}    ${prefixes}    ${node}=127.0.0.1    ${session}=session    ${domain}=global    ${origin}=LOCAL
     [Documentation]    Based on ODL version decide if bindings will be added with or without origin type (introduced in Fluorine)
-    CompareStream.Run_Keyword_If_At_Least_Fluorine    Add Bindings Fluorine    ${sgt}    ${prefixes}    ${node}    ${session}    ${domain}
+    [Arguments]    ${sgt}    ${prefixes}    ${node}=127.0.0.1    ${session}=session    ${domain}=global    ${origin}=LOCAL
+    CompareStream.Run_Keyword_If_At_Least_Fluorine
+    ...    Add Bindings Fluorine
+    ...    ${sgt}
+    ...    ${prefixes}
+    ...    ${node}
+    ...    ${session}
+    ...    ${domain}
     ...    ${origin}
-    CompareStream.Run_Keyword_If_At_Most_Oxygen    Add Bindings Oxygen    ${sgt}    ${prefixes}    ${node}    ${session}    ${domain}
+    CompareStream.Run_Keyword_If_At_Most_Oxygen
+    ...    Add Bindings Oxygen
+    ...    ${sgt}
+    ...    ${prefixes}
+    ...    ${node}
+    ...    ${session}
+    ...    ${domain}
 
 Add Bindings Fluorine
-    [Arguments]    ${sgt}    ${prefixes}    ${node}    ${session}    ${domain}    ${origin}
     [Documentation]    Add/Update one or more bindings with origin type via RPC to Master DB of the node
+    [Arguments]    ${sgt}    ${prefixes}    ${node}    ${session}    ${domain}    ${origin}
     ${data} =    Sxp.Add Bindings Xml Fluorine    ${node}    ${domain}    ${sgt}    ${prefixes}    ${origin}
     Post To Controller    ${session}    add-bindings    ${data}
 
 Add Bindings Oxygen
-    [Arguments]    ${sgt}    ${prefixes}    ${node}    ${session}    ${domain}
     [Documentation]    Add/Update one or more bindings without origin type via RPC to Master DB of the node
+    [Arguments]    ${sgt}    ${prefixes}    ${node}    ${session}    ${domain}
     ${data} =    Sxp.Add Bindings Xml Oxygen    ${node}    ${domain}    ${sgt}    ${prefixes}
     Post To Controller    ${session}    add-bindings    ${data}
 
 Get Bindings
-    [Arguments]    ${node}=127.0.0.1    ${session}=session    ${domain}=global    ${scope}=all
     [Documentation]    Gets all binding via RPC from Master DB of node
+    [Arguments]    ${node}=127.0.0.1    ${session}=session    ${domain}=global    ${scope}=all
     ${data} =    Sxp.Get Bindings From Node Xml    ${node}    ${scope}    ${domain}
-    ${resp} =    TemplatedRequests.Post_To_Uri    ${REST_CONTEXT}:get-node-bindings    data=${data}    accept=${ACCEPT_JSON}    content_type=${HEADERS_XML}    session=${session}
-    [Return]    ${resp}
+    ${resp} =    TemplatedRequests.Post_To_Uri
+    ...    ${REST_CONTEXT}:get-node-bindings
+    ...    data=${data}
+    ...    accept=${ACCEPT_JSON}
+    ...    content_type=${HEADERS_XML}
+    ...    session=${session}
+    RETURN    ${resp}
 
 Clean Bindings
-    [Arguments]    ${node}=127.0.0.1    ${session}=session    ${domain}=global    ${scope}=local
     [Documentation]    Delete all bindings via RPC from Master DB of node
+    [Arguments]    ${node}=127.0.0.1    ${session}=session    ${domain}=global    ${scope}=local
     ${resp} =    Get Bindings    ${node}    ${session}    ${domain}    ${scope}
     @{bindings} =    Sxp.Parse Bindings    ${resp}
     FOR    ${binding}    IN    @{bindings}
@@ -117,34 +150,38 @@ Clean Bindings
     END
 
 Delete Bindings
-    [Arguments]    ${sgt}    ${prefixes}    ${node}=127.0.0.1    ${domain}=global    ${session}=session
     [Documentation]    Delete one or more bindings via RPC from Master DB of node
+    [Arguments]    ${sgt}    ${prefixes}    ${node}=127.0.0.1    ${domain}=global    ${session}=session
     ${data} =    Sxp.Delete Bindings Xml    ${node}    ${domain}    ${sgt}    ${prefixes}
     Post To Controller    ${session}    delete-bindings    ${data}
 
 Add PeerGroup
-    [Arguments]    ${name}    ${peers}=    ${node}=127.0.0.1    ${session}=session
     [Documentation]    Adds new PeerGroup via RPC to Node
+    [Arguments]    ${name}    ${peers}=    ${node}=127.0.0.1    ${session}=session
     ${data} =    Sxp.Add Peer Group Xml    ${name}    ${peers}    ${node}
     Post To Controller    ${session}    add-peer-group    ${data}
 
 Delete Peer Group
-    [Arguments]    ${name}    ${node}=127.0.0.1    ${session}=session
     [Documentation]    Delete PeerGroup via RPC from Node
+    [Arguments]    ${name}    ${node}=127.0.0.1    ${session}=session
     ${data} =    Sxp.Delete Peer Group Xml    ${name}    ${node}
     Post To Controller    ${session}    delete-peer-group    ${data}
 
 Get Peer Groups
-    [Arguments]    ${node}=127.0.0.1    ${session}=session
     [Documentation]    Gets all PeerGroups via RPC from node
+    [Arguments]    ${node}=127.0.0.1    ${session}=session
     ${data} =    Sxp.Get Peer Groups From Node Xml    ${node}
-    ${resp} =    RequestsLibrary.Post Request    ${session}    ${REST_CONTEXT}:get-peer-groups    data=${data}    headers=${HEADERS_XML}
+    ${resp} =    RequestsLibrary.Post Request
+    ...    ${session}
+    ...    ${REST_CONTEXT}:get-peer-groups
+    ...    data=${data}
+    ...    headers=${HEADERS_XML}
     BuiltIn.Should be Equal As Strings    ${resp.status_code}    200
-    [Return]    ${resp.text}
+    RETURN    ${resp.text}
 
 Clean Peer Groups
-    [Arguments]    ${node}=127.0.0.1    ${session}=session
     [Documentation]    Delete all PeerGroups via RPC from node
+    [Arguments]    ${node}=127.0.0.1    ${session}=session
     ${resp} =    Get Peer Groups    ${node}    ${session}
     @{prefixes} =    Sxp.Parse Peer Groups    ${resp}
     FOR    ${group}    IN    @{prefixes}
@@ -152,90 +189,101 @@ Clean Peer Groups
     END
 
 Add Filter
-    [Arguments]    ${name}    ${type}    ${entries}    ${node}=127.0.0.1    ${session}=session    ${policy}=auto-update
     [Documentation]    Add Filter via RPC from Node
+    [Arguments]    ${name}    ${type}    ${entries}    ${node}=127.0.0.1    ${session}=session    ${policy}=auto-update
     ${data} =    Prepare Add/Update Filter Data    ${name}    ${type}    ${entries}    ${node}    ${policy}
     Post To Controller    ${session}    add-filter    ${data}
 
 Update Filter
-    [Arguments]    ${name}    ${type}    ${entries}    ${node}=127.0.0.1    ${session}=session    ${policy}=auto-update
     [Documentation]    Update Filter via RPC
+    [Arguments]    ${name}    ${type}    ${entries}    ${node}=127.0.0.1    ${session}=session    ${policy}=auto-update
     ${data} =    Prepare Add/Update Filter Data    ${name}    ${type}    ${entries}    ${node}    ${policy}
     Post To Controller    ${session}    update-filter    ${data}
 
 Prepare Add/Update Filter Data
-    [Arguments]    ${name}    ${type}    ${entries}    ${node}=127.0.0.1    ${policy}=auto-update
     [Documentation]    Prepare XML payload for add-filter and update-filter RPCs. Payloads for both RPCs are the same.
-    ${data} =    CompareStream.Run_Keyword_If_At_Least_Else    carbon    Add Filter Xml    ${name}    ${type}    ${entries}
-    ...    ${node}    ${policy}
-    ...    ELSE    Add Filter Xml    ${name}    ${type}    ${entries}    ${node}
-    [Return]    ${data}
+    [Arguments]    ${name}    ${type}    ${entries}    ${node}=127.0.0.1    ${policy}=auto-update
+    ${data} =    CompareStream.Run_Keyword_If_At_Least_Else
+    ...    carbon
+    ...    Add Filter Xml
+    ...    ${name}
+    ...    ${type}
+    ...    ${entries}
+    ...    ${node}
+    ...    ${policy}
+    ...    ELSE
+    ...    Add Filter Xml
+    ...    ${name}
+    ...    ${type}
+    ...    ${entries}
+    ...    ${node}
+    RETURN    ${data}
 
 Add Domain Filter
-    [Arguments]    ${name}    ${domains}    ${entries}    ${node}=127.0.0.1    ${filter_name}=base-domain-filter    ${session}=session
     [Documentation]    Add Domain Filter via RPC from Node
+    [Arguments]    ${name}    ${domains}    ${entries}    ${node}=127.0.0.1    ${filter_name}=base-domain-filter    ${session}=session
     ${data} =    Sxp.Add Domain Filter Xml    ${name}    ${domains}    ${entries}    ${node}    ${filter_name}
     Post To Controller    ${session}    add-domain-filter    ${data}
 
 Delete Filter
-    [Arguments]    ${name}    ${type}    ${node}=127.0.0.1    ${session}=session
     [Documentation]    Delete Filter via RPC from Node
+    [Arguments]    ${name}    ${type}    ${node}=127.0.0.1    ${session}=session
     ${data} =    Sxp.Delete Filter Xml    ${name}    ${type}    ${node}
     Post To Controller    ${session}    delete-filter    ${data}
 
 Delete Domain Filter
-    [Arguments]    ${name}    ${node}=127.0.0.1    ${filter_name}=base-domain-filter    ${session}=session
     [Documentation]    Delete Filter via RPC from Node
+    [Arguments]    ${name}    ${node}=127.0.0.1    ${filter_name}=base-domain-filter    ${session}=session
     ${data} =    Sxp.Delete Domain Filter Xml    ${name}    ${node}    ${filter_name}
     Post To Controller    ${session}    delete-domain-filter    ${data}
 
 Should Contain Binding
-    [Arguments]    ${resp}    ${sgt}    ${prefix}
     [Documentation]    Tests if data contains specified binding
+    [Arguments]    ${resp}    ${sgt}    ${prefix}
     ${out} =    Sxp.Find Binding    ${resp}    ${sgt}    ${prefix}
     BuiltIn.Should Be True    ${out}    Doesn't have ${sgt} ${prefix}
 
 Should Not Contain Binding
-    [Arguments]    ${resp}    ${sgt}    ${prefix}
     [Documentation]    Tests if data doesn't contains specified binding
+    [Arguments]    ${resp}    ${sgt}    ${prefix}
     ${out} =    Sxp.Find Binding    ${resp}    ${sgt}    ${prefix}
     BuiltIn.Should Not Be True    ${out}    Should't have ${sgt} ${prefix}
 
 Should Contain Connection
-    [Arguments]    ${resp}    ${ip}    ${port}    ${mode}    ${version}    ${state}=none
     [Documentation]    Test if data contains specified connection
+    [Arguments]    ${resp}    ${ip}    ${port}    ${mode}    ${version}    ${state}=none
     ${out} =    Sxp.Find Connection    ${resp}    ${version}    ${mode}    ${ip}    ${port}
     ...    ${state}
     BuiltIn.Should Be True    ${out}    Doesn't have ${ip}:${port} ${mode} ${version} in state ${state}
 
 Should Not Contain Connection
-    [Arguments]    ${resp}    ${ip}    ${port}    ${mode}    ${version}    ${state}=none
     [Documentation]    Test if data doesn't contains specified connection
+    [Arguments]    ${resp}    ${ip}    ${port}    ${mode}    ${version}    ${state}=none
     ${out} =    Sxp.Find Connection    ${resp}    ${version}    ${mode}    ${ip}    ${port}
     ...    ${state}
     BuiltIn.Should Not Be True    ${out}    Shouldn't have ${ip}:${port} ${mode} ${version}
 
 Bindings Should Contain
-    [Arguments]    ${sgt}    ${prefix}    ${domain}=global    ${scope}=all
     [Documentation]    Retrieves bindings and verifies they contain given binding
+    [Arguments]    ${sgt}    ${prefix}    ${domain}=global    ${scope}=all
     ${resp} =    Get Bindings    domain=${domain}    scope=${scope}
     Should Contain Binding    ${resp}    ${sgt}    ${prefix}
 
 Bindings Should Not Contain
-    [Arguments]    ${sgt}    ${prefix}    ${domain}=global    ${scope}=all
     [Documentation]    Retrieves bindings and verifies they do not contain given binding
+    [Arguments]    ${sgt}    ${prefix}    ${domain}=global    ${scope}=all
     ${resp} =    Get Bindings    domain=${domain}    scope=${scope}
     Should Not Contain Binding    ${resp}    ${sgt}    ${prefix}
 
 Connections Should Contain
-    [Arguments]    ${ip}    ${port}    ${mode}    ${version}    ${state}=none
     [Documentation]    Retrieves connections and verifies they contain given connection
+    [Arguments]    ${ip}    ${port}    ${mode}    ${version}    ${state}=none
     ${resp} =    Get Connections
     Should Contain Connection    ${resp}    ${ip}    ${port}    ${mode}    ${version}    ${state}
 
 Connections Should Not Contain
-    [Arguments]    ${ip}    ${port}    ${mode}    ${version}    ${state}=none
     [Documentation]    Retrieves connections and verifies they do not contain given connection
+    [Arguments]    ${ip}    ${port}    ${mode}    ${version}    ${state}=none
     ${resp} =    Get Connections
     Should Not Contain Connection    ${resp}    ${ip}    ${port}    ${mode}    ${version}    ${state}
 
@@ -259,51 +307,65 @@ Setup Topology Complex
     Add Bindings    10    10.0.0.0/8    127.0.0.1
 
 Verify Snapshot Was Pushed
-    [Arguments]    ${snapshot_string}=22-sxp-controller-one-node.xml
     [Documentation]    Will succeed if the ${snapshot_string} is found in the karaf logs
-    ${output} =    Utils.Run Command On Controller    ${ODL_SYSTEM_IP}    cat ${WORKSPACE}/${BUNDLEFOLDER}/data/log/karaf.log* | grep -c 'Successfully pushed configuration snapshot.*${snapshot_string}'
+    [Arguments]    ${snapshot_string}=22-sxp-controller-one-node.xml
+    ${output} =    Utils.Run Command On Controller
+    ...    ${ODL_SYSTEM_IP}
+    ...    cat ${WORKSPACE}/${BUNDLEFOLDER}/data/log/karaf.log* | grep -c 'Successfully pushed configuration snapshot.*${snapshot_string}'
     BuiltIn.Should Not Be Equal As Strings    ${output}    0
 
 Add Domain
-    [Arguments]    ${domain_name}    ${sgt}=None    ${prefixes}=None    ${origin}=LOCAL    ${node}=127.0.0.1    ${session}=session
     [Documentation]    Based on ODL version decide if domain's bindings will be added with or without origin type (introduced in Fluorine)
-    CompareStream.Run_Keyword_If_At_Least_Fluorine    Add Domain Fluorine    ${domain_name}    ${sgt}    ${prefixes}    ${origin}    ${node}
+    [Arguments]    ${domain_name}    ${sgt}=None    ${prefixes}=None    ${origin}=LOCAL    ${node}=127.0.0.1    ${session}=session
+    CompareStream.Run_Keyword_If_At_Least_Fluorine
+    ...    Add Domain Fluorine
+    ...    ${domain_name}
+    ...    ${sgt}
+    ...    ${prefixes}
+    ...    ${origin}
+    ...    ${node}
+    ...    ${session}
+    CompareStream.Run_Keyword_If_At_Most_Oxygen
+    ...    Add Domain Oxygen
+    ...    ${domain_name}
+    ...    ${sgt}
+    ...    ${prefixes}
+    ...    ${node}
     ...    ${session}
-    CompareStream.Run_Keyword_If_At_Most_Oxygen    Add Domain Oxygen    ${domain_name}    ${sgt}    ${prefixes}    ${node}    ${session}
 
 Add Domain Fluorine
-    [Arguments]    ${domain_name}    ${sgt}    ${prefixes}    ${origin}    ${node}    ${session}
     [Documentation]    Add Domain with bindings (with origin) via RPC
+    [Arguments]    ${domain_name}    ${sgt}    ${prefixes}    ${origin}    ${node}    ${session}
     ${data} =    Sxp.Add Domain Xml Fluorine    ${node}    ${domain_name}    ${sgt}    ${prefixes}    ${origin}
     Post To Controller    ${session}    add-domain    ${data}
 
 Add Domain Oxygen
-    [Arguments]    ${domain_name}    ${sgt}    ${prefixes}    ${node}    ${session}
     [Documentation]    Add Domain with bindings (without origin) via RPC
+    [Arguments]    ${domain_name}    ${sgt}    ${prefixes}    ${node}    ${session}
     ${data} =    Sxp.Add Domain Xml Oxygen    ${node}    ${domain_name}    ${sgt}    ${prefixes}
     Post To Controller    ${session}    add-domain    ${data}
 
 Delete Domain
-    [Arguments]    ${domain_name}    ${node}=127.0.0.1    ${session}=session
     [Documentation]    Delete Domain via RPC
+    [Arguments]    ${domain_name}    ${node}=127.0.0.1    ${session}=session
     ${data} =    Sxp.Delete Domain Xml    ${node}    ${domain_name}
     Post To Controller    ${session}    delete-domain    ${data}
 
 Add Bindings Range
-    [Arguments]    ${sgt}    ${start}    ${size}    ${node}=127.0.0.1
     [Documentation]    Add Bindings to Node specified by range
+    [Arguments]    ${sgt}    ${start}    ${size}    ${node}=127.0.0.1
     ${prefixes} =    Sxp.Prefix Range    ${start}    ${size}
     Add Bindings    ${sgt}    ${prefixes}    ${node}
 
 Delete Bindings Range
-    [Arguments]    ${sgt}    ${start}    ${size}    ${node}=127.0.0.1
     [Documentation]    Delete Bindings to Node specified by range
+    [Arguments]    ${sgt}    ${start}    ${size}    ${node}=127.0.0.1
     ${prefixes} =    Sxp.Prefix Range    ${start}    ${size}
     Delete Bindings    ${sgt}    ${prefixes}    ${node}
 
 Check Binding Range
-    [Arguments]    ${sgt}    ${start}    ${end}    ${node}=127.0.0.1
     [Documentation]    Check if Node contains Bindings specified by range
+    [Arguments]    ${sgt}    ${start}    ${end}    ${node}=127.0.0.1
     ${resp} =    Get Bindings    ${node}
     FOR    ${num}    IN RANGE    ${start}    ${end}
         ${ip} =    Sxp.Get Ip From Number    ${num}
@@ -311,8 +373,8 @@ Check Binding Range
     END
 
 Check Binding Range Negative
-    [Arguments]    ${sgt}    ${start}    ${end}    ${node}=127.0.0.1
     [Documentation]    Check if Node does not contains Bindings specified by range
+    [Arguments]    ${sgt}    ${start}    ${end}    ${node}=127.0.0.1
     ${resp} =    Get Bindings    ${node}
     FOR    ${num}    IN RANGE    ${start}    ${end}
         ${ip} =    Sxp.Get Ip From Number    ${num}
@@ -320,9 +382,14 @@ Check Binding Range Negative
     END
 
 Setup SXP Environment
-    [Arguments]    ${node_range}=1
     [Documentation]    Create session to Controller, ${node_range} parameter specifies number of localhost nodes to be created on ${ODL_SYSTEM_IP}.
-    RequestsLibrary.Create Session    session    http://${ODL_SYSTEM_IP}:${RESTCONFPORT}    auth=${AUTH}    timeout=${DEFAULT_TIMEOUT_HTTP}    max_retries=0
+    [Arguments]    ${node_range}=1
+    RequestsLibrary.Create Session
+    ...    session
+    ...    http://${ODL_SYSTEM_IP}:${RESTCONFPORT}
+    ...    auth=${AUTH}
+    ...    timeout=${DEFAULT_TIMEOUT_HTTP}
+    ...    max_retries=0
     FOR    ${num}    IN RANGE    1    ${node_range}+1
         ${node} =    Sxp.Get Ip From Number    ${num}
         Add Node    ${node}    retry_open_timer=${num}
@@ -330,30 +397,39 @@ Setup SXP Environment
     END
 
 Check Node Started
-    [Arguments]    ${node}    ${port}=64999    ${system}=${node}    ${session}=session    ${ip}=${node}
     [Documentation]    Verify that SxpNode has data writen to Operational datastore and is running
-    ${resp} =    RequestsLibrary.Get Request    ${session}    /restconf/operational/network-topology:network-topology/topology/sxp/node/${node}/
+    [Arguments]    ${node}    ${port}=64999    ${system}=${node}    ${session}=session    ${ip}=${node}
+    ${resp} =    RequestsLibrary.Get Request
+    ...    ${session}
+    ...    /restconf/operational/network-topology:network-topology/topology/sxp/node/${node}/
     BuiltIn.Should Be Equal As Strings    ${resp.status_code}    200
     ${rc} =    Get Node Running Status    ${node}    ${port}    ${system}    ${session}    ${ip}
     BuiltIn.Should Be Equal As Strings    ${rc}    0
 
 Check Node Stopped
-    [Arguments]    ${node}    ${port}=64999    ${system}=${node}    ${session}=session    ${ip}=${node}
     [Documentation]    Verify that SxpNode has data removed from Operational datastore and is not running
-    ${resp} =    RequestsLibrary.Get Request    ${session}    /restconf/operational/network-topology:network-topology/topology/sxp/node/${node}/
+    [Arguments]    ${node}    ${port}=64999    ${system}=${node}    ${session}=session    ${ip}=${node}
+    ${resp} =    RequestsLibrary.Get Request
+    ...    ${session}
+    ...    /restconf/operational/network-topology:network-topology/topology/sxp/node/${node}/
     BuiltIn.Should Be Equal As Strings    ${resp.status_code}    404
     ${rc} =    Get Node Running Status    ${node}    ${port}    ${system}    ${session}    ${ip}
     BuiltIn.Should Be Equal As Strings    ${rc}    1
 
 Get Node Running Status
-    [Arguments]    ${node}    ${port}    ${system}    ${session}    ${ip}
     [Documentation]    Get status if node is running by checking that netty server is running
-    ${rc} =    Utils.Run Command On Remote System    ${system}    netstat -tln | grep -q ${ip}:${port} && echo 0 || echo 1    ${ODL_SYSTEM_USER}    ${ODL_SYSTEM_PASSWORD}    prompt=${ODL_SYSTEM_PROMPT}
-    [Return]    ${rc}
+    [Arguments]    ${node}    ${port}    ${system}    ${session}    ${ip}
+    ${rc} =    Utils.Run Command On Remote System
+    ...    ${system}
+    ...    netstat -tln | grep -q ${ip}:${port} && echo 0 || echo 1
+    ...    ${ODL_SYSTEM_USER}
+    ...    ${ODL_SYSTEM_PASSWORD}
+    ...    prompt=${ODL_SYSTEM_PROMPT}
+    RETURN    ${rc}
 
 Clean SXP Environment
-    [Arguments]    ${node_range}=1
     [Documentation]    Destroy created sessions
+    [Arguments]    ${node_range}=1
     FOR    ${num}    IN RANGE    1    ${node_range}+1
         ${ip} =    Sxp.Get Ip From Number    ${num}
         Delete Node    ${ip}
@@ -362,21 +438,33 @@ Clean SXP Environment
     RequestsLibrary.Delete All Sessions
 
 Get Routing Configuration From Controller
-    [Arguments]    ${session}
     [Documentation]    Get Routing configuration from config DS
-    ${resp} =    RequestsLibrary.Get Request    ${session}    /restconf/config/sxp-cluster-route:sxp-cluster-route/    headers=${ACCEPT_XML}
+    [Arguments]    ${session}
+    ${resp} =    RequestsLibrary.Get Request
+    ...    ${session}
+    ...    /restconf/config/sxp-cluster-route:sxp-cluster-route/
+    ...    headers=${ACCEPT_XML}
     ${data} =    BuiltIn.Set Variable If    "${resp.status_code}" == "200"    ${resp.text}    ${EMPTY}
-    [Return]    ${data}
+    RETURN    ${data}
 
 Put Routing Configuration To Controller
-    [Arguments]    ${data}    ${session}
     [Documentation]    Put Routing configuration to Config DS
-    ${resp} =    RequestsLibrary.Put Request    ${session}    /restconf/config/sxp-cluster-route:sxp-cluster-route/    data=${data}    headers=${HEADERS_XML}
+    [Arguments]    ${data}    ${session}
+    ${resp} =    RequestsLibrary.Put Request
+    ...    ${session}
+    ...    /restconf/config/sxp-cluster-route:sxp-cluster-route/
+    ...    data=${data}
+    ...    headers=${HEADERS_XML}
     BuiltIn.Log    ${resp}
     BuiltIn.Should Match    "${resp.status_code}"    "20?"
 
 Clean Routing Configuration To Controller
-    [Arguments]    ${session}
     [Documentation]    Delete Routing configuration from Config DS
-    ${resp} =    RequestsLibrary.Get Request    ${session}    /restconf/config/sxp-cluster-route:sxp-cluster-route/    headers=${ACCEPT_XML}
-    BuiltIn.Run Keyword If    "${resp.status_code}" == "200"    RequestsLibrary.Delete Request    ${session}    /restconf/config/sxp-cluster-route:sxp-cluster-route/
+    [Arguments]    ${session}
+    ${resp} =    RequestsLibrary.Get Request
+    ...    ${session}
+    ...    /restconf/config/sxp-cluster-route:sxp-cluster-route/
+    ...    headers=${ACCEPT_XML}
+    IF    "${resp.status_code}" == "200"
+        RequestsLibrary.Delete Request    ${session}    /restconf/config/sxp-cluster-route:sxp-cluster-route/
+    END