From da22a7d1638fda9603b6dbf57ce64d3ca821e9f8 Mon Sep 17 00:00:00 2001 From: Case Kramer Date: Mon, 10 Nov 2025 16:00:38 -0500 Subject: [PATCH 1/3] Mock Data Generator QA items --- .../faker-mapping-selector.tsx | 1 - .../faker-schema-editor-screen.tsx | 40 ++++++------------- .../mock-data-generator-modal.spec.tsx | 2 +- .../schema-field-selector.tsx | 6 ++- 4 files changed, 17 insertions(+), 32 deletions(-) diff --git a/packages/compass-collection/src/components/mock-data-generator-modal/faker-mapping-selector.tsx b/packages/compass-collection/src/components/mock-data-generator-modal/faker-mapping-selector.tsx index aa08248262b..ba85c953ebe 100644 --- a/packages/compass-collection/src/components/mock-data-generator-modal/faker-mapping-selector.tsx +++ b/packages/compass-collection/src/components/mock-data-generator-modal/faker-mapping-selector.tsx @@ -27,7 +27,6 @@ const fieldMappingSelectorsStyles = css({ }); const labelStyles = css({ - color: palette.gray.dark1, fontWeight: 600, }); diff --git a/packages/compass-collection/src/components/mock-data-generator-modal/faker-schema-editor-screen.tsx b/packages/compass-collection/src/components/mock-data-generator-modal/faker-schema-editor-screen.tsx index 744a65efb57..48d80583b7a 100644 --- a/packages/compass-collection/src/components/mock-data-generator-modal/faker-schema-editor-screen.tsx +++ b/packages/compass-collection/src/components/mock-data-generator-modal/faker-schema-editor-screen.tsx @@ -4,6 +4,8 @@ import { ButtonSize, ButtonVariant, css, + Description, + H3, Link, palette, spacing, @@ -35,21 +37,12 @@ const innerEditorStyles = css({ }); const titleStyles = css({ - color: palette.black, fontWeight: 600, fontSize: '16px', lineHeight: '20px', marginBottom: 0, }); -const bodyStyles = css({ - color: palette.gray.dark1, -}); - -const confirmMappingsButtonStyles = css({ - width: '200px', -}); - const schemaEditorLoaderStyles = css({ display: 'flex', alignItems: 'center', @@ -58,10 +51,8 @@ const schemaEditorLoaderStyles = css({ const FakerSchemaEditorContent = ({ fakerSchema, - onSchemaConfirmed, }: { fakerSchema: FakerSchema; - onSchemaConfirmed: () => void; }) => { const track = useTelemetry(); const [fakerSchemaFormValues, setFakerSchemaFormValues] = @@ -176,14 +167,6 @@ const FakerSchemaEditorContent = ({ /> )} - ); }; @@ -198,16 +181,17 @@ const FakerSchemaEditorScreen = ({ return (
-

+

Confirm Field to Faker Function Mappings -

- - We have sampled your collection and created a schema based on your - documents. That schema has been sent to an LLM and it has returned the - following mapping between your schema fields and{' '} - faker functions - . - + + + We analyzed a sample of your data and used generative AI to suggest + the following mapping between your document fields and data simulation + functions.{' '} + + Learn more about MongoDB’s AI usage + +
{fakerSchemaGenerationState.status === 'in-progress' && (
{ expect(screen.getByTestId('faker-schema-editor')).to.exist; }); - userEvent.click(screen.getByText('Confirm mappings')); + userEvent.click(screen.getByText('Next')); await waitFor(() => { expect(screen.getByText('Specify Number of Documents to Generate')).to diff --git a/packages/compass-collection/src/components/mock-data-generator-modal/schema-field-selector.tsx b/packages/compass-collection/src/components/mock-data-generator-modal/schema-field-selector.tsx index c4bc6c7abf5..4a64cca267b 100644 --- a/packages/compass-collection/src/components/mock-data-generator-modal/schema-field-selector.tsx +++ b/packages/compass-collection/src/components/mock-data-generator-modal/schema-field-selector.tsx @@ -50,8 +50,11 @@ const activeStylesLight = css({ const activeStylesDark = css({ color: palette.white, + backgroundColor: palette.green.dark3, + fontWeight: 600, + '&:active,&:focus': { - backgroundColor: palette.gray.dark3, + backgroundColor: palette.green.dark3, color: palette.white, }, }); @@ -71,7 +74,6 @@ const hoverStylesDark = css({ }); const labelStyles = css({ - color: palette.gray.dark1, fontWeight: 600, }); From e632b485cd6b0d728705910e2cc7fb40a1a8ea09 Mon Sep 17 00:00:00 2001 From: Case Kramer Date: Tue, 11 Nov 2025 12:31:00 -0500 Subject: [PATCH 2/3] Update styling --- .../mock-data-generator-modal/document-count-screen.tsx | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/packages/compass-collection/src/components/mock-data-generator-modal/document-count-screen.tsx b/packages/compass-collection/src/components/mock-data-generator-modal/document-count-screen.tsx index e08be615fa3..0144d72f5aa 100644 --- a/packages/compass-collection/src/components/mock-data-generator-modal/document-count-screen.tsx +++ b/packages/compass-collection/src/components/mock-data-generator-modal/document-count-screen.tsx @@ -2,6 +2,7 @@ import { Body, compactBytes, css, + Description, palette, spacing, TextInput, @@ -21,7 +22,6 @@ const titleStyles = css({ }); const descriptionStyles = css({ - color: palette.gray.dark1, fontStyle: 'italic', }); @@ -107,9 +107,9 @@ const DocumentCountScreen = ({ Specify Number of Documents to Generate - + Indicate the amount of documents you want to generate below. - +
Date: Tue, 11 Nov 2025 13:56:49 -0500 Subject: [PATCH 3/3] cleanup --- .../mock-data-generator-modal/document-count-screen.tsx | 1 - .../mock-data-generator-modal/faker-mapping-selector.tsx | 1 - .../faker-schema-editor-screen.tsx | 8 -------- .../mock-data-generator-modal.tsx | 2 -- 4 files changed, 12 deletions(-) diff --git a/packages/compass-collection/src/components/mock-data-generator-modal/document-count-screen.tsx b/packages/compass-collection/src/components/mock-data-generator-modal/document-count-screen.tsx index 0144d72f5aa..a232600a30e 100644 --- a/packages/compass-collection/src/components/mock-data-generator-modal/document-count-screen.tsx +++ b/packages/compass-collection/src/components/mock-data-generator-modal/document-count-screen.tsx @@ -3,7 +3,6 @@ import { compactBytes, css, Description, - palette, spacing, TextInput, } from '@mongodb-js/compass-components'; diff --git a/packages/compass-collection/src/components/mock-data-generator-modal/faker-mapping-selector.tsx b/packages/compass-collection/src/components/mock-data-generator-modal/faker-mapping-selector.tsx index ba85c953ebe..5b6c289b5fa 100644 --- a/packages/compass-collection/src/components/mock-data-generator-modal/faker-mapping-selector.tsx +++ b/packages/compass-collection/src/components/mock-data-generator-modal/faker-mapping-selector.tsx @@ -6,7 +6,6 @@ import { css, Label, Option, - palette, Select, spacing, } from '@mongodb-js/compass-components'; diff --git a/packages/compass-collection/src/components/mock-data-generator-modal/faker-schema-editor-screen.tsx b/packages/compass-collection/src/components/mock-data-generator-modal/faker-schema-editor-screen.tsx index 48d80583b7a..646262059a1 100644 --- a/packages/compass-collection/src/components/mock-data-generator-modal/faker-schema-editor-screen.tsx +++ b/packages/compass-collection/src/components/mock-data-generator-modal/faker-schema-editor-screen.tsx @@ -1,13 +1,8 @@ import { - Body, - Button, - ButtonSize, - ButtonVariant, css, Description, H3, Link, - palette, spacing, SpinLoaderWithLabel, } from '@mongodb-js/compass-components'; @@ -172,10 +167,8 @@ const FakerSchemaEditorContent = ({ }; const FakerSchemaEditorScreen = ({ - onSchemaConfirmed, fakerSchemaGenerationState, }: { - onSchemaConfirmed: () => void; fakerSchemaGenerationState: MockDataGeneratorState; }) => { return ( @@ -204,7 +197,6 @@ const FakerSchemaEditorScreen = ({ {fakerSchemaGenerationState.status === 'completed' && ( )}
diff --git a/packages/compass-collection/src/components/mock-data-generator-modal/mock-data-generator-modal.tsx b/packages/compass-collection/src/components/mock-data-generator-modal/mock-data-generator-modal.tsx index 91da5568871..c4cfb06413f 100644 --- a/packages/compass-collection/src/components/mock-data-generator-modal/mock-data-generator-modal.tsx +++ b/packages/compass-collection/src/components/mock-data-generator-modal/mock-data-generator-modal.tsx @@ -96,7 +96,6 @@ const MockDataGeneratorModal = ({ case MockDataGeneratorStep.SCHEMA_EDITOR: return ( ); @@ -125,7 +124,6 @@ const MockDataGeneratorModal = ({ fakerSchemaGenerationState, documentCount, onDocumentCountChange, - onNextStep, ]); useTrackOnChange(