Merge pull request #484 from mhaskel/use_puppet_install_helper

Use puppet_install_helper

Use puppet_install_helper

Merge pull request #475 from DavidS/document-puppet4-compat

Document puppet 4 compatability in 4.6

Merge pull request #482 from DavidS/document_validate_slength_3rd_arg

Add validate_slength's optional 3rd arg to README

Merge pull request #479 from mklette/master

catch and rescue from looking up non-existent facts

Add validate_slength's optional 3rd arg to README

catch and rescue from looking up non-existent facts when looking for 'kind'

facter (2.x) only provides facts without interface suffix for
 * ipaddress
 * netmask

'macaddress' and 'network' facts will always have the related interface
name appended. in turns lookupvar throws errors when strict_variables is
enabled.

Merge pull request #478 from drewfisher314/ticket/2134-add-solaris-12

Add support for Solaris 12

Add support for Solaris 12

Merge pull request #477 from mhaskel/aio_fix

AIO uses puppet 4 so should return true for is_future_parser_enabled

AIO uses puppet 4 so should return true for is_future_parser_enabled

Merge pull request #474 from DavidS/fix-getvar-specs

(maint) getvar: update spec to match implementation

Document puppet 4 compatability in 4.6

(maint) getvar: update spec to match implementation

Merge pull request #470 from Yelp/getvar_future_parser

Also catch :undefined_variable as thrown by future parser

Merge pull request #472 from justinstoller/maint/master/set_puppet_version

(maint) update PUPPET_VERSION default to be 3.8.1

(maint) update PUPPET_VERSION default to be 3.8.1

Merge pull request #471 from puppetlabs/travisci_update

Updated travisci file to remove allow_failures on Puppet4

Also catch :undefined_variable as thrown by future parser

Merge pull request #469 from hunner/fix_time

Fix time() on 1.8.7

Fix time() on 1.8.7

The time() function takes an argument of a timezone, and always returns
time in epoch format. The epoch format is the number of seconds that
have elapsed since January 1, 1970 (midnight UTC/GMT), not counting leap
seconds. This means that it is universally the same regardless of
timezones.

I don't know what the timezone argument is supposed to do, and it is not
documented. So lets just make 1.8.7 work like > 1.8.7

Updated travisci file to remove allow_failures on Puppet4

Merge pull request #468 from kylog/camels-all-the-way-down

Fix spelling of camelcase

Merge pull request #466 from elyscape/fix/improve_fqdn_acceptance_tests

Improve fqdn_rotate/fqdn_rand_string acceptance tests

(doc) Fix spelling of camelcase in CHANGELOG.md

fqdn_rand_string: Add acceptance tests for custom charsets

fqdn_rotate: Add acceptance tests for custom seeds

Merge pull request #467 from DavidS/document-rspec-puppet-version-reqs

Gemfile: specify minimum rspec-puppet version

Gemfile: specify minimum rspec-puppet version

Only 2.2 contains all the features we're currently using. Documenting
that in the Gemfile should make that clear.

Merge pull request #462 from elyscape/fix/fqdn_rotate_seeds_with_argument

fqdn_rotate: Don't use the value itself as part of the random seed

fqdn_rotate: Add tests for custom seeds

fqdn_rotate: Improve documentation

fqdn_rotate: Don't use the value itself as part of the random seed

Previously, the random number generator was seeded with the array or
string to be rotated in addition to any values specifically provided for
seeding. This behavior is potentially insecure in that it allows an
attacker who can modify the source data to choose the post-shuffle
order.

Merge pull request #464 from DavidS/modules-1882-convert-to-rspec

(MODULES-1882) convert function tests to rspec-puppet

Remove unused puppet_spec code

This is copied and changed code from an older version of puppet's internal
test setup code. It does not work with puppet4.

Convert tests to use plain rspec-puppet

Tests in the new style produces the following documentation output:

abs
  should not eq nil
  should run abs() and raise an Puppet::ParseError
  should run abs(-34) and return 34
  should run abs("-34") and return 34
  should run abs(34) and return 34
  should run abs("34") and return 34

spec_helper: implement an easy way for specs to confine to puppet version

Usage:

    describe 'puppet3 behaviour', :unless => RSpec.configuration.puppet_future do
    describe 'puppet4 behaviour', :if => RSpec.configuration.puppet_future do

Add the missing symlinks to get puppet 4 pickup the functions from the environmentpath

root all the gitignore patterns to avoid unintentional matches

Remove duplicate rake task and enable metadata.json linting

The :validate task is already provided by
puppetlabs_spec_helper/rake_tasks and would check files twice.

The metadata.json linting is just good form.

Workaround the broken rspec-mocks support in rspec-puppet

Enable bundler caching on travis

Merge pull request #465 from igalic/patch-1

simplify mac address regex

simplify mac address regex

let the computer do the counting and repetition and case

Merge pull request #463 from CENGN/fix/master/file_line_multiple_after

 (MODULES-2071) Patch file_line provider to use multiple with after

 (MODULES-2071) Refactor file_line provider to contain logic to handle parameter multiple in function handle_create_with_after

Without this, file_line resource without the `match` parameter but with the `after` param will throw an error if there are multiple matches for the after expression. This patch creates the handling for the `multiple` parameter in handle_create_with_after. This allows you to add a line after the `after` expression if it appears at multiple points in a file.

Updated reference to `file_line` in the README to reflect that the multiple parameter can be set when using `after` and/or `match` as the matching regex.

Merge pull request #461 from DavidS/validate-hashes

validate_integer, validate_numeric: explicitely reject hashes in arrays

validate_integer, validate_numeric: explicitely reject hashes in arrays

Without this patch, Ruby 1.8's Hash#to_s behaviour causes [{1=>2}] to be
treated as "12" when validating values.

