assigneeSelector.spec.tsx 14 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403
  1. import {act, render, screen, userEvent, waitFor} from 'sentry-test/reactTestingLibrary';
  2. import {openInviteMembersModal} from 'sentry/actionCreators/modal';
  3. import AssigneeSelectorComponent from 'sentry/components/assigneeSelector';
  4. import {putSessionUserFirst} from 'sentry/components/assigneeSelectorDropdown';
  5. import ConfigStore from 'sentry/stores/configStore';
  6. import GroupStore from 'sentry/stores/groupStore';
  7. import IndicatorStore from 'sentry/stores/indicatorStore';
  8. import MemberListStore from 'sentry/stores/memberListStore';
  9. import ProjectsStore from 'sentry/stores/projectsStore';
  10. import TeamStore from 'sentry/stores/teamStore';
  11. jest.mock('sentry/actionCreators/modal', () => ({
  12. openInviteMembersModal: jest.fn(),
  13. }));
  14. describe('AssigneeSelector', () => {
  15. let assignMock;
  16. let assignGroup2Mock;
  17. let USER_1, USER_2, USER_3, USER_4;
  18. let TEAM_1;
  19. let PROJECT_1;
  20. let GROUP_1;
  21. let GROUP_2;
  22. beforeEach(() => {
  23. USER_1 = TestStubs.User({
  24. id: '1',
  25. name: 'Jane Bloggs',
  26. email: 'janebloggs@example.com',
  27. });
  28. USER_2 = TestStubs.User({
  29. id: '2',
  30. name: 'John Smith',
  31. email: 'johnsmith@example.com',
  32. });
  33. USER_3 = TestStubs.User({
  34. id: '3',
  35. name: 'J J',
  36. email: 'jj@example.com',
  37. });
  38. USER_4 = TestStubs.Member({
  39. id: '4',
  40. name: 'Jane Doe',
  41. email: 'janedoe@example.com',
  42. team_slug: 'cool-team2',
  43. });
  44. TEAM_1 = TestStubs.Team({
  45. id: '3',
  46. name: 'COOL TEAM',
  47. slug: 'cool-team',
  48. });
  49. PROJECT_1 = TestStubs.Project({
  50. teams: [TEAM_1],
  51. });
  52. GROUP_1 = TestStubs.Group({
  53. id: '1337',
  54. project: {
  55. id: PROJECT_1.id,
  56. slug: PROJECT_1.slug,
  57. },
  58. });
  59. GROUP_2 = TestStubs.Group({
  60. id: '1338',
  61. project: {
  62. id: PROJECT_1.id,
  63. slug: PROJECT_1.slug,
  64. },
  65. owners: [
  66. {
  67. type: 'suspectCommit',
  68. owner: `user:${USER_1.id}`,
  69. date_added: '',
  70. },
  71. ],
  72. });
  73. TeamStore.reset();
  74. TeamStore.setTeams([TEAM_1]);
  75. GroupStore.reset();
  76. GroupStore.loadInitialData([GROUP_1, GROUP_2]);
  77. jest.spyOn(MemberListStore, 'getAll').mockImplementation(() => []);
  78. jest.spyOn(ProjectsStore, 'getAll').mockImplementation(() => [PROJECT_1]);
  79. jest.spyOn(GroupStore, 'get').mockImplementation(() => GROUP_1);
  80. assignMock = MockApiClient.addMockResponse({
  81. method: 'PUT',
  82. url: `/organizations/org-slug/issues/${GROUP_1.id}/`,
  83. body: {
  84. ...GROUP_1,
  85. assignedTo: {...USER_1, type: 'user'},
  86. },
  87. });
  88. assignGroup2Mock = MockApiClient.addMockResponse({
  89. method: 'PUT',
  90. url: `/organizations/org-slug/issues/${GROUP_2.id}/`,
  91. body: {
  92. ...GROUP_2,
  93. assignedTo: {...USER_1, type: 'user'},
  94. },
  95. });
  96. MemberListStore.reset();
  97. });
  98. // Doesn't need to always be async, but it was easier to prevent flakes this way
  99. const openMenu = async () => {
  100. await userEvent.click(await screen.findByTestId('assignee-selector'), undefined);
  101. };
  102. afterEach(() => {
  103. MockApiClient.clearMockResponses();
  104. });
  105. describe('render with props', () => {
  106. it('renders members from the prop when present', async () => {
  107. MemberListStore.loadInitialData([USER_1]);
  108. render(<AssigneeSelectorComponent id={GROUP_1.id} memberList={[USER_2, USER_3]} />);
  109. await openMenu();
  110. expect(screen.queryByTestId('loading-indicator')).not.toBeInTheDocument();
  111. // 3 total items
  112. expect(screen.getAllByTestId('assignee-option')).toHaveLength(3);
  113. // 1 team
  114. expect(screen.getByText(`#${TEAM_1.slug}`)).toBeInTheDocument();
  115. // 2 Users
  116. expect(screen.getByText(USER_2.name)).toBeInTheDocument();
  117. expect(screen.getByText(USER_3.name)).toBeInTheDocument();
  118. });
  119. });
  120. describe('putSessionUserFirst()', () => {
  121. it('should place the session user at the top of the member list if present', () => {
  122. render(<AssigneeSelectorComponent id={GROUP_1.id} />);
  123. jest.spyOn(ConfigStore, 'get').mockImplementation(() => USER_2);
  124. expect(putSessionUserFirst([USER_1, USER_2])).toEqual([USER_2, USER_1]);
  125. (ConfigStore.get as jest.Mock).mockRestore();
  126. });
  127. it("should return the same member list if the session user isn't present", () => {
  128. render(<AssigneeSelectorComponent id={GROUP_1.id} />);
  129. jest.spyOn(ConfigStore, 'get').mockImplementation(() =>
  130. TestStubs.User({
  131. id: '555',
  132. name: 'Here Comes a New Challenger',
  133. email: 'guile@mail.us.af.mil',
  134. })
  135. );
  136. expect(putSessionUserFirst([USER_1, USER_2])).toEqual([USER_1, USER_2]);
  137. (ConfigStore.get as jest.Mock).mockRestore();
  138. });
  139. });
  140. it('should initially have loading state', async () => {
  141. render(<AssigneeSelectorComponent id={GROUP_1.id} />);
  142. await openMenu();
  143. expect(screen.getByTestId('loading-indicator')).toBeInTheDocument();
  144. });
  145. it('does not have loading state and shows member list after calling MemberListStore.loadInitialData', async () => {
  146. render(<AssigneeSelectorComponent id={GROUP_1.id} />);
  147. act(() => MemberListStore.loadInitialData([USER_1, USER_2]));
  148. await openMenu();
  149. expect(screen.queryByTestId('loading-indicator')).not.toBeInTheDocument();
  150. // 3 total items
  151. expect(screen.getAllByTestId('assignee-option')).toHaveLength(3);
  152. // 1 team
  153. expect(screen.getByText(`#${TEAM_1.slug}`)).toBeInTheDocument();
  154. // 2 Users including self
  155. expect(screen.getByText(`${USER_1.name} (You)`)).toBeInTheDocument();
  156. expect(screen.getByText(USER_2.name)).toBeInTheDocument();
  157. });
  158. it('does NOT update member list after initial load', async () => {
  159. render(<AssigneeSelectorComponent id={GROUP_1.id} />);
  160. act(() => MemberListStore.loadInitialData([USER_1, USER_2]));
  161. await openMenu();
  162. expect(screen.queryByTestId('loading-indicator')).not.toBeInTheDocument();
  163. expect(screen.getByText(`${USER_1.name} (You)`)).toBeInTheDocument();
  164. expect(screen.getByText(USER_2.name)).toBeInTheDocument();
  165. act(() => MemberListStore.loadInitialData([USER_1, USER_2, USER_3]));
  166. expect(screen.queryByTestId('loading-indicator')).not.toBeInTheDocument();
  167. expect(screen.getByText(`${USER_1.name} (You)`)).toBeInTheDocument();
  168. expect(screen.getByText(USER_2.name)).toBeInTheDocument();
  169. expect(screen.queryByText(USER_3.name)).not.toBeInTheDocument();
  170. });
  171. it('successfully assigns users', async () => {
  172. render(<AssigneeSelectorComponent id={GROUP_1.id} />);
  173. act(() => MemberListStore.loadInitialData([USER_1, USER_2]));
  174. await openMenu();
  175. expect(screen.queryByTestId('loading-indicator')).not.toBeInTheDocument();
  176. await userEvent.click(screen.getByText(`${USER_1.name} (You)`));
  177. expect(assignMock).toHaveBeenLastCalledWith(
  178. '/organizations/org-slug/issues/1337/',
  179. expect.objectContaining({
  180. data: {assignedTo: 'user:1', assignedBy: 'assignee_selector'},
  181. })
  182. );
  183. expect(await screen.findByTestId('letter_avatar-avatar')).toBeInTheDocument();
  184. // USER_1 initials
  185. expect(screen.getByTestId('assignee-selector')).toHaveTextContent('JB');
  186. });
  187. it('successfully assigns teams', async () => {
  188. MockApiClient.clearMockResponses();
  189. assignMock = MockApiClient.addMockResponse({
  190. method: 'PUT',
  191. url: `/organizations/org-slug/issues/${GROUP_1.id}/`,
  192. body: {
  193. ...GROUP_1,
  194. assignedTo: {...TEAM_1, type: 'team'},
  195. },
  196. });
  197. render(<AssigneeSelectorComponent id={GROUP_1.id} />);
  198. act(() => MemberListStore.loadInitialData([USER_1, USER_2]));
  199. await openMenu();
  200. expect(screen.queryByTestId('loading-indicator')).not.toBeInTheDocument();
  201. await userEvent.click(screen.getByText(`#${TEAM_1.slug}`));
  202. await waitFor(() =>
  203. expect(assignMock).toHaveBeenCalledWith(
  204. '/organizations/org-slug/issues/1337/',
  205. expect.objectContaining({
  206. data: {assignedTo: 'team:3', assignedBy: 'assignee_selector'},
  207. })
  208. )
  209. );
  210. expect(await screen.findByTestId('letter_avatar-avatar')).toBeInTheDocument();
  211. // TEAM_1 initials
  212. expect(screen.getByTestId('assignee-selector')).toHaveTextContent('CT');
  213. });
  214. it('successfully clears assignment', async () => {
  215. render(<AssigneeSelectorComponent id={GROUP_1.id} />);
  216. act(() => MemberListStore.loadInitialData([USER_1, USER_2]));
  217. await openMenu();
  218. // Assign first item in list, which is TEAM_1
  219. await userEvent.click(screen.getByText(`#${TEAM_1.slug}`));
  220. await waitFor(() =>
  221. expect(assignMock).toHaveBeenCalledWith(
  222. '/organizations/org-slug/issues/1337/',
  223. expect.objectContaining({
  224. data: {assignedTo: 'team:3', assignedBy: 'assignee_selector'},
  225. })
  226. )
  227. );
  228. await openMenu();
  229. await userEvent.click(screen.getByRole('button', {name: 'Clear Assignee'}));
  230. // api was called with empty string, clearing assignment
  231. await waitFor(() =>
  232. expect(assignMock).toHaveBeenLastCalledWith(
  233. '/organizations/org-slug/issues/1337/',
  234. expect.objectContaining({
  235. data: {assignedTo: '', assignedBy: 'assignee_selector'},
  236. })
  237. )
  238. );
  239. });
  240. it('shows invite member button', async () => {
  241. MemberListStore.loadInitialData([USER_1, USER_2]);
  242. render(<AssigneeSelectorComponent id={GROUP_1.id} />, {
  243. context: TestStubs.routerContext(),
  244. });
  245. jest.spyOn(ConfigStore, 'get').mockImplementation(() => true);
  246. await openMenu();
  247. expect(screen.queryByTestId('loading-indicator')).not.toBeInTheDocument();
  248. await userEvent.click(await screen.findByRole('link', {name: 'Invite Member'}));
  249. expect(openInviteMembersModal).toHaveBeenCalled();
  250. (ConfigStore.get as jest.Mock).mockRestore();
  251. });
  252. it('filters user by email and selects with keyboard', async () => {
  253. render(<AssigneeSelectorComponent id={GROUP_2.id} />);
  254. act(() => MemberListStore.loadInitialData([USER_1, USER_2]));
  255. await openMenu();
  256. expect(screen.queryByTestId('loading-indicator')).not.toBeInTheDocument();
  257. await userEvent.type(screen.getByRole('textbox'), 'JohnSmith@example.com');
  258. // 1 total item
  259. expect(screen.getByTestId('assignee-option')).toBeInTheDocument();
  260. expect(screen.getByText(`${USER_2.name}`)).toBeInTheDocument();
  261. await userEvent.keyboard('{enter}');
  262. await waitFor(() =>
  263. expect(assignGroup2Mock).toHaveBeenLastCalledWith(
  264. '/organizations/org-slug/issues/1338/',
  265. expect.objectContaining({
  266. data: {assignedTo: `user:${USER_2.id}`, assignedBy: 'assignee_selector'},
  267. })
  268. )
  269. );
  270. expect(await screen.findByTestId('letter_avatar-avatar')).toBeInTheDocument();
  271. // USER_2 initials
  272. expect(screen.getByTestId('assignee-selector')).toHaveTextContent('JB');
  273. });
  274. it('shows the correct toast for assigning to a non-team member', async () => {
  275. jest.spyOn(GroupStore, 'get').mockImplementation(() => GROUP_2);
  276. const addMessageSpy = jest.spyOn(IndicatorStore, 'addMessage');
  277. render(<AssigneeSelectorComponent id={GROUP_2.id} />);
  278. act(() => MemberListStore.loadInitialData([USER_1, USER_2, USER_3, USER_4]));
  279. assignMock = MockApiClient.addMockResponse({
  280. method: 'PUT',
  281. url: `/organizations/org-slug/issues/${GROUP_2.id}/`,
  282. statusCode: 400,
  283. body: {detail: 'Cannot assign to non-team member'},
  284. });
  285. expect(screen.getByTestId('suggested-avatar-stack')).toBeInTheDocument();
  286. await openMenu();
  287. expect(screen.queryByTestId('loading-indicator')).not.toBeInTheDocument();
  288. expect(screen.getByText(`#${TEAM_1.slug}`)).toBeInTheDocument();
  289. expect(await screen.findByText('Suggested Assignees')).toBeInTheDocument();
  290. const options = screen.getAllByTestId('assignee-option');
  291. expect(options[5]).toHaveTextContent('JD');
  292. await userEvent.click(options[4]);
  293. await waitFor(() => {
  294. expect(addMessageSpy).toHaveBeenCalledWith(
  295. 'Cannot assign to non-team member',
  296. 'error',
  297. {duration: 4000}
  298. );
  299. });
  300. });
  301. it('successfully shows suggested assignees', async () => {
  302. jest.spyOn(GroupStore, 'get').mockImplementation(() => GROUP_2);
  303. const onAssign = jest.fn();
  304. render(<AssigneeSelectorComponent id={GROUP_2.id} onAssign={onAssign} />);
  305. act(() => MemberListStore.loadInitialData([USER_1, USER_2, USER_3]));
  306. expect(screen.getByTestId('suggested-avatar-stack')).toBeInTheDocument();
  307. // Hover over avatar
  308. await userEvent.hover(screen.getByTestId('letter_avatar-avatar'));
  309. expect(await screen.findByText('Suggestion: Jane Bloggs')).toBeInTheDocument();
  310. expect(screen.getByText('commit data')).toBeInTheDocument();
  311. await openMenu();
  312. expect(screen.queryByTestId('loading-indicator')).not.toBeInTheDocument();
  313. expect(await screen.findByText('Suggested Assignees')).toBeInTheDocument();
  314. const options = screen.getAllByTestId('assignee-option');
  315. // Suggested assignee initials
  316. expect(options[0]).toHaveTextContent('JB');
  317. await userEvent.click(options[0]);
  318. await waitFor(() =>
  319. expect(assignGroup2Mock).toHaveBeenCalledWith(
  320. '/organizations/org-slug/issues/1338/',
  321. expect.objectContaining({
  322. data: {assignedTo: `user:${USER_1.id}`, assignedBy: 'assignee_selector'},
  323. })
  324. )
  325. );
  326. // Suggested assignees shouldn't show anymore because we assigned to the suggested actor
  327. expect(screen.queryByTestId('suggested-avatar-stack')).not.toBeInTheDocument();
  328. expect(onAssign).toHaveBeenCalledWith(
  329. 'member',
  330. expect.objectContaining({id: USER_1.id}),
  331. expect.objectContaining({id: USER_1.id})
  332. );
  333. });
  334. it('renders unassigned', async () => {
  335. jest.spyOn(GroupStore, 'get').mockImplementation(() => GROUP_1);
  336. render(<AssigneeSelectorComponent id={GROUP_1.id} />);
  337. await userEvent.hover(screen.getByTestId('unassigned'));
  338. expect(await screen.findByText('Unassigned')).toBeInTheDocument();
  339. });
  340. });