Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cleanup(priority-alerts): Remove priority-ga flag from the frontend #77829

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

snigdhas
Copy link
Member

The feature has been GA for a few weeks and alert migrations are complete. We can remove the flag now.

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Sep 19, 2024
Copy link

codecov bot commented Sep 19, 2024

❌ 13 Tests Failed:

Tests completed Failed Passed Skipped
7826 13 7813 0
View the top 3 failed tests by shortest run time
ProjectAlertsCreate Issue Alert can remove triggers ProjectAlertsCreate Issue Alert can remove triggers
Stack Traces | 0.534s run time
Error: Expected test not to call console.error().

If the error is expected, test for it explicitly by mocking it out using jest.spyOn(console, 'error').mockImplementation() and test that the warning occurs.

Warning: An update to PreviewIssues inside a test was not wrapped in act(...).

When testing, code that causes React state updates should be wrapped into act(...):

act(() => {
  /* fire events that update state */
});
/* assert on the output */

This ensures that you're testing the behavior the user would see in the browser. Learn more at https://reactjs.org/link/wrap-tests-with-act
    at PreviewIssues (.../rules/issue/previewIssues.tsx:1703:3)
    at ErrorBoundary (.../app/components/errorBoundary.tsx:976:1)
    at ol
    at Object.<anonymous>.process.env.NODE_ENV.target (.../components/list/index.tsx:449:3)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at div
    at form
    at Form (.../components/forms/form.tsx:1840:19)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at section
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at SentryDocumentTitle (.../app/components/sentryDocumentTitle.tsx:708:3)
    at IssueRuleEditor (.../rules/issue/index.tsx:13364:1)
    at Wrapper (.../app/utils/withProjects.tsx:176:13)
    at Wrapper (.../app/utils/withOrganization.tsx:171:24)
    at div
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at Create (.../views/alerts/create.tsx:1335:5)
    at AlertBuilderProjectProvider (.../alerts/builder/projectProvider.tsx:962:11)
    at NoProjectMessage (.../app/components/noProjectMessage.tsx:695:3)
    at AlertsContainer (.../views/alerts/index.tsx:205:3)
    at GlobalDrawer (.../components/globalDrawer/index.tsx:1038:3)
    at QueryClientProvider (.../sentry/sentry/node_modules/@.../react-query/src/QueryClientProvider.tsx:30:3)
    at ThemeProvider (.../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:125:32)
    at LegacyRouterProvider (.../js/sentry-test/reactTestingLibrary.tsx:72:3)
    at .../js/sentry-test/reactTestingLibrary.tsx:89:5
    at console.captureMessage [as error] (.../sentry/sentry/node_modules/jest-fail-on-console/index.js:83:25)
    at printWarning (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:86:30)
    at error (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:60:7)
    at warnIfUpdatesNotWrappedWithActDEV (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:27589:9)
    at scheduleUpdateOnFiber (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:25508:5)
    at dispatchSetState (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:17527:7)
    at .../rules/issue/previewIssues.tsx:1926:7

Warning: An update to PreviewIssues inside a test was not wrapped in act(...).

When testing, code that causes React state updates should be wrapped into act(...):

act(() => {
  /* fire events that update state */
});
/* assert on the output */

