Merge pull request #523 from DavidS/modules-2516-is_a

(MODULES-2561) add is_a function

Merge pull request #511 from dmitryilyin/parseyaml_default

[MODULES-2462] Improve parseyaml function

(MODULES-2516) Adds an is_a() function

The data type system is very hard to understand. Many people don't
understand why

    type_of([1,2,3]) == Array

will fail, but

    type_of([1,2,3]) <= Array

passes. This does a simpler validation that doesn't rely on explicit
data types. Instead, use

    $foo = [1,2,3]
    if $foo.is_a(Array) {
      notify { 'This is an array': }
    }

This is based on code by Ben Ford <ben.ford@puppetlabs.com>.

  * Added acceptance tests
  * Added dispatch
  * Improved unit tests
  * Added docs to README

Merge pull request #518 from logicminds/case_insensitive

accept any case of boolean strings

Merge pull request #519 from dmitryilyin/fetch

[MAINT] Improve 'try_get_value' readme

[MAINT] Improve 'try_get_value' readme

accept any case of boolean strings

  * previously the str2bool function did not accept 'TRUE' as a bool
    type.  This causes the function to now accept TRUE, FALSE strings
    as a boolean type in order to be converted to a proper boolean.
  * This would also cause Y,N, YES, NO to be accepted as boolean types
    as well.

Release Prep 4.9.0

Merge pull request #517 from DavidS/try_get_value-acceptance

(MAINT) fix up try_get_value acceptance test

(MAINT) fix up try_get_value acceptance test

Merge pull request #513 from dmitryilyin/fetch

Add a new function "try_get_value"

Add a new function "try_get_value"

* Extracts a value from a deeply-nested data structure
* Returns default if a value could not be extracted

Merge pull request #515 from jfautley/ticket/MODULES-2478-support_root_home_fact_on_AIX

(MODULES-2478) Make root_home fact work on AIX using native lsuser command

(MODULES-2478) Support root_home fact on AIX through "lsuser" command

Squashed, and amended test for comment lines.

Merge pull request #514 from DavidS/add-convert_base

Adds a convert_base function, which can convert numbers between bases

Adds a convert_base function, which can convert numbers between bases

Squashed, improved docs, updated error handling and unit tests by David S.

[MODULES-2462] Improve parseyaml function

* Add default value support
  Second argument will be returned if yaml cannot be parsed
  instead of false value
* Update tests

Merge pull request #512 from Jetroid/consistentreadme

Consistent Readme

Add consistent *Type* information

Remove trailing whitespace

Two functions had not been given any *Type* information. This commit fixes that.

Merge pull request #510 from DavidS/base64-unit-tests

(MAINT) improve base64 unit tests

(MAINT) improve base64 unit tests

Merge pull request #507 from Jetroid/mod2456

(MODULES-2456) Modify union to accept more than two arrays

(MODULE-2456) Modify union to accept more than two arrays

Add spec tests to test the new functionality:
 *Case for 3 arrays.
 *Case for 4 arrays.
Modify README to note new functionality.

This is for issue MODULE-2456, follow the precedent of MODULE-444.

This change allows union to be much more useful, unioning many arrays
in one line rather than in n lines. Additionally, as this is only added
functionality, and does not affect the 2 array case that all modules
currently using array are using, it should not affect any existing
modules utilizing union.

This is now useful, for example, for merging many arrays of resources
(eg: packages.) to generate just one list with no duplicates, to avoid
duplicate resource declarations.

Add a service_provider fact

This returns the default provider Puppet will choose to manage services
on this system by instantiating a dummy service resource type and
returning the provider chosen.

Co-Authored-By: Simon Fraser University <asa188@sfu.ca>

Merge pull request #505 from gibbsoft/dos2unix

(MODULES-2410) Add new functions dos2unix and unix2dos

(MODULES-2410) Add new functions dos2unix and unix2dos

Merge pull request #499 from jearls/2370-use-match-for-ensure-absent

[MODULES-2370] allow `match` parameter to influence `ensure => absent` behavior.

Merge pull request #503 from puppetlabs/4.8.x

Mergeback 4.8.x

Add puppet_version back to spec_helper

Merge pull request #502 from hunner/fix_after

Sometimes this exits 1

Sometimes this exits 1

Merge pull request #501 from hunner/bug_helper

Fix extraneous end

Fix extraneous end

Merge pull request #500 from hunner/prep_4.8.0

Prep 4.8.0

Prep 4.8.0

allow `match` parameter to influence `ensure => absent` behavior.

Split the `destroy` method of the file_type::ruby provider into two
private methods:  `handle_destroy_line` which is the same as the previous
`destroy` method, and `handle_destroy_with_match` which will destroy any
line which matches the `match` parameter, raising an error if multiple
lines match and the `multiple` parameter is not `true`.  This new
behavior is only used if the new boolean parameter `match_for_absence`
is `true` (it defaults to `false`).

Merge pull request #497 from domcleal/tickets/master/MODULES-2316

(MODULES-2316) Change file_type boolean parameter to symbols

(MODULES-2316) Change file_type boolean parameter to symbols

Puppet's boolean parameter type is only available in Puppet 3.3 and
higher, so change file_type's new "replace" parameter to a regular
parameter with true and false as possible values.  This matches the
existing "multiple" parameter.

Merge pull request #483 from nibalizer/load_metadata_json

Add load_metadata_json function

Add load_metadata_json function

This function loads the metadata.json into a puppet variable. This enables a number of neat things such as:

* Which version of the module am I using? 2.x? 3.x?
* Which author of the module am I using? puppetlabs? example42?

Merge pull request #496 from pizzaops/master

Remove colorful language from module.

Remove colorful language from module.

Merge pull request #494 from CENGN/fix/master/file_line_replace

[#puppethack] Adding replace attribute to file_line

Merge pull request #495 from puppetlabs/4.7.x

4.7.x

(MODULES-2024) Adding replace attribute to file_line

Merge pull request #493 from DavidS/fixup-acceptance-testing

(maint) use puppet's utility function instead of API that's not avail…

(maint) use puppet's utility function instead of API that's not available on all rubies

Merge pull request #492 from DavidS/fixup-acceptance-testing

Fixup acceptance testing

Merge pull request #491 from ekohl/style-fixes

Style fixes

Style fixes

(main) clean up fqdn_rand acceptance tests to work on windows

(maint) Remove failing acceptance test for ensure_packages

This only duplicates what's already being tested in ensure_packages_spec.rb
but doesn't work on all our supported platforms.

Merge pull request #489 from gcmalloc/master

adding support for hash in the size function

adding support for hash in the size function

Merge pull request #490 from hunner/fix_specifier

disable pw_hash test on sles, as it only supports md5

disable pw_hash test on sles, as it only supports md5

Merge pull request #487 from liv3d/fix_docs_in_upcase

Fix documentation error in upcase

Merge pull request #488 from cyberious/master

(maint) Fix test to not assume is_pe fact on > 4.0.0 puppet

(maint) Fix test to not assume is_pe fact on > 4.0.0 puppet

Fix documentation error in upcase

The documentation example shows an incorrect response when using the
function, this PR corrects the example to agree with what the function
actually does.

Merge pull request #481 from tphoney/release_4.7.0

prep work for 4.7.0

prep work for 4.7.0

add new improvements

puppet4 changes for testing

Merge pull request #485 from ghoneycutt/patch-2

Clarify that third argument to ensure_resource() is a hash

Clarify that third argument to ensure_resource() is a hash

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