2015-05-06 19:57:38 -04:00
|
|
|
package libnetwork
|
|
|
|
|
|
|
|
|
|
import (
|
|
|
|
|
"testing"
|
|
|
|
|
|
|
|
|
|
"github.com/docker/libnetwork/driverapi"
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
func TestDriverRegistration(t *testing.T) {
|
|
|
|
|
bridgeNetType := "bridge"
|
Make driver packages register themselves via DriverCallback
In the present code, each driver package provides a `New()` method
which constructs a driver of its type, which is then registered with
the controller.
However, this is not suitable for the `drivers/remote` package, since
it does not provide a (singleton) driver, but a mechanism for drivers
to be added dynamically. As a result, the implementation is oddly
dual-purpose, and a spurious `"remote"` driver is added to the
controller's list of available drivers.
Instead, it is better to provide the registration callback to each
package and let it register its own driver or drivers. That way, the
singleton driver packages can construct one and register it, and the
remote package can hook the callback up with whatever the dynamic
driver mechanism turns out to be.
NB there are some method signature changes; in particular to
controller.New, which can return an error if the built-in driver
packages fail to initialise.
Signed-off-by: Michael Bridgen <mikeb@squaremobius.net>
2015-05-11 08:46:29 -04:00
|
|
|
c, err := New()
|
|
|
|
|
if err != nil {
|
|
|
|
|
t.Fatal(err)
|
|
|
|
|
}
|
|
|
|
|
err = c.(*controller).RegisterDriver(bridgeNetType, nil)
|
2015-05-06 19:57:38 -04:00
|
|
|
if err == nil {
|
|
|
|
|
t.Fatalf("Expecting the RegisterDriver to fail for %s", bridgeNetType)
|
|
|
|
|
}
|
|
|
|
|
if _, ok := err.(driverapi.ErrActiveRegistration); !ok {
|
|
|
|
|
t.Fatalf("Failed for unexpected reason: %v", err)
|
|
|
|
|
}
|
|
|
|
|
err = c.(*controller).RegisterDriver("test-dummy", nil)
|
|
|
|
|
if err != nil {
|
|
|
|
|
t.Fatalf("Test failed with an error %v", err)
|
|
|
|
|
}
|
|
|
|
|
}
|