This ensures that you're testing the behavior the user would see in the browser. Learn more at https://reactjs.org/link/wrap-tests-with-act
    at PreviewIssues (.../rules/issue/previewIssues.tsx:1703:3)
    at ErrorBoundary (.../app/components/errorBoundary.tsx:976:1)
    at ol
    at Object.<anonymous>.process.env.NODE_ENV.target (.../components/list/index.tsx:449:3)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at div
    at form
    at Form (.../components/forms/form.tsx:1840:19)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at section
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at SentryDocumentTitle (.../app/components/sentryDocumentTitle.tsx:708:3)
    at IssueRuleEditor (.../rules/issue/index.tsx:13364:1)
    at Wrapper (.../app/utils/withProjects.tsx:176:13)
    at Wrapper (.../app/utils/withOrganization.tsx:171:24)
    at div
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at Create (.../views/alerts/create.tsx:1335:5)
    at AlertBuilderProjectProvider (.../alerts/builder/projectProvider.tsx:962:11)
    at NoProjectMessage (.../app/components/noProjectMessage.tsx:695:3)
    at AlertsContainer (.../views/alerts/index.tsx:205:3)
    at GlobalDrawer (.../components/globalDrawer/index.tsx:1038:3)
    at QueryClientProvider (.../sentry/sentry/node_modules/@.../react-query/src/QueryClientProvider.tsx:30:3)
    at ThemeProvider (.../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:125:32)
    at LegacyRouterProvider (.../js/sentry-test/reactTestingLibrary.tsx:72:3)
    at .../js/sentry-test/reactTestingLibrary.tsx:89:5
    at console.captureMessage [as error] (.../sentry/sentry/node_modules/jest-fail-on-console/index.js:83:25)
    at printWarning (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:86:30)
    at error (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:60:7)
    at warnIfUpdatesNotWrappedWithActDEV (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:27589:9)
    at scheduleUpdateOnFiber (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:25508:5)
    at dispatchSetState (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:17527:7)
    at .../rules/issue/previewIssues.tsx:1934:7

Warning: An update to PreviewIssues inside a test was not wrapped in act(...).

When testing, code that causes React state updates should be wrapped into act(...):

act(() => {
  /* fire events that update state */
});
/* assert on the output */

This ensures that you're testing the behavior the user would see in the browser. Learn more at https://reactjs.org/link/wrap-tests-with-act
    at PreviewIssues (.../rules/issue/previewIssues.tsx:1703:3)
    at ErrorBoundary (.../app/components/errorBoundary.tsx:976:1)
    at ol
    at Object.<anonymous>.process.env.NODE_ENV.target (.../components/list/index.tsx:449:3)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at div
    at form
    at Form (.../components/forms/form.tsx:1840:19)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at section
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at SentryDocumentTitle (.../app/components/sentryDocumentTitle.tsx:708:3)
    at IssueRuleEditor (.../rules/issue/index.tsx:13364:1)
    at Wrapper (.../app/utils/withProjects.tsx:176:13)
    at Wrapper (.../app/utils/withOrganization.tsx:171:24)
    at div
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at Create (.../views/alerts/create.tsx:1335:5)
    at AlertBuilderProjectProvider (.../alerts/builder/projectProvider.tsx:962:11)
    at NoProjectMessage (.../app/components/noProjectMessage.tsx:695:3)
    at AlertsContainer (.../views/alerts/index.tsx:205:3)
    at GlobalDrawer (.../components/globalDrawer/index.tsx:1038:3)
    at QueryClientProvider (.../sentry/sentry/node_modules/@.../react-query/src/QueryClientProvider.tsx:30:3)
    at ThemeProvider (.../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:125:32)
    at LegacyRouterProvider (.../js/sentry-test/reactTestingLibrary.tsx:72:3)
    at .../js/sentry-test/reactTestingLibrary.tsx:89:5
    at console.captureMessage [as error] (.../sentry/sentry/node_modules/jest-fail-on-console/index.js:83:25)
    at printWarning (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:86:30)
    at error (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:60:7)
    at warnIfUpdatesNotWrappedWithActDEV (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:27589:9)
    at scheduleUpdateOnFiber (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:25508:5)
    at dispatchSetState (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:17527:7)
    at .../rules/issue/previewIssues.tsx:1937:7

Warning: An update to PreviewIssues inside a test was not wrapped in act(...).

When testing, code that causes React state updates should be wrapped into act(...):

act(() => {
  /* fire events that update state */
});
/* assert on the output */

