Skip to content

fix: deprecate this action#129

Merged
thomasrockhu-codecov merged 1 commit into
mainfrom
th/force-deprecate-action
Nov 4, 2025
Merged

fix: deprecate this action#129
thomasrockhu-codecov merged 1 commit into
mainfrom
th/force-deprecate-action

Conversation

@thomasrockhu-codecov

@thomasrockhu-codecov thomasrockhu-codecov commented Oct 28, 2025

Copy link
Copy Markdown
Contributor

No description provided.

sentry[bot]
sentry Bot reviewed Oct 28, 2025
View reviewed changes
Comment thread src/index.ts Outdated
@sentry

sentry Bot commented Nov 4, 2025 •
edited
Loading

Copy link
Copy Markdown

❌ 4 Tests Failed:

Tests completed Failed Passed Skipped
4 4 0 0
View the top 3 failed test(s) by shortest run time 
 fully covered
Stack Traces | 0s run time 
Error: expect(received).toEqual(expected) // deep equality

Expected: false
Received: true
    at Object.<anonymous> (.../demo/coverage-test/coverage.test.ts:10:39)
    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:95:5)
    at _runTest (.../jest-circus/build/run.js:252:3)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126: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)
 subtracts 2 - 3 to equal -1
Stack Traces | 0s run time 
Error: expect(received).toBe(expected) // Object.is equality

Expected: 1
Received: -1
    at Object.<anonymous> (.../demo/calculator/calculator.test.ts:10:31)
    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:95:5)
    at _runTest (.../jest-circus/build/run.js:252:3)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126: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)
 adds 2 + 3 to equal 5
Stack Traces | 0.002s run time 
Error: expect(received).toBe(expected) // Object.is equality

Expected: 4
Received: 5
    at Object.<anonymous> (.../demo/calculator/calculator.test.ts:5:26)
    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:95:5)
    at _runTest (.../jest-circus/build/run.js:252:3)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126: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)
 test uncovered if
Stack Traces | 0.002s run time 
Error: expect(received).toEqual(expected) // deep equality

Expected: true
Received: false
    at Object.<anonymous> (.../demo/coverage-test/coverage.test.ts:5:38)
    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:95:5)
    at _runTest (.../jest-circus/build/run.js:252:3)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126: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)

To view more test analytics, go to the Prevent Tests Dashboard

@codecov-staging

codecov-staging Bot commented Nov 4, 2025 •
edited
Loading

Copy link
Copy Markdown

❌ 4 Tests Failed:

Tests completed Failed Passed Skipped
4 4 0 0
View the top 3 failed test(s) by shortest run time 
 fully covered
Stack Traces | 0s run time 
Error: expect(received).toEqual(expected) // deep equality

Expected: false
Received: true
    at Object.<anonymous> (.../demo/coverage-test/coverage.test.ts:10:39)
    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:95:5)
    at _runTest (.../jest-circus/build/run.js:252:3)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126: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)
 subtracts 2 - 3 to equal -1
Stack Traces | 0s run time 
Error: expect(received).toBe(expected) // Object.is equality

Expected: 1
Received: -1
    at Object.<anonymous> (.../demo/calculator/calculator.test.ts:10:31)
    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:95:5)
    at _runTest (.../jest-circus/build/run.js:252:3)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126: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)
 adds 2 + 3 to equal 5
Stack Traces | 0.002s run time 
Error: expect(received).toBe(expected) // Object.is equality

Expected: 4
Received: 5
    at Object.<anonymous> (.../demo/calculator/calculator.test.ts:5:26)
    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:95:5)
    at _runTest (.../jest-circus/build/run.js:252:3)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126: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)
 test uncovered if
Stack Traces | 0.002s run time 
Error: expect(received).toEqual(expected) // deep equality

Expected: true
Received: false
    at Object.<anonymous> (.../demo/coverage-test/coverage.test.ts:5:38)
    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:95:5)
    at _runTest (.../jest-circus/build/run.js:252:3)
    at _runTestsForDescribeBlock (.../jest-circus/build/run.js:126: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)

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

@thomasrockhu-codecov
thomasrockhu-codecov merged commit 3fef12b into main Nov 4, 2025
9 checks passed
@vivodi vivodi mentioned this pull request Nov 6, 2025
Merged
@Bastian-Krause

Bastian-Krause commented Nov 10, 2025

Copy link
Copy Markdown

Isn't it report_type: test_results, not report-type: test_results?

@Bastian-Krause

Bastian-Krause commented Nov 10, 2025 •
edited
Loading

Copy link
Copy Markdown

Adding to that: It's also not clear if codecov/codecov-action@v5 can be called twice, one time with report_type: test_results and another time with report_type: coverage, if both test results and coverage should be uploaded in a CI run. Can you clarify that?

stefanceriu added a commit to element-hq/element-x-ios that referenced this pull request Dec 16, 2025
@stefanceriu
Replace the now deprecated codecov/test-results-action
b0065ba 
As per codecov/test-results-action#129
stefanceriu added a commit to element-hq/element-x-ios that referenced this pull request Dec 16, 2025
@stefanceriu
Replace the now deprecated codecov/test-results-action
3e9a231 
As per codecov/test-results-action#129
@ReenigneArcher ReenigneArcher mentioned this pull request Dec 16, 2025
Merged
21 tasks
@crenshaw-dev crenshaw-dev mentioned this pull request Dec 17, 2025
Merged
@amkisko

amkisko commented Dec 17, 2025

Copy link
Copy Markdown

It's also not clear if codecov/codecov-action@v5 can be called twice

Waiting for the answer too

@thomasrockhu-codecov Linear

thomasrockhu-codecov commented Dec 17, 2025

Copy link
Copy Markdown
Contributor Author

Sorry, I didn't realize that was a question or to update the repo. Yes, the action should be called twice, once for test analytics and once for coverage

This was referenced Dec 19, 2025
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sentry sentry[bot] sentry[bot] left review comments
@calvin-codecov calvin-codecov calvin-codecov approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@thomasrockhu-codecov @Bastian-Krause @amkisko @calvin-codecov