nextcloud / polls

lib/Db/Model.php

Spacing +1 added lines, -1 removed lines

@@ -34,6 +34,6 @@
 	 * @param $value
 	 */
 	public function __set($name, $value) {
-		$this->setter($name, [$value]);
+		$this->setter($name, [ $value ]);
 	}
 }

Indentation +10 added lines, -10 removed lines

@@ -26,14 +26,14 @@
 use OCP\AppFramework\Db\Entity;
 
 abstract class Model extends Entity {
-	/**
-	 * FactoryMuffin checks for the existence of setters with method_exists($obj, $attr) but that returns false.
-	 * By overwriting the __set() magic method we can trigger the changed flag on $obj->attr assignment.
-	 *
-	 * @param $name
-	 * @param $value
-	 */
-	public function __set($name, $value) {
-		$this->setter($name, [$value]);
-	}
+    /**
+     * FactoryMuffin checks for the existence of setters with method_exists($obj, $attr) but that returns false.
+     * By overwriting the __set() magic method we can trigger the changed flag on $obj->attr assignment.
+     *
+     * @param $name
+     * @param $value
+     */
+    public function __set($name, $value) {
+        $this->setter($name, [$value]);
+    }
 }

tests/Unit/Factories/CommentFactory.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
  */
 $fm->define('OCA\Polls\Db\Comment')->setDefinitions([
 	'userId' => Faker::firstNameMale(),
-	'dt' => function () {
+	'dt' => function() {
 		$date = new DateTime('today');
 		return $date->format('Y-m-d H:i:s');
 	},

Indentation +6 added lines, -6 removed lines

@@ -27,10 +27,10 @@
  * General factory for the comment model.
  */
 $fm->define('OCA\Polls\Db\Comment')->setDefinitions([
-	'userId' => Faker::firstNameMale(),
-	'dt' => function () {
-		$date = new DateTime('today');
-		return $date->format('Y-m-d H:i:s');
-	},
-	'comment' => Faker::text(256)
+    'userId' => Faker::firstNameMale(),
+    'dt' => function () {
+        $date = new DateTime('today');
+        return $date->format('Y-m-d H:i:s');
+    },
+    'comment' => Faker::text(256)
 ]);

tests/Unit/Controller/PageControllerTest.php

Indentation +76 added lines, -76 removed lines

@@ -29,85 +29,85 @@
 
 class PageControllerTest extends UnitTestCase {
 
-	/** @var PageController */
-	private $controller;
+    /** @var PageController */
+    private $controller;
 
-	private $userId = 'john';
+    private $userId = 'john';
 
-	/**
-	 * {@inheritDoc}
-	 */
-	public function setUp() {
-		$request = $this->getMockBuilder('OCP\IRequest')
-			->disableOriginalConstructor()
-			->getMock();
-		$userManager = $this->getMockBuilder('OCP\IUserManager')
-			->disableOriginalConstructor()
-			->getMock();
-		$groupManager = $this->getMockBuilder('OCP\IGroupManager')
-			->disableOriginalConstructor()
-			->getMock();
-		$avatarManager = $this->getMockBuilder('OCP\IAvatarManager')
-			->disableOriginalConstructor()
-			->getMock();
-		$logger = $this->getMockBuilder('OCP\ILogger')
-			->disableOriginalConstructor()
-			->getMock();
-		$l10n = $this->getMockBuilder('OCP\IL10N')
-			->disableOriginalConstructor()
-			->getMock();
-		$urlGenerator = $this->getMockBuilder('OCP\IURLGenerator')
-			->disableOriginalConstructor()
-			->getMock();
-		$commentMapper = $this->getMockBuilder('OCA\Polls\Db\CommentMapper')
-			->disableOriginalConstructor()
-			->getMock();
-		$dateMapper = $this->getMockBuilder('OCA\Polls\Db\DateMapper')
-			->disableOriginalConstructor()
-			->getMock();
-		$eventMapper = $this->getMockBuilder('OCA\Polls\Db\EventMapper')
-			->disableOriginalConstructor()
-			->getMock();
-		$notificationMapper = $this->getMockBuilder('OCA\Polls\Db\NotificationMapper')
-			->disableOriginalConstructor()
-			->getMock();
-		$participationMapper = $this->getMockBuilder('OCA\Polls\Db\ParticipationMapper')
-			->disableOriginalConstructor()
-			->getMock();
-		$participationTextMapper = $this->getMockBuilder('OCA\Polls\Db\ParticipationTextMapper')
-			->disableOriginalConstructor()
-			->getMock();
-		$textMapper = $this->getMockBuilder('OCA\Polls\Db\TextMapper')
-			->disableOriginalConstructor()
-			->getMock();
+    /**
+     * {@inheritDoc}
+     */
+    public function setUp() {
+        $request = $this->getMockBuilder('OCP\IRequest')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $userManager = $this->getMockBuilder('OCP\IUserManager')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $groupManager = $this->getMockBuilder('OCP\IGroupManager')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $avatarManager = $this->getMockBuilder('OCP\IAvatarManager')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $logger = $this->getMockBuilder('OCP\ILogger')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $l10n = $this->getMockBuilder('OCP\IL10N')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $urlGenerator = $this->getMockBuilder('OCP\IURLGenerator')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $commentMapper = $this->getMockBuilder('OCA\Polls\Db\CommentMapper')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $dateMapper = $this->getMockBuilder('OCA\Polls\Db\DateMapper')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $eventMapper = $this->getMockBuilder('OCA\Polls\Db\EventMapper')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $notificationMapper = $this->getMockBuilder('OCA\Polls\Db\NotificationMapper')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $participationMapper = $this->getMockBuilder('OCA\Polls\Db\ParticipationMapper')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $participationTextMapper = $this->getMockBuilder('OCA\Polls\Db\ParticipationTextMapper')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $textMapper = $this->getMockBuilder('OCA\Polls\Db\TextMapper')
+            ->disableOriginalConstructor()
+            ->getMock();
 
-		$this->controller = new PageController(
-			'polls',
-			$request,
-			$userManager,
-			$groupManager,
-			$avatarManager,
-			$logger,
-			$l10n,
-			$urlGenerator,
-			$this->userId,
-			$commentMapper,
-			$dateMapper,
-			$eventMapper,
-			$notificationMapper,
-			$participationMapper,
-			$participationTextMapper,
-			$textMapper
-		);
-	}
+        $this->controller = new PageController(
+            'polls',
+            $request,
+            $userManager,
+            $groupManager,
+            $avatarManager,
+            $logger,
+            $l10n,
+            $urlGenerator,
+            $this->userId,
+            $commentMapper,
+            $dateMapper,
+            $eventMapper,
+            $notificationMapper,
+            $participationMapper,
+            $participationTextMapper,
+            $textMapper
+        );
+    }
 
-	/**
-	 * Basic controller index route test.
-	 */
-	public function testIndex() {
-		$result = $this->controller->index();
+    /**
+     * Basic controller index route test.
+     */
+    public function testIndex() {
+        $result = $this->controller->index();
 
-		$this->assertEquals('main.tmpl', $result->getTemplateName());
-		$this->assertInstanceOf(TemplateResponse::class, $result);
-	}
+        $this->assertEquals('main.tmpl', $result->getTemplateName());
+        $this->assertInstanceOf(TemplateResponse::class, $result);
+    }
 }

tests/Unit/UnitTestCase.php

Indentation +10 added lines, -10 removed lines

@@ -28,15 +28,15 @@
 
 abstract class UnitTestCase extends PHPUnit_Framework_TestCase {
 
-	/** @var FactoryMuffin */
-	protected $fm;
+    /** @var FactoryMuffin */
+    protected $fm;
 
-	/**
-	 * {@inheritDoc}
-	 */
-	public function setUp() {
-		parent::setUp();
-		$this->fm = new FactoryMuffin();
-		$this->fm->loadFactories(__DIR__ . '/Factories');
-	}
+    /**
+     * {@inheritDoc}
+     */
+    public function setUp() {
+        parent::setUp();
+        $this->fm = new FactoryMuffin();
+        $this->fm->loadFactories(__DIR__ . '/Factories');
+    }
 }

tests/Unit/Factories/ParticipationFactory.php

Indentation +6 added lines, -6 removed lines

@@ -27,10 +27,10 @@
  * General factory for the participation model.
  */
 $fm->define('OCA\Polls\Db\Participation')->setDefinitions([
-	'userId' => Faker::firstNameMale(),
-	'dt' => function () {
-		$date = new DateTime('today');
-		return $date->format('Y-m-d H:i:s');
-	},
-	'type' => 0
+    'userId' => Faker::firstNameMale(),
+    'dt' => function () {
+        $date = new DateTime('today');
+        return $date->format('Y-m-d H:i:s');
+    },
+    'type' => 0
 ]);

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
  */
 $fm->define('OCA\Polls\Db\Participation')->setDefinitions([
 	'userId' => Faker::firstNameMale(),
-	'dt' => function () {
+	'dt' => function() {
 		$date = new DateTime('today');
 		return $date->format('Y-m-d H:i:s');
 	},

tests/Unit/Factories/DateFactory.php

Indentation +4 added lines, -4 removed lines

@@ -25,8 +25,8 @@
  * General factory for the date model.
  */
 $fm->define('OCA\Polls\Db\Date')->setDefinitions([
-	'dt' => function () {
-		$date = new DateTime('today');
-		return $date->format('Y-m-d H:i:s');
-	}
+    'dt' => function () {
+        $date = new DateTime('today');
+        return $date->format('Y-m-d H:i:s');
+    }
 ]);

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
  * General factory for the date model.
  */
 $fm->define('OCA\Polls\Db\Date')->setDefinitions([
-	'dt' => function () {
+	'dt' => function() {
 		$date = new DateTime('today');
 		return $date->format('Y-m-d H:i:s');
 	}

tests/Unit/Db/CommentMapperTest.php

Indentation +54 added lines, -54 removed lines

@@ -33,65 +33,65 @@
 
 class CommentMapperTest extends UnitTestCase {
 
-	/** @var IDBConnection */
-	private $con;
-	/** @var CommentMapper */
-	private $commentMapper;
-	/** @var EventMapper */
-	private $eventMapper;
+    /** @var IDBConnection */
+    private $con;
+    /** @var CommentMapper */
+    private $commentMapper;
+    /** @var EventMapper */
+    private $eventMapper;
 
-	/**
-	 * {@inheritDoc}
-	 */
-	public function setUp() {
-		parent::setUp();
-		$this->con = \OC::$server->getDatabaseConnection();
-		$this->commentMapper = new CommentMapper($this->con);
-		$this->eventMapper = new EventMapper($this->con);
-	}
+    /**
+     * {@inheritDoc}
+     */
+    public function setUp() {
+        parent::setUp();
+        $this->con = \OC::$server->getDatabaseConnection();
+        $this->commentMapper = new CommentMapper($this->con);
+        $this->eventMapper = new EventMapper($this->con);
+    }
 
-	/**
-	 * Create some fake data and persist them to the database.
-	 *
-	 * @return Comment
-	 */
-	public function testCreate() {
-		/** @var Event $event */
-		$event = $this->fm->instance('OCA\Polls\Db\Event');
-		$this->assertInstanceOf(Event::class, $this->eventMapper->insert($event));
+    /**
+     * Create some fake data and persist them to the database.
+     *
+     * @return Comment
+     */
+    public function testCreate() {
+        /** @var Event $event */
+        $event = $this->fm->instance('OCA\Polls\Db\Event');
+        $this->assertInstanceOf(Event::class, $this->eventMapper->insert($event));
 
-		/** @var Comment $comment */
-		$comment = $this->fm->instance('OCA\Polls\Db\Comment');
-		$comment->setPollId($event->getId());
-		$this->assertInstanceOf(Comment::class, $this->commentMapper->insert($comment));
+        /** @var Comment $comment */
+        $comment = $this->fm->instance('OCA\Polls\Db\Comment');
+        $comment->setPollId($event->getId());
+        $this->assertInstanceOf(Comment::class, $this->commentMapper->insert($comment));
 
-		return $comment;
-	}
+        return $comment;
+    }
 
-	/**
-	 * Update the previously created entry and persist the changes.
-	 *
-	 * @depends testCreate
-	 * @param Comment $comment
-	 * @return Comment
-	 */
-	public function testUpdate(Comment $comment) {
-		$newComment = Faker::paragraph();
-		$comment->setComment($newComment());
-		$this->commentMapper->update($comment);
+    /**
+     * Update the previously created entry and persist the changes.
+     *
+     * @depends testCreate
+     * @param Comment $comment
+     * @return Comment
+     */
+    public function testUpdate(Comment $comment) {
+        $newComment = Faker::paragraph();
+        $comment->setComment($newComment());
+        $this->commentMapper->update($comment);
 
-		return $comment;
-	}
+        return $comment;
+    }
 
-	/**
-	 * Delete the previously created entries from the database.
-	 *
-	 * @depends testUpdate
-	 * @param Comment $comment
-	 */
-	public function testDelete(Comment $comment) {
-		$event = $this->eventMapper->find($comment->getPollId());
-		$this->commentMapper->delete($comment);
-		$this->eventMapper->delete($event);
-	}
+    /**
+     * Delete the previously created entries from the database.
+     *
+     * @depends testUpdate
+     * @param Comment $comment
+     */
+    public function testDelete(Comment $comment) {
+        $event = $this->eventMapper->find($comment->getPollId());
+        $this->commentMapper->delete($comment);
+        $this->eventMapper->delete($event);
+    }
 }

tests/Unit/Db/DateMapperTest.php

Indentation +54 added lines, -54 removed lines

@@ -33,65 +33,65 @@
 
 class DateMapperTest extends UnitTestCase {
 
-	/** @var IDBConnection */
-	private $con;
-	/** @var DateMapper */
-	private $dateMapper;
-	/** @var EventMapper */
-	private $eventMapper;
+    /** @var IDBConnection */
+    private $con;
+    /** @var DateMapper */
+    private $dateMapper;
+    /** @var EventMapper */
+    private $eventMapper;
 
-	/**
-	 * {@inheritDoc}
-	 */
-	public function setUp() {
-		parent::setUp();
-		$this->con = \OC::$server->getDatabaseConnection();
-		$this->dateMapper = new DateMapper($this->con);
-		$this->eventMapper = new EventMapper($this->con);
-	}
+    /**
+     * {@inheritDoc}
+     */
+    public function setUp() {
+        parent::setUp();
+        $this->con = \OC::$server->getDatabaseConnection();
+        $this->dateMapper = new DateMapper($this->con);
+        $this->eventMapper = new EventMapper($this->con);
+    }
 
-	/**
-	 * Create some fake data and persist them to the database.
-	 *
-	 * @return Date
-	 */
-	public function testCreate() {
-		/** @var Event $event */
-		$event = $this->fm->instance('OCA\Polls\Db\Event');
-		$this->assertInstanceOf(Event::class, $this->eventMapper->insert($event));
+    /**
+     * Create some fake data and persist them to the database.
+     *
+     * @return Date
+     */
+    public function testCreate() {
+        /** @var Event $event */
+        $event = $this->fm->instance('OCA\Polls\Db\Event');
+        $this->assertInstanceOf(Event::class, $this->eventMapper->insert($event));
 
-		/** @var Date $date */
-		$date = $this->fm->instance('OCA\Polls\Db\Date');
-		$date->setPollId($event->getId());
-		$this->assertInstanceOf(Date::class, $this->dateMapper->insert($date));
+        /** @var Date $date */
+        $date = $this->fm->instance('OCA\Polls\Db\Date');
+        $date->setPollId($event->getId());
+        $this->assertInstanceOf(Date::class, $this->dateMapper->insert($date));
 
-		return $date;
-	}
+        return $date;
+    }
 
-	/**
-	 * Update the previously created entry and persist the changes.
-	 *
-	 * @depends testCreate
-	 * @param Date $date
-	 * @return Date
-	 */
-	public function testUpdate(Date $date) {
-		$newDt = Faker::date('Y-m-d H:i:s');
-		$date->setDt($newDt());
-		$this->dateMapper->update($date);
+    /**
+     * Update the previously created entry and persist the changes.
+     *
+     * @depends testCreate
+     * @param Date $date
+     * @return Date
+     */
+    public function testUpdate(Date $date) {
+        $newDt = Faker::date('Y-m-d H:i:s');
+        $date->setDt($newDt());
+        $this->dateMapper->update($date);
 
-		return $date;
-	}
+        return $date;
+    }
 
-	/**
-	 * Delete the previously created entries from the database.
-	 *
-	 * @depends testUpdate
-	 * @param Date $date
-	 */
-	public function testDelete(Date $date) {
-		$event = $this->eventMapper->find($date->getPollId());
-		$this->dateMapper->delete($date);
-		$this->eventMapper->delete($event);
-	}
+    /**
+     * Delete the previously created entries from the database.
+     *
+     * @depends testUpdate
+     * @param Date $date
+     */
+    public function testDelete(Date $date) {
+        $event = $this->eventMapper->find($date->getPollId());
+        $this->dateMapper->delete($date);
+        $this->eventMapper->delete($event);
+    }
 }

tests/Unit/Db/NotificationMapperTest.php

Indentation +54 added lines, -54 removed lines

@@ -33,65 +33,65 @@
 
 class NotificationMapperTest extends UnitTestCase {
 
-	/** @var IDBConnection */
-	private $con;
-	/** @var NotificationMapper */
-	private $notificationMapper;
-	/** @var EventMapper */
-	private $eventMapper;
+    /** @var IDBConnection */
+    private $con;
+    /** @var NotificationMapper */
+    private $notificationMapper;
+    /** @var EventMapper */
+    private $eventMapper;
 
-	/**
-	 * {@inheritDoc}
-	 */
-	public function setUp() {
-		parent::setUp();
-		$this->con = \OC::$server->getDatabaseConnection();
-		$this->notificationMapper = new NotificationMapper($this->con);
-		$this->eventMapper = new EventMapper($this->con);
-	}
+    /**
+     * {@inheritDoc}
+     */
+    public function setUp() {
+        parent::setUp();
+        $this->con = \OC::$server->getDatabaseConnection();
+        $this->notificationMapper = new NotificationMapper($this->con);
+        $this->eventMapper = new EventMapper($this->con);
+    }
 
-	/**
-	 * Create some fake data and persist them to the database.
-	 *
-	 * @return Notification
-	 */
-	public function testCreate() {
-		/** @var Event $event */
-		$event = $this->fm->instance('OCA\Polls\Db\Event');
-		$this->assertInstanceOf(Event::class, $this->eventMapper->insert($event));
+    /**
+     * Create some fake data and persist them to the database.
+     *
+     * @return Notification
+     */
+    public function testCreate() {
+        /** @var Event $event */
+        $event = $this->fm->instance('OCA\Polls\Db\Event');
+        $this->assertInstanceOf(Event::class, $this->eventMapper->insert($event));
 
-		/** @var Notification $notification */
-		$notification = $this->fm->instance('OCA\Polls\Db\Notification');
-		$notification->setPollId($event->getId());
-		$this->assertInstanceOf(Notification::class, $this->notificationMapper->insert($notification));
+        /** @var Notification $notification */
+        $notification = $this->fm->instance('OCA\Polls\Db\Notification');
+        $notification->setPollId($event->getId());
+        $this->assertInstanceOf(Notification::class, $this->notificationMapper->insert($notification));
 
-		return $notification;
-	}
+        return $notification;
+    }
 
-	/**
-	 * Update the previously created entry and persist the changes.
-	 *
-	 * @depends testCreate
-	 * @param Notification $notification
-	 * @return Notification
-	 */
-	public function testUpdate(Notification $notification) {
-		$newUserId = Faker::firstNameMale();
-		$notification->setUserId($newUserId());
-		$this->notificationMapper->update($notification);
+    /**
+     * Update the previously created entry and persist the changes.
+     *
+     * @depends testCreate
+     * @param Notification $notification
+     * @return Notification
+     */
+    public function testUpdate(Notification $notification) {
+        $newUserId = Faker::firstNameMale();
+        $notification->setUserId($newUserId());
+        $this->notificationMapper->update($notification);
 
-		return $notification;
-	}
+        return $notification;
+    }
 
-	/**
-	 * Delete the previously created entries from the database.
-	 *
-	 * @depends testUpdate
-	 * @param Notification $notification
-	 */
-	public function testDelete(Notification $notification) {
-		$event = $this->eventMapper->find($notification->getPollId());
-		$this->notificationMapper->delete($notification);
-		$this->eventMapper->delete($event);
-	}
+    /**
+     * Delete the previously created entries from the database.
+     *
+     * @depends testUpdate
+     * @param Notification $notification
+     */
+    public function testDelete(Notification $notification) {
+        $event = $this->eventMapper->find($notification->getPollId());
+        $this->notificationMapper->delete($notification);
+        $this->eventMapper->delete($event);
+    }
 }