This ensures that you're testing the behavior the user would see in the browser. Learn more at https://reactjs.org/link/wrap-tests-with-act
    at PreviewIssues (.../rules/issue/previewIssues.tsx:1703:3)
    at ErrorBoundary (.../app/components/errorBoundary.tsx:976:1)
    at ol
    at Object.<anonymous>.process.env.NODE_ENV.target (.../components/list/index.tsx:449:3)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at div
    at form
    at Form (.../components/forms/form.tsx:1840:19)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at section
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at SentryDocumentTitle (.../app/components/sentryDocumentTitle.tsx:708:3)
    at IssueRuleEditor (.../rules/issue/index.tsx:13364:1)
    at Wrapper (.../app/utils/withProjects.tsx:176:13)
    at Wrapper (.../app/utils/withOrganization.tsx:171:24)
    at div
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at Create (.../views/alerts/create.tsx:1335:5)
    at AlertBuilderProjectProvider (.../alerts/builder/projectProvider.tsx:962:11)
    at NoProjectMessage (.../app/components/noProjectMessage.tsx:695:3)
    at AlertsContainer (.../views/alerts/index.tsx:205:3)
    at GlobalDrawer (.../components/globalDrawer/index.tsx:1038:3)
    at QueryClientProvider (.../sentry/sentry/node_modules/@.../react-query/src/QueryClientProvider.tsx:30:3)
    at ThemeProvider (.../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:125:32)
    at LegacyRouterProvider (.../js/sentry-test/reactTestingLibrary.tsx:72:3)
    at .../js/sentry-test/reactTestingLibrary.tsx:89:5
    at console.captureMessage [as error] (.../sentry/sentry/node_modules/jest-fail-on-console/index.js:83:25)
    at printWarning (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:86:30)
    at error (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:60:7)
    at warnIfUpdatesNotWrappedWithActDEV (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:27589:9)
    at scheduleUpdateOnFiber (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:25508:5)
    at dispatchSetState (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:17527:7)
    at .../rules/issue/previewIssues.tsx:1944:7

Warning: An update to PreviewIssues inside a test was not wrapped in act(...).

When testing, code that causes React state updates should be wrapped into act(...):

act(() => {
  /* fire events that update state */
});
/* assert on the output */

This ensures that you're testing the behavior the user would see in the browser. Learn more at https://reactjs.org/link/wrap-tests-with-act
    at PreviewIssues (.../rules/issue/previewIssues.tsx:1703:3)
    at ErrorBoundary (.../app/components/errorBoundary.tsx:976:1)
    at ol
    at Object.<anonymous>.process.env.NODE_ENV.target (.../components/list/index.tsx:449:3)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at div
    at form
    at Form (.../components/forms/form.tsx:1840:19)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at section
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at SentryDocumentTitle (.../app/components/sentryDocumentTitle.tsx:708:3)
    at IssueRuleEditor (.../rules/issue/index.tsx:13364:1)
    at Wrapper (.../app/utils/withProjects.tsx:176:13)
    at Wrapper (.../app/utils/withOrganization.tsx:171:24)
    at div
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at Create (.../views/alerts/create.tsx:1335:5)
    at AlertBuilderProjectProvider (.../alerts/builder/projectProvider.tsx:962:11)
    at NoProjectMessage (.../app/components/noProjectMessage.tsx:695:3)
    at AlertsContainer (.../views/alerts/index.tsx:205:3)
    at GlobalDrawer (.../components/globalDrawer/index.tsx:1038:3)
    at QueryClientProvider (.../sentry/sentry/node_modules/@.../react-query/src/QueryClientProvider.tsx:30:3)
    at ThemeProvider (.../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:125:32)
    at LegacyRouterProvider (.../js/sentry-test/reactTestingLibrary.tsx:72:3)
    at .../js/sentry-test/reactTestingLibrary.tsx:89:5
    at console.captureMessage [as error] (.../sentry/sentry/node_modules/jest-fail-on-console/index.js:83:25)
    at printWarning (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:86:30)
    at error (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:60:7)
    at warnIfUpdatesNotWrappedWithActDEV (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:27589:9)
    at scheduleUpdateOnFiber (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:25508:5)
    at dispatchSetState (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:17527:7)
    at .../rules/issue/previewIssues.tsx:1947:7