Add ability to unittest puppet 4

Merge pull request #460 from DavidS/fix-fqdn_rotate-seed

fqdn_rotate: reset srand seed correctly on old ruby versions

fqdn_rotate: reset srand seed correctly on old ruby versions

Without this, the global seed is reseeded on every use
of fqdn_rotate, which is a waste. Older rubies might even use a
time-base seed which adversly impacts the quality of the RNG.

Merge pull request #454 from elyscape/document_facts.d_cache

(FM-2130) Document new location of facts.d cache

Merge pull request #455 from hunner/change_type_of_test

Remove all the pops stuff

Merge pull request #458 from ghoneycutt/patch-1

Update CHANGELOG.md

Update CHANGELOG.md

Merge pull request #456 from jtappa/readme-edits

DOC-1504: Readme edits

DOC-1504: README edits

Remove all the pops stuff

The rspec-puppet matchers don't allow to check the return types, but
this is a pretty rare thing to need to do anyway, so probably not worth
patching rspec-puppet

(FM-2130) Document new location of facts.d cache

Merge pull request #449 from puppetlabs/modulesync_updates

sync via modulesync

sync via modulesync

Merge pull request #448 from DavidS/fix_range

range(): fix TypeError(can't convert nil into Integer) when using range ...

range(): fix TypeError(can't convert nil into Integer) when using range syntax

Merge pull request #447 from elyscape/fix_range

Restore removed functionality to range()

Merge pull request #446 from elyscape/fix_pw_hash

Fix pw_hash() on JRuby < 1.7.17

range: Clean up and clarify function contents

Revert "range: remove dead code"

This reverts commit 063c58a992c1b5441b7e7b2a2e4886531035bb25, which
actually removed non-dead code. Specifically, it removed the ability to
make calls such as `range('2..3')`, `range('2...3')`, and
`range('2-3')`.

cf. https://github.com/puppetlabs/puppetlabs-stdlib/pull/443#commitcomment-11055565

pw_hash: Fix functionality on JRuby < 1.7.17

The previous change to this function broke it on JRuby before 1.7.17 by
attempting to use a variable that wasn't defined (`salt`). To fix this,
define `salt` ahead of time and use that instead of building the salt
later.

cf. https://github.com/puppetlabs/puppetlabs-stdlib/pull/443#discussion_r29718588

Merge pull request #443 from DavidS/prep-work-for-new-specs

Prep work for new specs

validate_augeas: fix URL to docs

range: remove dead code

Since a ParseError is always thrown for zero arguments, the if and all
dependent code can be removed.

pw_hash: avoid ruby magic when running on java

spec_helper: Remove unneccesary stubbing

This only roots all Facter instances into memory, while something already
creates a new Facter instance each run.

spec_helper: use proper mocha import to avoid warning

spec_helper: set parser config if requested

spec_helper_acceptance: fix FUTURE_PARSER usage

Use the more common "yes", instead of "true" to detect FUTURE_PARSER.

specs: move function specs to where rspec-puppet expects them

specs: loosen certain error expectations to make tests pass on future parser

Merge pull request #441 from mrzarquon/change_stdlib_to_include

uses include type class declaration

uses include type class declaration

previous behavior used class { 'stdlib::stages':} which isn't singleton and could cause duplication resource declaration on the stages class.

Since many community modules work by calling 'include stdlib' we should make stdlib's include of stages singleton as well.

Merge pull request #440 from DavidS/fix-error-message

fqdn_rand_string: fix argument error message

fqdn_rand_string: fix argument error message

Merge pull request #439 from puppetlabs/4.6.x

4.6.x

Merge pull request #314 from amateo/feature/loadyaml_check_file

Check if file exists before loading with loadyaml. If not, return nil

Merge pull request #438 from hunner/fix_date

Fix the 4.6.0 release date

Fix the 4.6.0 release date

Merge pull request #437 from hunner/release_4.6.0

Prep for 4.6.0

Prep for 4.6.0

test case for when the file does not exist

Merge pull request #434 from bmjen/modules-2474

Modules-2474: Only runs enhanced salts functions test on systems that ...

Modules-2474: Only runs enhanced salts functional test on systems that support it.

Merge pull request #433 from cmurphy/fix_acceptance

Fix acceptance tests for #405

Fix acceptance tests for #405

This fixes the acceptance tests by:
 - Ensuring the fqdn_rand_string spec is passed undef as the second
   parameter so that the seed is not used as the charset
 - Ensuring the pw_hash spec is passed the key specifying the type of
   hash, rather than the value that will be used to generate the
   password
 - Expecting puppet to report nil instead of empty string for undef
   passwords
 - Removing the fqdn_rand_base64 test because there is no such function

Merge pull request #425 from jeffcoat/validate_augeas_spec

Fix off-by-one error in validate_augeas_spec.rb that was causing rspec failure

Even without the gem installed always, this is helpful to avoid errors should augeas be enabled/installed due to something else.

Merge pull request #431 from bmjen/file-line-refactor

File_line checks provided after param if no match is found

Merge pull request #432 from cmurphy/fix_acceptance_undefined_var

Fix unsupported platforms variable name in tests

Fix unsupported platforms variable name in tests

unsupported_platforms is not a valid identifier, and trying to use it
causes acceptance tests to error out before running any tests. The
correct identifier for the unsupported platforms constants is
UNSUPPORTED_PLATFORMS.

Add spec tests and pulled in PR #427
Changed append line to open in 'w' mode and have to rewrite lines in order to append new line

file_line honors after if match not found.

Merge pull request #430 from mhaskel/gjngeldenhuis-docupdate

Clarifying behaviour of attributes and adding an extra example.