organizationMembersList.spec.tsx 20 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696
  1. import {browserHistory} from 'react-router';
  2. import selectEvent from 'react-select-event';
  3. import {
  4. render,
  5. renderGlobalModal,
  6. screen,
  7. userEvent,
  8. waitFor,
  9. within,
  10. } from 'sentry-test/reactTestingLibrary';
  11. import {addErrorMessage, addSuccessMessage} from 'sentry/actionCreators/indicator';
  12. import ConfigStore from 'sentry/stores/configStore';
  13. import OrganizationsStore from 'sentry/stores/organizationsStore';
  14. import {trackAnalytics} from 'sentry/utils/analytics';
  15. import OrganizationMembersList from 'sentry/views/settings/organizationMembers/organizationMembersList';
  16. jest.mock('sentry/utils/analytics');
  17. jest.mock('sentry/api');
  18. jest.mock('sentry/actionCreators/indicator');
  19. const roles = [
  20. {
  21. id: 'admin',
  22. name: 'Admin',
  23. desc: 'This is the admin role',
  24. allowed: true,
  25. },
  26. {
  27. id: 'member',
  28. name: 'Member',
  29. desc: 'This is the member role',
  30. allowed: true,
  31. },
  32. {
  33. id: 'owner',
  34. name: 'Owner',
  35. desc: 'This is the owner role',
  36. allowed: true,
  37. },
  38. ];
  39. const missingMembers = [
  40. {
  41. integration: 'github',
  42. users: [
  43. {
  44. commitCount: 6,
  45. email: 'hello@sentry.io',
  46. externalId: 'hello',
  47. },
  48. {
  49. commitCount: 5,
  50. email: 'abcd@sentry.io',
  51. externalId: 'abcd',
  52. },
  53. {
  54. commitCount: 4,
  55. email: 'hola@sentry.io',
  56. externalId: 'hola',
  57. },
  58. {
  59. commitCount: 3,
  60. email: 'test@sentry.io',
  61. externalId: 'test',
  62. },
  63. {
  64. commitCount: 2,
  65. email: 'five@sentry.io',
  66. externalId: 'five',
  67. },
  68. ],
  69. },
  70. ];
  71. describe('OrganizationMembersList', function () {
  72. const members = TestStubs.Members();
  73. const ownerTeam = TestStubs.Team({slug: 'owner-team', orgRole: 'owner'});
  74. const member = TestStubs.Member({
  75. id: '5',
  76. email: 'member@sentry.io',
  77. teams: [ownerTeam.slug],
  78. teamRoles: [
  79. {
  80. teamSlug: ownerTeam.slug,
  81. role: null,
  82. },
  83. ],
  84. flags: {
  85. 'sso:linked': true,
  86. },
  87. groupOrgRoles: [
  88. {
  89. teamSlug: ownerTeam.slug,
  90. role: {id: 'owner'},
  91. },
  92. ],
  93. });
  94. const currentUser = members[1];
  95. const organization = TestStubs.Organization({
  96. access: ['member:admin', 'org:admin', 'member:write'],
  97. status: {
  98. id: 'active',
  99. },
  100. });
  101. const router = TestStubs.router();
  102. const defaultProps = {
  103. organization,
  104. router,
  105. location: router.location,
  106. routes: router.routes,
  107. route: router.routes[0],
  108. params: router.params,
  109. routeParams: router.params,
  110. };
  111. jest.spyOn(ConfigStore, 'get').mockImplementation(() => currentUser);
  112. afterAll(function () {
  113. (ConfigStore.get as jest.Mock).mockRestore();
  114. });
  115. beforeEach(function () {
  116. MockApiClient.clearMockResponses();
  117. MockApiClient.addMockResponse({
  118. url: '/organizations/org-slug/members/me/',
  119. method: 'GET',
  120. body: {roles},
  121. });
  122. MockApiClient.addMockResponse({
  123. url: '/organizations/org-slug/members/',
  124. method: 'GET',
  125. body: [...TestStubs.Members(), member],
  126. });
  127. MockApiClient.addMockResponse({
  128. url: `/organizations/org-slug/members/${member.id}/`,
  129. body: member,
  130. });
  131. MockApiClient.addMockResponse({
  132. url: '/organizations/org-slug/access-requests/',
  133. method: 'GET',
  134. body: [
  135. {
  136. id: 'pending-id',
  137. member: {
  138. id: 'pending-member-id',
  139. email: '',
  140. name: '',
  141. role: '',
  142. roleName: '',
  143. user: {
  144. id: '',
  145. name: 'sentry@test.com',
  146. },
  147. },
  148. team: TestStubs.Team(),
  149. },
  150. ],
  151. });
  152. MockApiClient.addMockResponse({
  153. url: '/organizations/org-slug/auth-provider/',
  154. method: 'GET',
  155. body: {
  156. ...TestStubs.AuthProvider(),
  157. require_link: true,
  158. },
  159. });
  160. MockApiClient.addMockResponse({
  161. url: '/organizations/org-slug/teams/',
  162. method: 'GET',
  163. body: [TestStubs.Team(), ownerTeam],
  164. });
  165. MockApiClient.addMockResponse({
  166. url: '/organizations/org-slug/invite-requests/',
  167. method: 'GET',
  168. body: [],
  169. });
  170. MockApiClient.addMockResponse({
  171. url: '/organizations/org-slug/missing-members/',
  172. method: 'GET',
  173. body: [],
  174. });
  175. (browserHistory.push as jest.Mock).mockReset();
  176. OrganizationsStore.load([organization]);
  177. });
  178. it('can remove a member', async function () {
  179. const deleteMock = MockApiClient.addMockResponse({
  180. url: `/organizations/org-slug/members/${members[0].id}/`,
  181. method: 'DELETE',
  182. });
  183. render(<OrganizationMembersList {...defaultProps} />, {
  184. context: TestStubs.routerContext([{organization}]),
  185. });
  186. await userEvent.click(screen.getAllByRole('button', {name: 'Remove'})[0]);
  187. renderGlobalModal();
  188. await userEvent.click(screen.getByTestId('confirm-button'));
  189. await waitFor(() => expect(addSuccessMessage).toHaveBeenCalled());
  190. expect(deleteMock).toHaveBeenCalled();
  191. expect(browserHistory.push).not.toHaveBeenCalled();
  192. expect(OrganizationsStore.getAll()).toEqual([organization]);
  193. });
  194. it('displays error message when failing to remove member', async function () {
  195. const deleteMock = MockApiClient.addMockResponse({
  196. url: `/organizations/org-slug/members/${members[0].id}/`,
  197. method: 'DELETE',
  198. statusCode: 500,
  199. });
  200. render(<OrganizationMembersList {...defaultProps} />, {
  201. context: TestStubs.routerContext([{organization}]),
  202. });
  203. await userEvent.click(screen.getAllByRole('button', {name: 'Remove'})[0]);
  204. renderGlobalModal();
  205. await userEvent.click(screen.getByTestId('confirm-button'));
  206. await waitFor(() => expect(addErrorMessage).toHaveBeenCalled());
  207. expect(deleteMock).toHaveBeenCalled();
  208. expect(browserHistory.push).not.toHaveBeenCalled();
  209. expect(OrganizationsStore.getAll()).toEqual([organization]);
  210. });
  211. it('can leave org', async function () {
  212. const deleteMock = MockApiClient.addMockResponse({
  213. url: `/organizations/org-slug/members/${members[1].id}/`,
  214. method: 'DELETE',
  215. });
  216. render(<OrganizationMembersList {...defaultProps} />, {
  217. context: TestStubs.routerContext([{organization}]),
  218. });
  219. await userEvent.click(screen.getAllByRole('button', {name: 'Leave'})[0]);
  220. renderGlobalModal();
  221. await userEvent.click(screen.getByTestId('confirm-button'));
  222. await waitFor(() => expect(addSuccessMessage).toHaveBeenCalled());
  223. expect(deleteMock).toHaveBeenCalled();
  224. expect(browserHistory.push).toHaveBeenCalledTimes(1);
  225. expect(browserHistory.push).toHaveBeenCalledWith('/organizations/new/');
  226. });
  227. it('can redirect to remaining org after leaving', async function () {
  228. const deleteMock = MockApiClient.addMockResponse({
  229. url: `/organizations/org-slug/members/${members[1].id}/`,
  230. method: 'DELETE',
  231. });
  232. const secondOrg = TestStubs.Organization({
  233. slug: 'org-two',
  234. status: {
  235. id: 'active',
  236. },
  237. });
  238. OrganizationsStore.addOrReplace(secondOrg);
  239. render(<OrganizationMembersList {...defaultProps} />, {
  240. context: TestStubs.routerContext([{organization}]),
  241. });
  242. await userEvent.click(screen.getAllByRole('button', {name: 'Leave'})[0]);
  243. renderGlobalModal();
  244. await userEvent.click(screen.getByTestId('confirm-button'));
  245. await waitFor(() => expect(addSuccessMessage).toHaveBeenCalled());
  246. expect(deleteMock).toHaveBeenCalled();
  247. expect(browserHistory.push).toHaveBeenCalledTimes(1);
  248. expect(browserHistory.push).toHaveBeenCalledWith(
  249. `/organizations/${secondOrg.slug}/issues/`
  250. );
  251. expect(OrganizationsStore.getAll()).toEqual([secondOrg]);
  252. });
  253. it('displays error message when failing to leave org', async function () {
  254. const deleteMock = MockApiClient.addMockResponse({
  255. url: `/organizations/org-slug/members/${members[1].id}/`,
  256. method: 'DELETE',
  257. statusCode: 500,
  258. });
  259. render(<OrganizationMembersList {...defaultProps} />, {
  260. context: TestStubs.routerContext([{organization}]),
  261. });
  262. await userEvent.click(screen.getAllByRole('button', {name: 'Leave'})[0]);
  263. renderGlobalModal();
  264. await userEvent.click(screen.getByTestId('confirm-button'));
  265. await waitFor(() => expect(addErrorMessage).toHaveBeenCalled());
  266. expect(deleteMock).toHaveBeenCalled();
  267. expect(browserHistory.push).not.toHaveBeenCalled();
  268. expect(OrganizationsStore.getAll()).toEqual([organization]);
  269. });
  270. it('can re-send SSO link to member', async function () {
  271. const inviteMock = MockApiClient.addMockResponse({
  272. url: `/organizations/org-slug/members/${members[0].id}/`,
  273. method: 'PUT',
  274. body: {
  275. id: '1234',
  276. },
  277. });
  278. render(<OrganizationMembersList {...defaultProps} />, {
  279. context: TestStubs.routerContext([{organization}]),
  280. });
  281. expect(inviteMock).not.toHaveBeenCalled();
  282. await userEvent.click(screen.getByRole('button', {name: 'Resend SSO link'}));
  283. expect(inviteMock).toHaveBeenCalled();
  284. });
  285. it('can re-send invite to member', async function () {
  286. const inviteMock = MockApiClient.addMockResponse({
  287. url: `/organizations/org-slug/members/${members[1].id}/`,
  288. method: 'PUT',
  289. body: {
  290. id: '1234',
  291. },
  292. });
  293. render(<OrganizationMembersList {...defaultProps} />, {
  294. context: TestStubs.routerContext([{organization}]),
  295. });
  296. expect(inviteMock).not.toHaveBeenCalled();
  297. await userEvent.click(screen.getByRole('button', {name: 'Resend invite'}));
  298. expect(inviteMock).toHaveBeenCalled();
  299. });
  300. it('can search organization members', async function () {
  301. const searchMock = MockApiClient.addMockResponse({
  302. url: '/organizations/org-slug/members/',
  303. body: [],
  304. });
  305. const routerContext = TestStubs.routerContext();
  306. render(<OrganizationMembersList {...defaultProps} />, {
  307. context: routerContext,
  308. });
  309. await userEvent.type(screen.getByPlaceholderText('Search Members'), 'member');
  310. expect(searchMock).toHaveBeenLastCalledWith(
  311. '/organizations/org-slug/members/',
  312. expect.objectContaining({
  313. method: 'GET',
  314. query: {
  315. query: 'member',
  316. },
  317. })
  318. );
  319. await userEvent.keyboard('{enter}');
  320. expect(routerContext.context.router.push).toHaveBeenCalledTimes(1);
  321. });
  322. it('can filter members', async function () {
  323. const searchMock = MockApiClient.addMockResponse({
  324. url: '/organizations/org-slug/members/',
  325. body: [],
  326. });
  327. const routerContext = TestStubs.routerContext();
  328. render(<OrganizationMembersList {...defaultProps} />, {
  329. context: routerContext,
  330. });
  331. await userEvent.click(screen.getByRole('button', {name: 'Filter'}));
  332. await userEvent.click(screen.getByRole('option', {name: 'Member'}));
  333. expect(searchMock).toHaveBeenLastCalledWith(
  334. '/organizations/org-slug/members/',
  335. expect.objectContaining({
  336. method: 'GET',
  337. query: {query: 'role:member'},
  338. })
  339. );
  340. await userEvent.click(screen.getByRole('option', {name: 'Member'}));
  341. for (const [filter, label] of [
  342. ['isInvited', 'Invited'],
  343. ['has2fa', '2FA'],
  344. ['ssoLinked', 'SSO Linked'],
  345. ]) {
  346. const filterSection = screen.getByRole('listbox', {name: label});
  347. await userEvent.click(
  348. within(filterSection).getByRole('option', {
  349. name: 'True',
  350. })
  351. );
  352. expect(searchMock).toHaveBeenLastCalledWith(
  353. '/organizations/org-slug/members/',
  354. expect.objectContaining({
  355. method: 'GET',
  356. query: {query: `${filter}:true`},
  357. })
  358. );
  359. await userEvent.click(
  360. within(filterSection).getByRole('option', {
  361. name: 'False',
  362. })
  363. );
  364. expect(searchMock).toHaveBeenLastCalledWith(
  365. '/organizations/org-slug/members/',
  366. expect.objectContaining({
  367. method: 'GET',
  368. query: {query: `${filter}:false`},
  369. })
  370. );
  371. await userEvent.click(
  372. within(filterSection).getByRole('option', {
  373. name: 'All',
  374. })
  375. );
  376. }
  377. });
  378. it('can filter members with org roles from team membership', async function () {
  379. const routerContext = TestStubs.routerContext();
  380. render(<OrganizationMembersList {...defaultProps} />, {
  381. context: routerContext,
  382. });
  383. await userEvent.click(screen.getByRole('button', {name: 'Filter'}));
  384. await userEvent.click(screen.getByRole('option', {name: 'Owner'}));
  385. await userEvent.click(screen.getByRole('button', {name: 'Filter'}));
  386. const owners = screen.queryAllByText('Owner');
  387. expect(owners).toHaveLength(3);
  388. });
  389. describe('OrganizationInviteRequests', function () {
  390. const inviteRequest = TestStubs.Member({
  391. id: '123',
  392. user: null,
  393. inviteStatus: 'requested_to_be_invited',
  394. inviter: TestStubs.User(),
  395. role: 'member',
  396. teams: [],
  397. });
  398. const joinRequest = TestStubs.Member({
  399. id: '456',
  400. user: null,
  401. email: 'test@gmail.com',
  402. inviteStatus: 'requested_to_join',
  403. role: 'member',
  404. teams: [],
  405. });
  406. it('disable buttons for no access', function () {
  407. const org = TestStubs.Organization({
  408. status: {
  409. id: 'active',
  410. },
  411. });
  412. MockApiClient.addMockResponse({
  413. url: '/organizations/org-slug/invite-requests/',
  414. method: 'GET',
  415. body: [inviteRequest],
  416. });
  417. MockApiClient.addMockResponse({
  418. url: `/organizations/org-slug/invite-requests/${inviteRequest.id}/`,
  419. method: 'PUT',
  420. });
  421. render(<OrganizationMembersList {...defaultProps} organization={org} />, {
  422. context: TestStubs.routerContext([{organization: org}]),
  423. });
  424. expect(screen.getByText('Pending Members')).toBeInTheDocument();
  425. expect(screen.getByRole('button', {name: 'Approve'})).toBeDisabled();
  426. });
  427. it('can approve invite request and update', async function () {
  428. const org = TestStubs.Organization({
  429. access: ['member:admin', 'org:admin', 'member:write'],
  430. status: {
  431. id: 'active',
  432. },
  433. });
  434. MockApiClient.addMockResponse({
  435. url: '/organizations/org-slug/invite-requests/',
  436. method: 'GET',
  437. body: [inviteRequest],
  438. });
  439. MockApiClient.addMockResponse({
  440. url: `/organizations/org-slug/invite-requests/${inviteRequest.id}/`,
  441. method: 'PUT',
  442. });
  443. render(<OrganizationMembersList {...defaultProps} />, {
  444. context: TestStubs.routerContext([{organization: org}]),
  445. });
  446. expect(screen.getByText('Pending Members')).toBeInTheDocument();
  447. await userEvent.click(screen.getByRole('button', {name: 'Approve'}));
  448. renderGlobalModal();
  449. await userEvent.click(screen.getByTestId('confirm-button'));
  450. expect(screen.queryByText('Pending Members')).not.toBeInTheDocument();
  451. expect(trackAnalytics).toHaveBeenCalledWith('invite_request.approved', {
  452. invite_status: inviteRequest.inviteStatus,
  453. member_id: parseInt(inviteRequest.id, 10),
  454. organization: org,
  455. });
  456. });
  457. it('can deny invite request and remove', async function () {
  458. const org = TestStubs.Organization({
  459. access: ['member:admin', 'org:admin', 'member:write'],
  460. status: {
  461. id: 'active',
  462. },
  463. });
  464. MockApiClient.addMockResponse({
  465. url: '/organizations/org-slug/invite-requests/',
  466. method: 'GET',
  467. body: [joinRequest],
  468. });
  469. MockApiClient.addMockResponse({
  470. url: `/organizations/org-slug/invite-requests/${joinRequest.id}/`,
  471. method: 'DELETE',
  472. });
  473. render(<OrganizationMembersList {...defaultProps} />, {
  474. context: TestStubs.routerContext([{organization: org}]),
  475. });
  476. expect(screen.getByText('Pending Members')).toBeInTheDocument();
  477. await userEvent.click(screen.getByRole('button', {name: 'Deny'}));
  478. expect(screen.queryByText('Pending Members')).not.toBeInTheDocument();
  479. expect(trackAnalytics).toHaveBeenCalledWith('invite_request.denied', {
  480. invite_status: joinRequest.inviteStatus,
  481. member_id: parseInt(joinRequest.id, 10),
  482. organization: org,
  483. });
  484. });
  485. it('can update invite requests', async function () {
  486. const org = TestStubs.Organization({
  487. access: ['member:admin', 'org:admin', 'member:write'],
  488. status: {
  489. id: 'active',
  490. },
  491. });
  492. MockApiClient.addMockResponse({
  493. url: '/organizations/org-slug/invite-requests/',
  494. method: 'GET',
  495. body: [inviteRequest],
  496. });
  497. const updateWithApprove = MockApiClient.addMockResponse({
  498. url: `/organizations/org-slug/invite-requests/${inviteRequest.id}/`,
  499. method: 'PUT',
  500. });
  501. render(<OrganizationMembersList {...defaultProps} />, {
  502. context: TestStubs.routerContext([{organization: org}]),
  503. });
  504. await selectEvent.select(screen.getAllByRole('textbox')[1], ['Admin']);
  505. await userEvent.click(screen.getByRole('button', {name: 'Approve'}));
  506. renderGlobalModal();
  507. await userEvent.click(screen.getByTestId('confirm-button'));
  508. expect(updateWithApprove).toHaveBeenCalledWith(
  509. `/organizations/org-slug/invite-requests/${inviteRequest.id}/`,
  510. expect.objectContaining({data: expect.objectContaining({role: 'admin'})})
  511. );
  512. });
  513. });
  514. describe('inviteBanner', function () {
  515. it('renders banner with feature flag', function () {
  516. MockApiClient.addMockResponse({
  517. url: '/organizations/org-slug/missing-members/',
  518. method: 'GET',
  519. body: missingMembers,
  520. });
  521. const org = TestStubs.Organization({
  522. features: ['integrations-gh-invite'],
  523. });
  524. render(<OrganizationMembersList {...defaultProps} organization={org} />, {
  525. context: TestStubs.routerContext([{organization: org}]),
  526. });
  527. expect(screen.getByTestId('invite-banner')).toBeInTheDocument();
  528. expect(screen.queryAllByTestId('invite-missing-member')).toHaveLength(5);
  529. expect(screen.getByText('See all 5 missing members')).toBeInTheDocument();
  530. });
  531. it('does not render banner if no missing members', function () {
  532. const org = TestStubs.Organization({
  533. features: ['integrations-gh-invite'],
  534. });
  535. render(<OrganizationMembersList {...defaultProps} organization={org} />, {
  536. context: TestStubs.routerContext([{organization: org}]),
  537. });
  538. expect(screen.queryByTestId('invite-banner')).not.toBeInTheDocument();
  539. });
  540. it('does not render banner if lacking org:write', function () {
  541. const org = TestStubs.Organization({
  542. features: ['integrations-gh-invite'],
  543. access: [],
  544. });
  545. render(<OrganizationMembersList {...defaultProps} organization={org} />, {
  546. context: TestStubs.routerContext([{organization: org}]),
  547. });
  548. expect(screen.queryByTestId('invite-banner')).not.toBeInTheDocument();
  549. });
  550. it('invites member from banner', async function () {
  551. MockApiClient.addMockResponse({
  552. url: '/organizations/org-slug/missing-members/',
  553. method: 'GET',
  554. body: missingMembers,
  555. });
  556. const newMember = TestStubs.Member({
  557. id: '6',
  558. email: 'hello@sentry.io',
  559. teams: [],
  560. teamRoles: [],
  561. flags: {
  562. 'sso:linked': true,
  563. 'idp:provisioned': false,
  564. },
  565. });
  566. MockApiClient.addMockResponse({
  567. url: '/organizations/org-slug/members/',
  568. method: 'POST',
  569. body: newMember,
  570. });
  571. const org = TestStubs.Organization({
  572. features: ['integrations-gh-invite'],
  573. });
  574. render(<OrganizationMembersList {...defaultProps} organization={org} />, {
  575. context: TestStubs.routerContext([{organization: org}]),
  576. });
  577. expect(screen.getByTestId('invite-banner')).toBeInTheDocument();
  578. expect(screen.queryAllByTestId('invite-missing-member')).toHaveLength(5);
  579. expect(screen.getByText('See all 5 missing members')).toBeInTheDocument();
  580. const inviteButton = screen.queryAllByTestId('invite-missing-member')[0];
  581. await userEvent.click(inviteButton);
  582. expect(screen.queryAllByTestId('invite-missing-member')).toHaveLength(4);
  583. expect(screen.getByText('See all 4 missing members')).toBeInTheDocument();
  584. });
  585. });
  586. });