Warning: An update to AlertBuilderProjectProvider inside a test was not wrapped in act(...).

When testing, code that causes React state updates should be wrapped into act(...):

act(() => {
  /* fire events that update state */
});
/* assert on the output */

This ensures that you're testing the behavior the user would see in the browser. Learn more at https://reactjs.org/link/wrap-tests-with-act
    at AlertBuilderProjectProvider (.../alerts/builder/projectProvider.tsx:962:11)
    at NoProjectMessage (.../app/components/noProjectMessage.tsx:695:3)
    at AlertsContainer (.../views/alerts/index.tsx:205:3)
    at GlobalDrawer (.../components/globalDrawer/index.tsx:1038:3)
    at QueryClientProvider (.../sentry/sentry/node_modules/@.../react-query/src/QueryClientProvider.tsx:30:3)
    at ThemeProvider (.../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:125:32)
    at LegacyRouterProvider (.../js/sentry-test/reactTestingLibrary.tsx:72:3)
    at .../js/sentry-test/reactTestingLibrary.tsx:89:5
    at console.captureMessage [as error] (.../sentry/sentry/node_modules/jest-fail-on-console/index.js:83:25)
    at printWarning (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:86:30)
    at error (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:60:7)
    at warnIfUpdatesNotWrappedWithActDEV (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:27589:9)
    at scheduleUpdateOnFiber (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:25508:5)
    at dispatchSetState (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:17527:7)
    at .../alerts/builder/projectProvider.tsx:1085:9

Warning: An update to PreviewIssues inside a test was not wrapped in act(...).

When testing, code that causes React state updates should be wrapped into act(...):

act(() => {
  /* fire events that update state */
});
/* assert on the output */

This ensures that you're testing the behavior the user would see in the browser. Learn more at https://reactjs.org/link/wrap-tests-with-act
    at PreviewIssues (.../rules/issue/previewIssues.tsx:1703:3)
    at ErrorBoundary (.../app/components/errorBoundary.tsx:976:1)
    at ol
    at Object.<anonymous>.process.env.NODE_ENV.target (.../components/list/index.tsx:449:3)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at div
    at form
    at Form (.../components/forms/form.tsx:1840:19)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at section
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at SentryDocumentTitle (.../app/components/sentryDocumentTitle.tsx:708:3)
    at IssueRuleEditor (.../rules/issue/index.tsx:13364:1)
    at Wrapper (.../app/utils/withProjects.tsx:176:13)
    at Wrapper (.../app/utils/withOrganization.tsx:171:24)
    at div
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at Create (.../views/alerts/create.tsx:1335:5)
    at AlertBuilderProjectProvider (.../alerts/builder/projectProvider.tsx:962:11)
    at NoProjectMessage (.../app/components/noProjectMessage.tsx:695:3)
    at AlertsContainer (.../views/alerts/index.tsx:205:3)
    at GlobalDrawer (.../components/globalDrawer/index.tsx:1038:3)
    at QueryClientProvider (.../sentry/sentry/node_modules/@.../react-query/src/QueryClientProvider.tsx:30:3)
    at ThemeProvider (.../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:125:32)
    at LegacyRouterProvider (.../js/sentry-test/reactTestingLibrary.tsx:72:3)
    at .../js/sentry-test/reactTestingLibrary.tsx:89:5
    at console.captureMessage [as error] (.../sentry/sentry/node_modules/jest-fail-on-console/index.js:83:25)
    at printWarning (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:86:30)
    at error (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:60:7)
    at warnIfUpdatesNotWrappedWithActDEV (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:27589:9)
    at scheduleUpdateOnFiber (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:25508:5)
    at dispatchSetState (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:17527:7)
    at .../rules/issue/previewIssues.tsx:1926:7

Warning: An update to PreviewIssues inside a test was not wrapped in act(...).

When testing, code that causes React state updates should be wrapped into act(...):

act(() => {
  /* fire events that update state */
});
/* assert on the output */

