@scality/data-browser-library 1.0.0-preview.13 → 1.0.0-preview.15
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/components/DataBrowserUI.d.ts +15 -8
- package/dist/components/DataBrowserUI.js +79 -55
- package/dist/components/Editor.d.ts +1 -1
- package/dist/components/Editor.js +3 -3
- package/dist/components/__tests__/BucketCreate.test.js +102 -102
- package/dist/components/__tests__/BucketDetails.test.js +121 -122
- package/dist/components/__tests__/BucketLifecycleFormPage.test.js +177 -177
- package/dist/components/__tests__/BucketLifecycleList.test.js +85 -85
- package/dist/components/__tests__/BucketList.test.js +175 -176
- package/dist/components/__tests__/BucketNotificationCreatePage.test.js +84 -84
- package/dist/components/__tests__/BucketOverview.test.js +256 -200
- package/dist/components/__tests__/BucketPolicyPage.test.js +62 -62
- package/dist/components/__tests__/BucketReplicationFormPage.test.js +542 -542
- package/dist/components/__tests__/BucketReplicationList.test.js +106 -106
- package/dist/components/__tests__/CreateFolderButton.test.js +56 -56
- package/dist/components/__tests__/DeleteBucketButton.test.js +62 -62
- package/dist/components/__tests__/DeleteBucketConfigRuleButton.test.js +47 -47
- package/dist/components/__tests__/DeleteObjectButton.test.js +63 -63
- package/dist/components/__tests__/EmptyBucketButton.test.js +56 -56
- package/dist/components/__tests__/MetadataSearch.test.js +65 -65
- package/dist/components/__tests__/ObjectList.test.js +251 -250
- package/dist/components/__tests__/UploadButton.test.js +45 -45
- package/dist/components/buckets/BucketCreate.d.ts +2 -2
- package/dist/components/buckets/BucketCreate.js +41 -41
- package/dist/components/buckets/BucketDetails.d.ts +2 -2
- package/dist/components/buckets/BucketDetails.js +48 -36
- package/dist/components/buckets/BucketLifecycleFormPage.js +161 -160
- package/dist/components/buckets/BucketLifecycleList.d.ts +2 -2
- package/dist/components/buckets/BucketLifecycleList.js +46 -46
- package/dist/components/buckets/BucketList.d.ts +2 -2
- package/dist/components/buckets/BucketList.js +28 -27
- package/dist/components/buckets/BucketLocation.js +3 -3
- package/dist/components/buckets/BucketOverview.d.ts +1 -1
- package/dist/components/buckets/BucketOverview.js +62 -62
- package/dist/components/buckets/BucketPage.js +19 -11
- package/dist/components/buckets/BucketPolicyButton.js +2 -2
- package/dist/components/buckets/BucketPolicyPage.js +27 -25
- package/dist/components/buckets/BucketReplicationFormPage.js +133 -132
- package/dist/components/buckets/BucketReplicationList.d.ts +2 -2
- package/dist/components/buckets/BucketReplicationList.js +41 -41
- package/dist/components/buckets/BucketVersioning.js +11 -11
- package/dist/components/buckets/DeleteBucketButton.js +5 -5
- package/dist/components/buckets/DeleteBucketConfigRuleButton.d.ts +2 -2
- package/dist/components/buckets/DeleteBucketConfigRuleButton.js +1 -1
- package/dist/components/buckets/EmptyBucketButton.js +19 -19
- package/dist/components/buckets/EmptyBucketSummary.d.ts +1 -1
- package/dist/components/buckets/EmptyBucketSummary.js +1 -1
- package/dist/components/buckets/EmptyBucketSummaryList.js +22 -22
- package/dist/components/buckets/__tests__/BucketVersioning.test.js +45 -45
- package/dist/components/buckets/notifications/BucketNotificationCreatePage.js +34 -33
- package/dist/components/buckets/notifications/EventsSection.js +144 -28
- package/dist/components/buckets/notifications/__tests__/events.test.d.ts +1 -0
- package/dist/components/buckets/notifications/__tests__/events.test.js +56 -0
- package/dist/components/buckets/notifications/events.d.ts +71 -7
- package/dist/components/buckets/notifications/events.js +98 -16
- package/dist/components/index.d.ts +23 -22
- package/dist/components/index.js +3 -2
- package/dist/components/layouts/ArrowNavigation.d.ts +1 -2
- package/dist/components/layouts/ArrowNavigation.js +3 -3
- package/dist/components/layouts/BrowserPageLayout.d.ts +2 -3
- package/dist/components/layouts/BrowserPageLayout.js +1 -1
- package/dist/components/objects/CreateFolderButton.d.ts +2 -2
- package/dist/components/objects/CreateFolderButton.js +9 -9
- package/dist/components/objects/DeleteObjectButton.d.ts +1 -1
- package/dist/components/objects/DeleteObjectButton.js +20 -20
- package/dist/components/objects/ObjectDetails/ObjectMetadata.d.ts +1 -1
- package/dist/components/objects/ObjectDetails/ObjectMetadata.js +56 -56
- package/dist/components/objects/ObjectDetails/ObjectSummary.d.ts +1 -1
- package/dist/components/objects/ObjectDetails/ObjectSummary.js +39 -39
- package/dist/components/objects/ObjectDetails/ObjectTags.d.ts +1 -1
- package/dist/components/objects/ObjectDetails/ObjectTags.js +25 -25
- package/dist/components/objects/ObjectDetails/__tests__/ObjectDetails.test.js +119 -119
- package/dist/components/objects/ObjectDetails/__tests__/ObjectSummary.test.js +211 -211
- package/dist/components/objects/ObjectDetails/index.d.ts +1 -1
- package/dist/components/objects/ObjectDetails/index.js +30 -30
- package/dist/components/objects/ObjectList.d.ts +5 -5
- package/dist/components/objects/ObjectList.js +112 -111
- package/dist/components/objects/ObjectLock/EditRetentionButton.js +3 -3
- package/dist/components/objects/ObjectLock/ObjectLockRetentionSettings.js +14 -14
- package/dist/components/objects/ObjectLock/ObjectLockSettings.d.ts +1 -1
- package/dist/components/objects/ObjectLock/ObjectLockSettings.js +29 -28
- package/dist/components/objects/ObjectLock/ObjectLockSettingsUtils.d.ts +1 -1
- package/dist/components/objects/ObjectLock/ObjectLockSettingsUtils.js +6 -6
- package/dist/components/objects/ObjectLock/__tests__/EditRetentionButton.test.js +50 -50
- package/dist/components/objects/ObjectLock/__tests__/ObjectLockSettings.test.js +77 -77
- package/dist/components/objects/ObjectPage.js +5 -4
- package/dist/components/objects/UploadButton.d.ts +3 -3
- package/dist/components/objects/UploadButton.js +5 -5
- package/dist/components/providers/DataBrowserProvider.d.ts +15 -4
- package/dist/components/providers/DataBrowserProvider.js +33 -11
- package/dist/components/search/MetadataSearch.js +26 -25
- package/dist/components/search/SearchHints.js +1 -1
- package/dist/components/ui/ArrayFieldActions.js +4 -4
- package/dist/components/ui/ConfirmDeleteRuleModal.d.ts +1 -1
- package/dist/components/ui/ConfirmDeleteRuleModal.js +1 -1
- package/dist/components/ui/DeleteObjectModalContent.d.ts +1 -1
- package/dist/components/ui/DeleteObjectModalContent.js +12 -12
- package/dist/components/ui/FilterFormSection.d.ts +2 -2
- package/dist/components/ui/FilterFormSection.js +29 -29
- package/dist/components/ui/Search.elements.d.ts +1 -1
- package/dist/components/ui/Search.elements.js +7 -7
- package/dist/components/ui/Table.elements.js +5 -5
- package/dist/config/factory.d.ts +23 -10
- package/dist/config/factory.js +22 -7
- package/dist/config/types.d.ts +20 -3
- package/dist/contexts/DataBrowserUICustomizationContext.d.ts +2 -2
- package/dist/hooks/__tests__/useISVBucketDetection.test.js +41 -41
- package/dist/hooks/__tests__/useIsBucketEmpty.test.js +25 -25
- package/dist/hooks/bucketConfiguration.d.ts +1 -1
- package/dist/hooks/bucketConfiguration.js +48 -48
- package/dist/hooks/bucketOperations.d.ts +1 -1
- package/dist/hooks/bucketOperations.js +6 -6
- package/dist/hooks/factories/__tests__/useCreateS3FunctionMutationHook.test.js +78 -78
- package/dist/hooks/factories/__tests__/useCreateS3InfiniteQueryHook.test.js +78 -78
- package/dist/hooks/factories/__tests__/useCreateS3LoginHook.test.js +42 -42
- package/dist/hooks/factories/__tests__/useCreateS3MutationHook.test.js +61 -61
- package/dist/hooks/factories/__tests__/useCreateS3QueryHook.test.js +63 -63
- package/dist/hooks/factories/index.d.ts +4 -4
- package/dist/hooks/factories/useCreateS3InfiniteQueryHook.d.ts +2 -2
- package/dist/hooks/factories/useCreateS3InfiniteQueryHook.js +15 -12
- package/dist/hooks/factories/useCreateS3LoginHook.d.ts +2 -2
- package/dist/hooks/factories/useCreateS3MutationHook.d.ts +3 -3
- package/dist/hooks/factories/useCreateS3MutationHook.js +6 -1
- package/dist/hooks/factories/useCreateS3QueryHook.d.ts +2 -2
- package/dist/hooks/factories/useCreateS3QueryHook.js +8 -5
- package/dist/hooks/index.d.ts +14 -13
- package/dist/hooks/index.js +2 -1
- package/dist/hooks/loginOperations.d.ts +1 -1
- package/dist/hooks/loginOperations.js +1 -1
- package/dist/hooks/objectOperations.d.ts +2 -2
- package/dist/hooks/objectOperations.js +49 -49
- package/dist/hooks/presignedOperations.d.ts +2 -2
- package/dist/hooks/presignedOperations.js +3 -3
- package/dist/hooks/useBatchObjectLegalHold.js +7 -4
- package/dist/hooks/useDataBrowserNavigate.d.ts +14 -0
- package/dist/hooks/useDataBrowserNavigate.js +24 -0
- package/dist/hooks/useDeleteBucketConfigRule.d.ts +2 -2
- package/dist/hooks/useDeleteBucketConfigRule.js +4 -4
- package/dist/hooks/useEmptyBucket.js +10 -10
- package/dist/hooks/useISVBucketDetection.js +3 -3
- package/dist/hooks/useIsBucketEmpty.js +4 -4
- package/dist/hooks/useLoginMutation.d.ts +1 -1
- package/dist/hooks/useLoginMutation.js +1 -1
- package/dist/hooks/useS3Client.d.ts +5 -0
- package/dist/hooks/useS3Client.js +3 -2
- package/dist/hooks/useS3ConfigSwitch.d.ts +11 -0
- package/dist/hooks/useS3ConfigSwitch.js +37 -0
- package/dist/index.d.ts +6 -6
- package/dist/test/msw/handlers/deleteBucket.d.ts +1 -1
- package/dist/test/msw/handlers/deleteBucket.js +20 -10
- package/dist/test/msw/handlers/getBucketAcl.d.ts +1 -1
- package/dist/test/msw/handlers/getBucketAcl.js +29 -17
- package/dist/test/msw/handlers/getBucketLocation.d.ts +1 -1
- package/dist/test/msw/handlers/getBucketLocation.js +29 -15
- package/dist/test/msw/handlers/getBucketPolicy.d.ts +1 -1
- package/dist/test/msw/handlers/getBucketPolicy.js +52 -32
- package/dist/test/msw/handlers/headObject.d.ts +1 -1
- package/dist/test/msw/handlers/headObject.js +31 -13
- package/dist/test/msw/handlers/listBuckets.d.ts +1 -1
- package/dist/test/msw/handlers/listBuckets.js +5 -3
- package/dist/test/msw/handlers/listObjectVersions.d.ts +1 -1
- package/dist/test/msw/handlers/listObjectVersions.js +38 -26
- package/dist/test/msw/handlers/listObjects.d.ts +1 -1
- package/dist/test/msw/handlers/listObjects.js +35 -23
- package/dist/test/msw/handlers/objectLegalHold.d.ts +1 -1
- package/dist/test/msw/handlers/objectLegalHold.js +31 -16
- package/dist/test/msw/handlers/objectRetention.d.ts +1 -1
- package/dist/test/msw/handlers/objectRetention.js +31 -17
- package/dist/test/msw/handlers/putBucketAcl.d.ts +1 -1
- package/dist/test/msw/handlers/putBucketAcl.js +29 -14
- package/dist/test/msw/handlers/putObject.d.ts +1 -1
- package/dist/test/msw/handlers/putObject.js +27 -12
- package/dist/test/msw/handlers.d.ts +3 -3
- package/dist/test/msw/handlers.js +72 -49
- package/dist/test/msw/index.d.ts +2 -2
- package/dist/test/msw/server.d.ts +1 -1
- package/dist/test/msw/server.js +1 -1
- package/dist/test/msw/utils.js +2 -2
- package/dist/test/setup.d.ts +1 -1
- package/dist/test/setup.js +19 -19
- package/dist/test/testUtils.d.ts +9 -15
- package/dist/test/testUtils.js +78 -92
- package/dist/test/utils/errorHandling.test.js +119 -119
- package/dist/types/index.d.ts +6 -31
- package/dist/utils/__tests__/s3ConfigIdentifier.test.d.ts +1 -0
- package/dist/utils/__tests__/s3ConfigIdentifier.test.js +429 -0
- package/dist/utils/constants.js +8 -8
- package/dist/utils/deletion/index.d.ts +2 -2
- package/dist/utils/deletion/messages.d.ts +1 -1
- package/dist/utils/deletion/messages.js +4 -4
- package/dist/utils/errorHandling.d.ts +3 -3
- package/dist/utils/errorHandling.js +6 -6
- package/dist/utils/hooks.js +8 -8
- package/dist/utils/index.d.ts +5 -4
- package/dist/utils/index.js +2 -0
- package/dist/utils/proxyMiddleware.d.ts +1 -1
- package/dist/utils/proxyMiddleware.js +6 -11
- package/dist/utils/s3Client.d.ts +2 -2
- package/dist/utils/s3Client.js +1 -1
- package/dist/utils/s3ConfigIdentifier.d.ts +68 -0
- package/dist/utils/s3ConfigIdentifier.js +55 -0
- package/dist/utils/s3RuleUtils.d.ts +5 -5
- package/dist/utils/s3RuleUtils.js +17 -17
- package/dist/utils/useSupportedNotificationEvents.d.ts +6 -0
- package/dist/utils/useSupportedNotificationEvents.js +7 -0
- package/package.json +2 -2
|
@@ -1,11 +1,11 @@
|
|
|
1
1
|
import { jsx } from "react/jsx-runtime";
|
|
2
2
|
import { fireEvent, render, screen, waitFor } from "@testing-library/react";
|
|
3
3
|
import user_event from "@testing-library/user-event";
|
|
4
|
-
import { MemoryRouter } from "react-router
|
|
4
|
+
import { MemoryRouter } from "react-router";
|
|
5
5
|
import { createMockMutationResult, createTestWrapper } from "../../test/testUtils.js";
|
|
6
6
|
import { BucketCreate } from "../buckets/BucketCreate.js";
|
|
7
7
|
import { useCreateBucket, useSetBucketObjectLockConfiguration, useSetBucketVersioning } from "../../hooks/index.js";
|
|
8
|
-
jest.mock(
|
|
8
|
+
jest.mock('../../hooks', ()=>({
|
|
9
9
|
useCreateBucket: jest.fn(),
|
|
10
10
|
useSetBucketVersioning: jest.fn(),
|
|
11
11
|
useSetBucketObjectLockConfiguration: jest.fn()
|
|
@@ -15,12 +15,12 @@ const mockUseSetBucketVersioning = jest.mocked(useSetBucketVersioning);
|
|
|
15
15
|
const mockUseSetBucketObjectLockConfiguration = jest.mocked(useSetBucketObjectLockConfiguration);
|
|
16
16
|
const mockNavigate = jest.fn();
|
|
17
17
|
const mockShowToast = jest.fn();
|
|
18
|
-
jest.mock(
|
|
19
|
-
...jest.requireActual(
|
|
18
|
+
jest.mock('react-router', ()=>({
|
|
19
|
+
...jest.requireActual('react-router'),
|
|
20
20
|
useNavigate: ()=>mockNavigate
|
|
21
21
|
}));
|
|
22
|
-
jest.mock(
|
|
23
|
-
...jest.requireActual(
|
|
22
|
+
jest.mock('@scality/core-ui', ()=>({
|
|
23
|
+
...jest.requireActual('@scality/core-ui'),
|
|
24
24
|
useToast: ()=>({
|
|
25
25
|
showToast: mockShowToast
|
|
26
26
|
})
|
|
@@ -51,108 +51,108 @@ const fillBucketName = async (name)=>{
|
|
|
51
51
|
};
|
|
52
52
|
const submitForm = async ()=>{
|
|
53
53
|
await waitFor(()=>{
|
|
54
|
-
const createButton = screen.getByRole(
|
|
54
|
+
const createButton = screen.getByRole('button', {
|
|
55
55
|
name: /^create$/i
|
|
56
56
|
});
|
|
57
57
|
expect(createButton).not.toBeDisabled();
|
|
58
58
|
}, {
|
|
59
59
|
timeout: 3000
|
|
60
60
|
});
|
|
61
|
-
const createButton = screen.getByRole(
|
|
61
|
+
const createButton = screen.getByRole('button', {
|
|
62
62
|
name: /^create$/i
|
|
63
63
|
});
|
|
64
64
|
await user_event.click(createButton);
|
|
65
65
|
};
|
|
66
|
-
describe(
|
|
66
|
+
describe('BucketCreate', ()=>{
|
|
67
67
|
beforeEach(()=>{
|
|
68
68
|
jest.clearAllMocks();
|
|
69
69
|
setupMocks();
|
|
70
70
|
});
|
|
71
|
-
it(
|
|
71
|
+
it('renders form with required fields', ()=>{
|
|
72
72
|
renderBucketCreate();
|
|
73
|
-
expect(screen.getByText(
|
|
73
|
+
expect(screen.getByText('Create a New Bucket')).toBeInTheDocument();
|
|
74
74
|
expect(screen.getByLabelText(/bucket name/i)).toBeInTheDocument();
|
|
75
|
-
expect(screen.getByText(
|
|
76
|
-
expect(screen.getByText(
|
|
75
|
+
expect(screen.getByText('Versioning')).toBeInTheDocument();
|
|
76
|
+
expect(screen.getByText('Object-lock')).toBeInTheDocument();
|
|
77
77
|
});
|
|
78
|
-
it(
|
|
78
|
+
it('shows validation error for invalid bucket name with uppercase', async ()=>{
|
|
79
79
|
renderBucketCreate();
|
|
80
|
-
await user_event.type(screen.getByLabelText(/bucket name/i),
|
|
80
|
+
await user_event.type(screen.getByLabelText(/bucket name/i), 'INVALID');
|
|
81
81
|
await user_event.tab();
|
|
82
82
|
await waitFor(()=>{
|
|
83
83
|
expect(screen.getByText(/bucket names can include only lowercase/i)).toBeInTheDocument();
|
|
84
84
|
});
|
|
85
85
|
});
|
|
86
|
-
it(
|
|
86
|
+
it('shows validation error for bucket name starting with hyphen', async ()=>{
|
|
87
87
|
renderBucketCreate();
|
|
88
|
-
await user_event.type(screen.getByLabelText(/bucket name/i),
|
|
88
|
+
await user_event.type(screen.getByLabelText(/bucket name/i), '-invalid');
|
|
89
89
|
await user_event.tab();
|
|
90
90
|
await waitFor(()=>{
|
|
91
91
|
expect(screen.getByText(/bucket names can include only lowercase/i)).toBeInTheDocument();
|
|
92
92
|
});
|
|
93
93
|
});
|
|
94
|
-
it(
|
|
94
|
+
it('shows validation error for bucket name ending with hyphen', async ()=>{
|
|
95
95
|
renderBucketCreate();
|
|
96
|
-
await user_event.type(screen.getByLabelText(/bucket name/i),
|
|
96
|
+
await user_event.type(screen.getByLabelText(/bucket name/i), 'invalid-');
|
|
97
97
|
await user_event.tab();
|
|
98
98
|
await waitFor(()=>{
|
|
99
99
|
expect(screen.getByText(/bucket names can include only lowercase/i)).toBeInTheDocument();
|
|
100
100
|
});
|
|
101
101
|
});
|
|
102
|
-
it(
|
|
102
|
+
it('shows validation error for bucket name with adjacent periods', async ()=>{
|
|
103
103
|
renderBucketCreate();
|
|
104
|
-
await user_event.type(screen.getByLabelText(/bucket name/i),
|
|
104
|
+
await user_event.type(screen.getByLabelText(/bucket name/i), 'test..bucket');
|
|
105
105
|
await user_event.tab();
|
|
106
106
|
await waitFor(()=>{
|
|
107
107
|
expect(screen.getByText(/bucket names cannot contain two adjacent periods/i)).toBeInTheDocument();
|
|
108
108
|
});
|
|
109
109
|
});
|
|
110
|
-
it(
|
|
110
|
+
it('shows validation error for bucket name formatted as IP address', async ()=>{
|
|
111
111
|
renderBucketCreate();
|
|
112
|
-
await user_event.type(screen.getByLabelText(/bucket name/i),
|
|
112
|
+
await user_event.type(screen.getByLabelText(/bucket name/i), '192.168.1.1');
|
|
113
113
|
await user_event.tab();
|
|
114
114
|
await waitFor(()=>{
|
|
115
115
|
expect(screen.getByText(/bucket names must not be formatted as an ip address/i)).toBeInTheDocument();
|
|
116
116
|
});
|
|
117
117
|
});
|
|
118
|
-
it(
|
|
118
|
+
it('shows validation error for bucket name with forbidden prefix', async ()=>{
|
|
119
119
|
renderBucketCreate();
|
|
120
|
-
await user_event.type(screen.getByLabelText(/bucket name/i),
|
|
120
|
+
await user_event.type(screen.getByLabelText(/bucket name/i), 'xn--test');
|
|
121
121
|
await user_event.tab();
|
|
122
122
|
await waitFor(()=>{
|
|
123
123
|
expect(screen.getByText(/bucket names must not start with/i)).toBeInTheDocument();
|
|
124
124
|
});
|
|
125
125
|
});
|
|
126
|
-
it(
|
|
126
|
+
it('shows validation error for bucket name with forbidden suffix', async ()=>{
|
|
127
127
|
renderBucketCreate();
|
|
128
|
-
await user_event.type(screen.getByLabelText(/bucket name/i),
|
|
128
|
+
await user_event.type(screen.getByLabelText(/bucket name/i), 'test-s3alias');
|
|
129
129
|
await user_event.tab();
|
|
130
130
|
await waitFor(()=>{
|
|
131
131
|
expect(screen.getByText(/bucket names must not end with/i)).toBeInTheDocument();
|
|
132
132
|
});
|
|
133
133
|
});
|
|
134
|
-
it(
|
|
134
|
+
it('shows validation error for duplicate bucket name when context provided', async ()=>{
|
|
135
135
|
renderBucketCreate({
|
|
136
136
|
validationContext: {
|
|
137
137
|
existingBuckets: [
|
|
138
|
-
|
|
139
|
-
|
|
138
|
+
'existing-bucket',
|
|
139
|
+
'another-bucket'
|
|
140
140
|
]
|
|
141
141
|
}
|
|
142
142
|
});
|
|
143
|
-
await user_event.type(screen.getByLabelText(/bucket name/i),
|
|
143
|
+
await user_event.type(screen.getByLabelText(/bucket name/i), 'existing-bucket');
|
|
144
144
|
await user_event.tab();
|
|
145
145
|
await waitFor(()=>{
|
|
146
146
|
expect(screen.getByText(/a bucket with this name already exists/i)).toBeInTheDocument();
|
|
147
147
|
});
|
|
148
148
|
});
|
|
149
|
-
it(
|
|
149
|
+
it('accepts valid bucket names with dots and hyphens', async ()=>{
|
|
150
150
|
renderBucketCreate();
|
|
151
151
|
const validNames = [
|
|
152
|
-
|
|
153
|
-
|
|
154
|
-
|
|
155
|
-
|
|
152
|
+
'my-bucket-123',
|
|
153
|
+
'my.bucket.name',
|
|
154
|
+
'test-bucket.example',
|
|
155
|
+
'123-bucket-456'
|
|
156
156
|
];
|
|
157
157
|
for (const name of validNames){
|
|
158
158
|
const input = screen.getByLabelText(/bucket name/i);
|
|
@@ -166,79 +166,79 @@ describe("BucketCreate", ()=>{
|
|
|
166
166
|
});
|
|
167
167
|
}
|
|
168
168
|
});
|
|
169
|
-
it(
|
|
169
|
+
it('allows bucket creation when name is unique', async ()=>{
|
|
170
170
|
renderBucketCreate({
|
|
171
171
|
validationContext: {
|
|
172
172
|
existingBuckets: [
|
|
173
|
-
|
|
174
|
-
|
|
173
|
+
'existing-bucket',
|
|
174
|
+
'another-bucket'
|
|
175
175
|
]
|
|
176
176
|
}
|
|
177
177
|
});
|
|
178
|
-
await fillBucketName(
|
|
178
|
+
await fillBucketName('new-unique-bucket');
|
|
179
179
|
await waitFor(()=>{
|
|
180
|
-
expect(screen.getByRole(
|
|
180
|
+
expect(screen.getByRole('button', {
|
|
181
181
|
name: /^create$/i
|
|
182
182
|
})).not.toBeDisabled();
|
|
183
183
|
});
|
|
184
184
|
});
|
|
185
|
-
it(
|
|
185
|
+
it('disables create button when form is invalid', ()=>{
|
|
186
186
|
renderBucketCreate();
|
|
187
|
-
expect(screen.getByRole(
|
|
187
|
+
expect(screen.getByRole('button', {
|
|
188
188
|
name: /^create$/i
|
|
189
189
|
})).toBeDisabled();
|
|
190
190
|
});
|
|
191
|
-
it(
|
|
191
|
+
it('enables create button when bucket name is valid', async ()=>{
|
|
192
192
|
renderBucketCreate();
|
|
193
|
-
await fillBucketName(
|
|
193
|
+
await fillBucketName('test-bucket');
|
|
194
194
|
await waitFor(()=>{
|
|
195
|
-
expect(screen.getByRole(
|
|
195
|
+
expect(screen.getByRole('button', {
|
|
196
196
|
name: /^create$/i
|
|
197
197
|
})).not.toBeDisabled();
|
|
198
198
|
});
|
|
199
199
|
});
|
|
200
|
-
it(
|
|
200
|
+
it('creates bucket with basic configuration', async ()=>{
|
|
201
201
|
renderBucketCreate();
|
|
202
|
-
await fillBucketName(
|
|
202
|
+
await fillBucketName('test-bucket');
|
|
203
203
|
await submitForm();
|
|
204
204
|
expect(mockMutations.createBucket).toHaveBeenCalledWith({
|
|
205
|
-
Bucket:
|
|
205
|
+
Bucket: 'test-bucket',
|
|
206
206
|
ObjectLockEnabledForBucket: void 0
|
|
207
207
|
}, expect.objectContaining({
|
|
208
208
|
onSuccess: expect.any(Function),
|
|
209
209
|
onError: expect.any(Function)
|
|
210
210
|
}));
|
|
211
211
|
});
|
|
212
|
-
it(
|
|
212
|
+
it('shows success toast and navigates after bucket creation', async ()=>{
|
|
213
213
|
renderBucketCreate();
|
|
214
|
-
await fillBucketName(
|
|
214
|
+
await fillBucketName('my-bucket');
|
|
215
215
|
await submitForm();
|
|
216
216
|
const successCallback = mockMutations.createBucket.mock.calls[0][1].onSuccess;
|
|
217
217
|
successCallback();
|
|
218
218
|
expect(mockShowToast).toHaveBeenCalledWith({
|
|
219
219
|
open: true,
|
|
220
220
|
message: 'Bucket "my-bucket" created successfully',
|
|
221
|
-
status:
|
|
221
|
+
status: 'success'
|
|
222
222
|
});
|
|
223
|
-
expect(mockNavigate).toHaveBeenCalledWith(
|
|
223
|
+
expect(mockNavigate).toHaveBeenCalledWith('/buckets/my-bucket');
|
|
224
224
|
});
|
|
225
|
-
it(
|
|
225
|
+
it('shows error toast when bucket creation fails', async ()=>{
|
|
226
226
|
renderBucketCreate();
|
|
227
|
-
await fillBucketName(
|
|
227
|
+
await fillBucketName('test-bucket');
|
|
228
228
|
await submitForm();
|
|
229
229
|
const errorCallback = mockMutations.createBucket.mock.calls[0][1].onError;
|
|
230
|
-
const error = new Error(
|
|
230
|
+
const error = new Error('Bucket already exists');
|
|
231
231
|
errorCallback(error);
|
|
232
232
|
expect(mockShowToast).toHaveBeenCalledWith({
|
|
233
233
|
open: true,
|
|
234
|
-
message:
|
|
235
|
-
status:
|
|
234
|
+
message: 'Bucket already exists',
|
|
235
|
+
status: 'error'
|
|
236
236
|
});
|
|
237
237
|
});
|
|
238
|
-
it(
|
|
238
|
+
it('enables versioning when requested', async ()=>{
|
|
239
239
|
renderBucketCreate();
|
|
240
|
-
await fillBucketName(
|
|
241
|
-
const versioningCheckbox = screen.getByRole(
|
|
240
|
+
await fillBucketName('test-bucket');
|
|
241
|
+
const versioningCheckbox = screen.getByRole('checkbox', {
|
|
242
242
|
name: /versioning/i
|
|
243
243
|
});
|
|
244
244
|
await user_event.click(versioningCheckbox);
|
|
@@ -246,32 +246,32 @@ describe("BucketCreate", ()=>{
|
|
|
246
246
|
const successCallback = mockMutations.createBucket.mock.calls[0][1].onSuccess;
|
|
247
247
|
successCallback();
|
|
248
248
|
expect(mockMutations.setVersioning).toHaveBeenCalledWith({
|
|
249
|
-
Bucket:
|
|
249
|
+
Bucket: 'test-bucket',
|
|
250
250
|
VersioningConfiguration: {
|
|
251
|
-
Status:
|
|
251
|
+
Status: 'Enabled'
|
|
252
252
|
}
|
|
253
253
|
}, expect.objectContaining({
|
|
254
254
|
onSuccess: expect.any(Function),
|
|
255
255
|
onError: expect.any(Function)
|
|
256
256
|
}));
|
|
257
257
|
});
|
|
258
|
-
it(
|
|
258
|
+
it('disables versioning checkbox when object lock is enabled', async ()=>{
|
|
259
259
|
renderBucketCreate();
|
|
260
|
-
await fillBucketName(
|
|
261
|
-
const objectLockCheckbox = screen.getByRole(
|
|
260
|
+
await fillBucketName('test-bucket');
|
|
261
|
+
const objectLockCheckbox = screen.getByRole('checkbox', {
|
|
262
262
|
name: /object-lock/i
|
|
263
263
|
});
|
|
264
264
|
await user_event.click(objectLockCheckbox);
|
|
265
|
-
const versioningCheckbox = screen.getByRole(
|
|
265
|
+
const versioningCheckbox = screen.getByRole('checkbox', {
|
|
266
266
|
name: /versioning/i
|
|
267
267
|
});
|
|
268
268
|
expect(versioningCheckbox).toBeDisabled();
|
|
269
269
|
expect(screen.getByText(/automatically activated when object-lock is enabled/i)).toBeInTheDocument();
|
|
270
270
|
});
|
|
271
|
-
it(
|
|
271
|
+
it('enables versioning automatically when object lock is enabled', async ()=>{
|
|
272
272
|
renderBucketCreate();
|
|
273
|
-
await fillBucketName(
|
|
274
|
-
const objectLockCheckbox = screen.getByRole(
|
|
273
|
+
await fillBucketName('test-bucket');
|
|
274
|
+
const objectLockCheckbox = screen.getByRole('checkbox', {
|
|
275
275
|
name: /object-lock/i
|
|
276
276
|
});
|
|
277
277
|
await user_event.click(objectLockCheckbox);
|
|
@@ -283,13 +283,13 @@ describe("BucketCreate", ()=>{
|
|
|
283
283
|
successCallback();
|
|
284
284
|
expect(mockMutations.setVersioning).not.toHaveBeenCalled();
|
|
285
285
|
});
|
|
286
|
-
it(
|
|
286
|
+
it('configures object lock retention when enabled', async ()=>{
|
|
287
287
|
renderBucketCreate();
|
|
288
|
-
await fillBucketName(
|
|
289
|
-
await user_event.click(screen.getByRole(
|
|
288
|
+
await fillBucketName('test-bucket');
|
|
289
|
+
await user_event.click(screen.getByRole('checkbox', {
|
|
290
290
|
name: /object-lock/i
|
|
291
291
|
}));
|
|
292
|
-
await user_event.click(screen.getByRole(
|
|
292
|
+
await user_event.click(screen.getByRole('checkbox', {
|
|
293
293
|
name: /default retention/i
|
|
294
294
|
}));
|
|
295
295
|
await submitForm();
|
|
@@ -297,12 +297,12 @@ describe("BucketCreate", ()=>{
|
|
|
297
297
|
createSuccessCallback();
|
|
298
298
|
expect(mockMutations.setVersioning).not.toHaveBeenCalled();
|
|
299
299
|
expect(mockMutations.setObjectLock).toHaveBeenCalledWith({
|
|
300
|
-
Bucket:
|
|
300
|
+
Bucket: 'test-bucket',
|
|
301
301
|
ObjectLockConfiguration: {
|
|
302
|
-
ObjectLockEnabled:
|
|
302
|
+
ObjectLockEnabled: 'Enabled',
|
|
303
303
|
Rule: {
|
|
304
304
|
DefaultRetention: {
|
|
305
|
-
Mode:
|
|
305
|
+
Mode: 'GOVERNANCE',
|
|
306
306
|
Days: 1,
|
|
307
307
|
Years: void 0
|
|
308
308
|
}
|
|
@@ -313,83 +313,83 @@ describe("BucketCreate", ()=>{
|
|
|
313
313
|
onError: expect.any(Function)
|
|
314
314
|
}));
|
|
315
315
|
});
|
|
316
|
-
it(
|
|
316
|
+
it('calls custom onSubmit when provided', async ()=>{
|
|
317
317
|
const customSubmit = jest.fn();
|
|
318
318
|
renderBucketCreate({
|
|
319
319
|
onSubmit: customSubmit
|
|
320
320
|
});
|
|
321
|
-
await fillBucketName(
|
|
321
|
+
await fillBucketName('test-bucket');
|
|
322
322
|
await submitForm();
|
|
323
323
|
expect(customSubmit).toHaveBeenCalledWith(expect.objectContaining({
|
|
324
|
-
name:
|
|
324
|
+
name: 'test-bucket',
|
|
325
325
|
isVersioning: false,
|
|
326
326
|
isObjectLockEnabled: false
|
|
327
327
|
}));
|
|
328
328
|
expect(mockMutations.createBucket).not.toHaveBeenCalled();
|
|
329
329
|
});
|
|
330
|
-
it(
|
|
330
|
+
it('calls custom onCancel when provided', ()=>{
|
|
331
331
|
const customCancel = jest.fn();
|
|
332
332
|
renderBucketCreate({
|
|
333
333
|
onCancel: customCancel
|
|
334
334
|
});
|
|
335
|
-
const cancelButton = screen.getByRole(
|
|
335
|
+
const cancelButton = screen.getByRole('button', {
|
|
336
336
|
name: /cancel/i
|
|
337
337
|
});
|
|
338
338
|
fireEvent.click(cancelButton);
|
|
339
339
|
expect(customCancel).toHaveBeenCalled();
|
|
340
340
|
expect(mockNavigate).not.toHaveBeenCalled();
|
|
341
341
|
});
|
|
342
|
-
it(
|
|
342
|
+
it('navigates to buckets list when cancel is clicked without custom handler', ()=>{
|
|
343
343
|
renderBucketCreate();
|
|
344
|
-
const cancelButton = screen.getByRole(
|
|
344
|
+
const cancelButton = screen.getByRole('button', {
|
|
345
345
|
name: /cancel/i
|
|
346
346
|
});
|
|
347
347
|
fireEvent.click(cancelButton);
|
|
348
|
-
expect(mockNavigate).toHaveBeenCalledWith(
|
|
348
|
+
expect(mockNavigate).toHaveBeenCalledWith('/buckets');
|
|
349
349
|
});
|
|
350
|
-
it(
|
|
350
|
+
it('renders injected children', ()=>{
|
|
351
351
|
renderBucketCreate({
|
|
352
352
|
children: /*#__PURE__*/ jsx("div", {
|
|
353
353
|
"data-testid": "custom-field",
|
|
354
354
|
children: "Custom Field"
|
|
355
355
|
})
|
|
356
356
|
});
|
|
357
|
-
expect(screen.getByTestId(
|
|
357
|
+
expect(screen.getByTestId('custom-field')).toBeInTheDocument();
|
|
358
358
|
});
|
|
359
|
-
it(
|
|
359
|
+
it('shows loading state during bucket creation', async ()=>{
|
|
360
360
|
renderBucketCreate();
|
|
361
|
-
await fillBucketName(
|
|
361
|
+
await fillBucketName('test-bucket');
|
|
362
362
|
await submitForm();
|
|
363
363
|
setupMocks({
|
|
364
364
|
isPending: true
|
|
365
365
|
});
|
|
366
366
|
expect(mockMutations.createBucket).toHaveBeenCalled();
|
|
367
367
|
});
|
|
368
|
-
it(
|
|
368
|
+
it('shows error toast when versioning configuration fails', async ()=>{
|
|
369
369
|
renderBucketCreate();
|
|
370
|
-
await fillBucketName(
|
|
371
|
-
await user_event.click(screen.getByRole(
|
|
370
|
+
await fillBucketName('test-bucket');
|
|
371
|
+
await user_event.click(screen.getByRole('checkbox', {
|
|
372
372
|
name: /versioning/i
|
|
373
373
|
}));
|
|
374
374
|
await submitForm();
|
|
375
375
|
const createSuccessCallback = mockMutations.createBucket.mock.calls[0][1].onSuccess;
|
|
376
376
|
createSuccessCallback();
|
|
377
377
|
const versioningErrorCallback = mockMutations.setVersioning.mock.calls[0][1].onError;
|
|
378
|
-
const error = new Error(
|
|
378
|
+
const error = new Error('Failed to enable versioning');
|
|
379
379
|
versioningErrorCallback(error);
|
|
380
380
|
expect(mockShowToast).toHaveBeenCalledWith({
|
|
381
381
|
open: true,
|
|
382
|
-
message:
|
|
383
|
-
status:
|
|
382
|
+
message: 'Failed to enable versioning',
|
|
383
|
+
status: 'error'
|
|
384
384
|
});
|
|
385
385
|
});
|
|
386
|
-
it(
|
|
386
|
+
it('shows error toast when object lock configuration fails', async ()=>{
|
|
387
387
|
renderBucketCreate();
|
|
388
|
-
await fillBucketName(
|
|
389
|
-
await user_event.click(screen.getByRole(
|
|
388
|
+
await fillBucketName('test-bucket');
|
|
389
|
+
await user_event.click(screen.getByRole('checkbox', {
|
|
390
390
|
name: /object-lock/i
|
|
391
391
|
}));
|
|
392
|
-
await user_event.click(screen.getByRole(
|
|
392
|
+
await user_event.click(screen.getByRole('checkbox', {
|
|
393
393
|
name: /default retention/i
|
|
394
394
|
}));
|
|
395
395
|
await submitForm();
|
|
@@ -397,12 +397,12 @@ describe("BucketCreate", ()=>{
|
|
|
397
397
|
createSuccessCallback();
|
|
398
398
|
expect(mockMutations.setVersioning).not.toHaveBeenCalled();
|
|
399
399
|
const objectLockErrorCallback = mockMutations.setObjectLock.mock.calls[0][1].onError;
|
|
400
|
-
const error = new Error(
|
|
400
|
+
const error = new Error('Failed to configure retention');
|
|
401
401
|
objectLockErrorCallback(error);
|
|
402
402
|
expect(mockShowToast).toHaveBeenCalledWith({
|
|
403
403
|
open: true,
|
|
404
|
-
message:
|
|
405
|
-
status:
|
|
404
|
+
message: 'Failed to configure retention',
|
|
405
|
+
status: 'error'
|
|
406
406
|
});
|
|
407
407
|
});
|
|
408
408
|
});
|