Merge pull request #118 from php-http/test-journal

test that journal is set up correctly

Author: Nyholm

Tests/Functional/ServiceInstantiationTest.php

@@ -2,7 +2,11 @@
 
 namespace Http\HttplugBundle\Tests\Functional;
 
+use Http\Client\HttpClient;
+use Http\HttplugBundle\Collector\DebugPluginCollector;
+use Http\HttplugBundle\Collector\PluginJournal;
 use Symfony\Bundle\FrameworkBundle\Test\WebTestCase;
+use Symfony\Component\HttpKernel\Profiler\Profiler;
 
 class ServiceInstantiationTest extends WebTestCase
 {
@@ -12,7 +16,7 @@ public function testHttpClient()
         $container = static::$kernel->getContainer();
         $this->assertTrue($container->has('httplug.client'));
         $client = $container->get('httplug.client');
-        $this->assertInstanceOf('Http\Client\HttpClient', $client);
+        $this->assertInstanceOf(HttpClient::class, $client);
     }
 
     public function testHttpClientNoDebug()
@@ -21,6 +25,25 @@ public function testHttpClientNoDebug()
         $container = static::$kernel->getContainer();
         $this->assertTrue($container->has('httplug.client'));
         $client = $container->get('httplug.client');
-        $this->assertInstanceOf('Http\Client\HttpClient', $client);
+        $this->assertInstanceOf(HttpClient::class, $client);
+    }
+
+    public function testDebugToolbar()
+    {
+        static::bootKernel(['debug' => true]);
+        $container = static::$kernel->getContainer();
+        $this->assertTrue($container->has('profiler'));
+        $profiler = $container->get('profiler');
+        $this->assertInstanceOf(Profiler::class, $profiler);
+        $this->assertTrue($profiler->has('httplug'));
+        $collector = $profiler->get('httplug');
+        $this->assertInstanceOf(DebugPluginCollector::class, $collector);
+        /** @var PluginJournal $journal */
+        $journal = $collector->getJournal();
+        $plugins = $journal->getPlugins('acme');
+        $this->assertEquals([
+            'httplug.plugin.stopwatch',
+            'httplug.plugin.redirect',
+        ], $plugins);
     }
 }

Tests/Resources/app/AppKernel.php

@@ -12,9 +12,14 @@ public function registerBundles()
     {
         $bundles =  [
             new \Symfony\Bundle\FrameworkBundle\FrameworkBundle(),
+            new \Symfony\Bundle\TwigBundle\TwigBundle(),
             new \Http\HttplugBundle\HttplugBundle(),
         ];
 
+        if (in_array($this->getEnvironment(), array('dev', 'test'))) {
+            $bundles[] = new Symfony\Bundle\WebProfilerBundle\WebProfilerBundle();
+        }
+
         return $bundles;
     }
 
@@ -23,7 +28,9 @@ public function registerBundles()
      */
     public function registerContainerConfiguration(LoaderInterface $loader)
     {
-        $loader->load(__DIR__.'/config/config.yml');
+        $this->isDebug()
+            ? $loader->load(__DIR__.'/config/config_debug.yml')
+            : $loader->load(__DIR__.'/config/config.yml');
     }
 
     /**

Tests/Resources/app/config/config.yml

@@ -2,5 +2,9 @@ framework:
   secret: php-http
   test: ~
 
-# Use auto discovery
-httplug: ~
+httplug:
+    clients:
+        acme:
+            factory: httplug.factory.curl
+            plugins:
+                - httplug.plugin.redirect

Tests/Resources/app/config/config_debug.yml

@@ -0,0 +1,5 @@
+imports:
+    - { resource: config.yml }
+
+framework:
+    profiler: ~