Fix rspec colour in jenkins

Signed-off-by: Ken Barber <ken@bob.sh>

Merge pull request #162 from kbarber/booleans_not_idempotent

Booleans not idempotent

Fix boolean rules being always recognized as changed.

String and boolean types were compared.
Had to adapt the tests which checked for booleans, when in
reality strings where present.

Add system tests for socket property

Signed-off-by: Ken Barber <ken@bob.sh>

Merge branch 'add_isfragment_option2'

* add_isfragment_option2:
  Add match rule for fragments.

Add match rule for fragments.

Via isfragment true/false property, toggles '-f'.

Same implementation as socket property, same bug
where the property is always recognized as changed.

Merge pull request #145 from ecbypi/ticket/20096-fedora-systemd-support

(20096) Support systemd on Fedora 15 and up

Merge pull request #161 from kbarber/standard_run_tests

Add tests for the recommended setup

Add tests for the recommended setup

Using the documented recommended setup, we test if it works with no error and
test if it is idempotent by running it again, looking for resource changes.

Signed-off-by: Ken Barber <ken@bob.sh>

Merge pull request #160 from kbarber/more_rspec_tests

Add more system tests: class testing in particular

Add more system tests: class testing in particular

Signed-off-by: Ken Barber <ken@bob.sh>

Merge branch 'absolute_path_to_iptables_for_tests'

* absolute_path_to_iptables_for_tests:
  Add an absolute path to iptables to make system tests pass

Add an absolute path to iptables to make system tests pass

Signed-off-by: Ken Barber <ken@bob.sh>

Merge branch '20125'

