Skip to content

Commit 7873b6e

Browse files
committed
nit
1 parent be20bc1 commit 7873b6e

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

packages/compass-collection/src/components/mock-data-generator-modal/mock-data-generator-modal.spec.tsx

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -211,6 +211,11 @@ describe('MockDataGeneratorModal', () => {
211211
).to.equal('true');
212212
});
213213

214+
it('displays the namespace', async () => {
215+
await renderModal();
216+
expect(screen.queryByText('test.collection')).to.exist;
217+
});
218+
214219
it('uses the appropriate copy when Generative AI sample document passing is enabled', async () => {
215220
await renderModal({ enableGenAISampleDocumentPassing: true });
216221
expect(screen.queryByText('Sample Documents Collected')).to.exist;
@@ -266,8 +271,6 @@ describe('MockDataGeneratorModal', () => {
266271
expect(screen.getByText('LLM Request failed. Please confirm again.')).to
267272
.exist;
268273
});
269-
270-
// todo: assert that closing then re-opening the modal after an LLM err removes the err message
271274
});
272275

273276
describe('on the generate data step', () => {

packages/compass-collection/src/components/mock-data-generator-modal/raw-schema-confirmation-screen.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -61,10 +61,10 @@ const RawSchemaConfirmationScreen = ({
6161
? 'A sample of document values from your collection will be sent to an LLM for processing.'
6262
: 'We have identified the following schema from your documents. This schema will be sent to an LLM for processing.';
6363

64-
// todo: should establish if unfinished schema analysis edge case should be handled within the modal or
65-
// from the button opening the modal
64+
// (CLOUDP-333853) todo: disable button that opens modal and update tooltip while schema analysis is underway.
65+
// after the todo this state should not be reachable
6666
const schemaAnalysisIncompletePlaceholder = (
67-
<Body>Schema analysis has yet to complete successfully.</Body>
67+
<Body>We are analyzing your collection.</Body>
6868
);
6969

7070
return (

0 commit comments

Comments
 (0)