rename caller to callingFn

Author: nicholas-codecov

src/shared/api/rejectNetworkError.test.ts

@@ -41,22 +41,22 @@ const parsingError: ParsingErrorObject = {
   errorName: 'Parsing Error',
   errorDetails: {
     error: Error('bad parsing'),
-    caller: 'TestQueryOpts',
+    callingFn: 'TestQueryOpts',
   },
 }
 
 const notFoundError: NotFoundErrorObject = {
   errorName: 'Not Found Error',
   errorDetails: {
-    caller: 'TestQueryOpts',
+    callingFn: 'TestQueryOpts',
   },
 }
 
 const ownerNotActivatedError: OwnerNotActivatedErrorObject = {
   errorName: 'Owner Not Activated',
   data: { detail: 'test' },
   errorDetails: {
-    caller: 'TestQueryOpts',
+    callingFn: 'TestQueryOpts',
   },
 }
 
@@ -76,7 +76,7 @@ describe('rejectNetworkError', () => {
         expect(mocks.addBreadcrumb).toHaveBeenCalledWith({
           category: 'network.error',
           level,
-          message: `${errorObject.errorDetails.caller} - ${errorObject.errorName}`,
+          message: `${errorObject.errorDetails.callingFn} - ${errorObject.errorName}`,
           data:
             'error' in errorObject.errorDetails
               ? errorObject.errorDetails.error
@@ -88,7 +88,7 @@ describe('rejectNetworkError', () => {
         _rejectNetworkError(errorObject).catch((_e) => {})
 
         expect(mocks.setTags).toHaveBeenCalledWith({
-          caller: errorObject.errorDetails.caller,
+          callingFn: errorObject.errorDetails.callingFn,
           errorName: errorObject.errorName,
         })
       })
@@ -102,7 +102,7 @@ describe('rejectNetworkError', () => {
       it('sets the fingerprint', () => {
         _rejectNetworkError(errorObject).catch((_e) => {
           expect(mocks.setFingerprint).toHaveBeenCalledWith([
-            `${errorObject.errorDetails.caller} - ${errorObject.errorName}`,
+            `${errorObject.errorDetails.callingFn} - ${errorObject.errorName}`,
           ])
         })
       })
@@ -111,15 +111,15 @@ describe('rejectNetworkError', () => {
         _rejectNetworkError(errorObject).catch((_e) => {})
 
         expect(mocks.captureMessage).toHaveBeenCalledWith(
-          `${errorObject.errorDetails.caller} - ${errorObject.errorName}`
+          `${errorObject.errorDetails.callingFn} - ${errorObject.errorName}`
         )
       })
 
       it('returns a rejected promise', () => {
         const result = _rejectNetworkError(errorObject)
 
         expect(result).rejects.toStrictEqual({
-          dev: `${errorObject.errorDetails.caller} - ${errorObject.errorName}`,
+          dev: `${errorObject.errorDetails.callingFn} - ${errorObject.errorName}`,
           data: 'data' in errorObject ? errorObject.data : undefined,
           status,
         })

src/shared/api/rejectNetworkError.ts

@@ -11,18 +11,18 @@ export type NetworkErrorName =
 
 export type ParsingErrorObject = {
   errorName: ParsingError
-  errorDetails: { error: Error; caller: string }
+  errorDetails: { error: Error; callingFn: string }
 }
 
 export type NotFoundErrorObject = {
   errorName: NotFoundError
-  errorDetails: { caller: string }
+  errorDetails: { callingFn: string }
 }
 
 export type OwnerNotActivatedErrorObject = {
   errorName: OwnerNotActivatedError
   data: { detail?: React.ReactNode }
-  errorDetails: { caller: string }
+  errorDetails: { callingFn: string }
 }
 
 type NetworkErrorObject =
@@ -65,10 +65,10 @@ export function determineStatusCode(errorName: NetworkErrorName) {
 export function _rejectNetworkError(error: NetworkErrorObject) {
   const {
     errorName,
-    errorDetails: { caller },
+    errorDetails: { callingFn },
   } = error
 
-  const devMsg = `${caller} - ${errorName}`
+  const devMsg = `${callingFn} - ${errorName}`
 
   Sentry.withScope((scope) => {
     const level = determineSentryLevel(errorName)
@@ -81,7 +81,7 @@ export function _rejectNetworkError(error: NetworkErrorObject) {
     })
 
     scope.setTags({
-      caller: caller,
+      callingFn: callingFn,
       errorName: errorName,
     })