Skip to content

Fix command run by planemo test --skip_venv#907

Merged
nsoranzo merged 1 commit intogalaxyproject:masterfrom
nsoranzo:fix_planemo_test_skip_venv
Jan 3, 2019
Merged

Fix command run by planemo test --skip_venv#907
nsoranzo merged 1 commit intogalaxyproject:masterfrom
nsoranzo:fix_planemo_test_skip_venv

Conversation

@nsoranzo
Copy link
Member

@nsoranzo nsoranzo commented Jan 3, 2019

Fix the following issue found in https://travis-ci.org/peterjc/galaxy_blast/jobs/474815361 :

$ planemo test --no_conda_auto_install --galaxy_root ${TRAVIS_BUILD_DIR}/galaxy-${GALAXY_BRANCH} --skip_venv tools/
Testing using galaxy_root /home/travis/build/peterjc/galaxy_blast/galaxy-master
Testing tools with command [cd /home/travis/build/peterjc/galaxy_blast/galaxy-master && COMMON_STARTUP_ARGS=--skip-venvexport COMMON_STARTUP_ARGSecho "Set COMMON_STARTUP_ARGS to ${COMMON_STARTUP_ARGS}" && sh run_tests.sh $COMMON_STARTUP_ARGS --report_file /home/travis/build/peterjc/galaxy_blast/tool_test_output.html --xunit_report_file /tmp/tmp0dVdsD/xunit.xml --structured_data_report_file /home/travis/build/peterjc/galaxy_blast/tool_test_output.json functional.test_toolbox]
/bin/sh: 1: COMMON_STARTUP_ARGSecho: not found

Introduced in commit 434099e .

Fix the following issue found in https://travis-ci.org/peterjc/galaxy_blast/jobs/474815361 :
```
$ planemo test --no_conda_auto_install --galaxy_root ${TRAVIS_BUILD_DIR}/galaxy-${GALAXY_BRANCH} --skip_venv tools/
Testing using galaxy_root /home/travis/build/peterjc/galaxy_blast/galaxy-master
Testing tools with command [cd /home/travis/build/peterjc/galaxy_blast/galaxy-master && COMMON_STARTUP_ARGS=--skip-venvexport COMMON_STARTUP_ARGSecho "Set COMMON_STARTUP_ARGS to ${COMMON_STARTUP_ARGS}" && sh run_tests.sh $COMMON_STARTUP_ARGS --report_file /home/travis/build/peterjc/galaxy_blast/tool_test_output.html --xunit_report_file /tmp/tmp0dVdsD/xunit.xml --structured_data_report_file /home/travis/build/peterjc/galaxy_blast/tool_test_output.json functional.test_toolbox]
/bin/sh: 1: COMMON_STARTUP_ARGSecho: not found
```

Introduced in commit 434099e .
Copy link
Member

@bgruening bgruening left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Upps. Thanks @nsoranzo!

@nsoranzo nsoranzo merged commit eba1aa0 into galaxyproject:master Jan 3, 2019
@nsoranzo nsoranzo deleted the fix_planemo_test_skip_venv branch January 3, 2019 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants