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.
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
ref: Remove repositoryDeprecated from api tests #2872
ref: Remove repositoryDeprecated from api tests #2872
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #2872 +/- ##
==========================================
- Coverage 98.48% 98.47% -0.02%
==========================================
Files 879 879
Lines 12954 12954
Branches 3408 3451 +43
==========================================
- Hits 12758 12756 -2
- Misses 192 194 +2
Partials 4 4
Continue to review full report in Codecov by Sentry.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found @@ Coverage Diff @@
## main #2872 +/- ##
==========================================
- Coverage 98.48% 98.47% -0.02%
==========================================
Files 879 879
Lines 12954 12954
Branches 3469 3408 -61
==========================================
- Hits 12758 12756 -2
- Misses 192 194 +2
Partials 4 4
Continue to review full report in Codecov by Sentry.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅ ✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #2872 +/- ##
==========================================
- Coverage 98.48% 98.47% -0.02%
==========================================
Files 879 879
Lines 12954 12954
Branches 3469 3409 -60
==========================================
- Hits 12758 12756 -2
- Misses 192 194 +2
Partials 4 4
Continue to review full report in Codecov by Sentry.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #2872 +/- ##
=======================================
- Coverage 98.49 98.47 -0.02
=======================================
Files 879 879
Lines 12954 12954
Branches 3451 3469 +18
=======================================
- Hits 12758 12756 -2
- Misses 192 194 +2
Partials 4 4
Continue to review full report in Codecov by Sentry.
|
Bundle ReportChanges will decrease total bundle size by 152 bytes ⬇️
|
Bundle ReportChanges will decrease total bundle size by 152 bytes ⬇️
|
✅ Deploy preview for gazebo ready!Previews expire after 1 month automatically.
|
@@ -407,6 +433,7 @@ describe('when using a graphql mutation', () => { | |||
|
|||
beforeAll(() => { | |||
const location = window.location | |||
// @ts-expect-error - test magic |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧙
function prefillMethod(method) { | ||
return (config) => | ||
function prefillMethod(method: string) { | ||
return (config: any) => |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's no type we can use here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Config isn't typed yet ... and until we can ... I'd rather this just be any for the time being ... even tho it hurts every bone in my body.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok as long as you feel bad about it 😆
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Feeling absolutely sick to my stomach about it
Description
This PR updates the
api
util file to TS, as well as removing therepositoryDeprecated
field in theapi
tests.Notable Changes
api
to TSrepositoryDeprecated