Skip to content

fix: deprecate this action #129

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.

Sign up for GitHub

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

Jump to bottom
Merged
thomasrockhu-codecov merged 1 commit into from
Nov 4, 2025
Merged

fix: deprecate this action #129

thomasrockhu-codecov merged 1 commit into from
Nov 4, 2025

Conversation

@thomasrockhu-codecov
Copy link
Contributor

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

No description provided.

sentry[bot]
sentry bot reviewed Oct 28, 2025
View reviewed changes
@sentry
Copy link

sentry bot commented Nov 4, 2025 •
edited
Loading

❌ 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
Copy link

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

❌ 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
vivodi added a commit to vivodi/Flexget that referenced this pull request Nov 6, 2025
@vivodi
chore: migrate to codecov/codecov-action
3882e35 
Since `codecov/test-results-action` has been deprecated (codecov/test-results-action#129), we should migrate to `codecov/codecov-action`.
vivodi added a commit to vivodi/Flexget that referenced this pull request Nov 8, 2025
@vivodi
chore: migrate to codecov/codecov-action
2e64120 
Since `codecov/test-results-action` has been deprecated (codecov/test-results-action#129), we should migrate to `codecov/codecov-action`.
@vivodi vivodi mentioned this pull request Nov 8, 2025
Closed
vivodi added a commit to vivodi/Flexget that referenced this pull request Nov 8, 2025
@vivodi
chore: migrate to codecov/codecov-action
f4739d0 
Since `codecov/test-results-action` has been deprecated (codecov/test-results-action#129), we should migrate to `codecov/codecov-action`.
vivodi added a commit to vivodi/Flexget that referenced this pull request Nov 8, 2025
@vivodi
chore: migrate to codecov/codecov-action
ead8746 
Since `codecov/test-results-action` has been deprecated (codecov/test-results-action#129), we should migrate to `codecov/codecov-action`.
vivodi added a commit to vivodi/Flexget that referenced this pull request Nov 8, 2025
@vivodi
chore: migrate to codecov/codecov-action
28c0f73 
Since `codecov/test-results-action` has been deprecated (codecov/test-results-action#129), we should migrate to `codecov/codecov-action`.
vivodi added a commit to vivodi/Flexget that referenced this pull request Nov 8, 2025
@vivodi
chore: migrate to codecov/codecov-action
2ecd0e5 
Since `codecov/test-results-action` has been deprecated (codecov/test-results-action#129), we should migrate to `codecov/codecov-action`.
vivodi added a commit to vivodi/Flexget that referenced this pull request Nov 8, 2025
@vivodi
chore: migrate to codecov/codecov-action
3e1d082 
Since `codecov/test-results-action` has been deprecated (codecov/test-results-action#129), we should migrate to `codecov/codecov-action`.
@Bastian-Krause
Copy link

Bastian-Krause commented Nov 10, 2025

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

@Bastian-Krause
Copy link

Bastian-Krause commented Nov 10, 2025 •
edited
Loading

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
Copy link

amkisko commented Dec 17, 2025

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

Waiting for the answer too

@thomasrockhu-codecov Linear
Copy link
Contributor Author

thomasrockhu-codecov commented Dec 17, 2025

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

@calvin-codecov calvin-codecov calvin-codecov approved these changes

+1 more reviewer

@sentry sentry[bot] sentry[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

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