Skip to content

Commit 5491de8

Browse files
committed
feat(pci-instances): add tests for missings ones
ref: #- Signed-off-by: Yann Allançon <yann.allancon.ext@corp.ovh.com>
1 parent d8b7bb2 commit 5491de8

2 files changed

Lines changed: 13 additions & 2 deletions

File tree

packages/manager/apps/pci-instances/src/domain/services/__tests__/instance.service.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ describe('instance.service', () => {
2222
'd2-2_GRA-STAGING-A',
2323
);
2424

25-
expect(result).toBe(`d2-2 ${CURRENT_DATE_MOCK}`);
25+
expect(result).toBe(`d2-2-${CURRENT_DATE_MOCK}`);
2626
});
2727

2828
it('returns null when regionalizedFlavorId is not in catalog', () => {
@@ -70,7 +70,7 @@ describe('instance.service', () => {
7070
orphanRegionalizedFlavorId,
7171
);
7272

73-
expect(result).toBe(`instance ${CURRENT_DATE_MOCK}`);
73+
expect(result).toBe(`instance-${CURRENT_DATE_MOCK}`);
7474
});
7575
});
7676
});

packages/manager/apps/pci-instances/src/pages/instances/create/components/__test__/Network.component.spec.tsx

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,10 @@ import { NAMESPACES } from '@ovh-ux/manager-common-translations';
1616
import { TestCreateInstanceFormWrapper } from '@/__tests__/CreateInstanceFormWrapper';
1717
import { getPrivateNetworks } from '@/data/api/privateNetworks';
1818
import { TPrivateNetwork } from '@/domain/entities/configuration';
19+
import { useInstancesCatalogWithSelect } from '@/data/hooks/catalog/useInstancesCatalogWithSelect';
1920

2021
const getPrivateNetworksMock = vi.fn();
22+
const useInstancesCatalogWithSelectMock = vi.fn();
2123

2224
const NETWORK_NAME = 'fake-name';
2325
const DEFAULT_VLAN_ID = 24;
@@ -33,6 +35,11 @@ vi.mock(
3335
vi.mock('@/data/api/privateNetworks');
3436
vi.mocked(getPrivateNetworks).mockImplementation(getPrivateNetworksMock);
3537

38+
vi.mock('@/data/hooks/catalog/useInstancesCatalogWithSelect');
39+
vi.mocked(useInstancesCatalogWithSelect).mockImplementation(
40+
useInstancesCatalogWithSelectMock,
41+
);
42+
3643
const setupTest = (
3744
{
3845
networks,
@@ -42,6 +49,9 @@ const setupTest = (
4249
},
4350
) => {
4451
getPrivateNetworksMock.mockReturnValue(networks);
52+
useInstancesCatalogWithSelectMock.mockReturnValue({
53+
data: 'region',
54+
});
4555

4656
renderWithMockedWrappers(
4757
<TestCreateInstanceFormWrapper
@@ -230,6 +240,7 @@ describe('Considering Network component', () => {
230240

231241
await waitFor(() => {
232242
const options = screen.getAllByText('test-network-metal');
243+
// eslint-disable-next-line max-nested-callbacks
233244
const optionWithValue = options.find((option) =>
234245
option.hasAttribute('selected'),
235246
);

0 commit comments

Comments
 (0)