Fix unit tests for simplebridge

@lk4d4 @icecrime @tiborvass

You’re right. I’ll put that together in the morning.

-Erik

> On Dec 22, 2014, at 7:45 PM, Arnaud Porterie <notifications@github.com> wrote:
> 
> In extensions/simplebridge/endpoint.go <https://github.com/shykes/docker/pull/129#discussion-diff-22202833>:
> 
> > @@ -120,7 +120,11 @@ func (b *BridgeEndpoint) configure(name string, s sandbox.Sandbox) error {
> >  	b.ip = ip
> >  	b.interfaceName = name
> >  
> > -	return s.AddIface(ns)
> > +	if s != nil { // this allows for pure endpoint testing.
> > +		return s.AddIface(ns)
> > +	} else {
> > +		return nil
> > +	}
> If this is about testing, shouldn't we instead have the test pass a non nil instance of sandbox.Sandbox (so basically just a DummySandbox which implements the interface by doing nothing) instead of modifying the tested code?
> 
> —
> Reply to this email directly or view it on GitHub <https://github.com/shykes/docker/pull/129/files#r22202833>.
>