DevelopTech / AgilityBundle

Tests/Manager/ProjectManagerTest.php

Spacing +1 added lines, -1 removed lines

@@ -121,7 +121,7 @@
      * @return Project|null
      */
     public function getProjectMock($id) {
-        if($id === 2) {
+        if ($id === 2) {
             return null;
         }
         return

Doc Comments +4 added lines, -1 removed lines

@@ -63,6 +63,9 @@ 
         $this->manager->editProject(2, 'Unknown project', 'Where is my project ?');
     }
 
+    /**
+     * @return \Doctrine\ORM\EntityManager
+     */
     public function getEntityManagerMock() {
         $entityManagerMock = $this
             ->getMockBuilder('Doctrine\ORM\EntityManager')
@@ -118,7 +121,7 @@ 
 
     /**
      * @param mixed $id
-     * @return Project|null
+     * @return null|\Developtech\AgilityBundle\Model\ProjectModel
      */
     public function getProjectMock($id) {
         if($id === 2) {

Unused Use Statements -3 removed lines

@@ -3,12 +3,9 @@
 namespace Developtech\AgilityBundle\Tests\Manager;
 
 use Developtech\AgilityBundle\Manager\ProjectManager;
-
 use Developtech\AgilityBundle\Entity\Project;
 use Developtech\AgilityBundle\Tests\Mock\User;
-
 use Developtech\AgilityBundle\Utils\Slugger;
-
 use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
 
 class ProjectManagerTest extends \PHPUnit_Framework_TestCase {

Manager/ProjectManager.php

Doc Comments +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
     protected $slugger;
 
     /**
-     * @param Doctrine\ORM\EntityManager $em
+     * @param EntityManager $em
      * @param Slugger $slugger
      */
     public function __construct(EntityManager $em, Slugger $slugger) {
@@ -51,7 +51,7 @@ 
      * @param string $name
 	 * @param string $description
      * @param UserInterface $productOwner
-     * @return \DevelopTech\AgilityBundle\ProjectModel
+     * @return \Developtech\AgilityBundle\Model\ProjectModel
      */
     public function createProject($name, $description, UserInterface $productOwner) {
         $project =

Indentation +4 added lines, -4 removed lines

@@ -49,7 +49,7 @@ 
 
     /**
      * @param string $name
-	 * @param string $description
+     * @param string $description
      * @param UserInterface $productOwner
      * @return \DevelopTech\AgilityBundle\ProjectModel
      */
@@ -58,7 +58,7 @@ 
             (new Project())
             ->setName($name)
             ->setSlug($this->slugger->slugify($name))
-			->setDescription($description)
+            ->setDescription($description)
             ->setProductOwner($productOwner)
         ;
 		
@@ -71,7 +71,7 @@ 
     /**
      * @param integer $id
      * @param string $name
-	 * @param string $description
+     * @param string $description
      * @param UserInterface $productOwner
      * @throws NotFoundHttpException
      * @return ProjectModel
@@ -83,7 +83,7 @@ 
         $project
             ->setName($name)
             ->setSlug($this->slugger->slugify($name))
-			->setDescription($description)
+            ->setDescription($description)
         ;
         if($productOwner !== null) {
             $project->setProductOwner($productOwner);

Spacing +2 added lines, -2 removed lines

@@ -41,7 +41,7 @@ 
      */
     public function getProject($slug) {
         $project = $this->em->getRepository(Project::class)->findOneBySlug($slug);
-        if($project === null) {
+        if ($project === null) {
             throw new NotFoundHttpException('Project not found');
         }
         return $project;
@@ -85,7 +85,7 @@ 
             ->setSlug($this->slugger->slugify($name))
 			->setDescription($description)
         ;
-        if($productOwner !== null) {
+        if ($productOwner !== null) {
             $project->setProductOwner($productOwner);
         }