feat(onboarding): use ProjectOverview endpoint for polling #86386

constantinius · 2025-03-05T15:23:06Z

Contributes to https://github.com/getsentry/projects/issues/789

codecov · 2025-03-05T15:27:35Z

❌ 3 Tests Failed:

Tests completed	Failed	Passed	Skipped
9635	3	9632	4

View the top 3 failed test(s) by shortest run time

ProjectInstallPlatform should render getting started docs for correct platform

Stack Traces | 0.137s 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.



Error: No mocked response found for request: GET .../org-slug/project-slug/overview/
    at Object.<anonymous> (.../views/projectInstall/platformOrIntegration.spec.tsx:51:37)
    at Promise.then.completed (.../jest-circus/build/utils.js:298:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../jest-circus/build/utils.js:231:10)
    at _callCircusTest (.../jest-circus/build/run.js:316:40)
    at processTicksAndRejections (node:internal/process/task_queues:105:5)
    at _runTest (.../jest-circus/build/run.js:252:3)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126:9)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:121:9)
    at run (.../jest-circus/build/run.js:71:3)
    at runAndTransformResultsToJestFormat (.../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../jest-runner/build/runTest.js:367:16)
    at runTest (.../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../jest-runner/build/testWorker.js:106:12)
    at console.captureMessage [as error] (.../node_modules/jest-fail-on-console/index.js:83:25)
    at Object.<anonymous> (.../app/__mocks__/api.tsx:45:15)
    at Promise.then.completed (.../jest-circus/build/utils.js:298:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../jest-circus/build/utils.js:231:10)
    at _callCircusHook (.../jest-circus/build/run.js:281:40)
    at runNextTicks (node:internal/process/task_queues:65:5)
    at listOnTimeout (node:internal/timers:555:9)
    at processTimers (node:internal/timers:529:7)
    at _runTest (.../jest-circus/build/run.js:254:5)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126:9)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:121:9)
    at run (.../jest-circus/build/run.js:71:3)
    at runAndTransformResultsToJestFormat (.../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../jest-runner/build/runTest.js:367:16)
    at runTest (.../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../jest-runner/build/testWorker.js:106:12)
    at flushUnexpectedConsoleCalls (.../node_modules/jest-fail-on-console/index.js:48:13)
    at Object.<anonymous> (.../node_modules/jest-fail-on-console/index.js:139:7)
    at Promise.then.completed (.../jest-circus/build/utils.js:298:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../jest-circus/build/utils.js:231:10)
    at _callCircusHook (.../jest-circus/build/run.js:281:40)
    at runNextTicks (node:internal/process/task_queues:65:5)
    at listOnTimeout (node:internal/timers:555:9)
    at processTimers (node:internal/timers:529:7)
    at _runTest (.../jest-circus/build/run.js:254:5)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126:9)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:121:9)
    at run (.../jest-circus/build/run.js:71:3)
    at runAndTransformResultsToJestFormat (.../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../jest-runner/build/runTest.js:367:16)
    at runTest (.../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../jest-runner/build/testWorker.js:106:12)

ProjectInstallPlatform should display info for a non-supported platform

Stack Traces | 0.191s 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.



Error: No mocked response found for request: GET .../org-slug/project-slug/overview/
    at Object.<anonymous> (.../views/projectInstall/platform.spec.tsx:107:37)
    at Promise.then.completed (.../jest-circus/build/utils.js:298:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../jest-circus/build/utils.js:231:10)
    at _callCircusTest (.../jest-circus/build/run.js:316:40)
    at runNextTicks (node:internal/process/task_queues:65:5)
    at processTimers (node:internal/timers:526:9)
    at _runTest (.../jest-circus/build/run.js:252:3)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126:9)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:121:9)
    at run (.../jest-circus/build/run.js:71:3)
    at runAndTransformResultsToJestFormat (.../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../jest-runner/build/runTest.js:367:16)
    at runTest (.../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../jest-runner/build/testWorker.js:106:12)
    at console.captureMessage [as error] (.../node_modules/jest-fail-on-console/index.js:83:25)
    at Object.<anonymous> (.../app/__mocks__/api.tsx:45:15)
    at Promise.then.completed (.../jest-circus/build/utils.js:298:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../jest-circus/build/utils.js:231:10)
    at _callCircusHook (.../jest-circus/build/run.js:281:40)
    at _runTest (.../jest-circus/build/run.js:254:5)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126:9)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:121:9)
    at run (.../jest-circus/build/run.js:71:3)
    at runAndTransformResultsToJestFormat (.../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../jest-runner/build/runTest.js:367:16)
    at runTest (.../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../jest-runner/build/testWorker.js:106:12)
    at flushUnexpectedConsoleCalls (.../node_modules/jest-fail-on-console/index.js:48:13)
    at Object.<anonymous> (.../node_modules/jest-fail-on-console/index.js:139:7)
    at Promise.then.completed (.../jest-circus/build/utils.js:298:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../jest-circus/build/utils.js:231:10)
    at _callCircusHook (.../jest-circus/build/run.js:281:40)
    at _runTest (.../jest-circus/build/run.js:254:5)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126:9)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:121:9)
    at run (.../jest-circus/build/run.js:71:3)
    at runAndTransformResultsToJestFormat (.../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../jest-runner/build/runTest.js:367:16)
    at runTest (.../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../jest-runner/build/testWorker.js:106:12)

