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

(MODULES-3043) add acceptance and reference test for choco install #14

Merged
merged 1 commit into from May 13, 2016

Conversation

ThoughtCrhyme
Copy link

@ThoughtCrhyme ThoughtCrhyme commented May 4, 2016

Added reference and acceptance tests for installing the Chocolatey application via the Chocolatey Module.

https://tickets.puppetlabs.com/browse/MODULES-3043

require 'master_manipulator'
test_name 'MODULES-3043 - C97739 - Install Client on Virgin System'

chocolatey_pp = 'include chocolatey'

Choose a reason for hiding this comment

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

This could be a failure point. I don't think we want to have this. I'd much prefer we use our internal chocolatey.nupkg that we use for our internal build systems.

Other than that, this looks pretty good.

chocolatey_download_url => 'file:///C:/chocolatey.nupkg',
use_7zip => false,
}
include chocolatey

Choose a reason for hiding this comment

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

Getting closer. I think you mean to remove this line (the lines above are basically the same).

@ThoughtCrhyme ThoughtCrhyme force-pushed the master branch 2 times, most recently from 4654921 to d50646e Compare May 10, 2016 18:19

url = get_latest_chocholatey_download_url


Choose a reason for hiding this comment

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

Unnecessary newline.

@ThoughtCrhyme ThoughtCrhyme force-pushed the master branch 2 times, most recently from f179019 to fcf7b40 Compare May 11, 2016 16:34
@cowofevil
Copy link

👍

Added reference and acceptance tests for installing the Chocolatey application via the Chocolatey Module.
@ferventcoder
Copy link

👍

@ferventcoder ferventcoder merged commit 1818523 into puppetlabs:master May 13, 2016
ThoughtCrhyme pushed a commit to ThoughtCrhyme/puppetlabs-chocolatey that referenced this pull request Mar 20, 2017
…subcommand

(QA-2149) Add config subcommand
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants