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

(maint) Fix package tests to remove hardcoding #613

Merged
merged 1 commit into from
Jan 9, 2019

Conversation

bmjen
Copy link
Contributor

@bmjen bmjen commented Jan 9, 2019

The vendored Gemfile name had a hardcoded value with the "latest ruby"
version. This caused breakage in package tests each time packaged ruby
versions were updated. This fix updates the tests to detect the latest version
from the pdk private cache.

@bmjen bmjen requested review from scotje and rodjek January 9, 2019 00:56
@bmjen
Copy link
Contributor Author

bmjen commented Jan 9, 2019

Ran the package tests changes against SHA "ea596e938f28de92f884e2abfd1356256641cd66" on redhat7 and windows2012.

@coveralls
Copy link

coveralls commented Jan 9, 2019

Coverage Status

Coverage remained the same at 92.887% when pulling 63927a4 on bmjen:fix-pkg-test into 26aad19 on puppetlabs:master.

The vendored Gemfile name had a hardcoded value with the "latest ruby"
version. This caused breakage in package tests each time packaged ruby
versions were updated. This fix updates the tests to detect the latest version
from the pdk private cache.
@scotje scotje merged commit db9a1f4 into puppetlabs:master Jan 9, 2019
@bmjen bmjen deleted the fix-pkg-test branch March 11, 2019 22:34
@chelnak chelnak added the maintenance Internal maintenance work that shouldn't appear in the changelog label Jan 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Internal maintenance work that shouldn't appear in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants