Merge branch 'feature/forms' into unstable

src/Muzich/CoreBundle/Controller/CoreController.php

@@ -122,6 +122,20 @@ class CoreController extends Controller
     $user = $this->getUser();
     $em = $this->getDoctrine()->getEntityManager();
     
+    /*
+     * Contrôle préléminaire si groupe précisé
+     */
+    $group = null;
+    if ($group_slug)
+    {
+      $group = $this->findGroupWithSlug($group_slug);
+      if (!$group->userCanAddElement($this->getUserId()))
+      {
+        $group = null;
+        throw  $this->createNotFoundException('Vous ne pouvez pas ajouter d\'éléments a ce groupe');
+      }
+    }
+    
     $element = new Element();
     $form = $this->createForm(
       new ElementAddForm(),
@@ -156,18 +170,10 @@ class CoreController extends Controller
         $factory->proceedFill($user);
         
         // Si on a précisé un groupe dans lequel mettre l'element
-        if ($group_slug)
+        if ($group)
         {
-          $group = $this->findGroupWithSlug($group_slug);
-          if ($group->userCanAddElement($this->getUserId()))
-          {
-            $element->setGroup($group);
-          }
-          else
-          {
-            throw $this->createNotFoundException('Vous ne pouvez ajouter d\'element a ce groupe.');
-          }
-          $redirect_url = $this->generateUrl('show_group', array('slug' => $group->getSlug()));
+          $element->setGroup($group);
+          $redirect_url = $this->generateUrl('show_group', array('slug' => $group_slug));
         }
         else
         {

src/Muzich/CoreBundle/ElementFactory/Site/base/VideoSiteFactory.php

@@ -21,14 +21,25 @@ class VideoSiteFactory extends BaseFactory
   {
     parent::__construct($element, $container);
     
-    // Configuration de VideoEmbed
-    define('SITEBASE', $container->getParameter('sitebase'));
-    define('VIDEO_EMBED_CONFIG_FILE', SITEBASE.$container->getParameter('video_embed_config_file'));
-    //to activate debug mode and false for production usage. it will write 
-    //to a log file when something goes wrong but should not produce 
-    //exceptions in production enviroment
-    define('DEBUG', $container->getParameter('video_embed_debug')); 
-    
+    //if ($container->getParameter('env') != 'test')
+    //{
+      // Configuration de VideoEmbed
+      if (!defined('SITEBASE')) 
+      {
+        define('SITEBASE', $container->getParameter('sitebase'));
+      }
+    //}
+      if (!defined('VIDEO_EMBED_CONFIG_FILE')) 
+      {
+      define('VIDEO_EMBED_CONFIG_FILE', SITEBASE.$container->getParameter('video_embed_config_file'));
+      }
+      //to activate debug mode and false for production usage. it will write 
+      //to a log file when something goes wrong but should not produce 
+      //exceptions in production enviroment
+      if (!defined('DEBUG')) 
+      {
+        define('DEBUG', $container->getParameter('video_embed_debug')); 
+      }
     try {
       $this->video_engine =  new VideoEmbed($this->element->getUrl());
     } catch (Exception $exc) {

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

@@ -141,9 +141,6 @@ class HomeControllerTest extends FunctionalTest
     $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
      */
@@ -175,9 +172,6 @@ class HomeControllerTest extends FunctionalTest
     $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
      */
@@ -190,10 +184,10 @@ class HomeControllerTest extends FunctionalTest
     );
     
     $this->isResponseSuccess();
-    
-    $this->notExist('li:contains("Mon bel element")');
+        
+    $this->notExist('li:contains("Mon bel element a4er563a1r")');
     $element = $this->getDoctrine()->getRepository('MuzichCoreBundle:Element')
-      ->findOneByName('Mon bel element')
+      ->findOneByName('Mon bel element a4er563a1r')
     ;
     $this->assertTrue(is_null($element));
     
@@ -221,9 +215,9 @@ class HomeControllerTest extends FunctionalTest
     
     $this->isResponseSuccess();
     
-    $this->notExist('li:contains("Mon bel element")');
+    $this->notExist('li:contains("Mon bel element gfez7f")');
     $element = $this->getDoctrine()->getRepository('MuzichCoreBundle:Element')
-      ->findOneByName('Mon bel element')
+      ->findOneByName('Mon bel element gfez7f')
     ;
     $this->assertTrue(is_null($element));
     
@@ -243,41 +237,110 @@ class HomeControllerTest extends FunctionalTest
     
   }
   
-  public function testAddElementAtGroupFailure()
+  /**
+   * L'ajout d'un Element a un de ses groupe ne doit pas poser de problème
+   */
+  public function testAddElementAtMyGroupSuccess()
   {
     $this->connectUser('bux', 'toor');
+    // Un groupe open, donc pas de soucis
+    $fan_de_psy = $this->getDoctrine()->getRepository('MuzichCoreBundle:Group')->findOneByName('Fans de psytrance');
+    
+    $hardtek = $this->getDoctrine()->getRepository('MuzichCoreBundle:Tag')->findOneByName('Hardtek');
+    $tribe = $this->getDoctrine()->getRepository('MuzichCoreBundle:Tag')->findOneByName('Tribe');
+        
+    $this->isResponseSuccess();
+    $this->procedure_add_element(
+      'Element mis dans le groupe de psytrance', 
+      'http://www.youtube.com/watch?v=WC8qb_of04E', 
+      array($hardtek->getId(), $tribe->getId()),
+      $fan_de_psy->getSlug()
+    );
+    
+    $this->isResponseRedirection();
+    $this->followRedirection();
+    $this->isResponseSuccess();
+    
+    $this->exist('li:contains("Element mis dans le groupe de psytrance")');
+    $element = $this->getDoctrine()->getRepository('MuzichCoreBundle:Element')
+      ->findOneByName('Element mis dans le groupe de psytrance')
+    ;
+    $this->assertTrue(!is_null($element));
+    
+    if (!is_null($element))
+    {
+      $this->assertEquals($fan_de_psy->getId(), $element->getGroup()->getId());
+    }
+    else
+    {
+      $this->assertTrue(false);
+    }
+    
+    $this->disconnectUser();
     
     /*
-     * Ajout d'un élément lié a un groupe (success)
+     * Ajout d'un element dans un groupe que l'on posséde.
      */
+    $this->connectUser('joelle', 'toor');
+    $this->isResponseSuccess();
+    
+    // Ce groupe appartient a joelle
+    $groupe_de_joelle = $this->getDoctrine()->getRepository('MuzichCoreBundle:Group')->findOneByName('Le groupe de joelle');
+    
     $this->procedure_add_element(
-      'Mon bel element', 
+      'Element mis dans le groupe de joelle', 
       'http://www.youtube.com/watch?v=WC8qb_of04E', 
       array($hardtek->getId(), $tribe->getId()),
-      $fan_de_psy->getId()
+      $groupe_de_joelle->getSlug()
     );
     
     $this->isResponseRedirection();
     $this->followRedirection();
     $this->isResponseSuccess();
     
-    $this->exist('li:contains("Mon bel element")');
+    $this->exist('li:contains("Element mis dans le groupe de joelle")');
     $element = $this->getDoctrine()->getRepository('MuzichCoreBundle:Element')
-      ->findOneByName('Mon bel element')
+      ->findOneByName('Element mis dans le groupe de joelle')
     ;
     $this->assertTrue(!is_null($element));
+    
     if (!is_null($element))
     {
-      $this->assertEquals($fan_de_psy->getId(), $element->getGroup()->getId());
+      $this->assertEquals($groupe_de_joelle->getId(), $element->getGroup()->getId());
+    }
+    else
+    {
+      $this->assertTrue(false);
     }
   }
   
   /**
-   * L'ajout d'un Element a un de ses groupe ne doit pas poser de problème
+   * L'ajout a un group qui n'est pas a sois, ou qui n'est pas open
+   * doit être impossible.
    */
-  public function testAddElementAtMyGroupSuccess()
+  public function testAddElementAtGroupFailure()
   {
+    $this->connectUser('bux', 'toor');
+    // Un groupe no open
+    $dudeldrum = $this->getDoctrine()->getRepository('MuzichCoreBundle:Group')->findOneByName('DUDELDRUM');
+    
+    $hardtek = $this->getDoctrine()->getRepository('MuzichCoreBundle:Tag')->findOneByName('Hardtek');
+    $tribe = $this->getDoctrine()->getRepository('MuzichCoreBundle:Tag')->findOneByName('Tribe');
+        
+    // Nous tentons d'ouvrir l'url d'ajout d'élément avec un groupe qui n'est pas ouvert
+    // et qui n'appartient pas a l'utilisateur connecté
+    $this->crawler = $this->client->request(
+      'POST', 
+      $this->generateUrl('element_add', array('group_slug' => $dudeldrum->getSlug())),
+      array(
+        'element_add[name]' => 'Yohoho trululu',
+        'element_add[url]'  => 'http://www.youtube.com/watch?v=WC8qb_of04E',
+        'element_add[tags]['.$hardtek->getId().']' => $hardtek->getId(),
+        'element_add[tags]['.$tribe->getId().']' => $tribe->getId()
+      )
+    );
     
+    $this->isResponseNotFound();
   }
   
 }

src/Muzich/CoreBundle/lib/FunctionalTest.php

@@ -67,6 +67,11 @@ class FunctionalTest extends WebTestCase
     $this->assertEquals($login, $user->getUsername());
   }
   
