minor #1580 Refactoring tests (carusogabriel)

This PR was merged into the 2.2.x-dev branch.

Discussion
----------

Refactoring tests

I've refactored some tests, using:
- `assertCount` instead of `count` function;
- `assertFalse` instead of strict comparison with `false` keyword;
- `assertArrayHasKey` instead of `isset` function.

Commits
-------

b163e44 Refactoring tests

tests/Silex/Tests/ApplicationTest.php

@@ -88,7 +88,7 @@ public function testGetRoutesWithNoRoutes()
 
         $routes = $app['routes'];
         $this->assertInstanceOf('Symfony\Component\Routing\RouteCollection', $routes);
-        $this->assertEquals(0, count($routes->all()));
+        $this->assertCount(0, $routes->all());
     }
 
     public function testGetRoutesWithRoutes()
@@ -105,9 +105,9 @@ public function testGetRoutesWithRoutes()
 
         $routes = $app['routes'];
         $this->assertInstanceOf('Symfony\Component\Routing\RouteCollection', $routes);
-        $this->assertEquals(0, count($routes->all()));
+        $this->assertCount(0, $routes->all());
         $app->flush();
-        $this->assertEquals(2, count($routes->all()));
+        $this->assertCount(2, $routes->all());
     }
 
     public function testOnCoreController()

tests/Silex/Tests/ControllerCollectionTest.php

@@ -30,7 +30,7 @@ public function testGetRouteCollectionWithNoRoutes()
     {
         $controllers = new ControllerCollection(new Route());
         $routes = $controllers->flush();
-        $this->assertEquals(0, count($routes->all()));
+        $this->assertCount(0, $routes->all());
     }
 
     public function testGetRouteCollectionWithRoutes()
@@ -40,7 +40,7 @@ public function testGetRouteCollectionWithRoutes()
         $controllers->match('/bar', function () {});
 
         $routes = $controllers->flush();
-        $this->assertEquals(2, count($routes->all()));
+        $this->assertCount(2, $routes->all());
     }
 
     public function testControllerFreezing()

tests/Silex/Tests/Provider/DoctrineServiceProviderTest.php

@@ -88,7 +88,7 @@ public function testLoggerLoading()
         }
 
         $app = new Application();
-        $this->assertTrue(isset($app['logger']));
+        $this->assertArrayHasKey('logger', $app);
         $this->assertNull($app['logger']);
         $app->register(new DoctrineServiceProvider(), array(
             'dbs.options' => array(

tests/Silex/Tests/Provider/ValidatorServiceProviderTest.php

@@ -110,8 +110,8 @@ public function testValidatorConstraint($email, $isValid, $nbGlobalError, $nbEma
         $form->submit(array('email' => $email));
 
         $this->assertEquals($isValid, $form->isValid());
-        $this->assertEquals($nbGlobalError, count($form->getErrors()));
-        $this->assertEquals($nbEmailError, count($form->offsetGet('email')->getErrors()));
+        $this->assertCount($nbGlobalError, $form->getErrors());
+        $this->assertCount($nbEmailError, $form->offsetGet('email')->getErrors());
     }
 
     public function testValidatorWillNotAddNonexistentTranslationFiles()

tests/Silex/Tests/StreamTest.php

@@ -28,7 +28,7 @@ public function testStreamReturnsStreamingResponse()
 
         $response = $app->stream();
         $this->assertInstanceOf('Symfony\Component\HttpFoundation\StreamedResponse', $response);
-        $this->assertSame(false, $response->getContent());
+        $this->assertFalse($response->getContent());
     }
 
     public function testStreamActuallyStreams()