Skip to content

(#15521) Convert to using host stub wrapper#984

Merged
joshcooper merged 1 commit intopuppetlabs:2.7.xfrom
kbarber:ticket/2.7.x/15521-use_dynamic_forge_locale_for_acceptance
Aug 13, 2012
Merged

(#15521) Convert to using host stub wrapper#984
joshcooper merged 1 commit intopuppetlabs:2.7.xfrom
kbarber:ticket/2.7.x/15521-use_dynamic_forge_locale_for_acceptance

Conversation

@kbarber
Copy link
Contributor

@kbarber kbarber commented Jul 30, 2012

NOTE: this depends on the following pull request to be merged first:

https://github.com/puppetlabs/puppet-acceptance/pull/260

This patch converts the module acceptance code to using the host stub wrapper
now provided in puppet-acceptance: stub_forge_on.

This will allow us to change the forge host we are stubbing dynamically as
apposed to manually changing it everywhere in acceptance code.

This also changes the cases where we have teardown code to use the new
teardown block.

Copy link
Member

Choose a reason for hiding this comment

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

good catch

This patch converts the module acceptance code to using the host stub wrapper
now provided in puppet-acceptance: `stub_forge_on`.

This will allow us to change the forge host we are stubbing dynamically as
apposed to manually changing it everywhere in acceptance code.

This also changes the cases where we have teardown code to use the new
`teardown` block, and to use `rm -rf` for file cleanup to avoid puppet backup
duplication issues.
@kbarber
Copy link
Contributor Author

kbarber commented Aug 10, 2012

@joshcooper if you have time, can you take a look at this again? Thanks mate.

@joshcooper
Copy link
Contributor

@kbarber will do today

@joshcooper
Copy link
Contributor

@kbarber actually, we're working through some function issues in 2.7.x, so whenever that gets cleared up.

@joshcooper joshcooper merged commit fcd4957 into puppetlabs:2.7.x Aug 13, 2012
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.

3 participants