Add missing tests for the resolving callbacks on the container

+ bug fix related to multiple calls to "resolving" and "afterResolving" callbacks

Author: imanghafoori1

src/Illuminate/Container/Container.php

@@ -261,7 +261,8 @@ protected function getClosure($abstract, $concrete)
                 return $container->build($concrete);
             }
 
-            return $container->make($concrete, $parameters);
+            // To prevent extra call to resolving callbacks, we make the object silently.
+            return $container->make($concrete, $parameters, true);
         };
     }
 
@@ -602,11 +603,12 @@ public function makeWith($abstract, array $parameters = [])
      *
      * @param  string  $abstract
      * @param  array  $parameters
+     * @param bool $silent
      * @return mixed
      */
-    public function make($abstract, array $parameters = [])
+    public function make($abstract, array $parameters = [], $silent = false)
     {
-        return $this->resolve($abstract, $parameters);
+        return $this->resolve($abstract, $parameters, $silent);
     }
 
     /**
@@ -630,9 +632,10 @@ public function get($id)
      *
      * @param  string  $abstract
      * @param  array  $parameters
+     * @param  bool  $silent
      * @return mixed
      */
-    protected function resolve($abstract, $parameters = [])
+    protected function resolve($abstract, $parameters = [], $silent = false)
     {
         $abstract = $this->getAlias($abstract);
 
@@ -674,8 +677,9 @@ protected function resolve($abstract, $parameters = [])
             $this->instances[$abstract] = $object;
         }
 
-        $this->fireResolvingCallbacks($abstract, $object);
-
+        if (! $silent) {
+            $this->fireResolvingCallbacks($abstract, $object);
+        }
         // Before returning, we will also set the resolved flag to "true" and pop off
         // the parameter overrides for this build. After those two things are done
         // we will be ready to return back the fully constructed class instance.

src/Illuminate/Contracts/Container/Container.php

@@ -113,7 +113,7 @@ public function factory($abstract);
      * @param  array  $parameters
      * @return mixed
      */
-    public function make($abstract, array $parameters = []);
+    public function make($abstract, array $parameters = [], $silent);
 
     /**
      * Call the given Closure / class@method and inject its dependencies.

src/Illuminate/Foundation/Application.php

@@ -722,17 +722,18 @@ public function registerDeferredProvider($provider, $service = null)
      *
      * @param  string  $abstract
      * @param  array  $parameters
+     * @param  bool   $silent
      * @return mixed
      */
-    public function make($abstract, array $parameters = [])
+    public function make($abstract, array $parameters = [], $silent = false)
     {
         $abstract = $this->getAlias($abstract);
 
         if (isset($this->deferredServices[$abstract]) && ! isset($this->instances[$abstract])) {
             $this->loadDeferredProvider($abstract);
         }
 
-        return parent::make($abstract, $parameters);
+        return parent::make($abstract, $parameters, $silent);
     }
 
     /**