Skip to content

chore: Notification services eslint cleanup #7480

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

Merged
Prithpal-Sooriya merged 4 commits into from
Dec 15, 2025
Merged

chore: Notification services eslint cleanup #7480

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
4625e40
Refactor: Add explicit return types and remove unused ESLint suppress…
cursoragent Dec 15, 2025
0fed078
feat: Add explicit return types to NotificationServicesPushController
cursoragent Dec 15, 2025
786c700
Refactor: Simplify test setup for notification services
cursoragent Dec 15, 2025
43b5233
Refactor: Move controller peer dependencies to direct
cursoragent Dec 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 0 additions & 15 deletions eslint-suppressions.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1915,21 +1915,6 @@
"count": 1
}
},
"packages/notification-services-controller/src/NotificationServicesPushController/mocks/mockResponse.ts": {
"@typescript-eslint/explicit-function-return-type": {
"count": 3
}
},
"packages/notification-services-controller/src/NotificationServicesPushController/services/endpoints.ts": {
"@typescript-eslint/explicit-function-return-type": {
"count": 2
}
},
"packages/notification-services-controller/src/NotificationServicesPushController/services/services.test.ts": {
"@typescript-eslint/explicit-function-return-type": {
"count": 4
}
},
"packages/notification-services-controller/src/NotificationServicesPushController/services/services.ts": {
"@typescript-eslint/naming-convention": {
"count": 1
Expand Down
6 changes: 3 additions & 3 deletions ...fication-services-controller/src/NotificationServicesPushController/mocks/mockResponse.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ type MockResponse = {

export const MOCK_REG_TOKEN = 'REG_TOKEN';

export const getMockUpdatePushNotificationLinksResponse = () => {
export const getMockUpdatePushNotificationLinksResponse = (): MockResponse => {
return {
url: REGISTRATION_TOKENS_ENDPOINT(),
requestMethod: 'POST',
Expand All @@ -27,15 +27,15 @@ export const MOCK_FCM_RESPONSE = {
},
};

export const getMockCreateFCMRegistrationTokenResponse = () => {
export const getMockCreateFCMRegistrationTokenResponse = (): MockResponse => {
return {
url: /^https:\/\/fcmregistrations\.googleapis\.com\/v1\/projects\/.*$/u,
requestMethod: 'POST',
response: MOCK_FCM_RESPONSE,
} satisfies MockResponse;
};

export const getMockDeleteFCMRegistrationTokenResponse = () => {
export const getMockDeleteFCMRegistrationTokenResponse = (): MockResponse => {
return {
url: /^https:\/\/fcmregistrations\.googleapis\.com\/v1\/projects\/.*$/u,
requestMethod: 'POST',
Expand Down
4 changes: 2 additions & 2 deletions ...fication-services-controller/src/NotificationServicesPushController/services/endpoints.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ const PUSH_API_ENV = {
prd: 'https://push.api.cx.metamask.io',
} satisfies Record<ENV, string>;

export const PUSH_API = (env: ENV = 'prd') =>
export const PUSH_API = (env: ENV = 'prd'): string =>
PUSH_API_ENV[env] ?? PUSH_API_ENV.prd;

export const REGISTRATION_TOKENS_ENDPOINT = (env: ENV = 'prd') =>
export const REGISTRATION_TOKENS_ENDPOINT = (env: ENV = 'prd'): string =>
`${PUSH_API(env)}/api/v2/token`;
8 changes: 6 additions & 2 deletions ...tion-services-controller/src/NotificationServicesPushController/services/services.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import { mockEndpointUpdatePushNotificationLinks } from '../__fixtures__/mockSer
import type { PushNotificationEnv } from '../types/firebase';

// Testing util to clean up verbose logs when testing errors
const mockErrorLog = () =>
const mockErrorLog = (): jest.SpyInstance =>
jest.spyOn(log, 'error').mockImplementation(jest.fn());

const MOCK_REG_TOKEN = 'REG_TOKEN';
Expand All @@ -19,7 +19,7 @@ const MOCK_JWT = 'MOCK_JWT';

describe('NotificationServicesPushController Services', () => {
describe('updateLinksAPI', () => {
const act = async () =>
const act = async (): Promise<boolean> =>
await updateLinksAPI({
bearerToken: MOCK_JWT,
addresses: MOCK_ADDRESSES,
Expand Down Expand Up @@ -54,6 +54,8 @@ describe('NotificationServicesPushController Services', () => {
});

describe('activatePushNotifications', () => {
// Internal testing utility - return type is inferred
// eslint-disable-next-line @typescript-eslint/explicit-function-return-type
const arrangeMocks = (override?: { mockPut?: { status: number } }) => {
const params = {
bearerToken: MOCK_JWT,
Expand Down Expand Up @@ -124,6 +126,8 @@ describe('NotificationServicesPushController Services', () => {
});

describe('deactivatePushNotifications', () => {
// Internal testing utility - return type is inferred
// eslint-disable-next-line @typescript-eslint/explicit-function-return-type
const arrangeMocks = () => {
const params = {
regToken: MOCK_REG_TOKEN,
Expand Down
Loading