testmonitor / devops-client

src/Actions/ManagesAccounts.php

<?php

namespace TestMonitor\DevOps\Actions;

use TestMonitor\DevOps\Transforms\TransformsAccounts;

trait ManagesAccounts
{
    use TransformsAccounts;

    /**
     * Get a list of of accounts.
     *
     * @throws \TestMonitor\DevOps\Exceptions\InvalidDataException
     *
     * @return \TestMonitor\DevOps\Resources\Account[]
     */
    public function accounts()
    {
        // First, establish your profile ID
        $user = $this->myself();

        // Second, retrieve for the accounts for this member
        $accounts = $this->request(

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

            'GET',
            'https://app.vssps.visualstudio.com/_apis/accounts',
            ['query' => ['memberId' => $user->id]]
        );

        if ($accounts === '[]') {
            return [];
        }

        return $this->fromDevOpsAccounts($accounts);
    }

    /**
     * Returns the profile of the authenticated user.
     *
     * @return \TestMonitor\DevOps\Resources\Profile
     */
    public function myself()
    {
        $response = $this->get('https://app.vssps.visualstudio.com/_apis/profile/profiles/me');

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

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