* 20125:
  (#20125) Add persistence support for Archlinux platform
  (#20125) Use PATH to find (ip|ip6|eb)tables for chain provider
  (#20125) Use PATH to find (ip|ip6)tables for (ip|ip6)tables provider
  (#20125) Add archlinux firewall class and unit test

(#20125) Add persistence support for Archlinux platform

This only works with facter 1.7.0-rc1 and up because os_key isn't
properly defined in facter 1.6.18 on Archlinux (it's set to Linux).

(#20125) Use PATH to find (ip|ip6|eb)tables for chain provider

(#20125) Use PATH to find (ip|ip6)tables for (ip|ip6)tables provider

(#20125) Add archlinux firewall class and unit test

Merge pull request #155 from bobtfish/error_reporting_fix

Error reporting fix

Update to also be for destination

Fix for rspec-system-puppet 0.3.x

Signed-off-by: Ken Barber <ken@bob.sh>

Use rspec-system-puppet 0.3.0

Signed-off-by: Ken Barber <ken@bob.sh>

Support for new prefabs from rspec-system

Signed-off-by: Ken Barber <ken@bob.sh>

Get rid of examples it isn't being maintained and it belongs in docs

I think the expectation that people should drop to looking at code for examples
is wrong anyway, we should express examples through documentation if we can.

Signed-off-by: Ken Barber <ken@bob.sh>

Merge branch 'rspec-system-puppet'

* rspec-system-puppet:
  Moved some of the puppet setup stuff in system tests to rspec-system-puppet

Moved some of the puppet setup stuff in system tests to rspec-system-puppet

Signed-off-by: Ken Barber <ken@bob.sh>

(20096) Support systemd on Fedora 15 and up

Add a check to see if running Fedora 15 in order to use init scripts
provided by systemd. This adds compatibility for systemd on Fedora,
which currently returns an incorrect failure message when persisting
rules.

Use latest rspec-system gem

Signed-off-by: Ken Barber <ken@bob.sh>

Merge branch 'rspec-system'

* rspec-system:
  Initial start on rspec-system tests

Initial start on rspec-system tests

This patch includes system tests using rspec-system. You can try these out
with:

    rake spec:system

Consult the docs in the README.md for details on how to run tests on
different OS variants.

Signed-off-by: Ken Barber <ken@bob.sh>

Merge branch 'ticket/master/release_021'

* ticket/master/release_021:
  Release 0.2.1

Release 0.2.1

Signed-off-by: Ken Barber <ken@bob.sh>

Merge branch 'maint/master/fix_spec_failures'

* maint/master/fix_spec_failures:
  Fix failing spec tests, due to dpkg change in iptables_persistent_version_spec

Fix failing spec tests, due to dpkg change in iptables_persistent_version_spec

Signed-off-by: Ken Barber <ken@bob.sh>

Merge pull request #140 from laurenrother/readme_cleanup

Update README to be consistent with module documentation template

Update README to be consistent with module documentation template

(GH-139) Throw away STDERR from dpkg-query in Fact

Newer versions of dpkg-query, as of Ubuntu 12.10, will make noise on STDERR
if the queried package isn't currently installed. Facter's `exec()` outputs
this without giving us a chance to catch it.

Pipe STDERR to `/dev/null` so that it's not seen by the end-user. STDOUT
will still be `nil` if the package isn't installed. It doesn't seem
reasonable to spec test for this without reaching deep into Facter, so I'm
not going to.

Fix links to forge and github issues in README

Signed-off-by: Ken Barber <ken@bob.sh>

Merge branch 'ticket/master/release_020'

* ticket/master/release_020:
  Release 0.2.0

Release 0.2.0

Signed-off-by: Ken Barber <ken@bob.sh>

Merge pull request #137 from dcarley/persist_iptables_tests

Tests for #persist_iptables

Merge pull request #138 from dcarley/129-autoreq_bad_return

(GH-129) Replace errant return in autoreq block

(GH-129) Replace errant return in autoreq block

It's not valid to use `return` within a block. We could use `next []`,
however it's probably better form to just always return the array, whether
it's populated or not. This will stop the error:

    err: Got an uncaught exception of type LocalJumpError: unexpected return

When one of the listed providers isn't selected. Which is suitable, because
this autorequire won't be suitable to any other future providers anyway.

Tests for #persist_iptables

Basic coverage of protocol and OS detection. Including older and newer
Debian versions. Nearly all based on expectations since there aren't any
return values.

Typo in #persist_iptables OS normalisation

Debian is upstream of Ubuntu. Not the other way around. Would have affected
users of Facter <1.6.2 which doesn't have osfamily. Discovered while writing
tests, yey tests.

Merge pull request #136 from dcarley/134-autorequire_packages

(GH-134) Autorequire iptables related packages

(GH-134) Refer to new classes in documentation.

(GH-134) Autorequire iptables related packages

autorequires from firewall and firewallchain resources to iptables and
iptables-persistent packages, when the appropriate provider is selected and
the packages are managed in the catalog. This will prevent failed rule
creation and persistence on fresh nodes where the packages may not be
pre-installed.

Merge pull request #133 from dcarley/native_persistence

Native persistence

Firewall and firewallchain persistence

Call the necessary commands from the provider to persist rules between
reboots. Tested on the following distros:

- CentOS 5.8
- CentOS 6.3
- Ubuntu 10.04
- Ubuntu 12.04
- Debian 6

Cavaets:

- Persistence may fail on the first run if Firewall resources are actioned
  before the Package resource.
- Older iptables-persistent doesn't support the restoration of ip6tables.
- ebtables cannot be restored.

Missing modulefile change

Signed-off-by: Ken Barber <ken@bob.sh>

Release 0.1.1

Signed-off-by: Ken Barber <ken@bob.sh>

Fix define_method for Ruby 1.9.x

Signed-off-by: Ken Barber <ken@bob.sh>

Merge branch 'ticket/master/128-puppet_30_broke_method_missing'

* ticket/master/128-puppet_30_broke_method_missing:
  (GH-128) Change method_missing to define_method

(GH-128) Change method_missing to define_method

Previously method_missing was enough to create dynamic methods but Puppet 3.0
broke that functionality. So here we used 'define_method' instead to work
around that.

Signed-off-by: Ken Barber <ken@bob.sh>

Firewall class manifests

Manifests for managing the required packages and services on supported Linux
operating systems. These will be required for persistence.

Change source to specific https target

This removes the warnings from rubygems-2.0.0 and ruby-2.0.0.

Signed-off-by: Ken Barber <ken@bob.sh>

Merge branch 'maint/master/fix_travis_gem_versions'

* maint/master/fix_travis_gem_versions:
  Fix gem versions for travis.yml

Fix gem versions for travis.yml

Signed-off-by: Ken Barber <ken@bob.sh>

Fix error reporting for insane hostnames.

If you put some really silly values in (e.g. /) into hostnames then
the error message s super super cryptic.

This patch fixes that, so it's at least obvious what / where / why it's
failing if you use --trace --debug

Merge branch 'maint/master/fix_forge_rendering'

* maint/master/fix_forge_rendering:
  New changelog

New changelog

Signed-off-by: Ken Barber <ken@bob.sh>

Merge branch 'maint/master/new_release_010'

* maint/master/new_release_010:
  Release 0.1.0

Release 0.1.0

Signed-off-by: Ken Barber <ken@bob.sh>

Update docs for source and dest - they are not arrays

Signed-off-by: Ken Barber <ken@bob.sh>

Merge pull request #105 from wuwx/master

ip6tables provider allways execute /sbin/iptables command

Merge pull request #110 from dcarley/expect_resolv_getaddress

Mock Resolv.getaddress in #host_to_ip

Mock Resolv.getaddress in #host_to_ip

Add an expect for Resolv.getaddress in Puppet::Util::Firewall#host_to_ip so
that the test can be run when disconnected from the net. Also isolates it
should should puppetlabs.com move to a different address.

Merge branch 'maint/master/add_puppet_31_to_travis'

* maint/master/add_puppet_31_to_travis:
  Add Puppet 3.1 to travis-ci test matrix

Add Puppet 3.1 to travis-ci test matrix

Signed-off-by: Ken Barber <ken@bob.sh>

Merge pull request #96 from sfozz/documentation-fixes

Add missing class declaration

Merge pull request #109 from dcarley/fix_tests_ruby_193

Fix tests for Ruby 1.9.3 from 3e13bf3

Fix tests for Ruby 1.9.3 from 3e13bf3

Changes in 3e13bf3 broke tests for Ruby 1.9.3 which doesn't support
Enumerable on Strings. Workaround this by casting everything as an array and
flattening to prevent existing arrays from being encapsulated.

Merge branch 'ticket/master/single_sport_dport_support_for_parsing'

* ticket/master/single_sport_dport_support_for_parsing:
  Add support for single --sport and --dport parsing

Add support for single --sport and --dport parsing

Previously if someone already had a rule with a single --sport or --dport we
would fail the parse. This now accepts parsing in the single variant, while
still supporting the multiport variant.

Signed-off-by: Ken Barber <ken@bob.sh>

Merge branch 'ticket/master/socket'

* ticket/master/socket:
  Add support for -m socket option
  Added type for socket
  Add tests for socket option

Add support for -m socket option

This adds support to boolean-style flags like '-m socket' as well.

Signed-off-by: Ken Barber <ken@bob.sh>

Added type for socket

Signed-off-by: Ken Barber <ken@bob.sh>

Add tests for socket option

Signed-off-by: Ken Barber <ken@bob.sh>

Merge branch 'standardize_travis'

* standardize_travis:
  Fix require of precise puppet library
  Update travis and gemfile to be like stdlib travis files
  Remove gemfile.lock and add to gitignore

Fix require of precise puppet library

rspec tests fail unless we added require 'puppet'.

Signed-off-by: Ken Barber <ken@bob.sh>

Update travis and gemfile to be like stdlib travis files

Remove gemfile.lock and add to gitignore

working with ip6tables support

Merge pull request #102 from kbarber/test_updates

Update test framework to the modern age

Update test framework to the modern age

* Install puppetalbs_spec_helper and removed the stuff we were using previously
* Get tests running on 3.0.x
* Update gemspecs to more recent revisions of test tooling

Signed-off-by: Ken Barber <ken@bob.sh>

Merge branch '14463-port_fixnums_to_strings'

Fixes #101 pull request.

(#14463) Fix to pass unit tests

* Add default protocol to fix the test for converting a string 'ssh' to a port
number was failing like so:
  1) Puppet::Type::Firewall dport should convert a port name for dport to its number
     Failure/Error: @resource[port] = 'ssh'
     Puppet::Error:
       Parameter dport failed: Munging failed for value "ssh" in class dport: no such service ssh/proto
     # ./lib/puppet/type/../../puppet/util/firewall.rb:84:in `getservbyname'
     # ./lib/puppet/type/../../puppet/util/firewall.rb:84:in `string_to_port'
     # ./lib/puppet/type/firewall.rb:164:in `unsafe_munge'
     # ./spec/unit/puppet/type/firewall_spec.rb:161

* Always convert the response .to_s

(#14463) Convert port Fixnum into strings

Avert errors like this:
Parameter dport failed: Munging failed for value 1194 in class dport: can’t convert Fixnum into String

Also, pass along the protocol so Socket can make well informed decisions.

Merge pull request #100 from mediatemple/16004-fix_uidgid

(#16004) uid/gid array_matching is contraindicated.

(#16004) array_matching is contraindicated.

With ":array_matching =>:all", uid/gid rules are reloaded with every
Puppet run.  This is ugly and annoying, and arguably wrong.

Add missing class declaration

README.markdown was missing details about declaring 'my_fw::pre'
and 'my_fw::post' which caused folks following the example to see
the following error:

Could not find dependency Class[My_w::Pre] for Firewall[BLAH]

Merge pull request #91 from saysjonathan/rspec_new_api

rspec 2.11 compatibility

rspec 2.11 compatibility

Merge pull request #89 from kbarber/ticket/master/10322-error_with_same_chain_diff_table

(#10322) Insert order hash included chains from different tables

(#10322) Insert order hash included chains from different tables

This fix corrects the insert_order handling to make sure that not only are
rules from the same chain evaulated, but we also check that the table
matches as well.

Merge pull request #87 from dcarley/15556-icmp6_codes

(#15556) Support for ICMP6 type code resolutions

(#15556) Support for ICMP6 type code resolutions

Add support for IPv6 ICMP code types as strings, which differ in mapping
from IPv4. A subset of the currently supported strings for IPv4 are
supported where applicable to the IPv6 specification.

Currently the only way of determining the protocol family is by whether the
provider is :iptables or :ip6tables. This can be changed within the type in
the future.

Merge pull request #86 from Whopper92/readme_best_practices

Update formatting of README to meet Puppet Labs best practices

Update formatting of README to meet Puppet Labs best practices