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

Update beaker tests + travis integration #180

Merged
merged 2 commits into from
Sep 5, 2015
Merged

Conversation

solarkennedy
Copy link
Contributor

This updates the beaker tests to the point where they work again.

@aj-jester this is ready for review. I mostly wanted this in perpetration for integrating your work with installing zips and downgrades.

If we want to be sure that our code can fetch consul from the internet, unzip, verify, downgrade, etc, then we should describe it in code. With these this infrastructure we should be able to do that reproducibly, and if it ever breaks we will know.

Of course you can run them manually yourself if you have docker installed with bundle exec rake beaker

- PUPPET_VERSION="~> 3.1.0"
- PUPPET_VERSION="~> 3.3.0"
- PUPPET_VERSION="~> 3.7.4" FUTURE_PARSER=yes STRICT_VARIABLES=yes
- PUPPET_VERSION="~> 4.0.0"
- PUPPET_VERSION="~> 4.1.0"
- BEAKER=true
global:

Choose a reason for hiding this comment

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

may be PUPPET_VERSION=* needs to move into global as well? Currently it seems to conflict between the 2 data types; array and hash.

@solarkennedy solarkennedy force-pushed the more_beaker branch 4 times, most recently from 03a5df9 to 778740a Compare August 25, 2015 15:17
@solarkennedy solarkennedy changed the title Update beaker tests - wip Update beaker tests + travis integration Aug 26, 2015
@aj-jester
Copy link

@solarkennedy niiiiiice! Let me try this again and I'll let you know in a few.

@solarkennedy
Copy link
Contributor Author

I'm going to go ahead and merge this in, as they do pass and work.
We can get more sophisticated as we go and want to ensure certain behaviors.

solarkennedy added a commit that referenced this pull request Sep 5, 2015
Update beaker tests + travis integration
@solarkennedy solarkennedy merged commit 3d503f4 into master Sep 5, 2015
@aj-jester
Copy link

@solarkennedy sorry for not getting back quickly. I had time to play around with it earlier this week and think it looks good. I'll play around with it some more this weekend. Thanks.

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.

2 participants