chore: enable codecov #12876

HuiSF · 2024-01-23T01:08:49Z

Description of changes

Enable the codecov flow again.

Temporarily disabled the the cache check on the generate package-list.json file during GH Action run, to let the Action run successfully with this PR. A separate PR is required to turn the cache check back on after merging this PR.

Issue #, if available

Description of how you validated changes

Tested GH Action with a feature branch with codecov enabled: [DO NOT MERGE] Testing #12874 (comment)

Checklist

PR description included
yarn test passes
Tests are changed or added
Relevant documentation is changed or added (and PR referenced)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

codecov-commenter · 2024-01-23T01:17:34Z

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (bdaee91) 83.65% compared to head (8e46c6d) 87.32%.
Report is 984 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #12876      +/-   ##
==========================================
+ Coverage   83.65%   87.32%   +3.67%     
==========================================
  Files         341      634     +293     
  Lines       21579    13400    -8179     
  Branches     4611     2450    -2161     
==========================================
- Hits        18051    11702    -6349     
+ Misses       3241     1698    -1543     
+ Partials      287        0     -287

Flag	Coverage Δ
adapter-nextjs	`98.64% <ø> (?)`
analytics	`62.75% <ø> (?)`
api	`∅ <ø> (?)`
api-graphql	`87.71% <ø> (?)`
api-rest	`96.28% <ø> (?)`
auth	`86.72% <ø> (?)`
aws-amplify	`100.00% <ø> (?)`
core	`88.02% <ø> (?)`
datastore-storage-adapter	`88.13% <ø> (?)`
geo	`90.12% <ø> (?)`
interactions	`96.10% <ø> (?)`
notifications	`90.35% <ø> (?)`
predictions	`86.00% <ø> (?)`
pubsub	`94.70% <ø> (?)`
rtn-push-notification	`100.00% <ø> (?)`
storage	`91.28% <ø> (?)`

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

ashika112

This is great Hui. Thanks 🚢

jimblanc

Nice!

.github/workflows/callable-unit-tests.yml