This ensures that you're testing the behavior the user would see in the browser. Learn more at https://reactjs.org/link/wrap-tests-with-act
    at PreviewIssues (.../rules/issue/previewIssues.tsx:1703:3)
    at ErrorBoundary (.../app/components/errorBoundary.tsx:976:1)
    at ol
    at Object.<anonymous>.process.env.NODE_ENV.target (.../components/list/index.tsx:449:3)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at div
    at form
    at Form (.../components/forms/form.tsx:1840:19)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at section
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at SentryDocumentTitle (.../app/components/sentryDocumentTitle.tsx:708:3)
    at IssueRuleEditor (.../rules/issue/index.tsx:13364:1)
    at Wrapper (.../app/utils/withProjects.tsx:176:13)
    at Wrapper (.../app/utils/withOrganization.tsx:171:24)
    at div
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at Create (.../views/alerts/create.tsx:1335:5)
    at AlertBuilderProjectProvider (.../alerts/builder/projectProvider.tsx:962:11)
    at NoProjectMessage (.../app/components/noProjectMessage.tsx:695:3)
    at AlertsContainer (.../views/alerts/index.tsx:205:3)
    at GlobalDrawer (.../components/globalDrawer/index.tsx:1038:3)
    at QueryClientProvider (.../sentry/sentry/node_modules/@.../react-query/src/QueryClientProvider.tsx:30:3)
    at ThemeProvider (.../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:125:32)
    at LegacyRouterProvider (.../js/sentry-test/reactTestingLibrary.tsx:72:3)
    at .../js/sentry-test/reactTestingLibrary.tsx:89:5
    at console.captureMessage [as error] (.../sentry/sentry/node_modules/jest-fail-on-console/index.js:83:25)
    at printWarning (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:86:30)
    at error (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:60:7)
    at warnIfUpdatesNotWrappedWithActDEV (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:27589:9)
    at scheduleUpdateOnFiber (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:25508:5)
    at dispatchSetState (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:17527:7)
    at .../rules/issue/previewIssues.tsx:1934:7

Warning: An update to PreviewIssues inside a test was not wrapped in act(...).

When testing, code that causes React state updates should be wrapped into act(...):

act(() => {
  /* fire events that update state */
});
/* assert on the output */

This ensures that you're testing the behavior the user would see in the browser. Learn more at https://reactjs.org/link/wrap-tests-with-act
    at PreviewIssues (.../rules/issue/previewIssues.tsx:1703:3)
    at ErrorBoundary (.../app/components/errorBoundary.tsx:976:1)
    at ol
    at Object.<anonymous>.process.env.NODE_ENV.target (.../components/list/index.tsx:449:3)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at div
    at form
    at Form (.../components/forms/form.tsx:1840:19)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at section
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at SentryDocumentTitle (.../app/components/sentryDocumentTitle.tsx:708:3)
    at IssueRuleEditor (.../rules/issue/index.tsx:13364:1)
    at Wrapper (.../app/utils/withProjects.tsx:176:13)
    at Wrapper (.../app/utils/withOrganization.tsx:171:24)
    at div
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at Create (.../views/alerts/create.tsx:1335:5)
    at AlertBuilderProjectProvider (.../alerts/builder/projectProvider.tsx:962:11)
    at NoProjectMessage (.../app/components/noProjectMessage.tsx:695:3)
    at AlertsContainer (.../views/alerts/index.tsx:205:3)
    at GlobalDrawer (.../components/globalDrawer/index.tsx:1038:3)
    at QueryClientProvider (.../sentry/sentry/node_modules/@.../react-query/src/QueryClientProvider.tsx:30:3)
    at ThemeProvider (.../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:125:32)
    at LegacyRouterProvider (.../js/sentry-test/reactTestingLibrary.tsx:72:3)
    at .../js/sentry-test/reactTestingLibrary.tsx:89:5
    at console.captureMessage [as error] (.../sentry/sentry/node_modules/jest-fail-on-console/index.js:83:25)
    at printWarning (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:86:30)
    at error (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:60:7)
    at warnIfUpdatesNotWrappedWithActDEV (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:27589:9)
    at scheduleUpdateOnFiber (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:25508:5)
    at dispatchSetState (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:17527:7)
    at .../rules/issue/previewIssues.tsx:1937:7

