forked from linode/manager
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
upcoming: [M3-7928] - Linode Create Refactor - Summary - Part 8 (lino…
…de#10334) * inital summary * more work and unit tests * Added changeset: Linode Create Refactor - Summary - Part 8 * only show backups if a type is selected * add placement group to summary * fix summary wrapping --------- Co-authored-by: Banks Nussman <[email protected]>
- Loading branch information
1 parent
100f3df
commit 7853f7b
Showing
5 changed files
with
374 additions
and
18 deletions.
There are no files selected for viewing
5 changes: 5 additions & 0 deletions
5
packages/manager/.changeset/pr-10334-upcoming-features-1711997020117.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
--- | ||
"@linode/manager": Upcoming Features | ||
--- | ||
|
||
Linode Create Refactor - Summary - Part 8 ([#10334](https://github.com/linode/manager/pull/10334)) |
28 changes: 28 additions & 0 deletions
28
packages/manager/src/features/Linodes/LinodeCreatev2/Actions.tsx
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
import { CreateLinodeRequest } from '@linode/api-v4'; | ||
import React from 'react'; | ||
import { useFormContext } from 'react-hook-form'; | ||
|
||
import { Box } from 'src/components/Box'; | ||
import { Button } from 'src/components/Button/Button'; | ||
import { useRestrictedGlobalGrantCheck } from 'src/hooks/useRestrictedGlobalGrantCheck'; | ||
|
||
export const Actions = () => { | ||
const { formState } = useFormContext<CreateLinodeRequest>(); | ||
|
||
const isLinodeCreateRestricted = useRestrictedGlobalGrantCheck({ | ||
globalGrantType: 'add_linodes', | ||
}); | ||
|
||
return ( | ||
<Box sx={{ display: 'flex', justifyContent: 'flex-end' }}> | ||
<Button | ||
buttonType="primary" | ||
disabled={isLinodeCreateRestricted} | ||
loading={formState.isSubmitting} | ||
type="submit" | ||
> | ||
Create Linode | ||
</Button> | ||
</Box> | ||
); | ||
}; |
200 changes: 200 additions & 0 deletions
200
packages/manager/src/features/Linodes/LinodeCreatev2/Summary.test.tsx
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,200 @@ | ||
import React from 'react'; | ||
|
||
import { imageFactory, regionFactory, typeFactory } from 'src/factories'; | ||
import { makeResourcePage } from 'src/mocks/serverHandlers'; | ||
import { HttpResponse, http, server } from 'src/mocks/testServer'; | ||
import { renderWithThemeAndHookFormContext } from 'src/utilities/testHelpers'; | ||
|
||
import { Summary } from './Summary'; | ||
|
||
describe('Linode Create v2 Summary', () => { | ||
it('should render a heading based on the Linode label', async () => { | ||
const label = 'my-linode-1'; | ||
const { findByText } = renderWithThemeAndHookFormContext({ | ||
component: <Summary />, | ||
useFormOptions: { defaultValues: { label } }, | ||
}); | ||
|
||
const heading = await findByText(`Summary ${label}`); | ||
|
||
expect(heading).toBeVisible(); | ||
expect(heading.tagName).toBe('H2'); | ||
}); | ||
|
||
it('should render "Private IP" if private ip is enabled', async () => { | ||
const { getByText } = renderWithThemeAndHookFormContext({ | ||
component: <Summary />, | ||
useFormOptions: { defaultValues: { private_ip: true } }, | ||
}); | ||
|
||
expect(getByText('Private IP')).toBeVisible(); | ||
}); | ||
|
||
it('should render "Backups" if backups are enabled and a type is selected', async () => { | ||
const type = typeFactory.build(); | ||
|
||
server.use( | ||
http.get('*/v4/linode/types/*', () => { | ||
return HttpResponse.json(type); | ||
}) | ||
); | ||
|
||
const { findByText } = renderWithThemeAndHookFormContext({ | ||
component: <Summary />, | ||
useFormOptions: { | ||
defaultValues: { backups_enabled: true, type: type.id }, | ||
}, | ||
}); | ||
|
||
await findByText('Backups'); | ||
}); | ||
|
||
it('should render an image label if an image is selected', async () => { | ||
const image = imageFactory.build(); | ||
|
||
server.use( | ||
http.get('*/v4/images/*', () => { | ||
return HttpResponse.json(image); | ||
}) | ||
); | ||
|
||
const { findByText } = renderWithThemeAndHookFormContext({ | ||
component: <Summary />, | ||
useFormOptions: { defaultValues: { image: image.id } }, | ||
}); | ||
|
||
await findByText(image.label); | ||
}); | ||
|
||
it('should render a region label if a region is selected', async () => { | ||
const region = regionFactory.build(); | ||
|
||
server.use( | ||
http.get('*/v4/regions', () => { | ||
return HttpResponse.json(makeResourcePage([region])); | ||
}) | ||
); | ||
|
||
const { findByText } = renderWithThemeAndHookFormContext({ | ||
component: <Summary />, | ||
useFormOptions: { defaultValues: { region: region.id } }, | ||
}); | ||
|
||
await findByText(region.label); | ||
}); | ||
|
||
it('should render a plan (type) label if a type is selected', async () => { | ||
const type = typeFactory.build(); | ||
|
||
server.use( | ||
http.get('*/v4/linode/types/*', () => { | ||
return HttpResponse.json(type); | ||
}) | ||
); | ||
|
||
const { findByText } = renderWithThemeAndHookFormContext({ | ||
component: <Summary />, | ||
useFormOptions: { defaultValues: { type: type.id } }, | ||
}); | ||
|
||
await findByText(type.label); | ||
}); | ||
|
||
it('should a monthly price if a region and plan are selected', async () => { | ||
const type = typeFactory.build({ price: { monthly: 5 } }); | ||
|
||
server.use( | ||
http.get('*/v4/linode/types/*', () => { | ||
return HttpResponse.json(type); | ||
}) | ||
); | ||
|
||
const { findByText } = renderWithThemeAndHookFormContext({ | ||
component: <Summary />, | ||
useFormOptions: { defaultValues: { region: 'us-east', type: type.id } }, | ||
}); | ||
|
||
await findByText('$5/month'); | ||
}); | ||
|
||
it('should render a DC specific price if the selected region has price overrides', async () => { | ||
const regionId = 'id-cgk'; | ||
|
||
const type = typeFactory.build({ | ||
price: { monthly: 5 }, | ||
region_prices: [{ id: regionId, monthly: 7 }], | ||
}); | ||
|
||
server.use( | ||
http.get('*/v4/linode/types/*', () => { | ||
return HttpResponse.json(type); | ||
}) | ||
); | ||
|
||
const { findByText } = renderWithThemeAndHookFormContext({ | ||
component: <Summary />, | ||
useFormOptions: { defaultValues: { region: regionId, type: type.id } }, | ||
}); | ||
|
||
await findByText('$7/month'); | ||
}); | ||
|
||
it('should render a backups price if backups are enabled, a type is selected, and a region is selected', async () => { | ||
const type = typeFactory.build({ | ||
addons: { | ||
backups: { price: { monthly: 2 } }, | ||
}, | ||
}); | ||
|
||
server.use( | ||
http.get('*/v4/linode/types/*', () => { | ||
return HttpResponse.json(type); | ||
}) | ||
); | ||
|
||
const { findByText } = renderWithThemeAndHookFormContext({ | ||
component: <Summary />, | ||
useFormOptions: { | ||
defaultValues: { | ||
backups_enabled: true, | ||
region: 'us-east', | ||
type: type.id, | ||
}, | ||
}, | ||
}); | ||
|
||
await findByText('$2/month'); | ||
}); | ||
|
||
it('should render a DC specific backups price if the region has overrides ', async () => { | ||
const regionId = 'id-cgk'; | ||
|
||
const type = typeFactory.build({ | ||
addons: { | ||
backups: { | ||
price: { monthly: 2 }, | ||
region_prices: [{ id: regionId, monthly: 4.2 }], | ||
}, | ||
}, | ||
}); | ||
|
||
server.use( | ||
http.get('*/v4/linode/types/*', () => { | ||
return HttpResponse.json(type); | ||
}) | ||
); | ||
|
||
const { findByText } = renderWithThemeAndHookFormContext({ | ||
component: <Summary />, | ||
useFormOptions: { | ||
defaultValues: { | ||
backups_enabled: true, | ||
region: regionId, | ||
type: type.id, | ||
}, | ||
}, | ||
}); | ||
|
||
await findByText('$4.20/month'); | ||
}); | ||
}); |
Oops, something went wrong.