diff options
author | Matt Robinson <matt@puppetlabs.com> | 2011-05-16 16:10:21 -0700 |
---|---|---|
committer | Matt Robinson <matt@puppetlabs.com> | 2011-05-17 09:57:26 -0700 |
commit | 9da1454e71b8330e929ac5616eefa44388c90832 (patch) | |
tree | 8055105961771b9b95a4b888c0599b26000603d6 | |
parent | 47e4ac948504aa6fbd3487ca1b1e19f3c8884a79 (diff) | |
download | puppet-9da1454e71b8330e929ac5616eefa44388c90832.tar.gz puppet-9da1454e71b8330e929ac5616eefa44388c90832.tar.xz puppet-9da1454e71b8330e929ac5616eefa44388c90832.zip |
(#7507) Add ability to filter Ruby 1.9 spec failures
By running:
rspec spec --tag ~@fails_on_ruby_1.9.2
We can now just run the specs that pass under Ruby 1.9. Obviously in
the long term we want to have all the specs passing, but until then we
need notification when we regress. From now on new code will be
required to pass under Ruby 1.9, and Jenkins will give us email
notification if it doesn't or if we break something that was already
working.
Reviewed-by: Daniel Pittman <daniel@puppetlabs.com>
59 files changed, 102 insertions, 102 deletions
diff --git a/spec/integration/application/doc_spec.rb b/spec/integration/application/doc_spec.rb index c1e463033..9412976f0 100755 --- a/spec/integration/application/doc_spec.rb +++ b/spec/integration/application/doc_spec.rb @@ -5,7 +5,7 @@ require 'puppet_spec/files' describe Puppet::Application::Doc do include PuppetSpec::Files - it "should not generate an error when module dir overlaps parent of site.pp (#4798)" do + it "should not generate an error when module dir overlaps parent of site.pp (#4798)", :'fails_on_ruby_1.9.2' => true do begin # Note: the directory structure below is more complex than it # needs to be, but it's representative of the directory structure diff --git a/spec/integration/network/formats_spec.rb b/spec/integration/network/formats_spec.rb index af505f267..214b2d78f 100755 --- a/spec/integration/network/formats_spec.rb +++ b/spec/integration/network/formats_spec.rb @@ -44,7 +44,7 @@ describe Puppet::Network::FormatHandler.format(:s) do end end -describe Puppet::Network::FormatHandler.format(:pson) do +describe Puppet::Network::FormatHandler.format(:pson), :'fails_on_ruby_1.9.2' => true do describe "when pson is absent", :if => (! Puppet.features.pson?) do before do diff --git a/spec/integration/network/server/mongrel_spec.rb b/spec/integration/network/server/mongrel_spec.rb index fd5903b9e..1ce59eb49 100755 --- a/spec/integration/network/server/mongrel_spec.rb +++ b/spec/integration/network/server/mongrel_spec.rb @@ -3,7 +3,7 @@ require 'spec_helper' require 'puppet/network/server' require 'socket' -describe Puppet::Network::Server do +describe Puppet::Network::Server, :'fails_on_ruby_1.9.2' => true do describe "when using mongrel", :if => Puppet.features.mongrel? do before :each do diff --git a/spec/integration/node_spec.rb b/spec/integration/node_spec.rb index e15ddfc1e..4ea6142e2 100755 --- a/spec/integration/node_spec.rb +++ b/spec/integration/node_spec.rb @@ -43,7 +43,7 @@ describe Puppet::Node do Puppet::Node.indirection.terminus(:ldap).should_not be_nil end - it "should be able to use the plain terminus" do + it "should be able to use the plain terminus", :'fails_on_ruby_1.9.2' => true do Puppet::Node.indirection.stubs(:terminus_class).returns :plain # Load now, before we stub the exists? method. diff --git a/spec/integration/provider/package_spec.rb b/spec/integration/provider/package_spec.rb index 2d75826e8..5fecdf13c 100755 --- a/spec/integration/provider/package_spec.rb +++ b/spec/integration/provider/package_spec.rb @@ -1,7 +1,7 @@ #!/usr/bin/env rspec require 'spec_helper' -describe "Package Provider" do +describe "Package Provider", :'fails_on_ruby_1.9.2' => true do Puppet::Type.type(:package).providers.each do |name| provider = Puppet::Type.type(:package).provider(name) diff --git a/spec/integration/provider/service/init_spec.rb b/spec/integration/provider/service/init_spec.rb index ef7d98b4c..c209475bf 100755 --- a/spec/integration/provider/service/init_spec.rb +++ b/spec/integration/provider/service/init_spec.rb @@ -3,14 +3,14 @@ require 'spec_helper' provider = Puppet::Type.type(:service).provider(:init) -describe provider do +describe provider, :'fails_on_ruby_1.9.2' => true do describe "when running on FreeBSD", :if => (Facter.value(:operatingsystem) == "FreeBSD") do it "should set its default path to include /etc/init.d and /usr/local/etc/init.d" do provider.defpath.should == ["/etc/rc.d", "/usr/local/etc/rc.d"] end end - describe "when running on HP-UX", :if => (Facter.value(:operatingsystem) == "HP-UX")do + describe "when running on HP-UX", :if => (Facter.value(:operatingsystem) == "HP-UX") do it "should set its default path to include /sbin/init.d" do provider.defpath.should == "/sbin/init.d" end diff --git a/spec/integration/transaction_spec.rb b/spec/integration/transaction_spec.rb index 0ff50f47c..00e9dbb8e 100755 --- a/spec/integration/transaction_spec.rb +++ b/spec/integration/transaction_spec.rb @@ -220,7 +220,7 @@ describe Puppet::Transaction do FileTest.should be_exists(newfile) end - it "should still trigger skipped resources" do + it "should still trigger skipped resources", :'fails_on_ruby_1.9.2' => true do catalog = mk_catalog catalog.add_resource(*Puppet::Type.type(:schedule).mkdefaultschedules) diff --git a/spec/integration/util/rdoc/parser_spec.rb b/spec/integration/util/rdoc/parser_spec.rb index ce316309b..bf1f0d268 100755 --- a/spec/integration/util/rdoc/parser_spec.rb +++ b/spec/integration/util/rdoc/parser_spec.rb @@ -8,7 +8,7 @@ require 'puppet/util/rdoc/code_objects' require 'rdoc/options' require 'rdoc/rdoc' -describe RDoc::Parser do +describe RDoc::Parser, :'fails_on_ruby_1.9.2' => true do require 'puppet_spec/files' include PuppetSpec::Files diff --git a/spec/unit/application/cert_spec.rb b/spec/unit/application/cert_spec.rb index 1b1c61ab4..7510f0783 100755 --- a/spec/unit/application/cert_spec.rb +++ b/spec/unit/application/cert_spec.rb @@ -2,7 +2,7 @@ require 'spec_helper' require 'puppet/application/cert' -describe Puppet::Application::Cert do +describe Puppet::Application::Cert, :'fails_on_ruby_1.9.2' => true do before :each do @cert_app = Puppet::Application[:cert] Puppet::Util::Log.stubs(:newdestination) diff --git a/spec/unit/application/face_base_spec.rb b/spec/unit/application/face_base_spec.rb index 2a9a22356..3318e061e 100755 --- a/spec/unit/application/face_base_spec.rb +++ b/spec/unit/application/face_base_spec.rb @@ -148,7 +148,7 @@ describe Puppet::Application::FaceBase do end end - it "should handle application-level options" do + it "should handle application-level options", :'fails_on_ruby_1.9.2' => true do app.command_line.stubs(:args).returns %w{basetest --verbose return_true} app.preinit app.parse_options diff --git a/spec/unit/application/facts_spec.rb b/spec/unit/application/facts_spec.rb index 2981dc805..e11ea165c 100755 --- a/spec/unit/application/facts_spec.rb +++ b/spec/unit/application/facts_spec.rb @@ -15,7 +15,7 @@ describe Puppet::Application::Facts do @logs.first.to_s.should =~ /puppet facts find takes 1 argument, but you gave 0/ end - it "should return facts if a key is given to find" do + it "should return facts if a key is given to find", :'fails_on_ruby_1.9.2' => true do subject.command_line.stubs(:args).returns %w{find whatever --render-as yaml} expect { diff --git a/spec/unit/application/resource_spec.rb b/spec/unit/application/resource_spec.rb index 9ee6dd71b..a6564cae5 100755 --- a/spec/unit/application/resource_spec.rb +++ b/spec/unit/application/resource_spec.rb @@ -35,13 +35,13 @@ describe Puppet::Application::Resource do end describe "in preinit" do - it "should set hosts to nil" do + it "should set hosts to nil", :'fails_on_ruby_1.9.2' => true do @resource.preinit @resource.host.should be_nil end - it "should init extra_params to empty array" do + it "should init extra_params to empty array", :'fails_on_ruby_1.9.2' => true do @resource.preinit @resource.extra_params.should == [] diff --git a/spec/unit/application_spec.rb b/spec/unit/application_spec.rb index fc1bbceb6..aed80e3e6 100755 --- a/spec/unit/application_spec.rb +++ b/spec/unit/application_spec.rb @@ -26,7 +26,7 @@ describe Puppet::Application do @klass.find("Agent").should == @klass::Agent end - it "should not find classes outside the namespace" do + it "should not find classes outside the namespace", :'fails_on_ruby_1.9.2' => true do expect { @klass.find("String") }.to exit_with 1 end @@ -222,7 +222,7 @@ describe Puppet::Application do end describe 'on POSIX systems', :if => Puppet.features.posix? do - it 'should signal process with HUP after block if restart requested during block execution' do + it 'should signal process with HUP after block if restart requested during block execution', :'fails_on_ruby_1.9.2' => true do Puppet::Application.run_status = nil target = mock 'target' target.expects(:some_method).once diff --git a/spec/unit/indirector/facts/couch_spec.rb b/spec/unit/indirector/facts/couch_spec.rb index d0862486c..013fa3c33 100755 --- a/spec/unit/indirector/facts/couch_spec.rb +++ b/spec/unit/indirector/facts/couch_spec.rb @@ -11,7 +11,7 @@ describe "Puppet::Node::Facts::Couch" do end end - describe "when couchdb is available", :if => Puppet.features.couchdb? do + describe "when couchdb is available", :if => Puppet.features.couchdb?, :'fails_on_ruby_1.9.2' => true do before do @mock_db = mock('couch db') mock_document = CouchRest::Document.new(:_id => fake_request.key, :facts => fake_request.values) diff --git a/spec/unit/indirector/ldap_spec.rb b/spec/unit/indirector/ldap_spec.rb index 2b40325de..d4c9ddcff 100755 --- a/spec/unit/indirector/ldap_spec.rb +++ b/spec/unit/indirector/ldap_spec.rb @@ -108,7 +108,7 @@ describe Puppet::Indirector::Ldap do end end - describe "when connecting to ldap", :if => Puppet.features.ldap? do + describe "when connecting to ldap", :if => Puppet.features.ldap?, :'fails_on_ruby_1.9.2' => true do it "should create and start a Util::Ldap::Connection instance" do conn = mock 'connection', :connection => "myconn", :start => nil Puppet::Util::Ldap::Connection.expects(:instance).returns conn diff --git a/spec/unit/indirector/report/yaml_spec.rb b/spec/unit/indirector/report/yaml_spec.rb index 7df4bb2e9..557795f21 100755 --- a/spec/unit/indirector/report/yaml_spec.rb +++ b/spec/unit/indirector/report/yaml_spec.rb @@ -22,7 +22,7 @@ describe Puppet::Transaction::Report::Yaml do Puppet::Transaction::Report::Yaml.name.should == :yaml end - it "should inconditionnally save/load from the --lastrunreport setting" do + it "should inconditionnally save/load from the --lastrunreport setting", :'fails_on_ruby_1.9.2' => true do indirection = stub 'indirection', :name => :my_yaml, :register_terminus_type => nil Puppet::Indirector::Indirection.stubs(:instance).with(:my_yaml).returns(indirection) store_class = Class.new(Puppet::Transaction::Report::Yaml) do diff --git a/spec/unit/indirector/resource/ral_spec.rb b/spec/unit/indirector/resource/ral_spec.rb index 61290f7a7..cf746cb0c 100755 --- a/spec/unit/indirector/resource/ral_spec.rb +++ b/spec/unit/indirector/resource/ral_spec.rb @@ -18,7 +18,7 @@ describe "Puppet::Resource::Ral" do Puppet::Resource::Ral.new.find(@request).should == my_resource end - it "if there is no instance, it should create one" do + it "if there is no instance, it should create one", :'fails_on_ruby_1.9.2' => true do wrong_instance = stub "wrong user", :name => "bob" require 'puppet/type/user' diff --git a/spec/unit/indirector/terminus_spec.rb b/spec/unit/indirector/terminus_spec.rb index 41770e1e3..33932cfca 100755 --- a/spec/unit/indirector/terminus_spec.rb +++ b/spec/unit/indirector/terminus_spec.rb @@ -4,7 +4,7 @@ require 'puppet/defaults' require 'puppet/indirector' require 'puppet/indirector/file' -describe Puppet::Indirector::Terminus do +describe Puppet::Indirector::Terminus, :'fails_on_ruby_1.9.2' => true do before :each do Puppet::Indirector::Terminus.stubs(:register_terminus_class) @indirection = stub 'indirection', :name => :my_stuff, :register_terminus_type => nil @@ -135,7 +135,7 @@ describe Puppet::Indirector::Terminus, " when managing terminus classes" do Puppet::Indirector::Terminus.terminus_class(:test1, :yay) end - it "should fail when no indirection can be found" do + it "should fail when no indirection can be found", :'fails_on_ruby_1.9.2' => true do Puppet::Indirector::Indirection.expects(:instance).with(:my_indirection).returns(nil) @abstract_terminus = Class.new(Puppet::Indirector::Terminus) do @@ -152,7 +152,7 @@ describe Puppet::Indirector::Terminus, " when managing terminus classes" do }.should raise_error(ArgumentError) end - it "should register the terminus class with the terminus base class" do + it "should register the terminus class with the terminus base class", :'fails_on_ruby_1.9.2' => true do Puppet::Indirector::Terminus.expects(:register_terminus_class).with do |type| type.indirection_name == :my_indirection and type.name == :test_terminus end @@ -219,7 +219,7 @@ describe Puppet::Indirector::Terminus, " when parsing class constants for indire end end -describe Puppet::Indirector::Terminus, " when creating terminus class types" do +describe Puppet::Indirector::Terminus, " when creating terminus class types", :'fails_on_ruby_1.9.2' => true do before do Puppet::Indirector::Terminus.stubs(:register_terminus_class) @subclass = Class.new(Puppet::Indirector::Terminus) do diff --git a/spec/unit/interface_spec.rb b/spec/unit/interface_spec.rb index e28e55aac..8bbbcc857 100755 --- a/spec/unit/interface_spec.rb +++ b/spec/unit/interface_spec.rb @@ -101,7 +101,7 @@ describe Puppet::Interface do should raise_error ArgumentError end - it "should instance-eval any provided block" do + it "should instance-eval any provided block", :'fails_on_ruby_1.9.2' => true do face = subject.new(:face_test_block, '0.0.1') do action(:something) do when_invoked { "foo" } @@ -140,7 +140,7 @@ describe Puppet::Interface do end end - describe "with face-level options" do + describe "with face-level options", :'fails_on_ruby_1.9.2' => true do it "should not return any action-level options" do face = subject.new(:with_options, '0.0.1') do option "--foo" @@ -191,7 +191,7 @@ describe Puppet::Interface do face end - describe "#options" do + describe "#options", :'fails_on_ruby_1.9.2' => true do it "should list inherited options" do face.options.should =~ [:inherited, :local] end @@ -213,7 +213,7 @@ describe Puppet::Interface do end end - describe "#get_option" do + describe "#get_option", :'fails_on_ruby_1.9.2' => true do it "should return an inherited option object" do face.get_option(:inherited).should be_an_instance_of subject::Option end diff --git a/spec/unit/network/handler/fileserver_spec.rb b/spec/unit/network/handler/fileserver_spec.rb index 52c4a71f5..08852634d 100755 --- a/spec/unit/network/handler/fileserver_spec.rb +++ b/spec/unit/network/handler/fileserver_spec.rb @@ -123,7 +123,7 @@ describe Puppet::Network::Handler::FileServer do list.sort.should == [ ["/aFile", "file"], ["/", "directory"] ].sort end - describe Puppet::Network::Handler::FileServer::PluginMount do + describe Puppet::Network::Handler::FileServer::PluginMount, :'fails_on_ruby_1.9.2' => true do PLUGINS = Puppet::Network::Handler::FileServer::PLUGINS # create a module plugin hierarchy diff --git a/spec/unit/network/http/mongrel/rest_spec.rb b/spec/unit/network/http/mongrel/rest_spec.rb index 3e454cf8f..85b9f7496 100755 --- a/spec/unit/network/http/mongrel/rest_spec.rb +++ b/spec/unit/network/http/mongrel/rest_spec.rb @@ -3,7 +3,7 @@ require 'spec_helper' require 'puppet/network/http' -describe "Puppet::Network::HTTP::MongrelREST", :if => Puppet.features.mongrel? do +describe "Puppet::Network::HTTP::MongrelREST", :if => Puppet.features.mongrel?, :'fails_on_ruby_1.9.2' => true do before do require 'puppet/network/http/mongrel/rest' end diff --git a/spec/unit/network/http/mongrel_spec.rb b/spec/unit/network/http/mongrel_spec.rb index 56d0afbed..6776fad61 100755 --- a/spec/unit/network/http/mongrel_spec.rb +++ b/spec/unit/network/http/mongrel_spec.rb @@ -6,15 +6,15 @@ require 'spec_helper' require 'puppet/network/http' -describe "Puppet::Network::HTTP::Mongrel", "after initializing", :if => Puppet.features.mongrel? do - it "should not be listening" do +describe "Puppet::Network::HTTP::Mongrel", "after initializing", :if => Puppet.features.mongrel?, :'fails_on_ruby_1.9.2' => true do + it "should not be listening", :'fails_on_ruby_1.9.2' => true do require 'puppet/network/http/mongrel' Puppet::Network::HTTP::Mongrel.new.should_not be_listening end end -describe "Puppet::Network::HTTP::Mongrel", "when turning on listening", :if => Puppet.features.mongrel? do +describe "Puppet::Network::HTTP::Mongrel", "when turning on listening", :if => Puppet.features.mongrel?, :'fails_on_ruby_1.9.2' => true do before do require 'puppet/network/http/mongrel' @@ -96,7 +96,7 @@ describe "Puppet::Network::HTTP::Mongrel", "when turning on listening", :if => P end end -describe "Puppet::Network::HTTP::Mongrel", "when turning off listening", :if => Puppet.features.mongrel? do +describe "Puppet::Network::HTTP::Mongrel", "when turning off listening", :if => Puppet.features.mongrel?, :'fails_on_ruby_1.9.2' => true do before do @mock_mongrel = mock('mongrel httpserver') @mock_mongrel.stubs(:run) diff --git a/spec/unit/network/http/webrick/rest_spec.rb b/spec/unit/network/http/webrick/rest_spec.rb index 267ddcc72..84a2b7791 100755 --- a/spec/unit/network/http/webrick/rest_spec.rb +++ b/spec/unit/network/http/webrick/rest_spec.rb @@ -9,7 +9,7 @@ describe Puppet::Network::HTTP::WEBrickREST do Puppet::Network::HTTP::WEBrickREST.ancestors.should be_include(Puppet::Network::HTTP::Handler) end - describe "when initializing" do + describe "when initializing", :'fails_on_ruby_1.9.2' => true do it "should call the Handler's initialization hook with its provided arguments as the server and handler" do Puppet::Network::HTTP::WEBrickREST.any_instance.expects(:initialize_for_puppet).with(:server => "my", :handler => "arguments") Puppet::Network::HTTP::WEBrickREST.new("my", "arguments") diff --git a/spec/unit/parser/ast/function_spec.rb b/spec/unit/parser/ast/function_spec.rb index c52e806e9..d683b122b 100755 --- a/spec/unit/parser/ast/function_spec.rb +++ b/spec/unit/parser/ast/function_spec.rb @@ -48,7 +48,7 @@ describe Puppet::Parser::AST::Function do lambda{ func.evaluate(@scope) }.should raise_error(Puppet::ParseError,"Function 'exist' must be the value of a statement") end - it "should evaluate its arguments" do + it "should evaluate its arguments", :'fails_on_ruby_1.9.2' => true do argument = stub 'arg' Puppet::Parser::Functions.stubs(:function).with("exist").returns(true) func = Puppet::Parser::AST::Function.new :name => "exist", :ftype => :statement, :arguments => argument diff --git a/spec/unit/parser/ast/selector_spec.rb b/spec/unit/parser/ast/selector_spec.rb index 1bf5f6757..76afec271 100755 --- a/spec/unit/parser/ast/selector_spec.rb +++ b/spec/unit/parser/ast/selector_spec.rb @@ -6,7 +6,7 @@ describe Puppet::Parser::AST::Selector do @scope = Puppet::Parser::Scope.new end - describe "when evaluating" do + describe "when evaluating", :'fails_on_ruby_1.9.2' => true do before :each do @param = stub 'param' diff --git a/spec/unit/parser/compiler_spec.rb b/spec/unit/parser/compiler_spec.rb index f4cf8b3a1..9ad754ad8 100755 --- a/spec/unit/parser/compiler_spec.rb +++ b/spec/unit/parser/compiler_spec.rb @@ -609,7 +609,7 @@ describe Puppet::Parser::Compiler do @compiler.evaluate_classes(%w{myclass}, @scope) end - it "should ensure each node class hash is in catalog and have appropriate parameters" do + it "should ensure each node class hash is in catalog and have appropriate parameters", :'fails_on_ruby_1.9.2' => true do klasses = {'foo'=>{'1'=>'one'}, 'bar::foo'=>{'2'=>'two'}, 'bar'=>{'1'=> [1,2,3], '2'=>{'foo'=>'bar'}}} @node.classes = klasses ast_obj = Puppet::Parser::AST::String.new(:value => 'foo') @@ -633,7 +633,7 @@ describe Puppet::Parser::Compiler do r2.tags.should =~ ['class', 'bar'] end - it "should ensure each node class is in catalog and has appropriate tags" do + it "should ensure each node class is in catalog and has appropriate tags", :'fails_on_ruby_1.9.2' => true do klasses = ['bar::foo'] @node.classes = klasses ast_obj = Puppet::Parser::AST::String.new(:value => 'foo') diff --git a/spec/unit/parser/functions/create_resources_spec.rb b/spec/unit/parser/functions/create_resources_spec.rb index 88a67e369..da76e75d0 100755 --- a/spec/unit/parser/functions/create_resources_spec.rb +++ b/spec/unit/parser/functions/create_resources_spec.rb @@ -114,7 +114,7 @@ notify{tester:} @scope.resource=Puppet::Parser::Resource.new('class', 't', :scope => @scope) Puppet::Parser::Functions.function(:create_resources) end - it 'should be able to create classes' do + it 'should be able to create classes', :'fails_on_ruby_1.9.2' => true do @scope.function_create_resources(['class', {'bar'=>{'one'=>'two'}}]) @scope.compiler.compile @compiler.catalog.resource(:notify, "test")['message'].should == 'two' @@ -123,7 +123,7 @@ notify{tester:} it 'should fail to create non-existing classes' do lambda { @scope.function_create_resources(['class', {'blah'=>{'one'=>'two'}}]) }.should raise_error(ArgumentError ,'could not find hostclass blah') end - it 'should be able to add edges' do + it 'should be able to add edges', :'fails_on_ruby_1.9.2' => true do @scope.function_create_resources(['class', {'bar'=>{'one'=>'two', 'require' => 'Notify[tester]'}}]) @scope.compiler.compile rg = @scope.compiler.catalog.to_ral.relationship_graph diff --git a/spec/unit/parser/functions/inline_template_spec.rb b/spec/unit/parser/functions/inline_template_spec.rb index a9ac0c2d0..47dcae15e 100755 --- a/spec/unit/parser/functions/inline_template_spec.rb +++ b/spec/unit/parser/functions/inline_template_spec.rb @@ -1,7 +1,7 @@ #!/usr/bin/env rspec require 'spec_helper' -describe "the inline_template function" do +describe "the inline_template function", :'fails_on_ruby_1.9.2' => true do before :all do Puppet::Parser::Functions.autoloader.loadall end diff --git a/spec/unit/parser/functions/shellquote_spec.rb b/spec/unit/parser/functions/shellquote_spec.rb index b100b4913..bfa2969ea 100755 --- a/spec/unit/parser/functions/shellquote_spec.rb +++ b/spec/unit/parser/functions/shellquote_spec.rb @@ -53,7 +53,7 @@ describe "the shellquote function" do result.should(eql( "'$PATH' 'foo$bar' '\"x$\"'")) end - it "should deal with apostrophes (single quotes)" do + it "should deal with apostrophes (single quotes)", :'fails_on_ruby_1.9.2' => true do result = @scope.function_shellquote( ["'foo'bar'", "`$'EDITOR'`"]) result.should(eql( @@ -65,12 +65,12 @@ describe "the shellquote function" do result.should(eql( "'`echo *`' '`ls \"$MAILPATH\"`'")) end - it "should deal with both single and double quotes" do + it "should deal with both single and double quotes", :'fails_on_ruby_1.9.2' => true do result = @scope.function_shellquote( ['\'foo"bar"xyzzy\'', '"foo\'bar\'xyzzy"']) result.should(eql( '"\'foo\\"bar\\"xyzzy\'" "\\"foo\'bar\'xyzzy\\""')) end - it "should handle multiple quotes *and* dollars and backquotes" do + it "should handle multiple quotes *and* dollars and backquotes", :'fails_on_ruby_1.9.2' => true do result = @scope.function_shellquote( ['\'foo"$x`bar`"xyzzy\'']) result.should(eql( '"\'foo\\"\\$x\\`bar\\`\\"xyzzy\'"')) end diff --git a/spec/unit/parser/functions/template_spec.rb b/spec/unit/parser/functions/template_spec.rb index e7ee974d3..6bce69534 100755 --- a/spec/unit/parser/functions/template_spec.rb +++ b/spec/unit/parser/functions/template_spec.rb @@ -1,7 +1,7 @@ #!/usr/bin/env rspec require 'spec_helper' -describe "the template function" do +describe "the template function", :'fails_on_ruby_1.9.2' => true do before :all do Puppet::Parser::Functions.autoloader.loadall end diff --git a/spec/unit/parser/resource_spec.rb b/spec/unit/parser/resource_spec.rb index b03c18e5f..064f27514 100755 --- a/spec/unit/parser/resource_spec.rb +++ b/spec/unit/parser/resource_spec.rb @@ -113,7 +113,7 @@ describe Puppet::Parser::Resource do @arguments = {:scope => @scope} end - it "should fail unless #{name.to_s} is specified" do + it "should fail unless #{name.to_s} is specified", :'fails_on_ruby_1.9.2' => true do lambda { Puppet::Parser::Resource.new('file', '/my/file') }.should raise_error(ArgumentError) end diff --git a/spec/unit/provider/mount/parsed_spec.rb b/spec/unit/provider/mount/parsed_spec.rb index 0293e0758..c38a3cfcb 100755 --- a/spec/unit/provider/mount/parsed_spec.rb +++ b/spec/unit/provider/mount/parsed_spec.rb @@ -39,7 +39,7 @@ FSTAB # it_should_behave_like "all parsedfile providers", # provider_class, my_fixtures('*.fstab') - describe "on Solaris", :if => Facter.value(:operatingsystem) == 'Solaris' do + describe "on Solaris", :if => Facter.value(:operatingsystem) == 'Solaris', :'fails_on_ruby_1.9.2' => true do before :each do @example_line = "/dev/dsk/c0d0s0 /dev/rdsk/c0d0s0 \t\t / \t ufs 1 no\t-" diff --git a/spec/unit/provider/package/macports_spec.rb b/spec/unit/provider/package/macports_spec.rb index 7d1acd537..8e08242d8 100755 --- a/spec/unit/provider/package/macports_spec.rb +++ b/spec/unit/provider/package/macports_spec.rb @@ -30,7 +30,7 @@ describe provider_class do it { should be_versionable } end - describe "when listing all instances" do + describe "when listing all instances", :'fails_on_ruby_1.9.2' => true do it "should call port -q installed" do provider_class.expects(:port).with("-q", :installed).returns("") provider_class.instances @@ -111,7 +111,7 @@ describe provider_class do provider.update end - it "should execute port install if the port is not installed" do + it "should execute port install if the port is not installed", :'fails_on_ruby_1.9.2' => true do resource[:name] = resource_name resource[:ensure] = :present provider.expects(:query).returns("") diff --git a/spec/unit/provider/service/init_spec.rb b/spec/unit/provider/service/init_spec.rb index d64e0fc5d..8374594e7 100755 --- a/spec/unit/provider/service/init_spec.rb +++ b/spec/unit/provider/service/init_spec.rb @@ -46,7 +46,7 @@ describe provider_class do results = (@services-exclude).collect {|x| "#{x}_instance"} @class.get_services(@class.defpath, exclude).should == results end - it "should omit a single service from the exclude list" do + it "should omit a single service from the exclude list", :'fails_on_ruby_1.9.2' => true do exclude = 'two' (@services-exclude.to_a).each do |inst| @class.expects(:new).with{|hash| hash[:name] == inst}.returns("#{inst}_instance") diff --git a/spec/unit/provider/service/src_spec.rb b/spec/unit/provider/service/src_spec.rb index 17f49994e..b45ca0c7c 100755 --- a/spec/unit/provider/service/src_spec.rb +++ b/spec/unit/provider/service/src_spec.rb @@ -44,7 +44,7 @@ describe provider_class do @provider.stop end - it "should execute status and return running if the subsystem is active" do + it "should execute status and return running if the subsystem is active", :'fails_on_ruby_1.9.2' => true do sample_output = <<_EOF_ Subsystem Group PID Status myservice tcpip 1234 active @@ -54,7 +54,7 @@ _EOF_ @provider.status.should == :running end - it "should execute status and return stopped if the subsystem is inoperative" do + it "should execute status and return stopped if the subsystem is inoperative", :'fails_on_ruby_1.9.2' => true do sample_output = <<_EOF_ Subsystem Group PID Status myservice tcpip inoperative @@ -64,7 +64,7 @@ _EOF_ @provider.status.should == :stopped end - it "should execute status and return nil if the status is not known" do + it "should execute status and return nil if the status is not known", :'fails_on_ruby_1.9.2' => true do sample_output = <<_EOF_ Subsystem Group PID Status myservice tcpip randomdata @@ -74,7 +74,7 @@ _EOF_ @provider.status.should == nil end - it "should execute restart which runs refresh" do + it "should execute restart which runs refresh", :'fails_on_ruby_1.9.2' => true do sample_output = <<_EOF_ #subsysname:synonym:cmdargs:path:uid:auditid:standin:standout:standerr:action:multi:contact:svrkey:svrmtype:priority:signorm:sigforce:display:waittime:grpname: myservice:::/usr/sbin/inetd:0:0:/dev/console:/dev/console:/dev/console:-O:-Q:-K:0:0:20:0:0:-d:20:tcpip: @@ -84,7 +84,7 @@ _EOF_ @provider.restart end - it "should execute restart which runs stopsrc then startsrc" do + it "should execute restart which runs stopsrc then startsrc", :'fails_on_ruby_1.9.2' => true do sample_output = <<_EOF_ #subsysname:synonym:cmdargs:path:uid:auditid:standin:standout:standerr:action:multi:contact:svrkey:svrmtype:priority:signorm:sigforce:display:waittime:grpname: myservice::--no-daemonize:/usr/sbin/puppetd:0:0:/dev/null:/var/log/puppet.log:/var/log/puppet.log:-O:-Q:-S:0:0:20:15:9:-d:20::" diff --git a/spec/unit/provider/user/user_role_add_spec.rb b/spec/unit/provider/user/user_role_add_spec.rb index b17ba68c8..5f2fc306e 100755 --- a/spec/unit/provider/user/user_role_add_spec.rb +++ b/spec/unit/provider/user/user_role_add_spec.rb @@ -40,7 +40,7 @@ describe provider_class do end end - describe "when calling transition" do + describe "when calling transition", :'fails_on_ruby_1.9.2' => true do it "should return the type set to whatever is passed in" do @provider.expects(:command).with(:modify).returns("foomod") @provider.transition("bar").include?("type=bar") @@ -120,7 +120,7 @@ describe provider_class do @provider.expects(:execute).with { |args| args.include?("-o") } end - it "should add -o when the user is being created" do + it "should add -o when the user is being created", :'fails_on_ruby_1.9.2' => true do @provider.stubs(:password=) @provider.create end diff --git a/spec/unit/provider/zpool/solaris_spec.rb b/spec/unit/provider/zpool/solaris_spec.rb index 7e3048a7a..39ee9c9ba 100755 --- a/spec/unit/provider/zpool/solaris_spec.rb +++ b/spec/unit/provider/zpool/solaris_spec.rb @@ -150,7 +150,7 @@ describe provider_class do end end - describe "when calling create" do + describe "when calling create", :'fails_on_ruby_1.9.2' => true do before do @resource.stubs(:[]).with(:pool).returns("mypool") @provider.stubs(:zpool) diff --git a/spec/unit/resource/status_spec.rb b/spec/unit/resource/status_spec.rb index bb88518c0..e5a9291db 100755 --- a/spec/unit/resource/status_spec.rb +++ b/spec/unit/resource/status_spec.rb @@ -140,7 +140,7 @@ describe Puppet::Resource::Status do end end - describe "When converting to YAML" do + describe "When converting to YAML", :'fails_on_ruby_1.9.2' => true do it "should include only documented attributes" do @status.file = "/foo.rb" @status.line = 27 diff --git a/spec/unit/resource_spec.rb b/spec/unit/resource_spec.rb index 79ea69e9c..5c8e8dcf9 100755 --- a/spec/unit/resource_spec.rb +++ b/spec/unit/resource_spec.rb @@ -516,7 +516,7 @@ type: File ) end - it "should align, sort and add trailing commas to attributes with ensure first" do + it "should align, sort and add trailing commas to attributes with ensure first", :'fails_on_ruby_1.9.2' => true do @resource.to_manifest.should == <<-HEREDOC.gsub(/^\s{8}/, '').gsub(/\n$/, '') one::two { '/my/file': ensure => 'present', diff --git a/spec/unit/simple_graph_spec.rb b/spec/unit/simple_graph_spec.rb index 472ebbd36..17e382fcc 100755 --- a/spec/unit/simple_graph_spec.rb +++ b/spec/unit/simple_graph_spec.rb @@ -441,7 +441,7 @@ describe Puppet::SimpleGraph do end end - describe "when matching edges" do + describe "when matching edges", :'fails_on_ruby_1.9.2' => true do before do @graph = Puppet::SimpleGraph.new @event = Puppet::Transaction::Event.new(:name => :yay, :resource => "a") diff --git a/spec/unit/ssl/certificate_authority/interface_spec.rb b/spec/unit/ssl/certificate_authority/interface_spec.rb index 9e858dd54..46273ccee 100755 --- a/spec/unit/ssl/certificate_authority/interface_spec.rb +++ b/spec/unit/ssl/certificate_authority/interface_spec.rb @@ -68,7 +68,7 @@ describe Puppet::SSL::CertificateAuthority::Interface do @class.new(:generate, :to => :all).subjects.should == :all end - it "should fail if the subjects setting isn't :all or an array" do + it "should fail if the subjects setting isn't :all or an array", :'fails_on_ruby_1.9.2' => true do lambda { @class.new(:generate, "other") }.should raise_error(ArgumentError) end end diff --git a/spec/unit/ssl/host_spec.rb b/spec/unit/ssl/host_spec.rb index d14da538e..c2d9690e6 100755 --- a/spec/unit/ssl/host_spec.rb +++ b/spec/unit/ssl/host_spec.rb @@ -517,7 +517,7 @@ describe Puppet::SSL::Host do Puppet::SSL::Host.search :for => Puppet::SSL::CertificateRequest end - it "should return a Host instance created with the name of each found instance" do + it "should return a Host instance created with the name of each found instance", :'fails_on_ruby_1.9.2' => true do key = stub 'key', :name => "key" cert = stub 'cert', :name => "cert" csr = stub 'csr', :name => "csr" diff --git a/spec/unit/transaction/event_spec.rb b/spec/unit/transaction/event_spec.rb index 1227802a7..0093baeb9 100755 --- a/spec/unit/transaction/event_spec.rb +++ b/spec/unit/transaction/event_spec.rb @@ -5,7 +5,7 @@ require 'puppet/transaction/event' describe Puppet::Transaction::Event do [:previous_value, :desired_value, :property, :resource, :name, :message, :file, :line, :tags, :audited].each do |attr| - it "should support #{attr}" do + it "should support #{attr}", :'fails_on_ruby_1.9.2' => true do event = Puppet::Transaction::Event.new event.send(attr.to_s + "=", "foo") event.send(attr).should == "foo" @@ -16,7 +16,7 @@ describe Puppet::Transaction::Event do Puppet::Transaction::Event.new(:property => :foo).property.should == "foo" end - it "should always convert the resource to a string" do + it "should always convert the resource to a string", :'fails_on_ruby_1.9.2' => true do Puppet::Transaction::Event.new(:resource => :foo).resource.should == "foo" end @@ -95,17 +95,17 @@ describe Puppet::Transaction::Event do end end - it "should use the source description as the source if one is set" do + it "should use the source description as the source if one is set", :'fails_on_ruby_1.9.2' => true do Puppet::Util::Log.expects(:new).with { |args| args[:source] == "/my/param" } Puppet::Transaction::Event.new(:source_description => "/my/param", :resource => "Foo[bar]", :property => "foo").send_log end - it "should use the property as the source if one is available and no source description is set" do + it "should use the property as the source if one is available and no source description is set", :'fails_on_ruby_1.9.2' => true do Puppet::Util::Log.expects(:new).with { |args| args[:source] == "foo" } Puppet::Transaction::Event.new(:resource => "Foo[bar]", :property => "foo").send_log end - it "should use the property as the source if one is available and no property or source description is set" do + it "should use the property as the source if one is available and no property or source description is set", :'fails_on_ruby_1.9.2' => true do Puppet::Util::Log.expects(:new).with { |args| args[:source] == "Foo[bar]" } Puppet::Transaction::Event.new(:resource => "Foo[bar]").send_log end diff --git a/spec/unit/type/augeas_spec.rb b/spec/unit/type/augeas_spec.rb index c8dc207f9..e80da9559 100755 --- a/spec/unit/type/augeas_spec.rb +++ b/spec/unit/type/augeas_spec.rb @@ -4,7 +4,7 @@ require 'spec_helper' augeas = Puppet::Type.type(:augeas) describe augeas do - describe "when augeas is present", :if => Puppet.features.augeas? do + describe "when augeas is present", :if => Puppet.features.augeas?, :'fails_on_ruby_1.9.2' => true do it "should have a default provider inheriting from Puppet::Provider" do augeas.defaultprovider.ancestors.should be_include(Puppet::Provider) end diff --git a/spec/unit/type/file_spec.rb b/spec/unit/type/file_spec.rb index 683c3654b..4c58cc47a 100755 --- a/spec/unit/type/file_spec.rb +++ b/spec/unit/type/file_spec.rb @@ -262,12 +262,12 @@ describe Puppet::Type.type(:file) do file[:path].should == "X:/foo/bar/baz" end - it "should leave a drive letter with a slash alone" do + it "should leave a drive letter with a slash alone", :'fails_on_ruby_1.9.2' => true do file = Puppet::Type::File.new(:path => "X:/") file[:path].should == "X:/" end - it "should add a slash to a drive letter" do + it "should add a slash to a drive letter", :'fails_on_ruby_1.9.2' => true do file = Puppet::Type::File.new(:path => "X:") file[:path].should == "X:/" end @@ -286,7 +286,7 @@ describe Puppet::Type.type(:file) do end describe "when using UNC filenames" do - describe "on Microsoft Windows systems", :if => Puppet.features.microsoft_windows? do + describe "on Microsoft Windows systems", :if => Puppet.features.microsoft_windows?, :'fails_on_ruby_1.9.2' => true do before do Puppet.features.stubs(:posix?).returns(false) Puppet.features.stubs(:microsoft_windows?).returns(true) diff --git a/spec/unit/type/group_spec.rb b/spec/unit/type/group_spec.rb index 42fd3fb7b..afe28247a 100755 --- a/spec/unit/type/group_spec.rb +++ b/spec/unit/type/group_spec.rb @@ -41,11 +41,11 @@ describe Puppet::Type.type(:group) do end end - it "should have a boolean method for determining if duplicates are allowed" do + it "should have a boolean method for determining if duplicates are allowed", :'fails_on_ruby_1.9.2' => true do @class.new(:name => "foo").methods.should be_include("allowdupe?") end - it "should have a boolean method for determining if system groups are allowed" do + it "should have a boolean method for determining if system groups are allowed", :'fails_on_ruby_1.9.2' => true do @class.new(:name => "foo").methods.should be_include("system?") end diff --git a/spec/unit/type/resources_spec.rb b/spec/unit/type/resources_spec.rb index aedc58c7c..48c068cfa 100755 --- a/spec/unit/type/resources_spec.rb +++ b/spec/unit/type/resources_spec.rb @@ -51,7 +51,7 @@ describe resources do @resources.generate.collect { |r| r.ref }.should_not include(@host1.ref) end - it "should not include the skipped users" do + it "should not include the skipped users", :'fails_on_ruby_1.9.2' => true do res = Puppet::Type.type(:resources).new :name => :user, :purge => true res.catalog = Puppet::Resource::Catalog.new diff --git a/spec/unit/type/schedule_spec.rb b/spec/unit/type/schedule_spec.rb index 33064ca6f..08ec70cd7 100755 --- a/spec/unit/type/schedule_spec.rb +++ b/spec/unit/type/schedule_spec.rb @@ -96,7 +96,7 @@ describe Puppet::Type.type(:schedule) do end end - describe Puppet::Type.type(:schedule), "when matching hourly by distance" do + describe Puppet::Type.type(:schedule), "when matching hourly by distance", :'fails_on_ruby_1.9.2' => true do include ScheduleTesting before do @@ -117,7 +117,7 @@ describe Puppet::Type.type(:schedule) do end end - describe Puppet::Type.type(:schedule), "when matching daily by distance" do + describe Puppet::Type.type(:schedule), "when matching daily by distance", :'fails_on_ruby_1.9.2' => true do include ScheduleTesting before do @@ -138,7 +138,7 @@ describe Puppet::Type.type(:schedule) do end end - describe Puppet::Type.type(:schedule), "when matching weekly by distance" do + describe Puppet::Type.type(:schedule), "when matching weekly by distance", :'fails_on_ruby_1.9.2' => true do include ScheduleTesting before do @@ -159,7 +159,7 @@ describe Puppet::Type.type(:schedule) do end end - describe Puppet::Type.type(:schedule), "when matching monthly by distance" do + describe Puppet::Type.type(:schedule), "when matching monthly by distance", :'fails_on_ruby_1.9.2' => true do include ScheduleTesting before do @@ -180,7 +180,7 @@ describe Puppet::Type.type(:schedule) do end end - describe Puppet::Type.type(:schedule), "when matching hourly by number" do + describe Puppet::Type.type(:schedule), "when matching hourly by number", :'fails_on_ruby_1.9.2' => true do include ScheduleTesting before do @@ -205,7 +205,7 @@ describe Puppet::Type.type(:schedule) do end end - describe Puppet::Type.type(:schedule), "when matching daily by number" do + describe Puppet::Type.type(:schedule), "when matching daily by number", :'fails_on_ruby_1.9.2' => true do include ScheduleTesting before do @@ -236,7 +236,7 @@ describe Puppet::Type.type(:schedule) do end end - describe Puppet::Type.type(:schedule), "when matching weekly by number" do + describe Puppet::Type.type(:schedule), "when matching weekly by number", :'fails_on_ruby_1.9.2' => true do include ScheduleTesting before do @@ -261,7 +261,7 @@ describe Puppet::Type.type(:schedule) do end end - describe Puppet::Type.type(:schedule), "when matching monthly by number" do + describe Puppet::Type.type(:schedule), "when matching monthly by number", :'fails_on_ruby_1.9.2' => true do include ScheduleTesting before do @@ -286,7 +286,7 @@ describe Puppet::Type.type(:schedule) do end end - describe Puppet::Type.type(:schedule), "when matching with a repeat greater than one" do + describe Puppet::Type.type(:schedule), "when matching with a repeat greater than one", :'fails_on_ruby_1.9.2' => true do include ScheduleTesting before do diff --git a/spec/unit/util/autoload_spec.rb b/spec/unit/util/autoload_spec.rb index 6d49b57dc..512f06c75 100755 --- a/spec/unit/util/autoload_spec.rb +++ b/spec/unit/util/autoload_spec.rb @@ -103,14 +103,14 @@ describe Puppet::Util::Autoload do end [RuntimeError, LoadError, SyntaxError].each do |error| - it "should die an if a #{error.to_s} exception is thrown" do + it "should die an if a #{error.to_s} exception is thrown", :'fails_on_ruby_1.9.2' => true do Kernel.expects(:require).raises error lambda { @autoload.loadall }.should raise_error(Puppet::Error) end end - it "should require the full path to the file" do + it "should require the full path to the file", :'fails_on_ruby_1.9.2' => true do Kernel.expects(:require).with("/path/to/file.rb") @autoload.loadall diff --git a/spec/unit/util/cacher_spec.rb b/spec/unit/util/cacher_spec.rb index 2e43b4e20..fe93afd2b 100755 --- a/spec/unit/util/cacher_spec.rb +++ b/spec/unit/util/cacher_spec.rb @@ -42,7 +42,7 @@ describe Puppet::Util::Cacher do Puppet::Util::Cacher.singleton_class.ancestors.should be_include(Puppet::Util::Cacher::Expirer) end - it "should support defining cached attributes" do + it "should support defining cached attributes", :'fails_on_ruby_1.9.2' => true do CacheTest.methods.should be_include("cached_attr") end diff --git a/spec/unit/util/file_locking_spec.rb b/spec/unit/util/file_locking_spec.rb index 261474263..1a12244b8 100755 --- a/spec/unit/util/file_locking_spec.rb +++ b/spec/unit/util/file_locking_spec.rb @@ -16,11 +16,11 @@ describe Puppet::Util::FileLocking do Puppet::Util::FileLocking.should respond_to(:writelock) end - it "should have an instance method for getting a read lock on files" do + it "should have an instance method for getting a read lock on files", :'fails_on_ruby_1.9.2' => true do FileLocker.new.private_methods.should be_include("readlock") end - it "should have an instance method for getting a write lock on files" do + it "should have an instance method for getting a write lock on files", :'fails_on_ruby_1.9.2' => true do FileLocker.new.private_methods.should be_include("writelock") end diff --git a/spec/unit/util/log_spec.rb b/spec/unit/util/log_spec.rb index 78411d187..1baa0d5af 100755 --- a/spec/unit/util/log_spec.rb +++ b/spec/unit/util/log_spec.rb @@ -93,7 +93,7 @@ describe Puppet::Util::Log do Puppet::Util::Log.new(:level => "notice", :message => :foo).level.should == :notice end - it "should fail if the level is not a symbol or string" do + it "should fail if the level is not a symbol or string", :'fails_on_ruby_1.9.2' => true do lambda { Puppet::Util::Log.new(:level => 50, :message => :foo) }.should raise_error(ArgumentError) end @@ -206,7 +206,7 @@ describe Puppet::Util::Log do end end - describe "to_yaml" do + describe "to_yaml", :'fails_on_ruby_1.9.2' => true do it "should not include the @version attribute" do log = Puppet::Util::Log.new(:level => "notice", :message => :foo, :version => 100) log.to_yaml_properties.should_not include('@version') diff --git a/spec/unit/util/network_device/cisco/device_spec.rb b/spec/unit/util/network_device/cisco/device_spec.rb index 33242a1ab..8971205d3 100755 --- a/spec/unit/util/network_device/cisco/device_spec.rb +++ b/spec/unit/util/network_device/cisco/device_spec.rb @@ -145,7 +145,7 @@ eos "ATM 0/1.1" => "ATM0/1.1", "VLAN99" => "VLAN99" }.each do |input,expected| - it "should canonicalize #{input} to #{expected}" do + it "should canonicalize #{input} to #{expected}", :'fails_on_ruby_1.9.2' => true do @cisco.canonalize_ifname(input).should == expected end end @@ -232,7 +232,7 @@ eos @cisco.parse_interface("FastEthernet0/1").should == { :ensure => :absent, :duplex => :auto, :speed => :auto } end - it "should be able to parse the sh vlan brief command output" do + it "should be able to parse the sh vlan brief command output", :'fails_on_ruby_1.9.2' => true do @cisco.stubs(:support_vlan_brief?).returns(true) @transport.stubs(:command).with("sh vlan brief").returns(<<eos) Switch#sh vlan brief diff --git a/spec/unit/util/network_device/transport/ssh_spec.rb b/spec/unit/util/network_device/transport/ssh_spec.rb index 18d22a953..8fc357db3 100755 --- a/spec/unit/util/network_device/transport/ssh_spec.rb +++ b/spec/unit/util/network_device/transport/ssh_spec.rb @@ -4,7 +4,7 @@ require File.dirname(__FILE__) + '/../../../../spec_helper' require 'puppet/util/network_device/transport/ssh' -describe Puppet::Util::NetworkDevice::Transport::Ssh, :if => Puppet.features.ssh? do +describe Puppet::Util::NetworkDevice::Transport::Ssh, :if => Puppet.features.ssh?, :'fails_on_ruby_1.9.2' => true do before(:each) do @transport = Puppet::Util::NetworkDevice::Transport::Ssh.new() diff --git a/spec/unit/util/pson_spec.rb b/spec/unit/util/pson_spec.rb index 63d085a66..9331b9416 100755 --- a/spec/unit/util/pson_spec.rb +++ b/spec/unit/util/pson_spec.rb @@ -7,7 +7,7 @@ class PsonUtil include Puppet::Util::Pson end -describe Puppet::Util::Pson do +describe Puppet::Util::Pson, :'fails_on_ruby_1.9.2' => true do it "should fail if no data is provided" do lambda { PsonUtil.new.pson_create("type" => "foo") }.should raise_error(ArgumentError) end diff --git a/spec/unit/util/queue/stomp_spec.rb b/spec/unit/util/queue/stomp_spec.rb index 99c77d0b4..6799becea 100755 --- a/spec/unit/util/queue/stomp_spec.rb +++ b/spec/unit/util/queue/stomp_spec.rb @@ -2,14 +2,14 @@ require 'spec_helper' require 'puppet/util/queue' -describe Puppet::Util::Queue, :if => Puppet.features.stomp? do +describe Puppet::Util::Queue, :if => Puppet.features.stomp?, :'fails_on_ruby_1.9.2' => true do it 'should load :stomp client appropriately' do Puppet.settings.stubs(:value).returns 'faux_queue_source' Puppet::Util::Queue.queue_type_to_class(:stomp).name.should == 'Puppet::Util::Queue::Stomp' end end -describe 'Puppet::Util::Queue::Stomp', :if => Puppet.features.stomp? do +describe 'Puppet::Util::Queue::Stomp', :if => Puppet.features.stomp?, :'fails_on_ruby_1.9.2' => true do before do # So we make sure we never create a real client instance. # Otherwise we'll try to connect, and that's bad. diff --git a/spec/unit/util/rdoc/parser_spec.rb b/spec/unit/util/rdoc/parser_spec.rb index c7f99051f..92b50e09b 100755 --- a/spec/unit/util/rdoc/parser_spec.rb +++ b/spec/unit/util/rdoc/parser_spec.rb @@ -7,7 +7,7 @@ require 'puppet/util/rdoc/code_objects' require 'rdoc/options' require 'rdoc/rdoc' -describe RDoc::Parser do +describe RDoc::Parser, :'fails_on_ruby_1.9.2' => true do before :each do File.stubs(:stat).with("init.pp") @top_level = stub_everything 'toplevel', :file_relative_name => "init.pp" diff --git a/spec/unit/util/rdoc_spec.rb b/spec/unit/util/rdoc_spec.rb index 067b5b8a7..deae4ef2c 100755 --- a/spec/unit/util/rdoc_spec.rb +++ b/spec/unit/util/rdoc_spec.rb @@ -6,7 +6,7 @@ require 'rdoc/rdoc' describe Puppet::Util::RDoc do - describe "when generating RDoc HTML documentation" do + describe "when generating RDoc HTML documentation", :'fails_on_ruby_1.9.2' => true do before :each do @rdoc = stub_everything 'rdoc' RDoc::RDoc.stubs(:new).returns(@rdoc) diff --git a/spec/unit/util/selinux_spec.rb b/spec/unit/util/selinux_spec.rb index 0eaf43cbb..cc70b53c4 100755 --- a/spec/unit/util/selinux_spec.rb +++ b/spec/unit/util/selinux_spec.rb @@ -33,7 +33,7 @@ describe Puppet::Util::SELinux do end end - describe "filesystem detection" do + describe "filesystem detection", :'fails_on_ruby_1.9.2' => true do before :each do fh = stub 'fh', :close => nil File.stubs(:open).with("/proc/mounts").returns fh @@ -192,7 +192,7 @@ describe Puppet::Util::SELinux do end end - describe "set_selinux_context" do + describe "set_selinux_context", :'fails_on_ruby_1.9.2' => true do before :each do fh = stub 'fh', :close => nil File.stubs(:open).with("/proc/mounts").returns fh |