TestMonitor\DoneDone\Actions\ManagesTasks

Complexity

Total Complexity

3

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	task()	0	5	1
A	createTask()	0	7	1
A	tasks()	0	7	1

<?php

namespace TestMonitor\DoneDone\Actions;

use TestMonitor\DoneDone\Resources\Task;
use TestMonitor\DoneDone\Transforms\TransformsTasks;

trait ManagesTasks
{
    use TransformsTasks;

    /**
     * Get a list of of tasks.
     *
     * @param int $accountId
     * @param int $projectId
     * @param int $page
     *
     * @return Task[]
     */
    public function tasks(int $accountId, int $projectId, int $page = 1): array
    {
        $result = $this->get("{$accountId}/internal-projects/{$projectId}/tasks", ['page' => $page]);

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return array_map(function ($task) {
            return $this->fromDoneDoneTask($task);
        }, $result['listTasks']);
    }

    /**
     * Get a single task.
     *
     * @param int $id
     * @param int $accountId
     * @param int $projectId
     *
     * @return Task
     */
    public function task(int $id, int $accountId, int $projectId): Task
    {
        $result = $this->get("{$accountId}/internal-projects/{$projectId}/tasks/{$id}");

        return $this->fromDoneDoneTask($result);
    }

    /**
     * Create a new task.
     *
     * @param \TestMonitor\DoneDone\Resources\Task $task
     * @param int $accountId
     * @param int $projectId
     *
     * @return Task
     */
    public function createTask(Task $task, int $accountId, int $projectId)
    {
        $result = $this->post("{$accountId}/internal-projects/{$projectId}/tasks/", [

It seems like post() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            'json' => $this->toDoneDoneTask($task),
        ]);

        return $this->task($result['id'], $accountId, $projectId);
    }
}