Skip to content
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.

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

Fixing unit tests for develop #36141

Merged
merged 3 commits into from
Sep 8, 2016

Conversation

meaksh
Copy link
Contributor

@meaksh meaksh commented Sep 8, 2016

What does this PR do?

This PR fixes some failing unit tests in develop branch

@cachedout
Copy link
Contributor

Go Go Jenkins!

@rallytime rallytime merged commit 235d630 into saltstack:develop Sep 8, 2016
gitebra pushed a commit to gitebra/salt that referenced this pull request Sep 9, 2016
* upstream/develop: (111 commits)
  Pylint fix
  Remove redundant write of public key.
  Fix breakage from updates to __utils__ (saltstack#36168)
  Remove unclosed backticks in walkthrough doc (saltstack#36170)
  fixed whitespace lint issues
  Added documentation and cleaned up some linting issues.
  Back-port saltstack#36067 to carbon (saltstack#36159)
  Use string.ascii_lowercase (saltstack#36163)
  Fixing unit tests for develop (saltstack#36141)
  Adds saltstack#36055 to release notes
  Pylint fix for saltstack#36072 (saltstack#36155)
  INFRA-3943 - Bug fix - too many volumes getting tagged (saltstack#36072)
  Remove unclosed backticks in walkthrough doc (saltstack#36154)
  Fix issue with cp.push (saltstack#36136)
  Document `owner` kwarg for `postgres_schema.present` state function (saltstack#36147)
  Ignore states that do not have a numeric jid, i.e. 'req'
  Fixed _interfaces_ifconfig output for SunOS test
  Fix tests that assert CommandExecutionError (saltstack#32485)
  Fix tests (saltstack#35693)
  Skip utils_test if timelib is not installed (saltstack#32699)
  ...
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.

None yet

3 participants