ProjectInstallPlatform should not render performance/session replay buttons for errors only self-hosted

Stack Traces | 0.406s 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.



Error: No mocked response found for request: GET .../org-slug/project-slug/overview/
    at Object.<anonymous> (.../views/projectInstall/platform.spec.tsx:142:37)
    at Promise.then.completed (.../jest-circus/build/utils.js:298:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../jest-circus/build/utils.js:231:10)
    at _callCircusTest (.../jest-circus/build/run.js:316:40)
    at _runTest (.../jest-circus/build/run.js:252:3)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126:9)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:121:9)
    at run (.../jest-circus/build/run.js:71:3)
    at runAndTransformResultsToJestFormat (.../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../jest-runner/build/runTest.js:367:16)
    at runTest (.../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../jest-runner/build/testWorker.js:106:12)
    at console.captureMessage [as error] (.../node_modules/jest-fail-on-console/index.js:83:25)
    at Object.<anonymous> (.../app/__mocks__/api.tsx:45:15)
    at Promise.then.completed (.../jest-circus/build/utils.js:298:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../jest-circus/build/utils.js:231:10)
    at _callCircusHook (.../jest-circus/build/run.js:281:40)
    at runNextTicks (node:internal/process/task_queues:65:5)
    at listOnTimeout (node:internal/timers:555:9)
    at processTimers (node:internal/timers:529:7)
    at _runTest (.../jest-circus/build/run.js:254:5)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126:9)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:121:9)
    at run (.../jest-circus/build/run.js:71:3)
    at runAndTransformResultsToJestFormat (.../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../jest-runner/build/runTest.js:367:16)
    at runTest (.../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../jest-runner/build/testWorker.js:106:12)
    at flushUnexpectedConsoleCalls (.../node_modules/jest-fail-on-console/index.js:48:13)
    at Object.<anonymous> (.../node_modules/jest-fail-on-console/index.js:139:7)
    at Promise.then.completed (.../jest-circus/build/utils.js:298:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../jest-circus/build/utils.js:231:10)
    at _callCircusHook (.../jest-circus/build/run.js:281:40)
    at runNextTicks (node:internal/process/task_queues:65:5)
    at listOnTimeout (node:internal/timers:555:9)
    at processTimers (node:internal/timers:529:7)
    at _runTest (.../jest-circus/build/run.js:254:5)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126:9)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:121:9)
    at run (.../jest-circus/build/run.js:71:3)
    at runAndTransformResultsToJestFormat (.../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../jest-runner/build/runTest.js:367:16)
    at runTest (.../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../jest-runner/build/testWorker.js:106:12)

To view more test analytics, go to the Test Analytics Dashboard
_{📋 Got 3 mins? Take this short survey to help us improve Test Analytics.}

codecov · 2025-03-05T17:09:18Z

Bundle Report

Changes will increase total bundle size by 13.4kB (0.03%) ⬆️. This is within the configured threshold ✅

Detailed changes