* chore(auth): debounce refreshAuthTokens (#12845) * chore: add debounce callback helper * chore: add unit tests * chore: debounce fetchAuthSession * chore: fix unit test * chore: fix bundle size limits * chore: update debounce logic * chore: update dedup logic * chore: debounce refreshAuthTokens * chore: fix bundle size * chore: address feedback * chore: fix unit test * chore: address feedback * chore: update yarn.lock * chore: address feedbak * fix(data): update DataStore to send correct Control Messages when starting (#12861) * chore(release): Publish [ci skip] - @aws-amplify/adapter-nextjs@1.0.13 - @aws-amplify/analytics@7.0.13 - @aws-amplify/api@6.0.13 - @aws-amplify/api-graphql@4.0.13 - @aws-amplify/api-rest@4.0.13 - @aws-amplify/auth@6.0.13 - aws-amplify@6.0.13 - @aws-amplify/core@6.0.13 - @aws-amplify/datastore@5.0.13 - @aws-amplify/datastore-storage-adapter@2.1.13 - @aws-amplify/geo@3.0.13 - @aws-amplify/interactions@6.0.13 - @aws-amplify/notifications@2.0.13 - @aws-amplify/predictions@6.0.13 - @aws-amplify/pubsub@6.0.13 - @aws-amplify/react-native@1.0.13 - @aws-amplify/react-native-example@0.0.14 - @aws-amplify/rtn-push-notification@1.2.13 - @aws-amplify/rtn-web-browser@1.0.13 - @aws-amplify/storage@6.0.13 - tsc-compliance-test@0.1.13 * chore(release): update API docs [ci skip] * chore: enable codecov (#12876) * docs(auth,analytics): adding in-line docs for public apis (#12882) * docs(auth,analytics): adding in-line docs for public apis * fix: unblock the max length lint * chore(react-native): use react-native 0.71.0 as the dev dep * fix: Default branch resolution when running E2E tests (#12910) * fix: Lambda auth config value (#12922) * chore: renable caching the package list in GH actions * chore: remove unused codecov package (has been deprecated) * feat(api): expose HTTP response from API errors (#12835) --------- Co-authored-by: Hui Zhao <10602282+HuiSF@users.noreply.github.com> Co-authored-by: Jim Blanchard <jim.l.blanchard@gmail.com> * chore: temporarily disable codecov GH action integration (#12928) * ci: run flaky data e2es without retry (#12758) * ci: run flaky data e2es without retry * only run on chrome * restore workflow --------- Co-authored-by: Aaron S <94858815+stocaaro@users.noreply.github.com> * fix(core): Amplify.configure dispatches Hub event with unparsed config object (#12930) * chore(repo): use typescript 5.0.2 across workspace * chore(repo): refactor tsconfig hierarchy - move the base tsconfig into the root of the workspace - unifying tsconfig tsconfig.build tsconfig.test and tsconfig.watch settings * chore(repo): promote rollup dependencies into workspace - remove unused build.js script * chore(repo): add build:watch using rollup * chore(repo): setup eslint * chore(adapter-nextjs): migrate to eslint * fix(repo): test-github-actions using js-yaml already removed API * chore(repo): add Code Spell Checker to recommended extension list * chore(repo): add formatOn actions to formattingToggle controlables * fix: not appending notification configs * chore(release): Publish [ci skip] - @aws-amplify/adapter-nextjs@1.0.14 - @aws-amplify/analytics@7.0.14 - @aws-amplify/api@6.0.14 - @aws-amplify/api-graphql@4.0.14 - @aws-amplify/api-rest@4.0.14 - @aws-amplify/auth@6.0.14 - aws-amplify@6.0.14 - @aws-amplify/core@6.0.14 - @aws-amplify/datastore@5.0.14 - @aws-amplify/datastore-storage-adapter@2.1.14 - @aws-amplify/geo@3.0.14 - @aws-amplify/interactions@6.0.14 - @aws-amplify/notifications@2.0.14 - @aws-amplify/predictions@6.0.14 - @aws-amplify/pubsub@6.0.14 - @aws-amplify/react-native@1.0.14 - @aws-amplify/react-native-example@0.0.15 - @aws-amplify/rtn-push-notification@1.2.14 - @aws-amplify/rtn-web-browser@1.0.14 - @aws-amplify/storage@6.0.14 - tsc-compliance-test@0.1.14 * Revert "chore(repo): use typescript 5.0.2 across workspace" (#12941) * Revert "chore(repo): use typescript 5.0.2 across workspace" This reverts commit e20782e. * chore(api-graphql): temporarily install rollup to package * fix(repo): rollup generated sourcemap has wrong src path (#12947) * fix(datastore): Treat head as optional to avoid errors (#12936) * fix(datastore): Treat head as optional to avoid errors * Add testing and fix follow-on issue * fix(api-graphql): wrong arguments for GET operation of a CPK model * fix: Remove special e2e treatment for validated tests (#12946) fix: Remove special integ treament for validated tests * fix(datastore): Limit docs integ test to one browser to improve stability (#12937) * fix(api-graphql): same results returned for queries on the same model with different selection set - caused by the incomplete GraphQL documents caching * Update branch to work with merged updates from main --------- Co-authored-by: israx <70438514+israx@users.noreply.github.com> Co-authored-by: David McAfee <mcafd@amazon.com> Co-authored-by: Jim Blanchard <jim.l.blanchard@gmail.com> Co-authored-by: aws-amplify-bot <aws@amazon.com> Co-authored-by: Hui Zhao <10602282+HuiSF@users.noreply.github.com> Co-authored-by: AllanZhengYP <zheallan@amazon.com> Co-authored-by: Hui Zhao <zhohz@amazon.com> Co-authored-by: Ivan Artemiev <29709626+iartemiev@users.noreply.github.com> Co-authored-by: Aaron S <94858815+stocaaro@users.noreply.github.com> Co-authored-by: ManojNB <manojnb95@gmail.com> Co-authored-by: ManojNB <manojnb@amazon.com> Co-authored-by: Francisco Rodriguez <frodriguez.cs@gmail.com>

HuiSF requested a review from a team as a code owner January 23, 2024 01:08

ashika112 previously approved these changes Jan 23, 2024

View reviewed changes

jimblanc previously approved these changes Jan 23, 2024

View reviewed changes

.github/workflows/callable-unit-tests.yml Outdated Show resolved Hide resolved

HuiSF dismissed stale reviews from jimblanc and ashika112 via 46aaa7b January 23, 2024 16:32

HuiSF force-pushed the hui/chore/enable-codecov branch from 722b9ad to 46aaa7b Compare January 23, 2024 16:32

HuiSF requested review from jimblanc and ashika112 January 23, 2024 16:33

jimblanc approved these changes Jan 23, 2024

View reviewed changes

AllanZhengYP approved these changes Jan 23, 2024

View reviewed changes

HuiSF force-pushed the hui/chore/enable-codecov branch from 46aaa7b to 6c0229a Compare January 23, 2024 18:23

cshfang approved these changes Jan 23, 2024

View reviewed changes

ashika112 approved these changes Jan 23, 2024

View reviewed changes

HuiSF force-pushed the hui/chore/enable-codecov branch from 6c0229a to da94095 Compare January 24, 2024 00:20

HuiSF closed this Jan 24, 2024

HuiSF reopened this Jan 24, 2024

chore: enable codecov

8e46c6d

HuiSF force-pushed the hui/chore/enable-codecov branch from da94095 to 8e46c6d Compare January 24, 2024 00:36

HuiSF merged commit 3629a7b into main Jan 24, 2024
30 checks passed

HuiSF deleted the hui/chore/enable-codecov branch January 24, 2024 18:17

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore: enable codecov #12876

chore: enable codecov #12876

HuiSF commented Jan 23, 2024

codecov-commenter commented Jan 23, 2024 •

edited

ashika112 left a comment

jimblanc left a comment

chore: enable codecov #12876

chore: enable codecov #12876

Conversation

HuiSF commented Jan 23, 2024

Description of changes

Issue #, if available

Description of how you validated changes

Checklist

codecov-commenter commented Jan 23, 2024 • edited

Codecov Report

ashika112 left a comment

Choose a reason for hiding this comment

jimblanc left a comment

Choose a reason for hiding this comment

codecov-commenter commented Jan 23, 2024 •

edited