#180 check if required shell variables are set on basic role
Merged 6 years ago by astepano. Opened 6 years ago by bgoncalv.
bgoncalv/standard-test-roles basic-role-check-test-name  into  master

@@ -3,6 +3,18 @@ 

  - block:

    - name: Execute tests

      shell: |

+       if [[ -z ${TEST} ]]; then

+           echo "FAIL: Test case name is not set" >> {{ remote_artifacts }}/test.log

+           exit

+       fi

+       if [[ -z ${TEST_DIR} ]]; then

+           echo "FAIL: Test directory for $TEST not found" >> {{ remote_artifacts }}/test.log

+           exit

+       fi

+       if [[ -z ${TEST_CMD} ]]; then

+           echo "FAIL: Does not know how to run $TEST" >> {{ remote_artifacts }}/test.log

+           exit

+       fi

        logfile={{ remote_artifacts }}/test.$(echo $TEST | sed -e 's/\//-/g').log

        exec 2>>$logfile 1>>$logfile

        cd $TEST_DIR
@@ -24,11 +36,11 @@ 

      #It is also possible to define the test as dictionary, in that case it is possible

      #to change test directory and test command line parameters while test name is item key

        TEST:

-         "{{item if item.keys is not defined else item.keys()[0]}}"

+         "{{ item if item.keys is not defined else (item.keys()|list)[0] }}"

        TEST_DIR:

-         "{{ tenv_workdir }}/{{ item if item.keys is not defined else item[item.keys()[0]]['dir']|default(item.keys()[0]) }}"

+         "{{ tenv_workdir }}/{{ item if item.keys is not defined else item[(item.keys()|list)[0]]['dir']|default((item.keys()|list)[0]) }}"

        TEST_CMD:

-         "{{'./runtest.sh' if item.keys is not defined else item[item.keys()[0]]['run']|default('./runtest.sh')}}"

+         "{{'./runtest.sh' if item.keys is not defined else item[(item.keys()|list)[0]]['run']|default('./runtest.sh')}}"

      with_items:

      - "{{ tests }}"

  

no initial comment

pretty please pagure-ci rebuild

pretty please pagure-ci rebuild

rebased onto be45aa3

6 years ago

Commit b353f4b fixes this pull-request

Pull-Request has been merged by astepano

6 years ago

Pull-Request has been merged by astepano

6 years ago