@scality/data-browser-library 1.0.4 → 1.0.6

package/dist/components/__tests__/ObjectList.test.js

@@ -1,5 +1,5 @@
 import { jsx, jsxs } from "react/jsx-runtime";
-import { fireEvent, render, renderHook, screen, waitFor } from "@testing-library/react";
+import { act, fireEvent, render, renderHook, screen, waitFor } from "@testing-library/react";
 import { MemoryRouter } from "react-router";
 import { DataBrowserUICustomizationProvider } from "../../contexts/DataBrowserUICustomizationContext.js";
 import { useListObjectVersions, useListObjects } from "../../hooks/index.js";
@@ -281,14 +281,42 @@ describe('ObjectList', ()=>{
                 expect(screen.getByText('Version ID')).toBeInTheDocument();
             });
         });
+        it('notifies parent to clear selection when toggling versions', async ()=>{
+            const onObjectSelect = jest.fn();
+            renderObjectList({
+                onObjectSelect
+            });
+            await waitFor(()=>{
+                expect(screen.getByText('file1.txt')).toBeInTheDocument();
+            });
+            const rows = screen.getAllByRole('row');
+            const objectRow = rows.find((row)=>row.textContent?.includes('file1.txt'));
+            expect(objectRow).toBeDefined();
+            fireEvent.click(objectRow);
+            expect(onObjectSelect).toHaveBeenCalledWith(expect.objectContaining({
+                Key: 'file1.txt',
+                type: 'object'
+            }));
+            onObjectSelect.mockClear();
+            const toggleLabel = screen.getByText(/list versions/i);
+            const toggle = toggleLabel.closest('label')?.querySelector('input[type="checkbox"]');
+            fireEvent.click(toggle);
+            await waitFor(()=>{
+                expect(screen.getByText('Version ID')).toBeInTheDocument();
+            });
+            expect(onObjectSelect).toHaveBeenCalledWith(null);
+        });
     });
     describe('Search Features', ()=>{
-        it('renders table search when metadata-search feature is disabled', async ()=>{
+        it('renders SearchInput when metadata-search feature is disabled', async ()=>{
             jest.spyOn(__rspack_external__hooks_useFeatures_js_a6a84786, 'useFeatures').mockReturnValue(false);
             renderObjectList();
             await waitFor(()=>{
                 expect(screen.getByRole('grid')).toBeInTheDocument();
             });
+            expect(screen.getByRole('searchbox', {
+                name: 'search'
+            })).toBeInTheDocument();
             expect(screen.queryByPlaceholderText(/Metadata Search/i)).not.toBeInTheDocument();
         });
         it('renders MetadataSearch component when metadata-search feature is enabled', async ()=>{
@@ -299,6 +327,70 @@ describe('ObjectList', ()=>{
             });
             expect(screen.getByPlaceholderText(/Metadata Search/i)).toBeInTheDocument();
         });
+        it('filters table rows when typing in search input', async ()=>{
+            jest.useFakeTimers();
+            jest.spyOn(__rspack_external__hooks_useFeatures_js_a6a84786, 'useFeatures').mockReturnValue(false);
+            renderObjectList();
+            await waitFor(()=>{
+                expect(screen.getByText('file1.txt')).toBeInTheDocument();
+                expect(screen.getByText('file2.txt')).toBeInTheDocument();
+                expect(screen.getByText('folder1/')).toBeInTheDocument();
+            });
+            const searchInput = screen.getByRole('searchbox', {
+                name: 'search'
+            });
+            fireEvent.change(searchInput, {
+                target: {
+                    value: 'file1'
+                }
+            });
+            act(()=>{
+                jest.advanceTimersByTime(300);
+            });
+            await waitFor(()=>{
+                expect(screen.getByText('file1.txt')).toBeInTheDocument();
+                expect(screen.queryByText('file2.txt')).not.toBeInTheDocument();
+                expect(screen.queryByText('folder1/')).not.toBeInTheDocument();
+            });
+            jest.useRealTimers();
+        });
+        it('shows all rows when search is cleared', async ()=>{
+            jest.useFakeTimers();
+            jest.spyOn(__rspack_external__hooks_useFeatures_js_a6a84786, 'useFeatures').mockReturnValue(false);
+            renderObjectList();
+            await waitFor(()=>{
+                expect(screen.getByText('file1.txt')).toBeInTheDocument();
+                expect(screen.getByText('file2.txt')).toBeInTheDocument();
+            });
+            const searchInput = screen.getByRole('searchbox', {
+                name: 'search'
+            });
+            fireEvent.change(searchInput, {
+                target: {
+                    value: 'file1'
+                }
+            });
+            act(()=>{
+                jest.advanceTimersByTime(300);
+            });
+            await waitFor(()=>{
+                expect(screen.queryByText('file2.txt')).not.toBeInTheDocument();
+            });
+            fireEvent.change(searchInput, {
+                target: {
+                    value: ''
+                }
+            });
+            act(()=>{
+                jest.advanceTimersByTime(300);
+            });
+            await waitFor(()=>{
+                expect(screen.getByText('file1.txt')).toBeInTheDocument();
+                expect(screen.getByText('file2.txt')).toBeInTheDocument();
+                expect(screen.getByText('folder1/')).toBeInTheDocument();
+            });
+            jest.useRealTimers();
+        });
     });
     describe('Multi-Selection', ()=>{
         it('integrates with MultiSelectableContent', async ()=>{

package/dist/components/buckets/BucketCreate.d.ts

@@ -5,6 +5,7 @@ export declare const bucketNameValidationSchema: Joi.StringSchema<string>;
 export declare const baseBucketCreateSchema: Joi.ObjectSchema<any>;
 export type BucketCreateFormData = {
     name: string;
+    locationConstraint?: string;
     isVersioning: boolean;
     isEncryptionEnabled: boolean;
     isObjectLockEnabled: boolean;

package/dist/components/buckets/BucketCreate.js

@@ -4,7 +4,9 @@ import { Checkbox, Form, FormGroup, FormSection, Stack, useToast } from "@scalit
 import { Button, Input } from "@scality/core-ui/dist/next";
 import { convertRemToPixels } from "@scality/core-ui/dist/utils";
 import joi from "joi";
-import { FormProvider, useForm } from "react-hook-form";
+import { useMemo } from "react";
+import { Controller, FormProvider, useForm } from "react-hook-form";
+import { useDataBrowserUICustomization } from "../../contexts/DataBrowserUICustomizationContext.js";
 import { useCreateBucket, useSetBucketEncryption, useSetBucketObjectLockConfiguration, useSetBucketVersioning } from "../../hooks/index.js";
 import { useDataBrowserNavigate } from "../../hooks/useDataBrowserNavigate.js";
 import ObjectLockRetentionSettings from "../objects/ObjectLock/ObjectLockRetentionSettings.js";
@@ -58,8 +60,12 @@ const baseBucketCreateSchema = joi.object({
 const BucketCreate = ({ subTitle, validationSchema, validationContext, defaultValues: customDefaultValues, children, onSubmit: onSubmitProp, onCancel: onCancelProp })=>{
     const navigate = useDataBrowserNavigate();
     const { showToast } = useToast();
+    const { locationSelector: LocationSelector, bucketCreateExtraFields, transformBucketCreateData, bucketCreateVersioning: VersioningRenderer } = useDataBrowserUICustomization();
     const baseDefaultValues = {
         name: '',
+        ...LocationSelector ? {
+            locationConstraint: ''
+        } : {},
         isVersioning: false,
         isEncryptionEnabled: false,
         isObjectLockEnabled: false,
@@ -69,15 +75,32 @@ const BucketCreate = ({ subTitle, validationSchema, validationContext, defaultVa
         retentionPeriodFrequencyChoice: 'DAYS',
         ...customDefaultValues
     };
+    const hasExtraFields = !!bucketCreateExtraFields;
+    const resolvedSchema = useMemo(()=>{
+        let schema = validationSchema || baseBucketCreateSchema;
+        if (LocationSelector) schema = schema.keys({
+            locationConstraint: joi.string().required().messages({
+                'string.empty': 'Location is required'
+            })
+        });
+        if (hasExtraFields) schema = schema.options({
+            allowUnknown: true
+        });
+        return schema;
+    }, [
+        validationSchema,
+        LocationSelector,
+        hasExtraFields
+    ]);
     const useFormMethods = useForm({
         mode: 'all',
-        resolver: joiResolver(validationSchema || baseBucketCreateSchema, {
+        resolver: joiResolver(resolvedSchema, {
             context: validationContext || {}
         }),
         defaultValues: baseDefaultValues,
         context: validationContext
     });
-    const { register, handleSubmit, formState, watch } = useFormMethods;
+    const { register, handleSubmit, formState, watch, control } = useFormMethods;
     const { isValid, errors } = formState;
     const isObjectLockEnabled = watch('isObjectLockEnabled');
     const isEncryptionEnabled = watch('isEncryptionEnabled');
@@ -106,15 +129,21 @@ const BucketCreate = ({ subTitle, validationSchema, validationContext, defaultVa
             status: 'error'
         });
     };
-    const onSubmit = (data)=>{
-        if (onSubmitProp) return void onSubmitProp(data);
+    const onSubmit = (rawData)=>{
+        if (onSubmitProp) return void onSubmitProp(rawData);
+        const data = transformBucketCreateData ? transformBucketCreateData(rawData) : rawData;
         const bucketName = data.name;
         const needsExplicitVersioning = data.isVersioning && !data.isObjectLockEnabled;
         const needsRetentionConfig = data.isObjectLockEnabled && data.isDefaultRetentionEnabled;
         const needsEncryption = data.isEncryptionEnabled;
         createBucket({
             Bucket: bucketName,
-            ObjectLockEnabledForBucket: data.isObjectLockEnabled || void 0
+            ObjectLockEnabledForBucket: data.isObjectLockEnabled || void 0,
+            ...data.locationConstraint ? {
+                CreateBucketConfiguration: {
+                    LocationConstraint: data.locationConstraint
+                }
+            } : {}
         }, {
             onSuccess: ()=>{
                 if (needsExplicitVersioning) setBucketVersioning({
@@ -236,6 +265,24 @@ const BucketCreate = ({ subTitle, validationSchema, validationContext, defaultVa
                                 helpErrorPosition: "bottom",
                                 error: errors.name ? errors.name?.type === 'string.pattern.base' ? bucketErrorMessage : errors.name?.message : void 0
                             }),
+                            LocationSelector && /*#__PURE__*/ jsx(FormGroup, {
+                                id: "locationConstraint",
+                                label: "Storage Service Location",
+                                required: true,
+                                direction: "horizontal",
+                                labelHelpTooltip: "Cannot be modified after creation",
+                                helpErrorPosition: "bottom",
+                                error: errors?.locationConstraint?.message,
+                                content: /*#__PURE__*/ jsx(Controller, {
+                                    name: "locationConstraint",
+                                    control: control,
+                                    render: ({ field })=>/*#__PURE__*/ jsx(LocationSelector, {
+                                            value: field.value ?? '',
+                                            onChange: field.onChange
+                                        })
+                                })
+                            }),
+                            bucketCreateExtraFields,
                             children,
                             s3Capabilities?.bucketEncryption && /*#__PURE__*/ jsx(FormGroup, {
                                 id: "isEncryptionEnabled",
@@ -248,7 +295,10 @@ const BucketCreate = ({ subTitle, validationSchema, validationContext, defaultVa
                                     ...register('isEncryptionEnabled')
                                 })
                             }),
-                            /*#__PURE__*/ jsx(FormGroup, {
+                            VersioningRenderer ? /*#__PURE__*/ jsx(VersioningRenderer, {
+                                isVersioning: isVersioning,
+                                isObjectLockEnabled: isObjectLockEnabled
+                            }) : /*#__PURE__*/ jsx(FormGroup, {
                                 id: "isVersioning",
                                 label: "Versioning",
                                 disabled: isObjectLockEnabled,

package/dist/components/buckets/BucketDetails.js

@@ -179,7 +179,6 @@ const DefaultTabs = ()=>{
                 id: 'overview',
                 label: 'Overview',
                 path: '',
-                withoutPadding: false,
                 query: {
                     tab: ''
                 },