Skip to content

Conversation

@Abdkhan14
Copy link
Contributor

@Abdkhan14 Abdkhan14 commented Dec 8, 2025

  • Seperated utils from traceGuards
  • Utils that are only used once are move to the respective files
  • Moved functions that are more of a util in traceGuards.tsx to traceApi/utils.tsx

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Dec 8, 2025
@codecov
Copy link

codecov bot commented Dec 8, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
12194 1 12193 10
View the top 1 failed test(s) by shortest run time
IssuesTraceTree errors only
Stack Traces | 0.006s run time
Error: expect(received).toMatchSnapshot()

Snapshot name: `IssuesTraceTree errors only 1`

- Snapshot  - 2
+ Received  + 0

@@ -9,9 +9,7 @@
    error
    error
    error
    error
    error
-   error
-   error
    collapsed
  ↵
    at Object.toMatchSnapshot (.../newTraceDetails/traceModels/issuesTraceTree.spec.tsx:102:65)
    at Promise.finally.completed (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:1559:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:1499:10)
    at _callCircusTest (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:1009:40)
    at processTicksAndRejections (node:internal/process/task_queues:105:5)
    at _runTest (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:949:3)
    at _runTestsForDescribeBlock (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:839:13)
    at _runTestsForDescribeBlock (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:829:11)
    at run (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:757:3)
    at runAndTransformResultsToJestFormat (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/jestAdapterInit.js:1920:21)
    at jestAdapter (.../sentry/node_modules/.pnpm/jest-circus@30.0.4_babel-plugin-macros@3.1..../jest-circus/build/runner.js:101:19)
    at runTestInternal (.../sentry/node_modules/.pnpm/jest-runner@30.0..../jest-runner/build/testWorker.js:272:16)
    at runTest (.../sentry/node_modules/.pnpm/jest-runner@30.0..../jest-runner/build/testWorker.js:340:7)
    at Object.worker (.../sentry/node_modules/.pnpm/jest-runner@30.0..../jest-runner/build/testWorker.js:494: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.

@Abdkhan14 Abdkhan14 changed the title feat(trace-tree-node): Moving utils away from traceguards file feat(trace-tree-node): Dissolving more typeguards Dec 10, 2025
@Abdkhan14 Abdkhan14 marked this pull request as ready for review December 10, 2025 00:16
@Abdkhan14 Abdkhan14 requested review from a team as code owners December 10, 2025 00:16
@Abdkhan14 Abdkhan14 requested a review from gggritso December 10, 2025 00:16
Removed 'color' and 'color-string' from ESM_NODE_MODULES.
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.

3 participants