testmonitor / devops-client

src/Actions/ManagesProjects.php

<?php

namespace TestMonitor\DevOps\Actions;

use TestMonitor\DevOps\Resources\Project;
use TestMonitor\DevOps\Transforms\TransformsProjects;

trait ManagesProjects
{
    use TransformsProjects;

    /**
     * Get a single project.
     *
     * @param string $id
     *
     * @throws \TestMonitor\DevOps\Exceptions\InvalidDataException
     *
     * @return \TestMonitor\DevOps\Resources\Project
     */
    public function project(string $id): Project
    {
        $response = $this->get("_apis/projects/{$id}");

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

        return $this->fromDevOpsProject($response);
    }

    /**
     * Get a list of of projects.
     *
     * @throws \TestMonitor\DevOps\Exceptions\InvalidDataException
     *
     * @return \TestMonitor\DevOps\Resources\Project[]
     */
    public function projects()
    {
        $response = $this->get('_apis/projects');

        return $this->fromDevOpsProjects($response['value']);
    }
}