123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340 |
- import {Team} from 'sentry-fixture/team';
- import {initializeOrg} from 'sentry-test/initializeOrg';
- import {
- render,
- renderGlobalModal,
- screen,
- userEvent,
- waitFor,
- } from 'sentry-test/reactTestingLibrary';
- import TeamStore from 'sentry/stores/teamStore';
- import type {Organization, Project} from 'sentry/types';
- import ProjectTeams from 'sentry/views/settings/project/projectTeams';
- describe('ProjectTeams', function () {
- let org: Organization;
- let project: Project;
- let routerContext: Record<string, any>;
- const team1WithAdmin = Team({
- access: ['team:read', 'team:write', 'team:admin'],
- });
- const team2WithAdmin = Team({
- id: '2',
- slug: 'team-slug-2',
- name: 'Team Name 2',
- hasAccess: true,
- access: ['team:read', 'team:write', 'team:admin'],
- });
- const team3NoAdmin = Team({
- id: '3',
- slug: 'team-slug-3',
- name: 'Team Name 3',
- hasAccess: true,
- access: ['team:read'],
- });
- beforeEach(function () {
- const initialData = initializeOrg();
- org = initialData.organization;
- project = {
- ...initialData.project,
- access: ['project:admin', 'project:write', 'project:admin'],
- };
- routerContext = initialData.routerContext;
- TeamStore.loadInitialData([team1WithAdmin, team2WithAdmin]);
- MockApiClient.addMockResponse({
- url: `/projects/${org.slug}/${project.slug}/`,
- method: 'GET',
- body: project,
- });
- MockApiClient.addMockResponse({
- url: `/projects/${org.slug}/${project.slug}/teams/`,
- method: 'GET',
- body: [team1WithAdmin],
- });
- MockApiClient.addMockResponse({
- url: `/organizations/${org.slug}/teams/`,
- method: 'GET',
- body: [team1WithAdmin, team2WithAdmin],
- });
- });
- afterEach(function () {
- MockApiClient.clearMockResponses();
- });
- it('renders', function () {
- render(
- <ProjectTeams
- {...TestStubs.routeComponentProps()}
- params={{projectId: project.slug}}
- organization={org}
- project={project}
- />
- );
- });
- it('can remove a team from project', async function () {
- MockApiClient.addMockResponse({
- url: `/projects/${org.slug}/${project.slug}/teams/`,
- method: 'GET',
- body: [team1WithAdmin, team2WithAdmin],
- });
- const endpoint1 = `/projects/${org.slug}/${project.slug}/teams/${team1WithAdmin.slug}/`;
- const mock1 = MockApiClient.addMockResponse({
- url: endpoint1,
- method: 'DELETE',
- statusCode: 200,
- });
- const endpoint2 = `/projects/${org.slug}/${project.slug}/teams/${team2WithAdmin.slug}/`;
- const mock2 = MockApiClient.addMockResponse({
- url: endpoint2,
- method: 'DELETE',
- statusCode: 200,
- });
- render(
- <ProjectTeams
- {...TestStubs.routeComponentProps()}
- params={{projectId: project.slug}}
- organization={org}
- project={project}
- />
- );
- expect(mock1).not.toHaveBeenCalled();
- await userEvent.click(screen.getAllByRole('button', {name: 'Remove'})[0]);
- expect(mock1).toHaveBeenCalledWith(
- endpoint1,
- expect.objectContaining({
- method: 'DELETE',
- })
- );
- expect(screen.queryByText('#team-slug')).not.toBeInTheDocument();
- // Remove second team
- await userEvent.click(screen.getAllByRole('button', {name: 'Remove'})[0]);
- // Modal opens because this is the last team in project
- renderGlobalModal();
- expect(screen.getByRole('dialog')).toBeInTheDocument();
- await userEvent.click(screen.getByTestId('confirm-button'));
- expect(mock2).toHaveBeenCalledWith(
- endpoint2,
- expect.objectContaining({
- method: 'DELETE',
- })
- );
- });
- it('cannot remove a team without admin scopes', async function () {
- MockApiClient.addMockResponse({
- url: `/projects/${org.slug}/${project.slug}/teams/`,
- method: 'GET',
- body: [team1WithAdmin, team2WithAdmin, team3NoAdmin],
- });
- const endpoint1 = `/projects/${org.slug}/${project.slug}/teams/${team1WithAdmin.slug}/`;
- const mock1 = MockApiClient.addMockResponse({
- url: endpoint1,
- method: 'DELETE',
- statusCode: 200,
- });
- const endpoint3 = `/projects/${org.slug}/${project.slug}/teams/${team3NoAdmin.slug}/`;
- const mock3 = MockApiClient.addMockResponse({
- url: endpoint3,
- method: 'DELETE',
- statusCode: 200,
- });
- render(
- <ProjectTeams
- {...TestStubs.routeComponentProps()}
- params={{projectId: project.slug}}
- organization={org}
- project={project}
- />
- );
- // Remove first team
- await userEvent.click(screen.getAllByRole('button', {name: 'Remove'})[0]);
- expect(mock1).toHaveBeenCalledWith(
- endpoint1,
- expect.objectContaining({
- method: 'DELETE',
- })
- );
- expect(screen.queryByText('#team-slug')).not.toBeInTheDocument();
- // Remove third team, but button should be disabled
- await userEvent.click(screen.getAllByRole('button', {name: 'Remove'})[1]);
- expect(mock3).not.toHaveBeenCalled();
- });
- it('removes team from project when project team is not in org list', async function () {
- MockApiClient.addMockResponse({
- url: `/projects/${org.slug}/${project.slug}/teams/`,
- method: 'GET',
- body: [team1WithAdmin, team2WithAdmin],
- });
- const endpoint1 = `/projects/${org.slug}/${project.slug}/teams/${team1WithAdmin.slug}/`;
- const mock1 = MockApiClient.addMockResponse({
- url: endpoint1,
- method: 'DELETE',
- });
- const endpoint2 = `/projects/${org.slug}/${project.slug}/teams/${team2WithAdmin.slug}/`;
- const mock2 = MockApiClient.addMockResponse({
- url: endpoint2,
- method: 'DELETE',
- });
- MockApiClient.addMockResponse({
- url: `/organizations/${org.slug}/teams/`,
- method: 'GET',
- body: [team3NoAdmin],
- });
- render(
- <ProjectTeams
- {...TestStubs.routeComponentProps()}
- params={{projectId: project.slug}}
- organization={org}
- project={project}
- />
- );
- expect(mock1).not.toHaveBeenCalled();
- // Remove first team
- await userEvent.click(screen.getAllByRole('button', {name: 'Remove'})[0]);
- expect(mock1).toHaveBeenCalledWith(
- endpoint1,
- expect.objectContaining({
- method: 'DELETE',
- })
- );
- expect(screen.queryByText('#team-slug')).not.toBeInTheDocument();
- // Remove second team
- await userEvent.click(screen.getAllByRole('button', {name: 'Remove'})[0]);
- // Modal opens because this is the last team in project
- renderGlobalModal();
- expect(screen.getByRole('dialog')).toBeInTheDocument();
- // Click confirm
- await userEvent.click(screen.getByTestId('confirm-button'));
- expect(mock2).toHaveBeenCalledWith(
- endpoint2,
- expect.objectContaining({
- method: 'DELETE',
- })
- );
- });
- it('can associate a team with project', async function () {
- const endpoint = `/projects/${org.slug}/${project.slug}/teams/${team2WithAdmin.slug}/`;
- const mock = MockApiClient.addMockResponse({
- url: endpoint,
- method: 'POST',
- statusCode: 200,
- });
- render(
- <ProjectTeams
- {...TestStubs.routeComponentProps()}
- params={{projectId: project.slug}}
- organization={org}
- project={project}
- />
- );
- expect(mock).not.toHaveBeenCalled();
- // Add a team
- await userEvent.click(screen.getAllByRole('button', {name: 'Add Team'})[1]);
- await userEvent.click(screen.getByText('#team-slug-2'));
- expect(mock).toHaveBeenCalledWith(
- endpoint,
- expect.objectContaining({
- method: 'POST',
- })
- );
- });
- it('creates a new team adds it to current project using the "create team modal" in dropdown', async function () {
- MockApiClient.addMockResponse({
- url: '/internal/health/',
- body: {},
- });
- MockApiClient.addMockResponse({
- url: '/assistant/',
- body: {},
- });
- MockApiClient.addMockResponse({
- url: '/organizations/',
- body: [org],
- });
- const addTeamToProject = MockApiClient.addMockResponse({
- url: `/projects/${org.slug}/${project.slug}/teams/new-team/`,
- method: 'POST',
- });
- const createTeam = MockApiClient.addMockResponse({
- url: `/organizations/${org.slug}/teams/`,
- method: 'POST',
- body: {slug: 'new-team'},
- });
- render(
- <ProjectTeams
- {...TestStubs.routeComponentProps()}
- params={{projectId: project.slug}}
- project={project}
- organization={org}
- />,
- {context: routerContext}
- );
- // Add new team
- await userEvent.click(screen.getAllByRole('button', {name: 'Add Team'})[1]);
- // XXX(epurkhiser): Create Team should really be a button
- await userEvent.click(screen.getByRole('link', {name: 'Create Team'}));
- renderGlobalModal();
- await screen.findByRole('dialog');
- await userEvent.type(screen.getByRole('textbox', {name: 'Team Name'}), 'new-team');
- await userEvent.click(screen.getByRole('button', {name: 'Create Team'}));
- await waitFor(() => expect(createTeam).toHaveBeenCalledTimes(1));
- expect(createTeam).toHaveBeenCalledWith(
- '/organizations/org-slug/teams/',
- expect.objectContaining({
- data: {slug: 'new-team'},
- })
- );
- expect(addTeamToProject).toHaveBeenCalledTimes(1);
- expect(addTeamToProject).toHaveBeenCalledWith(
- '/projects/org-slug/project-slug/teams/new-team/',
- expect.anything()
- );
- });
- });
|