diff --git a/.travis.yml b/.travis.yml index 7a03776..e84e077 100644 --- a/.travis.yml +++ b/.travis.yml @@ -7,7 +7,7 @@ sudo: false language: ruby cache: bundler script: "bundle exec rake validate && bundle exec rake lint && bundle exec rake spec SPEC_OPTS='--format documentation'" -bundler_args: --without rake +bundler_args: --without system_tests matrix: fast_finish: true include: diff --git a/Gemfile b/Gemfile index 42468eb..99014ac 100644 --- a/Gemfile +++ b/Gemfile @@ -1,20 +1,16 @@ source 'https://rubygems.org' -group :test do - gem 'rest-client', '~> 1.6.8', :require => false, :platforms => :ruby_18 -end - -group :rake, :test do - gem 'puppetlabs_spec_helper', '>=0.8.2', :require => false - gem 'puppet-blacksmith', :require => false - gem 'beaker', :require => false - gem 'beaker-rspec', :require => false -end - group :rake do - gem 'rspec-puppet', '>=2.1.0', :require => false - gem 'rake', '>=0.9.2.2' - gem 'puppet-lint', '>=1.0.1' + gem 'puppetlabs_spec_helper', '~> 1.1', :require => false + gem 'rspec-puppet', '~> 2.3', :require => false + gem 'puppet-lint', '~> 2.0', :require => false + gem 'puppet-blacksmith', :require => false + gem 'rake', :require => false +end + +group :system_tests do + gem 'beaker-rspec', :require => false + gem 'beaker', :require => false end if puppetversion = ENV['PUPPET_GEM_VERSION'] diff --git a/Rakefile b/Rakefile index c2f0e10..e0fd413 100644 --- a/Rakefile +++ b/Rakefile @@ -12,8 +12,9 @@ task :default => [:clean, :spec] PuppetLint.configuration.fail_on_warnings = true PuppetLint.configuration.send('relative') +PuppetLint.configuration.send('disable_140chars') PuppetLint.configuration.send('disable_class_inherits_from_params_class') PuppetLint.configuration.send('disable_class_parameter_defaults') PuppetLint.configuration.send('disable_documentation') PuppetLint.configuration.send('disable_single_quote_string_with_variables') -PuppetLint.configuration.ignore_paths = ["spec/**/*.pp", "pkg/**/*.pp"] +PuppetLint.configuration.ignore_paths = ["spec/**/*.pp", "pkg/**/*.pp", "vendor/**/*.pp"] diff --git a/spec/classes/nginx_spec.rb b/spec/classes/nginx_spec.rb index 114fbeb..b5b9811 100644 --- a/spec/classes/nginx_spec.rb +++ b/spec/classes/nginx_spec.rb @@ -21,11 +21,11 @@ describe 'nginx' do it { is_expected.to compile.with_all_deps } it { is_expected.to contain_class('nginx') } it { is_expected.to contain_anchor('nginx::begin') } - it { is_expected.to contain_nginx__package.that_requires('Anchor[nginx::begin]') } - it { is_expected.to contain_nginx__config.that_requires('Class[nginx::package]') } - it { is_expected.to contain_nginx__service.that_subscribes_to('Anchor[nginx::begin]') } - it { is_expected.to contain_nginx__service.that_subscribes_to('Class[nginx::package]') } - it { is_expected.to contain_nginx__service.that_subscribes_to('Class[nginx::config]') } + it { is_expected.to contain_class('nginx::package').that_requires('Anchor[nginx::begin]') } + it { is_expected.to contain_class('nginx::config').that_requires('Class[nginx::package]') } + it { is_expected.to contain_class('nginx::service').that_subscribes_to('Anchor[nginx::begin]') } + it { is_expected.to contain_class('nginx::service').that_subscribes_to('Class[nginx::package]') } + it { is_expected.to contain_class('nginx::service').that_subscribes_to('Class[nginx::config]') } it { is_expected.to contain_anchor('nginx::end').that_requires('Class[nginx::service]') } it { is_expected.to contain_nginx__resource__upstream("upstream1") } it { is_expected.to contain_nginx__resource__vhost("test2.local") }