Merge branch 'feature/test' into unstable

app/config/config_dev.yml

@@ -21,3 +21,6 @@ monolog:
 
 assetic:
     use_controller: true
+
+parameters:
+    env: dev

app/config/config_prod.yml

@@ -16,3 +16,6 @@ monolog:
             type:  stream
             path:  %kernel.logs_dir%/%kernel.environment%.log
             level: debug
+
+parameters:
+    env: prod

app/config/config_test.yml

@@ -12,3 +12,6 @@ web_profiler:
 
 swiftmailer:
     disable_delivery: true
+
+parameters:
+    env: test

src/Muzich/CoreBundle/Controller/CoreController.php

@@ -138,6 +138,19 @@ class CoreController extends Controller
       $form->bindRequest($this->getRequest());
       if ($form->isValid())
       {
+        
+        /**
+         * Bug lors des tests: L'user n'est pas 'lié' a celui en base par doctrine.
+         * Docrine le voit si on faire une requete directe.
+         */
+        if ($this->container->getParameter('env') == 'test')
+        {
+          $user = $this->getDoctrine()->getRepository('MuzichCoreBundle:User')->findOneById(
+            $this->container->get('security.context')->getToken()->getUser()->getId(),
+            array()
+          )->getSingleResult();
+        }
+        
         // On utilise le gestionnaire d'élément
         $factory = new ElementManager($element, $em, $this->container);
         $factory->proceedFill($user);

src/Muzich/CoreBundle/Tests/Controller/HomeControllerTest.php

@@ -131,4 +131,153 @@ class HomeControllerTest extends FunctionalTest
     }
   }
   
+  /**
+   * Ajouts d'éléments et tests de cas refusés
+   */
+  public function testAddElementSuccess()
+  {
+    $this->connectUser('bux', 'toor');
+    
+    $hardtek = $this->getDoctrine()->getRepository('MuzichCoreBundle:Tag')->findOneByName('Hardtek');
+    $tribe = $this->getDoctrine()->getRepository('MuzichCoreBundle:Tag')->findOneByName('Tribe');
+    
+    // Un groupe open
+    $fan_de_psy = $this->getDoctrine()->getRepository('MuzichCoreBundle:Group')->findOneByName('Fans de psytrance');
+    
+    /*
+     *  Ajout d'un élément avec succés
+     */
+    $this->procedure_add_element(
+      'Mon bel element', 
+      'http://www.youtube.com/watch?v=WC8qb_of04E', 
+      array($hardtek->getId(), $tribe->getId())
+    );
+    
+    $this->isResponseRedirection();
+    $this->followRedirection();
+    $this->isResponseSuccess();
+    
+    $this->exist('li:contains("Mon bel element")');
+    $element = $this->getDoctrine()->getRepository('MuzichCoreBundle:Element')
+      ->findOneByName('Mon bel element')
+    ;
+    $this->assertTrue(!is_null($element));
+        
+  }
+  
+  /**
+   * Ajouts d'éléments et tests de cas refusés
+   */
+  public function testAddElementFailure()
+  {
+    $this->connectUser('bux', 'toor');
+    
+    $hardtek = $this->getDoctrine()->getRepository('MuzichCoreBundle:Tag')->findOneByName('Hardtek');
+    $tribe = $this->getDoctrine()->getRepository('MuzichCoreBundle:Tag')->findOneByName('Tribe');
+    
+    // Un groupe no open
+    $dudeldrum = $this->getDoctrine()->getRepository('MuzichCoreBundle:Group')->findOneByName('DUDELDRUM');
+    
+    /*
+     *  Ajouts d'éléments avec echec
+     */
+    
+    // Nom trop court
+    $this->procedure_add_element(
+      'Mo', 
+      'http://www.youtube.com/watch?v=WC8qb_of04E', 
+      array($hardtek->getId(), $tribe->getId())
+    );
+    
+    $this->isResponseSuccess();
+    
+    $this->notExist('li:contains("Mon bel element")');
+    $element = $this->getDoctrine()->getRepository('MuzichCoreBundle:Element')
+      ->findOneByName('Mon bel element')
+    ;
+    $this->assertTrue(is_null($element));
+    
+    // Nom trop long
+    $this->procedure_add_element(
+      'Mon bel element mais qui a un nom trop court la vache oui trop long hohoho', 
+      'http://www.youtube.com/watch?v=WC8qb_of04E', 
+      array($hardtek->getId(), $tribe->getId())
+    );
+    
+    $this->isResponseSuccess();
+    
+    $this->notExist('li:contains("Mon bel element mais qui a un nom trop court la vache oui trop long hohoho")');
+    $element = $this->getDoctrine()->getRepository('MuzichCoreBundle:Element')
+      ->findOneByName('Mon bel element mais qui a un nom trop court la vache oui trop long hohoho')
+    ;
+    $this->assertTrue(is_null($element));
+    
+    // Pas d'url
+    $this->procedure_add_element(
+      'Mon bel element', 
+      '', 
+      array($hardtek->getId(), $tribe->getId())
+    );
+    
+    $this->isResponseSuccess();
+    
+    $this->notExist('li:contains("Mon bel element")');
+    $element = $this->getDoctrine()->getRepository('MuzichCoreBundle:Element')
+      ->findOneByName('Mon bel element')
+    ;
+    $this->assertTrue(is_null($element));
+    
+    // Pas de nom
+    $this->procedure_add_element(
+      '', 
+      'http://www.youtube.com/watch?v=WC8qb_of04E', 
+      array($hardtek->getId(), $tribe->getId())
+    );
+    
+    $this->isResponseSuccess();
+    
+    $element = $this->getDoctrine()->getRepository('MuzichCoreBundle:Element')
+      ->findOneByName('')
+    ;
+    $this->assertTrue(is_null($element));
+    
+  }
+  
+  public function testAddElementAtGroupFailure()
+  {
+    $this->connectUser('bux', 'toor');
+    
+    /*
+     * Ajout d'un élément lié a un groupe (success)
+     */
+    $this->procedure_add_element(
+      'Mon bel element', 
+      'http://www.youtube.com/watch?v=WC8qb_of04E', 
+      array($hardtek->getId(), $tribe->getId()),
+      $fan_de_psy->getId()
+    );
+    
+    $this->isResponseRedirection();
+    $this->followRedirection();
+    $this->isResponseSuccess();
+    
+    $this->exist('li:contains("Mon bel element")');
+    $element = $this->getDoctrine()->getRepository('MuzichCoreBundle:Element')
+      ->findOneByName('Mon bel element')
+    ;
+    $this->assertTrue(!is_null($element));
+    if (!is_null($element))
+    {
+      $this->assertEquals($fan_de_psy->getId(), $element->getGroup()->getId());
+    }
+  }
+  
+  /**
+   * L'ajout d'un Element a un de ses groupe ne doit pas poser de problème
+   */
+  public function testAddElementAtMyGroupSuccess()
+  {
+    
+  }
+  
 }

