On Mar 26, 2010, at 9:30 AM, Markus Roberts wrote:
>> brice:tickets/0.25.x/3396 (Faster event propagation) succeeded:
> * 7c3cd8c Fix inefficient SimpleGraph#matching_edge
> 7771 examples, 28 failures, 41 pending
> 4 more in unit/simple_graph.rb (4 total)
This is not good. Looks like we are missing Luke's glue event patch,
or
there is something at play here.
The event-->events patch appears to be there; the problem appears to
be with the event.source patch (all of the errors look like this):
1)
NoMethodError in 'Puppet::SimpleGraph when matching edges should
match edges whose source matches the source of the event'
undefined method `inject' for #<Puppet::Transaction::Event:0x250b610>
/Users/markus/projects/puppet/lib/puppet/simple_graph.rb:164:in
`matching_edges'
spec/unit/simple_graph.rb:361:
/Library/Ruby/Gems/1.8/gems/rspec-1.2.9/lib/spec/example/
example_methods.rb:40:in `instance_eval'
/Library/Ruby/Gems/1.8/gems/rspec-1.2.9/lib/spec/example/
example_methods.rb:40:in `execute'
/System/Library/Frameworks/Ruby.framework/Versions/1.8/usr/lib/ruby/
1.8/timeout.rb:53:in `timeout'
/Library/Ruby/Gems/1.8/gems/rspec-1.2.9/lib/spec/example/
example_methods.rb:37:in `execute'
/Library/Ruby/Gems/1.8/gems/rspec-1.2.9/lib/spec/example/
example_group_methods.rb:214:in `run_examples'
/Library/Ruby/Gems/1.8/gems/rspec-1.2.9/lib/spec/example/
example_group_methods.rb:212:in `each'
/Library/Ruby/Gems/1.8/gems/rspec-1.2.9/lib/spec/example/
example_group_methods.rb:212:in `run_examples'
/Library/Ruby/Gems/1.8/gems/rspec-1.2.9/lib/spec/example/
example_group_methods.rb:103:in `run'
/Users/markus/projects/puppet/spec/monkey_patches/
add_confine_and_runnable_to_rspec_dsl.rb:22:in `run'
/Users/markus/projects/puppet/spec/monkey_patches/
add_confine_and_runnable_to_rspec_dsl.rb:17:in `each'
/Users/markus/projects/puppet/spec/monkey_patches/
add_confine_and_runnable_to_rspec_dsl.rb:17:in `run'
/Library/Ruby/Gems/1.8/gems/rspec-1.2.9/lib/spec/runner/options.rb:
151:in `run_examples'
/Library/Ruby/Gems/1.8/gems/rspec-1.2.9/lib/spec/runner.rb:61:in `run'
/Library/Ruby/Gems/1.8/gems/rspec-1.2.9/lib/spec/runner.rb:45:in
`autorun'
spec/unit/simple_graph.rb:9:
I've confirmed that my code still just calls matching_edges with one
event but the SimpleGraph class expects multiple of them, thus the
inject is failing (because it's being called on one event instead of
many).
I think the problem is that I refactored queue_event after Brice
refactored matching_edges, or something similar.
Brice, can you, in your patch, remove the optimizations in
EventManager#queue_event and change it so it passes the event list to
matching_edges, instead of one event at a time? I think that's the
best move, because otherwise we're going to keep dancing around each
other in our refactors.
--
Criminal: A person with predatory instincts who has not sufficient
capital to form a corporation. --Howard Scott
---------------------------------------------------------------------
Luke Kanies -|- http://puppetlabs.com -|- +1(615)594-8199
--
You received this message because you are subscribed to the Google Groups "Puppet
Developers" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/puppet-dev?hl=en.
