Skip to content
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

chore: Remove barrel files from services collection (part 3) #3780

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
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
2 changes: 1 addition & 1 deletion src/App.tsx
Original file line number Diff line number Diff line change
@@ -11,7 +11,7 @@ import BaseLayout from 'layouts/BaseLayout'
import EnterpriseLoginLayout from 'layouts/EnterpriseLoginLayout'
import LoginLayout from 'layouts/LoginLayout'
import { useLocationParams } from 'services/navigation/useLocationParams'
import { ToastNotificationProvider } from 'services/toastNotification'
import { ToastNotificationProvider } from 'services/toastNotification/context'
import { useInternalUser, useUser } from 'services/user'
import { isProvider } from 'shared/api/helpers'
import 'ui/Table/Table.css'
2 changes: 1 addition & 1 deletion src/layouts/BaseLayout/BaseLayout.tsx
Original file line number Diff line number Diff line change
@@ -14,7 +14,7 @@ import { RepoBreadcrumbProvider } from 'pages/RepoPage/context'
import TermsOfService from 'pages/TermsOfService'
import { useEventContext } from 'services/events/hooks'
import { useImpersonate } from 'services/impersonate/useImpersonate'
import { useTracking } from 'services/tracking'
import { useTracking } from 'services/tracking/useTracking'
import GlobalBanners from 'shared/GlobalBanners'
import GlobalTopBanners from 'shared/GlobalTopBanners'
import LoadingLogo from 'ui/LoadingLogo'
2 changes: 1 addition & 1 deletion src/layouts/ToastNotifications/NotificationItem.jsx
Original file line number Diff line number Diff line change
@@ -2,7 +2,7 @@ import PropType from 'prop-types'
import { useCallback, useEffect } from 'react'

import Message from 'old_ui/Message'
import { notificationPropType } from 'services/toastNotification'
import { notificationPropType } from 'services/toastNotification/propTypes'

function NotificationItem({ notification, removeNotification }) {
const close = useCallback(() => {
2 changes: 1 addition & 1 deletion src/layouts/ToastNotifications/ToastNotifications.jsx
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import {
useNotifications,
useRemoveNotification,
} from 'services/toastNotification'
} from 'services/toastNotification/context'

import NotificationItem from './NotificationItem'

4 changes: 2 additions & 2 deletions src/layouts/ToastNotifications/ToastNotifications.test.jsx
Original file line number Diff line number Diff line change
@@ -4,7 +4,7 @@ import { vi } from 'vitest'
import {
useNotifications,
useRemoveNotification,
} from 'services/toastNotification'
} from 'services/toastNotification/context'

import ToastNotifications from './ToastNotifications'

@@ -23,7 +23,7 @@ const notifications = [
},
]

vi.mock('services/toastNotification')
vi.mock('services/toastNotification/context')

