From c58478c0abfb418048d2ee9c6297a30793f10f80 Mon Sep 17 00:00:00 2001
From: Frederic Guillot <fred@kanboard.net>
Date: Fri, 15 Jan 2016 21:03:19 -0500
Subject: Move analytic logic to separate classes

---
 .../EstimatedTimeComparisonAnalyticTest.php        | 99 ++++++++++++++++++++++
 .../Analytic/TaskDistributionAnalyticTest.php      | 62 ++++++++++++++
 .../Analytic/UserDistributionAnalyticTest.php      | 83 ++++++++++++++++++
 3 files changed, 244 insertions(+)
 create mode 100644 tests/units/Analytic/EstimatedTimeComparisonAnalyticTest.php
 create mode 100644 tests/units/Analytic/TaskDistributionAnalyticTest.php
 create mode 100644 tests/units/Analytic/UserDistributionAnalyticTest.php

(limited to 'tests/units/Analytic')

diff --git a/tests/units/Analytic/EstimatedTimeComparisonAnalyticTest.php b/tests/units/Analytic/EstimatedTimeComparisonAnalyticTest.php
new file mode 100644
index 00000000..2ca631b1
--- /dev/null
+++ b/tests/units/Analytic/EstimatedTimeComparisonAnalyticTest.php
@@ -0,0 +1,99 @@
+<?php
+
+require_once __DIR__.'/../Base.php';
+
+use Kanboard\Model\TaskCreation;
+use Kanboard\Model\Project;
+use Kanboard\Analytic\EstimatedTimeComparisonAnalytic;
+
+class EstimatedTimeComparisonAnalyticTest extends Base
+{
+    public function testBuild()
+    {
+        $taskCreationModel = new TaskCreation($this->container);
+        $projectModel = new Project($this->container);
+        $estimatedTimeComparisonAnalytic = new EstimatedTimeComparisonAnalytic($this->container);
+
+        $this->assertEquals(1, $projectModel->create(array('name' => 'test1')));
+        $this->assertEquals(2, $projectModel->create(array('name' => 'test1')));
+
+        $this->assertNotFalse($taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'time_estimated' => 5.5)));
+        $this->assertNotFalse($taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'time_estimated' => 1.75)));
+        $this->assertNotFalse($taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'time_estimated' => 1.25, 'is_active' => 0)));
+
+        $this->assertNotFalse($taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'time_spent' => 8.25)));
+        $this->assertNotFalse($taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'time_spent' => 0.25)));
+        $this->assertNotFalse($taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'time_spent' => 0.5, 'is_active' => 0)));
+
+        $expected = array(
+            'open' => array(
+                'time_spent' => 8.5,
+                'time_estimated' => 7.25,
+            ),
+            'closed' => array(
+                'time_spent' => 0.5,
+                'time_estimated' => 1.25,
+            )
+        );
+
+        $this->assertEquals($expected, $estimatedTimeComparisonAnalytic->build(1));
+    }
+
+    public function testBuildWithNoClosedTask()
+    {
+        $taskCreationModel = new TaskCreation($this->container);
+        $projectModel = new Project($this->container);
+        $estimatedTimeComparisonAnalytic = new EstimatedTimeComparisonAnalytic($this->container);
+
+        $this->assertEquals(1, $projectModel->create(array('name' => 'test1')));
+        $this->assertEquals(2, $projectModel->create(array('name' => 'test1')));
+
+        $this->assertNotFalse($taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'time_estimated' => 5.5)));
+        $this->assertNotFalse($taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'time_estimated' => 1.75)));
+
+        $this->assertNotFalse($taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'time_spent' => 8.25)));
+        $this->assertNotFalse($taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'time_spent' => 0.25)));
+
+        $expected = array(
+            'open' => array(
+                'time_spent' => 8.5,
+                'time_estimated' => 7.25,
+            ),
+            'closed' => array(
+                'time_spent' => 0,
+                'time_estimated' => 0,
+            )
+        );
+
+        $this->assertEquals($expected, $estimatedTimeComparisonAnalytic->build(1));
+    }
+
+    public function testBuildWithOnlyClosedTask()
+    {
+        $taskCreationModel = new TaskCreation($this->container);
+        $projectModel = new Project($this->container);
+        $estimatedTimeComparisonAnalytic = new EstimatedTimeComparisonAnalytic($this->container);
+
+        $this->assertEquals(1, $projectModel->create(array('name' => 'test1')));
+        $this->assertEquals(2, $projectModel->create(array('name' => 'test1')));
+
+        $this->assertNotFalse($taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'time_estimated' => 5.5, 'is_active' => 0)));
+        $this->assertNotFalse($taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'time_estimated' => 1.75, 'is_active' => 0)));
+
+        $this->assertNotFalse($taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'time_spent' => 8.25, 'is_active' => 0)));
+        $this->assertNotFalse($taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'time_spent' => 0.25, 'is_active' => 0)));
+
+        $expected = array(
+            'closed' => array(
+                'time_spent' => 8.5,
+                'time_estimated' => 7.25,
+            ),
+            'open' => array(
+                'time_spent' => 0,
+                'time_estimated' => 0,
+            )
+        );
+
+        $this->assertEquals($expected, $estimatedTimeComparisonAnalytic->build(1));
+    }
+}
diff --git a/tests/units/Analytic/TaskDistributionAnalyticTest.php b/tests/units/Analytic/TaskDistributionAnalyticTest.php
new file mode 100644
index 00000000..531101ef
--- /dev/null
+++ b/tests/units/Analytic/TaskDistributionAnalyticTest.php
@@ -0,0 +1,62 @@
+<?php
+
+require_once __DIR__.'/../Base.php';
+
+use Kanboard\Model\TaskCreation;
+use Kanboard\Model\Project;
+use Kanboard\Analytic\TaskDistributionAnalytic;
+
+class TaskDistributionAnalyticTest extends Base
+{
+    public function testBuild()
+    {
+        $projectModel = new Project($this->container);
+        $taskDistributionModel = new TaskDistributionAnalytic($this->container);
+
+        $this->assertEquals(1, $projectModel->create(array('name' => 'test1')));
+        $this->assertEquals(2, $projectModel->create(array('name' => 'test1')));
+
+        $this->createTasks(1, 20, 1);
+        $this->createTasks(2, 30, 1);
+        $this->createTasks(3, 40, 1);
+        $this->createTasks(4, 10, 1);
+
+        $expected = array(
+            array(
+                'column_title' => 'Backlog',
+                'nb_tasks' => 20,
+                'percentage' => 20.0,
+            ),
+            array(
+                'column_title' => 'Ready',
+                'nb_tasks' => 30,
+                'percentage' => 30.0,
+            ),
+            array(
+                'column_title' => 'Work in progress',
+                'nb_tasks' => 40,
+                'percentage' => 40.0,
+            ),
+            array(
+                'column_title' => 'Done',
+                'nb_tasks' => 10,
+                'percentage' => 10.0,
+            )
+        );
+
+        $this->assertEquals($expected, $taskDistributionModel->build(1));
+    }
+
+    private function createTasks($column_id, $nb_active, $nb_inactive)
+    {
+        $taskCreationModel = new TaskCreation($this->container);
+
+        for ($i = 0; $i < $nb_active; $i++) {
+            $this->assertNotFalse($taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'column_id' => $column_id, 'is_active' => 1)));
+        }
+
+        for ($i = 0; $i < $nb_inactive; $i++) {
+            $this->assertNotFalse($taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'column_id' => $column_id, 'is_active' => 0)));
+        }
+    }
+}
diff --git a/tests/units/Analytic/UserDistributionAnalyticTest.php b/tests/units/Analytic/UserDistributionAnalyticTest.php
new file mode 100644
index 00000000..bdc136e1
--- /dev/null
+++ b/tests/units/Analytic/UserDistributionAnalyticTest.php
@@ -0,0 +1,83 @@
+<?php
+
+require_once __DIR__.'/../Base.php';
+
+use Kanboard\Model\TaskCreation;
+use Kanboard\Model\Project;
+use Kanboard\Model\ProjectUserRole;
+use Kanboard\Model\User;
+use Kanboard\Analytic\UserDistributionAnalytic;
+use Kanboard\Core\Security\Role;
+
+class UserDistributionAnalyticTest extends Base
+{
+    public function testBuild()
+    {
+        $projectModel = new Project($this->container);
+        $projectUserRoleModel = new ProjectUserRole($this->container);
+        $userModel = new User($this->container);
+        $userDistributionModel = new UserDistributionAnalytic($this->container);
+
+        $this->assertEquals(1, $projectModel->create(array('name' => 'test1')));
+        $this->assertEquals(2, $projectModel->create(array('name' => 'test1')));
+
+        $this->assertEquals(2, $userModel->create(array('username' => 'user1')));
+        $this->assertEquals(3, $userModel->create(array('username' => 'user2')));
+        $this->assertEquals(4, $userModel->create(array('username' => 'user3')));
+        $this->assertEquals(5, $userModel->create(array('username' => 'user4')));
+
+        $this->assertTrue($projectUserRoleModel->addUser(1, 2, Role::PROJECT_MEMBER));
+        $this->assertTrue($projectUserRoleModel->addUser(1, 3, Role::PROJECT_MEMBER));
+        $this->assertTrue($projectUserRoleModel->addUser(1, 4, Role::PROJECT_MEMBER));
+        $this->assertTrue($projectUserRoleModel->addUser(1, 5, Role::PROJECT_MEMBER));
+
+        $this->createTasks(0, 10, 1);
+        $this->createTasks(2, 30, 1);
+        $this->createTasks(3, 40, 1);
+        $this->createTasks(4, 10, 1);
+        $this->createTasks(5, 10, 1);
+
+        $expected = array(
+            array(
+                'user' => 'Unassigned',
+                'nb_tasks' => 10,
+                'percentage' => 10.0,
+            ),
+            array(
+                'user' => 'user1',
+                'nb_tasks' => 30,
+                'percentage' => 30.0,
+            ),
+            array(
+                'user' => 'user2',
+                'nb_tasks' => 40,
+                'percentage' => 40.0,
+            ),
+            array(
+                'user' => 'user3',
+                'nb_tasks' => 10,
+                'percentage' => 10.0,
+            ),
+            array(
+                'user' => 'user4',
+                'nb_tasks' => 10,
+                'percentage' => 10.0,
+            )
+        );
+
+        $this->assertEquals($expected, $userDistributionModel->build(1));
+    }
+
+    private function createTasks($user_id, $nb_active, $nb_inactive)
+    {
+        $taskCreationModel = new TaskCreation($this->container);
+
+        for ($i = 0; $i < $nb_active; $i++) {
+            $this->assertNotFalse($taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'owner_id' => $user_id, 'is_active' => 1)));
+        }
+
+        for ($i = 0; $i < $nb_inactive; $i++) {
+            $this->assertNotFalse($taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'owner_id' => $user_id, 'is_active' => 0)));
+        }
+    }
+}
-- 
cgit v1.2.3