projectGeneralSettings.spec.jsx 12 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368
  1. import {browserHistory} from 'react-router';
  2. import selectEvent from 'react-select-event';
  3. import {
  4. act,
  5. fireEvent,
  6. render,
  7. renderGlobalModal,
  8. screen,
  9. userEvent,
  10. waitFor,
  11. } from 'sentry-test/reactTestingLibrary';
  12. import {addErrorMessage, addSuccessMessage} from 'sentry/actionCreators/indicator';
  13. import {removePageFiltersStorage} from 'sentry/components/organizations/pageFilters/persistence';
  14. import ProjectsStore from 'sentry/stores/projectsStore';
  15. import ProjectContext from 'sentry/views/projects/projectContext';
  16. import ProjectGeneralSettings from 'sentry/views/settings/projectGeneralSettings';
  17. jest.mock('sentry/actionCreators/indicator');
  18. jest.mock('sentry/components/organizations/pageFilters/persistence');
  19. function getField(role, name) {
  20. return screen.getByRole(role, {name});
  21. }
  22. describe('projectGeneralSettings', function () {
  23. const org = TestStubs.Organization();
  24. const project = TestStubs.ProjectDetails();
  25. const groupingConfigs = TestStubs.GroupingConfigs();
  26. const groupingEnhancements = TestStubs.GroupingEnhancements();
  27. let routerContext;
  28. let putMock;
  29. beforeEach(function () {
  30. jest.spyOn(window.location, 'assign');
  31. routerContext = TestStubs.routerContext([
  32. {
  33. router: TestStubs.router({
  34. params: {
  35. projectId: project.slug,
  36. },
  37. }),
  38. },
  39. ]);
  40. MockApiClient.clearMockResponses();
  41. MockApiClient.addMockResponse({
  42. url: '/grouping-configs/',
  43. method: 'GET',
  44. body: groupingConfigs,
  45. });
  46. MockApiClient.addMockResponse({
  47. url: '/grouping-enhancements/',
  48. method: 'GET',
  49. body: groupingEnhancements,
  50. });
  51. MockApiClient.addMockResponse({
  52. url: `/projects/${org.slug}/${project.slug}/`,
  53. method: 'GET',
  54. body: project,
  55. });
  56. MockApiClient.addMockResponse({
  57. url: `/projects/${org.slug}/${project.slug}/environments/`,
  58. method: 'GET',
  59. body: [],
  60. });
  61. MockApiClient.addMockResponse({
  62. url: `/organizations/${org.slug}/users/`,
  63. method: 'GET',
  64. body: [],
  65. });
  66. });
  67. afterEach(function () {
  68. window.location.assign.mockRestore();
  69. MockApiClient.clearMockResponses();
  70. addSuccessMessage.mockReset();
  71. addErrorMessage.mockReset();
  72. });
  73. it('renders form fields', function () {
  74. render(<ProjectGeneralSettings params={{projectId: project.slug}} />);
  75. expect(getField('textbox', 'Name')).toHaveValue('Project Name');
  76. expect(getField('textbox', 'Subject Prefix')).toHaveValue('[my-org]');
  77. // Step 19 of the auto resolve slider equates to 48 hours. This is
  78. // different from thee actual field value (which will be 48)
  79. expect(getField('slider', 'Auto Resolve')).toHaveValue('19');
  80. expect(getField('textbox', 'Allowed Domains')).toHaveValue(
  81. 'example.com\nhttps://example.com'
  82. );
  83. expect(getField('checkbox', 'Enable JavaScript source fetching')).toBeChecked();
  84. expect(getField('textbox', 'Security Token')).toHaveValue('security-token');
  85. expect(getField('textbox', 'Security Token Header')).toHaveValue('x-security-header');
  86. expect(getField('checkbox', 'Verify TLS/SSL')).toBeChecked();
  87. });
  88. it('disables scrapeJavaScript when equivalent org setting is false', function () {
  89. routerContext.context.organization.scrapeJavaScript = false;
  90. render(<ProjectGeneralSettings params={{projectId: project.slug}} />, {
  91. context: routerContext,
  92. });
  93. expect(getField('checkbox', 'Enable JavaScript source fetching')).toBeDisabled();
  94. expect(getField('checkbox', 'Enable JavaScript source fetching')).not.toBeChecked();
  95. });
  96. it('project admins can remove project', function () {
  97. const deleteMock = MockApiClient.addMockResponse({
  98. url: `/projects/${org.slug}/${project.slug}/`,
  99. method: 'DELETE',
  100. });
  101. render(<ProjectGeneralSettings params={{projectId: project.slug}} />);
  102. userEvent.click(screen.getByRole('button', {name: 'Remove Project'}));
  103. // Click confirmation button
  104. renderGlobalModal();
  105. userEvent.click(screen.getByTestId('confirm-button'));
  106. expect(deleteMock).toHaveBeenCalled();
  107. expect(removePageFiltersStorage).toHaveBeenCalledWith('org-slug');
  108. });
  109. it('project admins can transfer project', async function () {
  110. const deleteMock = MockApiClient.addMockResponse({
  111. url: `/projects/${org.slug}/${project.slug}/transfer/`,
  112. method: 'POST',
  113. });
  114. render(<ProjectGeneralSettings params={{projectId: project.slug}} />);
  115. userEvent.click(screen.getByRole('button', {name: 'Transfer Project'}));
  116. // Click confirmation button
  117. renderGlobalModal();
  118. userEvent.type(getField('textbox', 'Organization Owner'), 'billy@sentry.io');
  119. userEvent.click(screen.getByTestId('confirm-button'));
  120. await waitFor(() =>
  121. expect(deleteMock).toHaveBeenCalledWith(
  122. `/projects/${org.slug}/${project.slug}/transfer/`,
  123. expect.objectContaining({
  124. method: 'POST',
  125. data: {
  126. email: 'billy@sentry.io',
  127. },
  128. })
  129. )
  130. );
  131. expect(addSuccessMessage).toHaveBeenCalled();
  132. });
  133. it('handles errors on transfer project', async function () {
  134. const deleteMock = MockApiClient.addMockResponse({
  135. url: `/projects/${org.slug}/${project.slug}/transfer/`,
  136. method: 'POST',
  137. statusCode: 400,
  138. body: {detail: 'An organization owner could not be found'},
  139. });
  140. render(<ProjectGeneralSettings params={{projectId: project.slug}} />);
  141. userEvent.click(screen.getByRole('button', {name: 'Transfer Project'}));
  142. // Click confirmation button
  143. renderGlobalModal();
  144. userEvent.type(getField('textbox', 'Organization Owner'), 'billy@sentry.io');
  145. userEvent.click(screen.getByTestId('confirm-button'));
  146. await waitFor(() => expect(deleteMock).toHaveBeenCalled());
  147. expect(addSuccessMessage).not.toHaveBeenCalled();
  148. expect(addErrorMessage).toHaveBeenCalled();
  149. // Check the error message
  150. const {container} = render(addErrorMessage.mock.calls[0][0]);
  151. expect(container).toHaveTextContent(
  152. 'Error transferring project-slug. An organization owner could not be found'
  153. );
  154. });
  155. it('displays transfer/remove message for non-admins', function () {
  156. routerContext.context.organization.access = ['org:read'];
  157. const {container} = render(
  158. <ProjectGeneralSettings params={{projectId: project.slug}} />,
  159. {context: routerContext}
  160. );
  161. expect(container).toHaveTextContent(
  162. 'You do not have the required permission to remove this project.'
  163. );
  164. expect(container).toHaveTextContent(
  165. 'You do not have the required permission to transfer this project.'
  166. );
  167. });
  168. it('disables the form for users without write permissions', function () {
  169. routerContext.context.organization.access = ['org:read'];
  170. render(<ProjectGeneralSettings params={{projectId: project.slug}} />, {
  171. context: routerContext,
  172. });
  173. // no textboxes are enabled
  174. screen.queryAllByRole('textbox').forEach(textbox => expect(textbox).toBeDisabled());
  175. expect(screen.getByTestId('project-permission-alert')).toHaveTextContent(
  176. 'These settings can only be edited by users with the organization owner, manager, or admin role.'
  177. );
  178. });
  179. it('changing project platform updates ProjectsStore', async function () {
  180. const params = {projectId: project.slug};
  181. ProjectsStore.loadInitialData([project]);
  182. putMock = MockApiClient.addMockResponse({
  183. url: `/projects/${org.slug}/${project.slug}/`,
  184. method: 'PUT',
  185. body: {
  186. ...project,
  187. platform: 'javascript',
  188. },
  189. });
  190. render(
  191. <ProjectContext orgId={org.slug} projectId={project.slug}>
  192. <ProjectGeneralSettings
  193. routes={[]}
  194. location={routerContext.context.location}
  195. params={params}
  196. />
  197. </ProjectContext>,
  198. {context: routerContext}
  199. );
  200. const platformSelect = await screen.findByRole('textbox', {name: 'Platform'});
  201. await selectEvent.select(platformSelect, ['React']);
  202. expect(putMock).toHaveBeenCalled();
  203. // updates ProjectsStore
  204. expect(ProjectsStore.itemsById['2'].platform).toBe('javascript');
  205. });
  206. it('changing name updates ProjectsStore', async function () {
  207. const params = {projectId: project.slug};
  208. ProjectsStore.loadInitialData([project]);
  209. putMock = MockApiClient.addMockResponse({
  210. url: `/projects/${org.slug}/${project.slug}/`,
  211. method: 'PUT',
  212. body: {
  213. ...project,
  214. slug: 'new-project',
  215. },
  216. });
  217. render(
  218. <ProjectContext orgId={org.slug} projectId={project.slug}>
  219. <ProjectGeneralSettings
  220. routes={[]}
  221. location={routerContext.context.location}
  222. params={params}
  223. />
  224. </ProjectContext>,
  225. {context: routerContext}
  226. );
  227. userEvent.type(await screen.findByRole('textbox', {name: 'Name'}), 'New Project');
  228. // Slug does not save on blur
  229. expect(putMock).not.toHaveBeenCalled();
  230. // Saves when clicking save
  231. userEvent.click(screen.getByRole('button', {name: 'Save'}));
  232. // Redirects the user
  233. await waitFor(() => expect(browserHistory.replace).toHaveBeenCalled());
  234. expect(ProjectsStore.itemsById['2'].slug).toBe('new-project');
  235. });
  236. describe('Non-"save on blur" Field', function () {
  237. beforeEach(function () {
  238. const params = {projectId: project.slug};
  239. ProjectsStore.loadInitialData([project]);
  240. putMock = MockApiClient.addMockResponse({
  241. url: `/projects/${org.slug}/${project.slug}/`,
  242. method: 'PUT',
  243. body: {
  244. ...project,
  245. slug: 'new-project',
  246. },
  247. });
  248. render(
  249. <ProjectContext orgId={org.slug} projectId={project.slug}>
  250. <ProjectGeneralSettings
  251. routes={[]}
  252. location={routerContext.context.location}
  253. params={params}
  254. />
  255. </ProjectContext>,
  256. {context: routerContext}
  257. );
  258. });
  259. it('can cancel unsaved changes for a field', async function () {
  260. expect(screen.queryByRole('button', {name: 'Cancel'})).not.toBeInTheDocument();
  261. const autoResolveSlider = getField('slider', 'Auto Resolve');
  262. expect(autoResolveSlider).toHaveValue('19');
  263. // Change value
  264. fireEvent.change(autoResolveSlider, {target: {value: '12'}});
  265. expect(autoResolveSlider).toHaveValue('12');
  266. // Click cancel
  267. userEvent.click(screen.getByRole('button', {name: 'Cancel'}));
  268. await act(tick);
  269. // Cancel row should disappear
  270. expect(screen.queryByRole('button', {name: 'Cancel'})).not.toBeInTheDocument();
  271. // Value should be reverted
  272. expect(autoResolveSlider).toHaveValue('19');
  273. // PUT should not be called
  274. expect(putMock).not.toHaveBeenCalled();
  275. });
  276. it('saves when value is changed and "Save" clicked', async function () {
  277. expect(screen.queryByRole('button', {name: 'Save'})).not.toBeInTheDocument();
  278. const autoResolveSlider = getField('slider', 'Auto Resolve');
  279. expect(autoResolveSlider).toHaveValue('19');
  280. // Change value
  281. fireEvent.change(autoResolveSlider, {target: {value: '12'}});
  282. expect(autoResolveSlider).toHaveValue('12');
  283. // Should not have put mock called yet
  284. expect(putMock).not.toHaveBeenCalled();
  285. // Click "Save"
  286. userEvent.click(screen.queryByRole('button', {name: 'Save'}));
  287. await act(tick);
  288. // API endpoint should have been called
  289. expect(putMock).toHaveBeenCalledWith(
  290. expect.anything(),
  291. expect.objectContaining({
  292. data: {
  293. resolveAge: 12,
  294. },
  295. })
  296. );
  297. expect(screen.queryByRole('button', {name: 'Save'})).not.toBeInTheDocument();
  298. });
  299. });
  300. });