describe('ToastNotifications', () => {
const removeNotification = vi.fn()
Original file line number Diff line number Diff line change
@@ -5,7 +5,7 @@ import { useForm } from 'react-hook-form'
import { useParams } from 'react-router-dom'
import { z } from 'zod'

import { useAddNotification } from 'services/toastNotification'
import { useAddNotification } from 'services/toastNotification/context'
import { useUpdateProfile } from 'services/user'
import Button from 'ui/Button'
import TextInput from 'ui/TextInput'
Original file line number Diff line number Diff line change
@@ -5,11 +5,11 @@ import { graphql, HttpResponse } from 'msw'
import { setupServer } from 'msw/node'
import { MemoryRouter, Route } from 'react-router-dom'

import { useAddNotification } from 'services/toastNotification'
import { useAddNotification } from 'services/toastNotification/context'

import DetailsSection from './DetailsSection'

vi.mock('services/toastNotification')
vi.mock('services/toastNotification/context')

const queryClient = new QueryClient({
defaultOptions: { queries: { retry: false } },
Original file line number Diff line number Diff line change
@@ -8,14 +8,14 @@ import { setupServer } from 'msw/node'
import { MemoryRouter, Route } from 'react-router-dom'
import { type Mock } from 'vitest'

import { useAddNotification } from 'services/toastNotification'
import { useAddNotification } from 'services/toastNotification/context'

import {
SaveOktaConfigMessage,
useUpdateOktaConfig,
} from './useUpdateOktaConfig'

vi.mock('services/toastNotification')
vi.mock('services/toastNotification/context')
const mockedToastNotification = useAddNotification as Mock

const queryClientV5 = new QueryClientV5({
Original file line number Diff line number Diff line change
@@ -4,7 +4,7 @@ import {
} from '@tanstack/react-queryV5'
import z from 'zod'

import { useAddNotification } from 'services/toastNotification'
import { useAddNotification } from 'services/toastNotification/context'
import Api from 'shared/api'
import { Provider } from 'shared/api/helpers'
import { rejectNetworkError } from 'shared/api/rejectNetworkError'
Original file line number Diff line number Diff line change
@@ -6,12 +6,12 @@ import { setupServer } from 'msw/node'
import { Suspense } from 'react'
import { MemoryRouter, Route } from 'react-router-dom'

import { useAddNotification } from 'services/toastNotification'
import { useAddNotification } from 'services/toastNotification/context'
import { useFlags } from 'shared/featureFlags'

import OrgUploadToken from './OrgUploadToken'

vi.mock('services/toastNotification')
vi.mock('services/toastNotification/context')
vi.mock('shared/featureFlags')
vi.mock('./TokenlessSection', () => ({ default: () => 'TokenlessSection' }))

Original file line number Diff line number Diff line change
@@ -16,8 +16,8 @@ const mocks = vi.hoisted(() => ({
useFlags: vi.fn(),
}))

vi.mock('services/toastNotification', async () => {
const actual = await vi.importActual('services/toastNotification')
vi.mock('services/toastNotification/context', async () => {
const actual = await vi.importActual('services/toastNotification/context')
return {
...actual,
useAddNotification: mocks.useAddNotification,
Original file line number Diff line number Diff line change
@@ -13,8 +13,8 @@ const mocks = vi.hoisted(() => ({
useAddNotification: vi.fn(),
}))

vi.mock('services/toastNotification', async () => {
const actual = await vi.importActual('services/toastNotification')
vi.mock('services/toastNotification/context', async () => {
const actual = await vi.importActual('services/toastNotification/context')
return {
...actual,
useAddNotification: mocks.useAddNotification,
Original file line number Diff line number Diff line change
@@ -2,7 +2,7 @@ import { useQueryClient } from '@tanstack/react-query'
import { useMutation as useMutationV5 } from '@tanstack/react-queryV5'
import { z } from 'zod'

import { useAddNotification } from 'services/toastNotification'
import { useAddNotification } from 'services/toastNotification/context'
import Api from 'shared/api'
import { Provider } from 'shared/api/helpers'
import { rejectNetworkError } from 'shared/api/rejectNetworkError'
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { useRegenerateOrgUploadToken } from 'services/orgUploadToken/useRegenerateOrgUploadToken'
import { useAddNotification } from 'services/toastNotification'
import { useAddNotification } from 'services/toastNotification/context'

export default function useGenerateOrgUploadToken() {
const addToast = useAddNotification()
Original file line number Diff line number Diff line change
@@ -4,7 +4,7 @@ import { useForm } from 'react-hook-form'
import { z } from 'zod'

import Card from 'old_ui/Card'
import { useAddNotification } from 'services/toastNotification'
import { useAddNotification } from 'services/toastNotification/context'
import { useUpdateProfile } from 'services/user'
import Button from 'ui/Button'
import TextInput from 'ui/TextInput'
Original file line number Diff line number Diff line change
@@ -4,11 +4,11 @@ import userEvent from '@testing-library/user-event'
import { graphql, HttpResponse } from 'msw'
import { setupServer } from 'msw/node'

import { useAddNotification } from 'services/toastNotification'
import { useAddNotification } from 'services/toastNotification/context'

import NameEmailCard from './NameEmailCard'

vi.mock('services/toastNotification')
vi.mock('services/toastNotification/context')

const currentUser = {
name: 'donald duck',
2 changes: 1 addition & 1 deletion src/pages/OwnerPage/OwnerPage.jsx
Original file line number Diff line number Diff line change
@@ -7,7 +7,7 @@ import { useOwnerPageData } from 'pages/OwnerPage/hooks'
import { useAccountDetails } from 'services/account/useAccountDetails'
import { useSentryToken } from 'services/account/useSentryToken'
import { useLocationParams } from 'services/navigation/useLocationParams'
import { renderToast } from 'services/toast'
import { renderToast } from 'services/toast/renderToast'
import { ActiveContext } from 'shared/context'
import ListRepo from 'shared/ListRepo'

4 changes: 2 additions & 2 deletions src/pages/OwnerPage/OwnerPage.test.jsx
Original file line number Diff line number Diff line change
@@ -11,8 +11,8 @@ const mocks = vi.hoisted(() => ({
renderToast: vi.fn(),
}))

vi.mock('services/toast', async () => {
const actual = await vi.importActual('services/toast')
vi.mock('services/toast/renderToast', async () => {
const actual = await vi.importActual('services/toast/renderToast')
return {
...actual,
renderToast: mocks.renderToast,
Original file line number Diff line number Diff line change
@@ -36,8 +36,8 @@ vi.mock('services/account/useCancelPlan', async () => {
}
})

vi.mock('services/toastNotification', async () => {
const actual = await vi.importActual('services/toastNotification')
vi.mock('services/toastNotification/context', async () => {
const actual = await vi.importActual('services/toastNotification/context')
return {
...actual,
useAddNotification: mocks.useAddNotification,
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import { useHistory, useParams } from 'react-router-dom'

import { useCancelPlan } from 'services/account/useCancelPlan'
import { useAddNotification } from 'services/toastNotification'
import { useAddNotification } from 'services/toastNotification/context'

import { useBarecancel } from './useBarecancel'

Original file line number Diff line number Diff line change
@@ -9,7 +9,7 @@ const mocks = vi.hoisted(() => ({
useParams: vi.fn(),
}))

vi.mock('services/toastNotification')
vi.mock('services/toastNotification/context')

vi.mock('react-router-dom', async () => {
const actual = await vi.requireActual('react-router-dom')
Original file line number Diff line number Diff line change
@@ -4,7 +4,7 @@ import { useHistory, useParams } from 'react-router-dom'

import { useSetCrumbs } from 'pages/PlanPage/context'
import { useNavLinks } from 'services/navigation/useNavLinks'
import { useAddNotification } from 'services/toastNotification'
import { useAddNotification } from 'services/toastNotification/context'
import Api from 'shared/api'
import A from 'ui/A'
import Button from 'ui/Button'
Original file line number Diff line number Diff line change
@@ -10,7 +10,7 @@ import SpecialOffer from './SpecialOffer'
const mockBody = vi.fn()
const mockToast = vi.fn()

vi.mock('services/toastNotification', () => ({
vi.mock('services/toastNotification/context', () => ({
useAddNotification: () => (data) => mockToast(data),
}))

Original file line number Diff line number Diff line change
@@ -17,8 +17,8 @@ const mocks = vi.hoisted(() => ({
}))

vi.mock('@stripe/react-stripe-js')
vi.mock('services/toastNotification', async () => {
const actual = await vi.importActual('services/toastNotification')
vi.mock('services/toastNotification/context', async () => {
const actual = await vi.importActual('services/toastNotification/context')
return {
...actual,
useAddNotification: mocks.useAddNotification,
Original file line number Diff line number Diff line change
@@ -15,8 +15,8 @@ const mocks = vi.hoisted(() => ({
useAddNotification: vi.fn(),
}))

vi.mock('services/toastNotification', async () => {
const actual = await vi.importActual('services/toastNotification')
vi.mock('services/toastNotification/context', async () => {
const actual = await vi.importActual('services/toastNotification/context')
return {
...actual,
useAddNotification: mocks.useAddNotification,
Original file line number Diff line number Diff line change
@@ -16,8 +16,8 @@ const mocks = vi.hoisted(() => ({
useAddNotification: vi.fn(),
}))

vi.mock('services/toastNotification', async () => {
const actual = await vi.importActual('services/toastNotification')
vi.mock('services/toastNotification/context', async () => {
const actual = await vi.importActual('services/toastNotification/context')
return {
...actual,
useAddNotification: mocks.useAddNotification,
Original file line number Diff line number Diff line change
@@ -21,8 +21,8 @@ const mocks = vi.hoisted(() => ({
useAddNotification: vi.fn(),
}))

vi.mock('services/toastNotification', async () => {
const actual = await vi.importActual('services/toastNotification')
vi.mock('services/toastNotification/context', async () => {
const actual = await vi.importActual('services/toastNotification/context')
return {
...actual,
useAddNotification: mocks.useAddNotification,
Original file line number Diff line number Diff line change
@@ -3,7 +3,7 @@ import { useHistory, useParams } from 'react-router-dom'

import { useSetPlanUpdatedNotification } from 'pages/PlanPage/context'
import { useUpgradePlan } from 'services/account/useUpgradePlan'
import { useAddNotification } from 'services/toastNotification'
import { useAddNotification } from 'services/toastNotification/context'
import { Provider } from 'shared/api/helpers'

import { UpgradeFormFields } from '../UpgradeForm'
Original file line number Diff line number Diff line change
@@ -12,8 +12,8 @@ const mocks = vi.hoisted(() => ({
useAddNotification: vi.fn(),
}))

vi.mock('services/toastNotification', async () => {
const actual = await vi.importActual('services/toastNotification')
vi.mock('services/toastNotification/context', async () => {
const actual = await vi.importActual('services/toastNotification/context')
return {
...actual,
useAddNotification: mocks.useAddNotification,
Original file line number Diff line number Diff line change
@@ -11,8 +11,8 @@ const mocks = vi.hoisted(() => ({
useAddNotification: vi.fn(),
}))

vi.mock('services/toastNotification', async () => {
const actual = await vi.importActual('services/toastNotification')
vi.mock('services/toastNotification/context', async () => {
const actual = await vi.importActual('services/toastNotification/context')
return {
...actual,
useAddNotification: mocks.useAddNotification,
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { useUpdateRepo } from 'services/repo'
import { useAddNotification } from 'services/toastNotification'
import { useAddNotification } from 'services/toastNotification/context'

export function useRepoActivation() {
const addToast = useAddNotification()
Original file line number Diff line number Diff line change
@@ -10,8 +10,8 @@ const mocks = vi.hoisted(() => ({
useAddNotification: vi.fn(),
}))

vi.mock('services/toastNotification', async () => {
const actual = await vi.importActual('services/toastNotification')
vi.mock('services/toastNotification/context', async () => {
const actual = await vi.importActual('services/toastNotification/context')
return {
...actual,
useAddNotification: mocks.useAddNotification,
Original file line number Diff line number Diff line change
@@ -4,7 +4,7 @@ import { useParams } from 'react-router-dom'

import { useBranches } from 'services/branches/useBranches'
import { useUpdateRepo } from 'services/repo'
import { useAddNotification } from 'services/toastNotification'
import { useAddNotification } from 'services/toastNotification/context'
import Icon from 'ui/Icon'
import Select from 'ui/Select'
import SettingsDescriptor from 'ui/SettingsDescriptor'
Original file line number Diff line number Diff line change
@@ -12,8 +12,8 @@ const mocks = vi.hoisted(() => ({
useIntersection: vi.fn(),
}))

vi.mock('services/toastNotification', async () => {
const actual = await vi.importActual('services/toastNotification')
vi.mock('services/toastNotification/context', async () => {
const actual = await vi.importActual('services/toastNotification/context')
return {
...actual,
useAddNotification: mocks.useAddNotification,
Original file line number Diff line number Diff line change
@@ -2,7 +2,7 @@ import PropTypes from 'prop-types'
import { useState } from 'react'
import { useParams } from 'react-router'

import { useRegenerateRepoUploadToken } from 'services/repoUploadToken'
import { useRegenerateRepoUploadToken } from 'services/repoUploadToken/useRegenerateRepoUploadToken'
import { useUploadTokenRequired } from 'services/uploadTokenRequired'
import A from 'ui/A'
import Button from 'ui/Button'
Original file line number Diff line number Diff line change
@@ -12,8 +12,8 @@ const mocks = vi.hoisted(() => ({
useAddNotification: vi.fn(),
}))

vi.mock('services/toastNotification', async () => {
const actual = await vi.importActual('services/toastNotification')
vi.mock('services/toastNotification/context', async () => {
const actual = await vi.importActual('services/toastNotification/context')
return {
...actual,
useAddNotification: mocks.useAddNotification,
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import PropTypes from 'prop-types'

import { useRegenerateRepositoryToken } from 'services/repositoryToken'
import { useRegenerateRepositoryToken } from 'services/repositoryToken/useRegenerateRepositoryToken'
import Button from 'ui/Button'
import Modal from 'ui/Modal'

Original file line number Diff line number Diff line change
@@ -12,8 +12,8 @@ const mocks = vi.hoisted(() => ({
useAddNotification: vi.fn(),
}))

vi.mock('services/toastNotification', async () => {
const actual = await vi.importActual('services/toastNotification')
vi.mock('services/toastNotification/context', async () => {
const actual = await vi.importActual('services/toastNotification/context')
return {
...actual,
useAddNotification: mocks.useAddNotification,
Loading
Oops, something went wrong.