Add methods for the team model

[JSReds]

Change-type: minor

Resolves: #
HQ:
See:
Depends-on:
Change-type: major|minor|patch

---

Contributor checklist

• Includes tests
• Includes typings
• Includes updated documentation
• Includes updated build output

[JSReds]

TODO:

Add getAll and assignToFleet methods

[thgreasi]

I would be fine if we always did a GET request to confirm that the org exists and give a more helpful error in case it doesn't.
The isId in most cases is an optimization for methods that the UI uses in batch actions.

[thgreasi]

How about using a $filter in team.getAllByOrganization(orgId); so that we don't have to fetch all teams of the org?
FWIW Given that the API does block this, I would be fine even if we didn't have this check.

[thgreasi]

Suggested change

	: { handle: organizationSlugOrId },
	: {
	$any: {
	$alias: 'bto',
	$expr: {
	bto: { handle: organizationSlugOrId },
	},
	},
	},

[thgreasi]

Suggested change

	throw new errors.BalenaInvalidParameterError('id', teamId);
	throw new errors.BalenaInvalidParameterError('teamId', teamId);

[thgreasi]

• Mark it as async
• then add a throw in case organizationSlugOrId is unexpected

[thgreasi]

Let's avoid the JOIN :)

Suggested change

	const team = await get(teamId, {
	$select: 'id',
	$expand: { belongs_to__organization: { $select: 'id' } },
	});
	const team = await get(teamId, {
	$select: ['id', 'belongs_to__organization'],
	});

[thgreasi]

log 🚓

[thgreasi]

If we do

		const teamOptions = {
			$select: ['id', 'belongs_to__organization'],
		} satisfies PineOptions<Team>;
		const team = (await get(teamId, teamOptions)) as PineTypedResult<Team, typeof teamOptions>;

then this will be fully typed and we should be able to do just:

Suggested change

	const orgId =
	Array.isArray(team.belongs_to__organization) &&
	team.belongs_to__organization[0] &&
	'id' in team.belongs_to__organization[0]
	? team.belongs_to__organization[0].id
	: undefined;
	const orgId = team.belongs_to__organization.__id;

[thgreasi]

You can probably also expand the teams w/ a $filter in the original pine query so that we do just 1 request.

[thgreasi]

afaik the API will just return 200 when the team (or any OData DELETE to a resource that) doesn't exist, so let's remove the try catch

Suggested change

	if (isNotFoundResponse(err)) {
	throw new Error(`Team not found: ${teamId}`);
	}