+1, with one comment below. -- Jacob Helwig
On Fri, 01 Oct 2010 14:08:37 -0400, Nick Lewis wrote: > > These shared behaviors were leaking before blocks into the specs > using them. Wrapping them in an enclosing describe fixes this and > alleviates the need to destroy_all before each spec. > > Signed-off-by: Nick Lewis <[email protected]> > --- > app/models/node.rb | 2 +- > spec/shared_behaviors/controller_mixins.rb | 108 ++++++++++++++------------- > spec/shared_behaviors/sorted_index.rb | 18 ++--- > 3 files changed, 65 insertions(+), 63 deletions(-) > > diff --git a/app/models/node.rb b/app/models/node.rb > index 3ecbb0d..c384004 100644 > --- a/app/models/node.rb > +++ b/app/models/node.rb > @@ -97,7 +97,7 @@ class Node < ActiveRecord::Base > end > > def inherited_classes > - (node_group_list.map(&:first)- [self]).map(&:node_classes).flatten.uniq > + (node_group_list.map(&:first) - [self]).map(&:node_classes).flatten.uniq No longer necessary after previous commit's comment here?
signature.asc
Description: Digital signature