Warning: An update to PreviewIssues inside a test was not wrapped in act(...).

When testing, code that causes React state updates should be wrapped into act(...):

act(() => {
  /* fire events that update state */
});
/* assert on the output */

This ensures that you're testing the behavior the user would see in the browser. Learn more at https://reactjs.org/link/wrap-tests-with-act
    at PreviewIssues (.../rules/issue/previewIssues.tsx:1703:3)
    at ErrorBoundary (.../app/components/errorBoundary.tsx:976:1)
    at ol
    at Object.<anonymous>.process.env.NODE_ENV.target (.../components/list/index.tsx:449:3)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at div
    at form
    at Form (.../components/forms/form.tsx:1840:19)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at section
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at SentryDocumentTitle (.../app/components/sentryDocumentTitle.tsx:708:3)
    at IssueRuleEditor (.../rules/issue/index.tsx:13364:1)
    at Wrapper (.../app/utils/withProjects.tsx:176:13)
    at Wrapper (.../app/utils/withOrganization.tsx:171:24)
    at div
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at Create (.../views/alerts/create.tsx:1335:5)
    at AlertBuilderProjectProvider (.../alerts/builder/projectProvider.tsx:962:11)
    at NoProjectMessage (.../app/components/noProjectMessage.tsx:695:3)
    at AlertsContainer (.../views/alerts/index.tsx:205:3)
    at GlobalDrawer (.../components/globalDrawer/index.tsx:1038:3)
    at QueryClientProvider (.../sentry/sentry/node_modules/@.../react-query/src/QueryClientProvider.tsx:30:3)
    at ThemeProvider (.../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:125:32)
    at LegacyRouterProvider (.../js/sentry-test/reactTestingLibrary.tsx:72:3)
    at .../js/sentry-test/reactTestingLibrary.tsx:89:5
    at console.captureMessage [as error] (.../sentry/sentry/node_modules/jest-fail-on-console/index.js:83:25)
    at printWarning (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:86:30)
    at error (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:60:7)
    at warnIfUpdatesNotWrappedWithActDEV (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:27589:9)
    at scheduleUpdateOnFiber (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:25508:5)
    at dispatchSetState (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:17527:7)
    at .../rules/issue/previewIssues.tsx:1944:7

Warning: An update to PreviewIssues inside a test was not wrapped in act(...).

When testing, code that causes React state updates should be wrapped into act(...):

act(() => {
  /* fire events that update state */
});
/* assert on the output */

