chore: remove ts-ignore and adapt tests

[gastonfournier]

About the changes

Just using proper types and adapting a few tests

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
unleash-monorepo-frontend	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 19, 2023 2:18pm

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
unleash-docs	⬜️ Ignored (Inspect)			Oct 19, 2023 2:18pm

[gastonfournier]

FYI @chriswk as you're the last to edit this test, it was 2 because when creating a project with an invalid user (one without an id), the code will just ignore adding the user as owner without failing:

unleash/src/lib/services/access-service.ts

Lines 561 to 567 in 1335da6

	// TODO: remove this when all users is guaranteed to have a unique id.
	if (owner.id) {
	this.logger.info(
	`Making ${owner.id} admin of ${projectId} via roleId=${ownerRole.id}`,
	);
	await this.store.addUserToRole(owner.id, ownerRole.id, projectId);
	}

[nunogois]

LGTM!