Bundle name	Size	Change
app-webpack-bundle-array-push	42.47MB	13.4kB (0.03%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: app-webpack-bundle-array-push

Assets Changed:

Asset Name	Size Change	Total Size	Change (%)
*`chunks/app_components_avatar_actorAvatar_tsx-app_components_avatar_avatarList_tsx-app_components_cha-7ec24d..js`** (New)	1.38MB	1.38MB	100.0% 🚀
*`chunks/app_components_assigneeBadge_tsx-app_components_charts_components_markLine_tsx-app_components-38886d..js`** (New)	1.16MB	1.16MB	100.0% 🚀
`chunks/app_actionCreators_redirectToProject_tsx-app_actionCreators_sudoModal_tsx-app_components_aler-d18754.*.js`	-6 bytes	453.19kB	-0.0%
`chunks/app_views_issueDetails_groupEventDetails_groupEventDetails_tsx.*.js`	-265 bytes	361.32kB	-0.07%
`chunks/app_actionCreators_preferences_tsx-app_components_avatar_actorAvatar_tsx-app_components_confi-71c9ce.*.js`	191 bytes	301.39kB	0.06%
`chunks/app_utils_duration_intervalToMilliseconds_tsx-app_views_dashboards_dashboard_tsx-app_views_da-03e4f7.*.js`	27 bytes	293.85kB	0.01%
*`chunks/app_actionCreators_members_tsx-app_components_avatar_avatarList_tsx-app_components_deprecated-76913d..js`** (New)	162.41kB	162.41kB	100.0% 🚀
*`chunks/app_components_feedback_widget_uskWidget_tsx-app_views_projectDetail_index_tsx.3fb9407eb8a2132bde57.js`** (New)	125.93kB	125.93kB	100.0% 🚀
`entrypoints/app.js`	41 bytes	108.5kB	0.04%
`chunks/app_views_insights_pages_backend_backendOverviewPage_tsx-node_modules_lodash_partialRight_js.*.js`	482 bytes	44.82kB	1.09%
`chunks/app_views_settings_project_projectOwnership_index_tsx.*.js`	-4 bytes	40.44kB	-0.01%
`entrypoints/pipeline.js`	41 bytes	37.41kB	0.11%
`chunks/app_components_compactSelect_list_tsx-app_views_settings_components_text_textBlock_tsx.*.js`	-3.46kB	37.3kB	-8.48%
*`chunks/app_components_charts_eventsRequest_tsx-app_components_feedback_widget_uskWidget_tsx--fbb1b9.f829f62db89184ceaab8.js`** (New)	31.1kB	31.1kB	100.0% 🚀
*`chunks/app_components_emptyStateWarning_tsx-app_components_modals_bulkEditMonitorsModal_tsx-app_util-63f737..js`** (New)	28.58kB	28.58kB	100.0% 🚀
*`chunks/app_components_feedback_widget_uskWidget_tsx-app_views_performance_transactionSummary-51d702.0d8e93ca22b39576eda0.js`** (New)	25.7kB	25.7kB	100.0% 🚀
`chunks/app_components_modals_widgetViewerModal_tsx.*.js`	-10 bytes	25.19kB	-0.04%
*`chunks/app_components_feedback_widget_uskWidget_tsx-app_components_performance_transactionSe-feae7d.d39e88a21a399a713441.js`** (New)	24.18kB	24.18kB	100.0% 🚀
`chunks/app_components_tours_tour_stories_tsx.*.js`	-5.49kB	21.97kB	-20.0%
`chunks/app_views_settings_organizationRelay_index_tsx.*.js`	-4 bytes	21.83kB	-0.02%
*`chunks/app_components_feedback_widget_uskWidget_tsx-app_utils_discover_discoverQuery_tsx-app-c5fefe.18f77be69fe17ddab774.js`** (New)	17.41kB	17.41kB	100.0% 🚀
*`chunks/app_components_feedback_widget_uskWidget_tsx-app_components_gridEditable_sortLink_tsx-d372f4.e2357db1a2c83cbd98fd.js`** (New)	16.21kB	16.21kB	100.0% 🚀
`chunks/app_components_onboarding_gettingStartedDoc_sdkDocumentation_tsx-app_components_onboarding_us-035896.*.js`	9 bytes	14.29kB	0.06%
*`chunks/app_components_feedback_widget_uskWidget_tsx-app_components_performance_transactionSe-f61270.4889b2e20d274411f343.js`** (New)	13.36kB	13.36kB	100.0% 🚀
`chunks/app_views_dashboards_widgets_bigNumberWidget_bigNumberWidgetVisualization_stories_tsx-app_vie-ded72c.*.js`	-69 bytes	12.86kB	-0.53%
`chunks/app_components_charts_components_dataZoomInside_tsx-app_components_charts_components_toolBox_-01ad31.*.js`	-69 bytes	12.45kB	-0.55%
*`chunks/app_components_charts_lineChart_tsx-app_components_feedback_widget_uskWidget_tsx-app_-26fc43.f504f8b330601746cebc.js`** (New)	12.34kB	12.34kB	100.0% 🚀
`chunks/app_views_settings_project_projectOwnership_ownerInput_tsx.*.js`	-4 bytes	11.77kB	-0.03%
*`chunks/app_components_tabs_tabList_tsx-app_components_tabs_tabPanels_tsx-app_utils_useOverlay_tsx..js`** (New)	11.3kB	11.3kB	100.0% 🚀
`chunks/app_views_dashboards_widgets_widget_widget_tsx.*.js`	-69 bytes	7.13kB	-0.96%
~~*`chunks/app_components_avatar_actorAvatar_tsx-app_components_avatar_avatarList_tsx-app_components_cha-e41958..js`*~~ (Deleted)*	-1.37MB	0 bytes	-100.0% 🗑️
~~*`chunks/app_components_assigneeBadge_tsx-app_components_charts_components_markLine_tsx-app_components-4285bc..js`*~~ (Deleted)*	-1.16MB	0 bytes	-100.0% 🗑️
~~*`chunks/app_actionCreators_members_tsx-app_components_avatar_avatarList_tsx-app_components_deprecated-c50dcc..js`*~~ (Deleted)*	-156.64kB	0 bytes	-100.0% 🗑️
~~*`chunks/app_components_feedback_widget_uskWidget_tsx-app_views_projectDetail_index_tsx.12e6348ae875388f1348.js`*~~ (Deleted)*	-125.93kB	0 bytes	-100.0% 🗑️
~~*`chunks/app_components_charts_eventsRequest_tsx-app_components_feedback_widget_uskWidget_tsx--fbb1b9.1032b843ea48495ab70c.js`*~~ (Deleted)*	-31.1kB	0 bytes	-100.0% 🗑️
~~*`chunks/app_components_feedback_widget_uskWidget_tsx-app_views_performance_transactionSummary-51d702.b1d4abae5582a56db62d.js`*~~ (Deleted)*	-25.7kB	0 bytes	-100.0% 🗑️
~~*`chunks/app_components_emptyStateWarning_tsx-app_components_modals_bulkEditMonitorsModal_tsx..js`*~~ (Deleted)*	-25.1kB	0 bytes	-100.0% 🗑️
~~*`chunks/app_components_feedback_widget_uskWidget_tsx-app_components_performance_transactionSe-feae7d.7bee8d7060d8968bc9a3.js`*~~ (Deleted)*	-24.18kB	0 bytes	-100.0% 🗑️
~~*`chunks/app_components_feedback_widget_uskWidget_tsx-app_utils_discover_discoverQuery_tsx-app-c5fefe.6eecdc978ef4ba44ced7.js`*~~ (Deleted)*	-17.41kB	0 bytes	-100.0% 🗑️
~~*`chunks/app_components_feedback_widget_uskWidget_tsx-app_components_gridEditable_sortLink_tsx-d372f4.9b8b163e3c61f4cd95bc.js`*~~ (Deleted)*	-16.21kB	0 bytes	-100.0% 🗑️
~~*`chunks/app_components_feedback_widget_uskWidget_tsx-app_components_performance_transactionSe-f61270.87b98e1746bf12afcdda.js`*~~ (Deleted)*	-13.36kB	0 bytes	-100.0% 🗑️
~~*`chunks/app_components_charts_lineChart_tsx-app_components_feedback_widget_uskWidget_tsx-app_-26fc43.c39a93965b9c6a2768f4.js`*~~ (Deleted)*	-12.34kB	0 bytes	-100.0% 🗑️
~~*`chunks/app_components_tabs_tabList_tsx-app_components_tabs_tabPanels_tsx..js`*~~ (Deleted)*	-7.8kB	0 bytes	-100.0% 🗑️

Files in chunks/app_components_onboarding_gettingStartedDoc_sdkDocumentation_tsx-app_components_onboarding_us-035896.*.js:

./app/components/onboarding/useRecentCreatedProject.tsx → Total Size: 1.41kB

feat(onboarding): use ProjectOverview endpoint for polling

f30c5bf

github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 5, 2025

constantinius requested review from priscilawebdev, ArthurKnaus and a team March 5, 2025 15:23

vgrozdanic approved these changes Mar 5, 2025

View reviewed changes

test(onboarding): fix testing to mock overview responses

4a684da

vercel bot deployed to Preview March 5, 2025 17:04 View deployment

constantinius merged commit 5b9dfa2 into master Mar 6, 2025
43 checks passed

constantinius deleted the constantinius/feat/onboarding/use-project-overview-in-polling branch March 6, 2025 07:56

philipphofmann pushed a commit that referenced this pull request Mar 6, 2025

feat(onboarding): use ProjectOverview endpoint for polling (#86386)

f1ca758

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(onboarding): use ProjectOverview endpoint for polling #86386

feat(onboarding): use ProjectOverview endpoint for polling #86386

constantinius commented Mar 5, 2025 •

edited

Loading

codecov bot commented Mar 5, 2025

codecov bot commented Mar 5, 2025 •

edited

Loading

Assets Changed:

feat(onboarding): use ProjectOverview endpoint for polling #86386

feat(onboarding): use ProjectOverview endpoint for polling #86386

Conversation

constantinius commented Mar 5, 2025 • edited Loading

codecov bot commented Mar 5, 2025

❌ 3 Tests Failed:

codecov bot commented Mar 5, 2025 • edited Loading

Bundle Report

Affected Assets, Files, and Routes:

Assets Changed:

constantinius commented Mar 5, 2025 •

edited

Loading

codecov bot commented Mar 5, 2025 •

edited

Loading