src/Muzich/CoreBundle/lib/FunctionalTest.php

@@ -20,6 +20,13 @@ class FunctionalTest extends WebTestCase
    */
   protected $crawler;
   
+  protected function outputDebug()
+  {
+    unlink('/home/bux/.debug/out.html');
+    $monfichier = fopen('/home/bux/.debug/out.html', 'a+');
+    fwrite($monfichier, $this->client->getResponse()->getContent());
+  }
+  
   /**
    * Retourne l'objet User
    * 
@@ -107,6 +114,30 @@ class FunctionalTest extends WebTestCase
   }
   
   /**
+   * Procédure d'ajout d'un élément a partir de la page home
+   * 
+   * @param string $name
+   * @param string $url
+   * @param array $tags
+   * @param int $group_id 
+   */
+  protected function procedure_add_element($name, $url, $tags, $group_id = '')
+  {
+    $this->crawler = $this->client->request('GET', $this->generateUrl('home'));
+    $this->isResponseSuccess();
+    
+    $form = $this->selectForm('form[action="'.$this->generateUrl('element_add').'"] input[type="submit"]');
+    $form['element_add[name]'] = $name;
+    $form['element_add[url]'] = $url;
+    foreach ($tags as $tag_id)
+    {
+      $form['element_add[tags]['.$tag_id.']'] = $tag_id;
+    }
+    $form['element_add[group]'] = $group_id;
+    $this->submit($form);
+  }
+  
+  /**
    * Generates a URL from the given parameters.
    *
    * @param string $route
@@ -150,6 +181,16 @@ class FunctionalTest extends WebTestCase
   }
   
   /**
+   * Test l'inexistance d'un element
+   * 
+   * @param string $filter 
+   */
+  protected function notExist($filter)
+  {
+    $this->assertFalse($this->crawler->filter($filter)->count() > 0);
+  }
+  
+  /**
    * Retourne un objet lien
    * 
    * @param string $filter

src/Muzich/CoreBundle/lib/Test/Client.php

@@ -9,32 +9,32 @@ class Client extends BaseClient
   static protected $connection;
   protected $requested;
 
-    protected function doRequest($request)
-    {
-        if ($this->requested) {
-            $this->kernel->shutdown();
-            $this->kernel->boot();
-        }
-
-        $this->injectConnection();
-        $this->requested = true;
+  protected function doRequest($request)
+  {
+    if ($this->requested) {
+      $this->kernel->shutdown();
+      $this->kernel->boot();
+    }
 
-        return $this->kernel->handle($request);
+    $this->injectConnection();
+    $this->requested = true;
+
+    return $this->kernel->handle($request);
+  }
+
+  protected function injectConnection()
+  {
+    if (null === self::$connection) {
+      self::$connection = $this->getContainer()->get('doctrine.dbal.default_connection');
+    } else {
+      if (! $this->requested) {
+          self::$connection->rollback();
+      }
+      $this->getContainer()->set('doctrine.dbal.default_connection', self::$connection);
     }
 
-    protected function injectConnection()
-    {
-        if (null === self::$connection) {
-            self::$connection = $this->getContainer()->get('doctrine.dbal.default_connection');
-        } else {
-            if (! $this->requested) {
-                self::$connection->rollback();
-            }
-            $this->getContainer()->set('doctrine.dbal.default_connection', self::$connection);
-        }
-
-        if (! $this->requested) {
-            self::$connection->beginTransaction();
-        }
+    if (! $this->requested) {
+      self::$connection->beginTransaction();
     }
+  }
 }