This ensures that you're testing the behavior the user would see in the browser. Learn more at https://reactjs.org/link/wrap-tests-with-act
    at PreviewIssues (.../rules/issue/previewIssues.tsx:1703:3)
    at ErrorBoundary (.../app/components/errorBoundary.tsx:976:1)
    at ol
    at Object.<anonymous>.process.env.NODE_ENV.target (.../components/list/index.tsx:449:3)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at div
    at form
    at Form (.../components/forms/form.tsx:1840:19)
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at section
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at SentryDocumentTitle (.../app/components/sentryDocumentTitle.tsx:708:3)
    at IssueRuleEditor (.../rules/issue/index.tsx:13364:1)
    at Wrapper (.../app/utils/withProjects.tsx:176:13)
    at Wrapper (.../app/utils/withOrganization.tsx:171:24)
    at div
    at .../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:62:23
    at Create (.../views/alerts/create.tsx:1335:5)
    at AlertBuilderProjectProvider (.../alerts/builder/projectProvider.tsx:962:11)
    at NoProjectMessage (.../app/components/noProjectMessage.tsx:695:3)
    at AlertsContainer (.../views/alerts/index.tsx:205:3)
    at GlobalDrawer (.../components/globalDrawer/index.tsx:1038:3)
    at QueryClientProvider (.../sentry/sentry/node_modules/@.../react-query/src/QueryClientProvider.tsx:30:3)
    at ThemeProvider (.../sentry/sentry/node_modules/@.../react/dist/emotion-element-48d2c2e4.cjs.dev.js:125:32)
    at LegacyRouterProvider (.../js/sentry-test/reactTestingLibrary.tsx:72:3)
    at .../js/sentry-test/reactTestingLibrary.tsx:89:5
    at console.captureMessage [as error] (.../sentry/sentry/node_modules/jest-fail-on-console/index.js:83:25)
    at printWarning (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:86:30)
    at error (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:60:7)
    at warnIfUpdatesNotWrappedWithActDEV (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:27589:9)
    at scheduleUpdateOnFiber (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:25508:5)
    at dispatchSetState (.../sentry/sentry/node_modules/react-dom/cjs/react-dom.development.js:17527:7)
    at .../rules/issue/previewIssues.tsx:1947:7
    at flushUnexpectedConsoleCalls (.../sentry/sentry/node_modules/jest-fail-on-console/index.js:48:13)
    at Object.<anonymous> (.../sentry/sentry/node_modules/jest-fail-on-console/index.js:139:7)
    at Promise.then.completed (.../sentry/sentry/node_modules/jest-circus/build/utils.js:298:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../sentry/sentry/node_modules/jest-circus/build/utils.js:231:10)
    at _callCircusHook (.../sentry/sentry/node_modules/jest-circus/build/run.js:281:40)
    at _runTest (.../sentry/sentry/node_modules/jest-circus/build/run.js:254:5)
    at _runTestsForDescribeBlock (.../sentry/sentry/node_modules/jest-circus/build/run.js:126:9)
    at _runTestsForDescribeBlock (.../sentry/sentry/node_modules/jest-circus/build/run.js:121:9)
    at _runTestsForDescribeBlock (.../sentry/sentry/node_modules/jest-circus/build/run.js:121:9)
    at run (.../sentry/sentry/node_modules/jest-circus/build/run.js:71:3)
    at runAndTransformResultsToJestFormat (.../sentry/sentry/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../sentry/sentry/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../sentry/sentry/node_modules/jest-runner/build/runTest.js:367:16)
    at runTest (.../sentry/sentry/node_modules/jest-runner/build/runTest.js:444:34)
    at Object.worker (.../sentry/sentry/node_modules/jest-runner/build/testWorker.js:106:12)
ProjectAlertsCreate test preview chart invalid preview alert ProjectAlertsCreate test preview chart invalid preview alert
Stack Traces | 0.74s run time
TestingLibraryElementError: Unable to find an element with the text: Select a condition to generate a preview. This could be because the text is broken up by multiple elements. In this case, you can provide a function for your text matcher to make your matcher more flexible.

Ignored nodes: comments, script, style
...
    at Object.getElementError (.../sentry/sentry/node_modules/@.../dom/dist/config.js:37:19)
    at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:76:38
    at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:52:17
    at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:95:19
    at Object.<anonymous> (.../views/alerts/create.spec.tsx:501:42)
ProjectAlertsCreate displays noisy alert checkbox for no conditions + filters ProjectAlertsCreate displays noisy alert checkbox for no conditions + filters
Stack Traces | 0.909s run time
TestingLibraryElementError: Unable to find an element with the text: /Alerts without conditions can fire too frequently/. This could be because the text is broken up by multiple elements. In this case, you can provide a function for your text matcher to make your matcher more flexible.

Ignored nodes: comments, script, style
...
    at Object.getElementError (.../sentry/sentry/node_modules/@.../dom/dist/config.js:37:19)
    at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:76:38
    at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:52:17
    at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:95:19
    at Object.<anonymous> (.../views/alerts/create.spec.tsx:572:40)

To view individual test run time comparison to the main branch, go to the Test Analytics Dashboard

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant