OK, but you are modifying the source to test the target.
Seems to clever for a spec ;-)
(And it's a big side-effect : we like to avoid side effect)

Max

On Jan 11, 2013, at 9:21 AM, llh1 <notifications@github.com> wrote:

> In spec/actions/plate_transfer_spec.rb:
> 
> > @@ -91,7 +94,9 @@ def self.should_transfer
> >              it "should save the transfered plates" do
> >                store.with_session do |s|
> >                  source, target = [source_id, target_id].map { |i| s.plate[i] }
> > -                target[:B1].should == source[:C3] 
> > +                target[:B1].should == source[:C3].each do |aliquot|
> > +                 aliquot.type = aliquot_type
> I know = it's an assignment and not a comparison... the objective wasn't to compare anything doing aliquot.type = aliquot_type. I just check the target corresponds to the source + the aliquot_type filled.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>