Ensure plugin managers validate service instances

@ezimuel noted when working on zend-validator updates that
`setService()` was no longer raising an exception when the plugin
instance was not valid for the plugin manager. This patch fixes the
regression.

Author: weierophinney

src/AbstractPluginManager.php

@@ -92,6 +92,28 @@ public function __construct($configInstanceOrParentLocator = null, array $config
             : $this;
     }
 
+    /**
+     * Override configure() to validate service instances.
+     *
+     * If an instance passed in the `services` configuration is invalid for the
+     * plugin manager, this method will raise an InvalidServiceException.
+     *
+     * {@inheritDoc}
+     * @throws InvalidServiceException
+     */
+    public function configure(array $config)
+    {
+        if (isset($config['services'])) {
+            foreach ($config['services'] as $service) {
+                $this->validate($service);
+            }
+        }
+
+        parent::configure($config);
+
+        return $this;
+    }
+
     /**
      * {@inheritDoc}
      *

test/AbstractPluginManagerTest.php

@@ -333,4 +333,20 @@ public function testValidateWillFallBackToValidatePluginWhenDefinedAndEmitDeprec
         $this->assertTrue($assertionCalled, 'Assertion was not called by validatePlugin!');
         $this->assertTrue($errorHandlerCalled, 'Error handler was not triggered by validatePlugin!');
     }
+
+    public function testSetServiceShouldRaiseExceptionForInvalidPlugin()
+    {
+        $pluginManager = new TestAsset\SimplePluginManager(new ServiceManager());
+        $this->setExpectedException(InvalidServiceException::class);
+        $pluginManager->setService(stdClass::class, new stdClass());
+    }
+
+    public function testPassingServiceInstanceViaConfigureShouldRaiseExceptionForInvalidPlugin()
+    {
+        $pluginManager = new TestAsset\SimplePluginManager(new ServiceManager());
+        $this->setExpectedException(InvalidServiceException::class);
+        $pluginManager->configure(['services' => [
+            stdClass::class => new stdClass(),
+        ]]);
+    }
 }