Skip to content

Commit bf4dff0

Browse files
nicholas-codecovnicholas-codecov
authored
ref: Move callingFn from string literals to a shared const (#3770)
1 parent 5a632e9 commit bf4dff0

File tree

116 files changed

+394
-732
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

116 files changed

+394
-732
lines changed

‎src/layouts/BaseLayout/hooks/NavigatorDataQueryOpts.ts

+2-4
Original file line numberDiff line numberDiff line change
@@ -67,15 +67,13 @@ export const NavigatorDataQueryOpts = ({
6767
repo,
6868
},
6969
}).then((res) => {
70+
const callingFn = 'NavigatorDataQueryOpts'
7071
const parsedData = RequestSchema.safeParse(res.data)
7172

7273
if (!parsedData.success) {
7374
return rejectNetworkError({
7475
errorName: 'Parsing Error',
75-
errorDetails: {
76-
callingFn: 'NavigatorDataQueryOpts',
77-
error: parsedData.error,
78-
},
76+
errorDetails: { callingFn, error: parsedData.error },
7977
})
8078
}
8179

‎src/pages/AccountSettings/tabs/OktaAccess/hooks/useUpdateOktaConfig.tsx

+3-4
Original file line numberDiff line numberDiff line change
@@ -103,14 +103,13 @@ export const useUpdateOktaConfig = ({ provider, owner }: URLParams) => {
103103
})
104104
},
105105
onSuccess: ({ data }) => {
106+
const callingFn = 'useUpdateOktaConfig'
106107
const parsedData = ResponseSchema.safeParse(data)
108+
107109
if (!parsedData.success) {
108110
return rejectNetworkError({
109111
errorName: 'Parsing Error',
110-
errorDetails: {
111-
callingFn: 'useUpdateOktaConfig',
112-
error: parsedData.error,
113-
},
112+
errorDetails: { callingFn, error: parsedData.error },
114113
})
115114
}
116115

‎src/pages/AccountSettings/tabs/OktaAccess/queries/OktaConfigQueryOpts.tsx

+2-4
Original file line numberDiff line numberDiff line change
@@ -62,15 +62,13 @@ export function OktaConfigQueryOpts({
6262
username,
6363
},
6464
}).then((res) => {
65+
const callingFn = 'OktaConfigQueryOpts'
6566
const parsedRes = OktaConfigRequestSchema.safeParse(res?.data)
6667

6768
if (!parsedRes.success) {
6869
return rejectNetworkError({
6970
errorName: 'Parsing Error',
70-
errorDetails: {
71-
callingFn: 'OktaConfigQueryOpts',
72-
error: parsedRes.error,
73-
},
71+
errorDetails: { callingFn, error: parsedRes.error },
7472
})
7573
}
7674

‎src/pages/AccountSettings/tabs/OrgUploadToken/TokenlessSection/useSetUploadTokenRequired.tsx

+3-4
Original file line numberDiff line numberDiff line change
@@ -81,14 +81,13 @@ export const useSetUploadTokenRequired = ({
8181
},
8282
mutationPath: 'setUploadTokenRequired',
8383
}).then((res) => {
84+
const callingFn = 'useSetUploadTokenRequired'
8485
const parsedData = ResponseSchema.safeParse(res.data)
86+
8587
if (!parsedData.success) {
8688
return rejectNetworkError({
8789
errorName: 'Parsing Error',
88-
errorDetails: {
89-
callingFn: 'useSetUploadTokenRequired',
90-
error: parsedData.error,
91-
},
90+
errorDetails: { callingFn, error: parsedData.error },
9291
})
9392
}
9493

‎src/pages/AdminSettings/AdminAccess/AdminAccessTable/useAdminAccessList.ts

+2-4
Original file line numberDiff line numberDiff line change
@@ -38,15 +38,13 @@ export const useAdminAccessList = () => {
3838
path: `/users?is_admin=true&page=${pageParam}`,
3939
signal,
4040
}).then((res) => {
41+
const callingFn = 'useAdminAccessList'
4142
const parsedData = RequestSchema.safeParse(res)
4243

4344
if (!parsedData.success) {
4445
return rejectNetworkError({
4546
errorName: 'Parsing Error',
46-
errorDetails: {
47-
callingFn: 'useAdminAccessList',
48-
error: parsedData.error,
49-
},
47+
errorDetails: { callingFn, error: parsedData.error },
5048
})
5149
}
5250
return parsedData.data

‎src/pages/CommitDetailPage/Header/HeaderDefault/queries/CommitHeaderDataQueryOpts.tsx

+4-6
Original file line numberDiff line numberDiff line change
@@ -104,15 +104,13 @@ export const CommitHeaderDataQueryOpts = ({
104104
commitId,
105105
},
106106
}).then((res) => {
107+
const callingFn = 'CommitHeaderDataQueryOpts'
107108
const parsedData = CommitHeaderDataSchema.safeParse(res?.data)
108109

109110
if (!parsedData.success) {
110111
return rejectNetworkError({
111112
errorName: 'Parsing Error',
112-
errorDetails: {
113-
callingFn: 'CommitHeaderDataQueryOpts',
114-
error: parsedData.error,
115-
},
113+
errorDetails: { callingFn, error: parsedData.error },
116114
})
117115
}
118116

@@ -121,14 +119,14 @@ export const CommitHeaderDataQueryOpts = ({
121119
if (data?.owner?.repository?.__typename === 'NotFoundError') {
122120
return rejectNetworkError({
123121
errorName: 'Not Found Error',
124-
errorDetails: { callingFn: 'CommitHeaderDataQueryOpts' },
122+
errorDetails: { callingFn },
125123
})
126124
}
127125

128126
if (data?.owner?.repository?.__typename === 'OwnerNotActivatedError') {
129127
return rejectNetworkError({
130128
errorName: 'Owner Not Activated',
131-
errorDetails: { callingFn: 'CommitHeaderDataQueryOpts' },
129+
errorDetails: { callingFn },
132130
data: {
133131
detail: (
134132
<p>

‎src/pages/CommitDetailPage/Header/HeaderTeam/queries/CommitHeaderDataTeamQueryOpts.tsx

+4-6
Original file line numberDiff line numberDiff line change
@@ -163,15 +163,13 @@ export const CommitHeaderDataTeamQueryOpts = ({
163163
commitId,
164164
},
165165
}).then((res) => {
166+
const callingFn = 'CommitHeaderDataTeamQueryOpts'
166167
const parsedData = CommitHeaderDataTeamSchema.safeParse(res?.data)
167168

168169
if (!parsedData.success) {
169170
return rejectNetworkError({
170171
errorName: 'Parsing Error',
171-
errorDetails: {
172-
callingFn: 'CommitHeaderDataTeamQueryOpts',
173-
error: parsedData.error,
174-
},
172+
errorDetails: { callingFn, error: parsedData.error },
175173
})
176174
}
177175

@@ -180,14 +178,14 @@ export const CommitHeaderDataTeamQueryOpts = ({
180178
if (data?.owner?.repository?.__typename === 'NotFoundError') {
181179
return rejectNetworkError({
182180
errorName: 'Not Found Error',
183-
errorDetails: { callingFn: 'CommitHeaderDataTeamQueryOpts' },
181+
errorDetails: { callingFn },
184182
})
185183
}
186184

187185
if (data?.owner?.repository?.__typename === 'OwnerNotActivatedError') {
188186
return rejectNetworkError({
189187
errorName: 'Owner Not Activated',
190-
errorDetails: { callingFn: 'CommitHeaderDataTeamQueryOpts' },
188+
errorDetails: { callingFn },
191189
data: {
192190
detail: (
193191
<p>

‎src/pages/CommitDetailPage/queries/CommitPageDataQueryOpts.tsx

+4-6
Original file line numberDiff line numberDiff line change
@@ -157,15 +157,13 @@ export const CommitPageDataQueryOpts = ({
157157
commitId,
158158
},
159159
}).then((res) => {
160+
const callingFn = 'CommitPageDataQueryOpts'
160161
const parsedData = CommitPageDataSchema.safeParse(res?.data)
161162

162163
if (!parsedData.success) {
163164
return rejectNetworkError({
164165
errorName: 'Parsing Error',
165-
errorDetails: {
166-
callingFn: 'CommitPageDataQueryOpts',
167-
error: parsedData.error,
168-
},
166+
errorDetails: { callingFn, error: parsedData.error },
169167
})
170168
}
171169

@@ -174,14 +172,14 @@ export const CommitPageDataQueryOpts = ({
174172
if (data?.owner?.repository?.__typename === 'NotFoundError') {
175173
return rejectNetworkError({
176174
errorName: 'Not Found Error',
177-
errorDetails: { callingFn: 'CommitPageDataQueryOpts' },
175+
errorDetails: { callingFn },
178176
})
179177
}
180178

181179
if (data?.owner?.repository?.__typename === 'OwnerNotActivatedError') {
182180
return rejectNetworkError({
183181
errorName: 'Owner Not Activated',
184-
errorDetails: { callingFn: 'CommitPageDataQueryOpts' },
182+
errorDetails: { callingFn },
185183
data: {
186184
detail: (
187185
<p>

‎src/pages/PlanPage/queries/PlanPageDataQueryOpts.ts

+2-4
Original file line numberDiff line numberDiff line change
@@ -44,15 +44,13 @@ export function PlanPageDataQueryOpts({
4444
variables,
4545
signal,
4646
}).then((res) => {
47+
const callingFn = 'PlanPageDataQueryOpts'
4748
const parsedRes = RequestSchema.safeParse(res?.data)
4849

4950
if (!parsedRes.success) {
5051
return rejectNetworkError({
5152
errorName: 'Parsing Error',
52-
errorDetails: {
53-
callingFn: 'PlanPageDataQueryOpts',
54-
error: parsedRes.error,
55-
},
53+
errorDetails: { callingFn, error: parsedRes.error },
5654
})
5755
}
5856

‎src/pages/PlanPage/subRoutes/CurrentOrgPlan/queries/EnterpriseAccountDetailsQueryOpts.ts

+2-4
Original file line numberDiff line numberDiff line change
@@ -56,17 +56,15 @@ export function EnterpriseAccountDetailsQueryOpts({
5656
owner,
5757
},
5858
}).then((res) => {
59+
const callingFn = 'EnterpriseAccountDetailsQueryOpts'
5960
const parsedRes = EnterpriseAccountDetailsRequestSchema.safeParse(
6061
res?.data
6162
)
6263

6364
if (!parsedRes.success) {
6465
return rejectNetworkError({
6566
errorName: 'Parsing Error',
66-
errorDetails: {
67-
callingFn: 'EnterpriseAccountDetailsQueryOpts',
68-
error: parsedRes.error,
69-
},
67+
errorDetails: { callingFn, error: parsedRes.error },
7068
})
7169
}
7270

‎src/pages/PlanPage/subRoutes/CurrentOrgPlan/queries/InfiniteAccountOrganizationsQueryOpts.ts

+3-7
Original file line numberDiff line numberDiff line change
@@ -98,15 +98,13 @@ export function InfiniteAccountOrganizationsQueryOpts({
9898
after,
9999
},
100100
}).then((res) => {
101+
const callingFn = 'InfiniteAccountOrganizationsQueryOpts'
101102
const parsedRes = RequestSchema.safeParse(res.data)
102103

103104
if (!parsedRes.success) {
104105
return rejectNetworkError({
105106
errorName: 'Parsing Error',
106-
errorDetails: {
107-
callingFn: 'InfiniteAccountOrganizationsQueryOpts',
108-
error: parsedRes.error,
109-
},
107+
errorDetails: { callingFn, error: parsedRes.error },
110108
})
111109
}
112110

@@ -115,9 +113,7 @@ export function InfiniteAccountOrganizationsQueryOpts({
115113
if (!account) {
116114
return rejectNetworkError({
117115
errorName: 'Not Found Error',
118-
errorDetails: {
119-
callingFn: 'InfiniteAccountOrganizationsQueryOpts',
120-
},
116+
errorDetails: { callingFn },
121117
})
122118
}
123119

‎src/pages/PlanPage/subRoutes/CurrentOrgPlan/useCurrentOrgPlanPageData.tsx

+3-4
Original file line numberDiff line numberDiff line change
@@ -69,14 +69,13 @@ export const useCurrentOrgPlanPageData = ({
6969
owner,
7070
},
7171
}).then((res) => {
72+
const callingFn = 'useCurrentOrgPlanPageData'
7273
const parsedRes = CurrentOrgPlanPageDataSchema.safeParse(res?.data)
74+
7375
if (!parsedRes.success) {
7476
return rejectNetworkError({
7577
errorName: 'Parsing Error',
76-
errorDetails: {
77-
callingFn: 'useCurrentOrgPlanPageData',
78-
error: parsedRes.error,
79-
},
78+
errorDetails: { callingFn, error: parsedRes.error },
8079
})
8180
}
8281

‎src/pages/PullRequestPage/Header/HeaderDefault/queries/PullHeadDataQueryOpts.tsx

+4-6
Original file line numberDiff line numberDiff line change
@@ -108,15 +108,13 @@ export const PullHeadDataQueryOpts = ({
108108
pullId: parseInt(pullId, 10),
109109
},
110110
}).then((res) => {
111+
const callingFn = 'PullHeadDataQueryOpts'
111112
const parsedData = PullHeadDataSchema.safeParse(res?.data)
112113

113114
if (!parsedData.success) {
114115
return rejectNetworkError({
115116
errorName: 'Parsing Error',
116-
errorDetails: {
117-
callingFn: 'PullHeadDataQueryOpts',
118-
error: parsedData.error,
119-
},
117+
errorDetails: { callingFn, error: parsedData.error },
120118
})
121119
}
122120

@@ -125,14 +123,14 @@ export const PullHeadDataQueryOpts = ({
125123
if (data?.owner?.repository?.__typename === 'NotFoundError') {
126124
return rejectNetworkError({
127125
errorName: 'Not Found Error',
128-
errorDetails: { callingFn: 'PullHeadDataQueryOpts' },
126+
errorDetails: { callingFn },
129127
})
130128
}
131129

132130
if (data?.owner?.repository?.__typename === 'OwnerNotActivatedError') {
133131
return rejectNetworkError({
134132
errorName: 'Owner Not Activated',
135-
errorDetails: { callingFn: 'PullHeadDataQueryOpts' },
133+
errorDetails: { callingFn },
136134
data: {
137135
detail: (
138136
<p>

‎src/pages/PullRequestPage/Header/HeaderTeam/queries/PullHeadDataTeamQueryOpts.tsx

+4-6
Original file line numberDiff line numberDiff line change
@@ -160,15 +160,13 @@ export const PullHeadDataTeamQueryOpts = ({
160160
pullId: parseInt(pullId, 10),
161161
},
162162
}).then((res) => {
163+
const callingFn = 'PullHeadDataTeamQueryOpts'
163164
const parsedData = PullHeadDataSchema.safeParse(res?.data)
164165

165166
if (!parsedData.success) {
166167
return rejectNetworkError({
167168
errorName: 'Parsing Error',
168-
errorDetails: {
169-
callingFn: 'PullHeadDataTeamQueryOpts',
170-
error: parsedData.error,
171-
},
169+
errorDetails: { callingFn, error: parsedData.error },
172170
})
173171
}
174172

@@ -177,14 +175,14 @@ export const PullHeadDataTeamQueryOpts = ({
177175
if (data?.owner?.repository?.__typename === 'NotFoundError') {
178176
return rejectNetworkError({
179177
errorName: 'Not Found Error',
180-
errorDetails: { callingFn: 'PullHeadDataTeamQueryOpts' },
178+
errorDetails: { callingFn },
181179
})
182180
}
183181

184182
if (data?.owner?.repository?.__typename === 'OwnerNotActivatedError') {
185183
return rejectNetworkError({
186184
errorName: 'Owner Not Activated',
187-
errorDetails: { callingFn: 'PullHeadDataTeamQueryOpts' },
185+
errorDetails: { callingFn },
188186
data: {
189187
detail: (
190188
<p>

‎src/pages/PullRequestPage/PullCoverage/routes/ComponentsTab/queries/ComponentComparisonQueryOpts.tsx

+4-6
Original file line numberDiff line numberDiff line change
@@ -172,15 +172,13 @@ export function ComponentComparisonQueryOpts({
172172
pullId: parseInt(pullId, 10),
173173
},
174174
}).then((res) => {
175+
const callingFn = 'ComponentComparisonQueryOpts'
175176
const parsedRes = ComponentComparisonSchema.safeParse(res?.data)
176177

177178
if (!parsedRes.success) {
178179
return rejectNetworkError({
179180
errorName: 'Parsing Error',
180-
errorDetails: {
181-
callingFn: 'ComponentComparisonQueryOpts',
182-
error: parsedRes.error,
183-
},
181+
errorDetails: { callingFn, error: parsedRes.error },
184182
})
185183
}
186184

@@ -189,14 +187,14 @@ export function ComponentComparisonQueryOpts({
189187
if (data?.owner?.repository?.__typename === 'NotFoundError') {
190188
return rejectNetworkError({
191189
errorName: 'Not Found Error',
192-
errorDetails: { callingFn: 'ComponentComparisonQueryOpts' },
190+
errorDetails: { callingFn },
193191
})
194192
}
195193

196194
if (data?.owner?.repository?.__typename === 'OwnerNotActivatedError') {
197195
return rejectNetworkError({
198196
errorName: 'Owner Not Activated',
199-
errorDetails: { callingFn: 'ComponentComparisonQueryOpts' },
197+
errorDetails: { callingFn },
200198
data: {
201199
detail: (
202200
<p>

0 commit comments

Comments
 (0)