+  protected function disconnectUser()
+  {
+    $this->crawler = $this->client->request('GET', $this->generateUrl('fos_user_security_logout'));
+  }
+  
   protected function validate_registrate_user_form($form, $username, $email, $pass1, $pass2)
   {
     $form['fos_user_registration_form[username]'] = $username;
@@ -114,26 +119,35 @@ class FunctionalTest extends WebTestCase
   }
   
   /**
-   * Procédure d'ajout d'un élément a partir de la page home
+   * Procédure d'ajout d'un élément
    * 
    * @param string $name
    * @param string $url
    * @param array $tags
    * @param int $group_id 
    */
-  protected function procedure_add_element($name, $url, $tags, $group_id = '')
+  protected function procedure_add_element($name, $url, $tags, $group_slug = null)
   {
-    $this->crawler = $this->client->request('GET', $this->generateUrl('home'));
+    if (!$group_slug)
+    {
+      $this->crawler = $this->client->request('GET', $this->generateUrl('home'));
+      $form_url = $this->generateUrl('element_add');
+    }
+    else
+    {
+      $this->crawler = $this->client->request('GET', $this->generateUrl('show_group', array('slug' => $group_slug)));
+      $form_url = $this->generateUrl('element_add', array('group_slug' => $group_slug));
+    }
     $this->isResponseSuccess();
     
-    $form = $this->selectForm('form[action="'.$this->generateUrl('element_add').'"] input[type="submit"]');
+    $form = $this->selectForm('form[action="'.$form_url.'"] 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);
   }
   
@@ -161,6 +175,12 @@ class FunctionalTest extends WebTestCase
     return $this->getContainer()->get('session');
   }
   
+  protected function clickOnLink($link)
+  {
+    $this->crawler = $this->client->click($link);
+  }
+
+
   /**
    *
    * @return \Symfony\Bundle\DoctrineBundle\Registry
@@ -256,7 +276,7 @@ class FunctionalTest extends WebTestCase
    */
   protected function isResponseRedirection()
   {
-    $this->client->getResponse()->isRedirection();
+    $this->assertTrue($this->client->getResponse()->isRedirection());
   }
   
   /**
@@ -264,6 +284,14 @@ class FunctionalTest extends WebTestCase
    */
   protected function isResponseSuccess()
   {
-    $this->client->getResponse()->isSuccessful();
+    $this->assertTrue($this->client->getResponse()->isSuccessful());
+  }
+  
+  /**
+   * Contrôle que le CodeStatus de la Response correspond bien a celle d'un Ok
+   */
+  protected function isResponseNotFound()
+  {
+    $this->assertTrue($this->client->getResponse()->isNotFound());
   }
 }