Skip to content

st2-self-check should ignore any pack actions which name doesn't start with "test_" #4622

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Kami merged 7 commits into from
Apr 9, 2019
Merged

st2-self-check should ignore any pack actions which name doesn't start with "test_" #4622

Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
b3e8d7f
st2-self-check should ignore any pack actions which name doesn't start
Kami Apr 8, 2019
bb8be32
Don't run WinRM tests when run_windows_tests variable is false (defau…
Kami Apr 8, 2019
a26af65
Fix action reference.
Kami Apr 8, 2019
21fac2c
Update the paramiko runner code to replace all occurances of \r\n in the
Kami Apr 8, 2019
48eb609
Merge branch 'master' of github.com:StackStorm/st2 into update_st2_se…
Kami Apr 9, 2019
3de29f1
Merge branch 'update_st2_self_check' of github.com:StackStorm/st2 int…
Kami Apr 9, 2019
2c66d53
Revert "Update the paramiko runner code to replace all occurances of …
Kami Apr 9, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 11 additions & 2 deletions st2common/bin/st2-self-check
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -119,18 +119,27 @@ rm -R st2tests/
popd

# Retrieve test action list
TEST_ACTION_LIST=`st2 action list --pack=tests -w 90 | awk '{ print $2 }' | grep -v "|" | grep -v "ref"`
TEST_ACTION_LIST=`st2 action list --pack=tests -w 90 | awk '{ print $2 }' | grep -v "|" | grep -v "ref" | grep tests.test_`

# Run all the tests
for TEST in $TEST_ACTION_LIST
do
# If -w option is not set, skip Windows related tests
# because smbclient is not installed by default.
if [ ${RUN_WINDOWS_TESTS} = "false" ] && [ ${TEST} = "tests.test_windows_runners" ]; then
echo "Skipping ${TEST}..."
continue
fi

if [ ${RUN_WINDOWS_TESTS} = "false" ] && [ ${TEST} = "tests.test_winrm_runners" ]; then
Copy link
Member Author

@Kami Kami Apr 9, 2019

Choose a reason for hiding this comment

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

Just a heads up - that's something we missed while working on original WinRM runner PRs.

Those tests only works on servers which have access to a configured Windows host which is not the case for default StackStorm installation and st2-self-check script.

echo "Skipping ${TEST}..."
continue
fi

if [ ${RUN_WINDOWS_TESTS} = "false" ] && [ ${TEST} = "tests.test_winrm_large_script" ]; then
echo "Skipping ${TEST}..."
continue
fi

# Skip Mistral Inquiry test if we're not running Mistral tests
if [ ${RUN_MISTRAL_TESTS} = "false" ] && [ ${TEST} = "tests.test_inquiry_mistral" ]; then
echo "Skipping ${TEST}..."
Expand Down