fix whitespace

This class should be private

Provides a more useful error message when users try to use this class by itself. Without this fix, you'll get something like

```
Error: Evaluation Error: Operator '[]' is not applicable to an Undef Value. at /etc/puppetlabs/code/environments/production/modules/apt/manifests/update.pp:7:8"?
```

Merge pull request #654 from puppetlabs/hunner_msync

(MODULES-4098) Sync the rest of the files

(MODULES-4098) Sync the rest of the files

Merge pull request #650 from puppetlabs/hunner_msync

(MODULES-4097) Sync travis.yml

Merge pull request #652 from puppetlabs/implement-beaker-module_install_helper

[MODULES-4224] Implement beaker-module_install_helper

(MODULES-4097) Sync travis.yml

[MODULES-4224] Implement beaker-module_install_helper

Merge pull request #646 from puppetlabs/modsync

(FM-5972) gettext and spec.opts

gettext and spec.opts

Merge pull request #641 from DavidS/use-stdlib-deprecation

Use stdlib deprecation

Fix for tests against PR-641

Merge pull request #644 from eputnam/spec_opts

(maint) updates spec.opts in prep for parallel_spec

removes spec.opts

Merge pull request #643 from puppetlabs/hunner_msync

(MODULES-3631) msync Gemfile for 1.9 frozen strings

(MODULES-3631) msync Gemfile for 1.9 frozen strings

Merge pull request #638 from lrrb/master

remove port number from hostname

Use the stdlib deprecation() function

Update to require using stdlib 4.13 for deprecation function

remove port number from hostname

Solve the issue reported in https://tickets.puppetlabs.com/browse/MODULES-4104 :  remove the port number from repository location in order to get the host name of the repository.

Merge pull request #636 from puppetlabs/hunner_msync

(MODULES-3704) Update gemfile template to be identical

(MODULES-3704) Update gemfile template to be identical

Merge pull request #635 from lkoranda/fix_string_comparison_error

Avoid string comparison error

Avoiding string comparison error

Signed-off-by: Lukas Koranda <lkoranda@redhat.com>

mocha version update for test hanging issue

Merge pull request #633 from puppetlabs/Paula_modulesync

(MODULES-3983) Update parallel_tests for ruby 2.0.0

Merge pull request #631 from wilson208/key-server-timeout-fix

[MODULES-3562] Implement retry for tests which require modules to pull key from keyserver

(MODULES-3983) Update parallel_tests for ruby 2.0.0

Implement retry on tests which pull key from a key server which sometimes times out (transient error)

Fix tests to check for idempotency correctly.

Merge pull request #629 from puppetlabs/modulesync

Update modulesync_config [51f469d]

Merge pull request #628 from cpavanrun/master

Undef default for $notify_update in source.pp results in problem with Puppet 3.7.2

Update modulesync_config [51f469d]

Merge pull request #624 from seffenberg-naspers/fix_apt_key_with_userinfo

apt_key now really works with userinfo for http(s)

On Puppet 3.7.2 the undef default for $notify_update will overwrite the default boolean setting for $notify_update in apt::settings resulting in an error 400: "" is not a boolean

Merge pull request #627 from puppetlabs/release

Release Mergeback

Merge pull request #626 from underscorgan/maint/master/update-gpg-key

(maint) Update the gpg key

(maint) Update the gpg key

Merge pull request #625 from HelenCampbell/rprep

Release prep 2.3.0

Release prep

apt_key now really works with userinfo for http(s)

now http and https sources for apt_key can take a userinfo
as it should be already since #384

```
apt_key { '...':
  source => 'https://username:password@myhost.com/mypath/key.gpg'
  ..
}
```

Merge pull request #616 from ghoneycutt/add_confs

Add ability to specify a hash of apt::conf defines

Add ability to specify a hash of apt::conf defines

Merge pull request #622 from puppetlabs/modulesync

Update modulesync_config [a3fe424]

Update modulesync_config [a3fe424]

Merge pull request #621 from puppetlabs/modulesync

Update modulesync_config [0d59329]

Update modulesync_config [0d59329]

Merge pull request #618 from puppetlabs/update-modulesync

(MAINT) Update for modulesync_config 72d19f184

(MAINT) Update for modulesync_config 72d19f184

Merge pull request #619 from DavidS/fix

Fix syntax error

Fix syntax error

Merge pull request #617 from hunner/strict_error

(MODULES-3414) Limit non-strict parsing to pre-3.5.0 only

(MODULES-3414) Limit non-strict parsing to pre-3.5.0 only

Puppet 3.5.0 introduced strict variables and the module handles strict
variables by using the defined() function. This does not work on prior
versions of puppet so we must gate based on that version.

Puppet 4 series has a new setting `strict` that may be set to enforce
strict variables while `strict_variables` remains unset (see PUP-6358)
which causes this conditional to erroniously use non-strict 3.5-era
parsing and fail.

The new conditional corrects the cases such that strict variable
behavior happens on versions 3.5.0 and later.

Merge pull request #615 from DavidS/modulesync

