diff options
author | Luke Kanies <luke@reductivelabs.com> | 2010-03-22 16:42:03 -0700 |
---|---|---|
committer | test branch <puppet-dev@googlegroups.com> | 2010-02-17 06:50:53 -0800 |
commit | 5aa26d08892010fc62de6adc278f07c8748d75e0 (patch) | |
tree | 237ab3c0f09ab8ef42ef1b98efb4847f70382baf /spec | |
parent | ec7ea27e356dc159b37cc2a09b6f14841e796ab2 (diff) | |
download | puppet-5aa26d08892010fc62de6adc278f07c8748d75e0.tar.gz puppet-5aa26d08892010fc62de6adc278f07c8748d75e0.tar.xz puppet-5aa26d08892010fc62de6adc278f07c8748d75e0.zip |
Changing method profile for other event queueing
It wasn't clear in the first refactor if this was
necessary, but doing the performance optimization
made it clear it was.
Signed-off-by: Luke Kanies <luke@reductivelabs.com>
Diffstat (limited to 'spec')
-rwxr-xr-x | spec/unit/transaction/event_manager.rb | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/spec/unit/transaction/event_manager.rb b/spec/unit/transaction/event_manager.rb index f808377ee..6b9fe0345 100755 --- a/spec/unit/transaction/event_manager.rb +++ b/spec/unit/transaction/event_manager.rb @@ -46,8 +46,8 @@ describe Puppet::Transaction::EventManager do @graph.expects(:matching_edges).with { |event, resource| event == @event }.returns [edge1, edge2] - @manager.expects(:queue_event_for_resource).with(@resource, edge1.target, edge1.callback, @event) - @manager.expects(:queue_event_for_resource).with(@resource, edge2.target, edge2.callback, @event) + @manager.expects(:queue_events_for_resource).with(@resource, edge1.target, edge1.callback, [@event]) + @manager.expects(:queue_events_for_resource).with(@resource, edge2.target, edge2.callback, [@event]) @manager.queue_events(@resource, [@event]) end @@ -57,7 +57,7 @@ describe Puppet::Transaction::EventManager do @resource.expects(:self_refresh?).returns true @resource.expects(:deleting?).returns false - @manager.expects(:queue_event_for_resource).with(@resource, @resource, :refresh, @event) + @manager.expects(:queue_events_for_resource).with(@resource, @resource, :refresh, [@event]) @manager.queue_events(@resource, [@event]) end @@ -67,7 +67,7 @@ describe Puppet::Transaction::EventManager do @resource.expects(:self_refresh?).returns false @resource.stubs(:deleting?).returns false - @manager.expects(:queue_event_for_resource).never + @manager.expects(:queue_events_for_resource).never @manager.queue_events(@resource, [@event]) end @@ -77,7 +77,7 @@ describe Puppet::Transaction::EventManager do @resource.expects(:self_refresh?).returns true @resource.expects(:deleting?).returns true - @manager.expects(:queue_event_for_resource).never + @manager.expects(:queue_events_for_resource).never @manager.queue_events(@resource, [@event]) end @@ -87,7 +87,7 @@ describe Puppet::Transaction::EventManager do @graph.expects(:matching_edges).returns [edge1] - @manager.expects(:queue_event_for_resource).never + @manager.expects(:queue_events_for_resource).never @manager.queue_events(@resource, [@event]) end @@ -97,7 +97,7 @@ describe Puppet::Transaction::EventManager do @graph.expects(:matching_edges).returns [edge1] - @manager.expects(:queue_event_for_resource).never + @manager.expects(:queue_events_for_resource).never @manager.queue_events(@resource, [@event]) end @@ -117,7 +117,7 @@ describe Puppet::Transaction::EventManager do target = stub("target") 2.times do |i| - @manager.queue_event_for_resource(stub("source", :info => nil), target, "callback#{i}", ["event#{i}"]) + @manager.queue_events_for_resource(stub("source", :info => nil), target, "callback#{i}", ["event#{i}"]) end @manager.queued_events(target) { |callback, events| } @@ -128,7 +128,7 @@ describe Puppet::Transaction::EventManager do source = stub 'source' source.expects(:info) - @manager.queue_event_for_resource(source, target, "callback", ["event"]) + @manager.queue_events_for_resource(source, target, "callback", ["event"]) @manager.queued_events(target) { |callback, events| } end |