Skip to content

Integration tests for checking simple outputter return data#12707

Merged
s0undt3ch merged 5 commits into
saltstack:developfrom
rallytime:output_tests
May 12, 2014
Merged

Integration tests for checking simple outputter return data#12707
s0undt3ch merged 5 commits into
saltstack:developfrom
rallytime:output_tests

Conversation

@rallytime
Copy link
Copy Markdown
Contributor

Closes #12464

@ghost
Copy link
Copy Markdown

ghost commented May 12, 2014

Test PASSed.
Refer to this link for build results: http://jenkins.saltstack.com/job/salt-pr-build/4270/

@ghost
Copy link
Copy Markdown

ghost commented May 12, 2014

Test FAILed.
Refer to this link for build results: http://cookingwithsalt.org/job/salt/job/pr/job/build/35174/

@ghost
Copy link
Copy Markdown

ghost commented May 12, 2014

Test Failed.

If the failures are unrelated to your code, don't stress, a core developer will know these apart.
In the future, if possible, please branch off a passing commit to avoid false positives.
Refer to this link for build results: http://jenkins.saltstack.com/job/salt-pr-build/4271/

s0undt3ch added a commit that referenced this pull request May 12, 2014
Integration tests for checking simple outputter return data
@s0undt3ch s0undt3ch merged commit 3a4405e into saltstack:develop May 12, 2014
@rallytime rallytime deleted the output_tests branch May 12, 2014 18:49
@ghost
Copy link
Copy Markdown

ghost commented May 12, 2014

Test FAILed.
Refer to this link for build results: http://cookingwithsalt.org/job/salt/job/pr/job/build/35176/

@techdragon
Copy link
Copy Markdown
Contributor

I may be able to use this either as an example or adapt it with some extra code to build a proper test case for #11337 and all the other commands not yet given individual issues, to ensure that all commands return the correct 0/1 exit codes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test outputters directly

3 participants