(MODULES-3581) modulesync [067d08a]

(MODULES-3581) modulesync [067d08a]

Merge pull request #613 from nicobn/master

Fixed "unless" test condition for ppa repository

Replaced `-s` with `-f` in ppa rspec tests

See 38263ac768349bf0eab56194aa70f4a78fb664af. The previous commit makes the tests fail.

After the repository is added, the ${::apt::sources_list_d}/${sources_list_d_filename} file is created as an empty file.
The unless condition of Exec["add-apt-repository-${name}"] calls test -s, which returns 1 if the file is empty. Because
the file is empty, the unless condition is never true and the repository is added on every execution. This commit
replaces the -s test condition with -f, which is true if the file exists or false otherwise.

Merge pull request #612 from puppetlabs/modulesync

{maint} modulesync 0794b2c

{maint} modulesync 0794b2c

Merge pull request #611 from mattkenn4545/master

Fix TYPO

Fix TYPO

Ubuntu not Ubunto

Merge pull request #610 from puppetlabs/unused-lsb

apt/params: Remove unused LSB facts

Merge pull request #609 from hfm/fix-regexp

Fix regexp for $ensure params

apt/params: Remove unused LSB facts

Fix regexp for $ensure params

/\Aabsent|present\Z/ match wrong values like 'absentaaa' or 'aaapresent'.

And add tests to the context of 'invalid ensure'.

Merge pull request #608 from DavidS/use-more-pool

Update tests to use the pool servers instead of ubuntu's

Update tests to use the pool servers instead of ubuntu's

Using the ubuntu keyserver causes stability issues during CI runs,
hopefully the pool servers are less fragile.

Merge pull request #606 from DavidS/use-key-server-pool

Use hkps.pool.sks-keyservers.net instead of pgp.mit.edu

Remove useless, failing test; cleanup whitespace

Use hkps.pool.sks-keyservers.net instead of pgp.mit.edu

The MIT server is sometimes unreliable during tests. The pool should
provide a much stabler service.

Merge pull request #607 from HelenCampbell/XenialTestFix

Removal of a breaking test for Xenial

Removal of a breaking test for Xenial

Merge pull request #605 from imphil/master

Install software-properties-common for xenial

Install software-properties-common for xenial

Instead of adding every newer release of Ubuntu, let's assume that the
package name that is used today will be used in the future, until the
next change is necessary.

Merge pull request #604 from tdb/fix_version_check

Fix version check on 16.04.

Fix version check on 16.04.

The old test obviously broke on 16.04. Switch to using versioncmp
since it does the right thing for us.

Merge pull request #602 from DG-i/master

apt::setting expects priority to be an integer, set defaults accordingly

apt::setting expects priority to be an integer, set defaults accordingly

Signed-off-by: Mathias Merscher <Mathias.Merscher@dg-i.net>

Merge pull request #600 from puppetlabs/update-msync

Update to newest modulesync_configs [9ca280f]

Update to newest modulesync_configs [9ca280f]

Merge pull request #596 from danielhoherd/master

Expose notify_update to apt::source

Merge pull request #599 from DavidS/fix-strict-var-testing

Fix STRICT_VARIABLE testing

(maint) clean up key_compat_specs

(maint) Update ke_compat_spec to run under STRICT_VARIABLES

Merge pull request #581 from robinelfrink/master

Make apt_updates facts use /usr/bin/apt-get.

Exposing notify_update to apt::source

Merge pull request #595 from danielhoherd/master

Typo: missing colon

Typo: missing colon

Merge pull request #593 from puppetlabs/2.2.x

Mergeback 2.2.x

Merge pull request #592 from HelenCampbell/2.2.2

Prep for 2.2.2 Release

Prep for 2.2.2 Release

Merge pull request #590 from DavidS/debian-8-support

Update metadata to note Debian 8 support

Update metadata to note Debian 8 support

Merge pull request #589 from DavidS/fm-4046-update-msync

(FM-4046) Update to current msync configs [006831f]

(FM-4046) Update to current msync configs [006831f]

This moves all copyright statements to the NOTICE file in accordance with the ASFs guidelines on applying the Apache-2.0 license.

Make apt_updates facts use /usr/bin/apt-get.

/usr/lib/update-notifier/apt-check is not available on all systems,
but /usr/bin/apt-get is.

Merge pull request #582 from imphil/master

Look for correct sources.list.d file for apt::ppa

Look for correct sources.list.d file for apt::ppa

In Ubuntu 15.10 the path of the apt sources file, which is generated by
apt-add-repository, changed to include the distid. This breaks apt::ppa
idempotency, since it does not recognize the repository is already
added.

Reported on puppet-users as well:
https://groups.google.com/forum/#!topic/puppet-users/YzeMyZYUo98

Ensure PPAs in tests have valid form

"man apt-add-repository" notes: "REPOSITORY can be either a line that
can be added directly to sources.list(5), in the form ppa:<user>/<ppa-name>
for adding Personal  Package  Archives".

Fix the tests to always use the format ppa:<user>/<ppa-name> when adding
PPAs.