@mintlify/previewing 4.0.1463 → 4.0.1464

package/dist/__tests__/packages-dev-snippets.test.js

@@ -100,18 +100,21 @@ const fakeBuild = async ({ islands, outDir, }) => {
         missingLocalFiles: [],
     };
 };
-const until = async (condition) => {
-    for (let i = 0; i < 300; i++) {
-        if (condition())
-            return;
-        await new Promise((resolve) => setTimeout(resolve, 10));
-    }
-    throw new Error('condition not met');
-};
 describe('package snippet membership in the mint dev watcher', () => {
     let root;
     let dev;
     let builds;
+    const buildWaiters = new Set();
+    const untilBuilt = (count) => new Promise((resolve) => {
+        const check = () => {
+            if (builds < count)
+                return;
+            buildWaiters.delete(check);
+            resolve();
+        };
+        buildWaiters.add(check);
+        check();
+    });
     const docs = (file) => join(testPaths.cmd, file);
     const output = (file) => fse.readFile(join(file.startsWith('snippets/') ? testPaths.public : testPaths.props, file), 'utf8');
     const outputs = (files) => Promise.all(files.map(output));
@@ -147,7 +150,7 @@ describe('package snippet membership in the mint dev watcher', () => {
     const editSnippet = async (file, content) => {
         const settled = builds + 1;
         await saveSnippet(file, content);
-        await until(() => builds >= settled);
+        await untilBuilt(settled);
     };
     const compilePage = async (file) => {
         const { pageContent } = await createPage(file, await fse.readFile(docs(file), 'utf8'), testPaths.cmd, [], []);
@@ -193,6 +196,19 @@ describe('package snippet membership in the mint dev watcher', () => {
         await generatePagesWithImports(new Set(snippets));
         builds = 0;
     };
+    beforeAll(async () => {
+        // getMdx loads its syntax highlighter on first use. Pay that once here instead of in
+        // whichever test renders first.
+        await getMdx({
+            path: 'warm-up',
+            content: '',
+            metadata: {},
+            snippets: [],
+            subdomain: 'local',
+            codeStyling: undefined,
+            tailwindSelectors: [],
+        });
+    });
     beforeEach(async () => {
         root = await mkdtemp(join(tmpdir(), 'packages-dev-snippets-'));
         testPaths.cmd = join(root, 'docs');
@@ -211,6 +227,8 @@ describe('package snippet membership in the mint dev watcher', () => {
             propsPath: testPaths.props,
             onRebuilt: () => {
                 builds++;
+                for (const check of buildWaiters)
+                    check();
             },
             regenerateSnippets: regenerateAllSnippets,
             reporter: {
@@ -221,11 +239,12 @@ describe('package snippet membership in the mint dev watcher', () => {
                 islandErrors: () => { },
                 configError: () => { },
             },
-            debounceMs: 200,
+            debounceMs: 0,
             build: fakeBuild,
         });
     });
     afterEach(async () => {
+        buildWaiters.clear();
         releaseBuild();
         await dev.close();
         setPackageImportOptions(undefined);
@@ -282,7 +301,7 @@ describe('package snippet membership in the mint dev watcher', () => {
         const settled = builds + 1;
         await fse.remove(docs('snippets/toggle.jsx'));
         await dev.onFileChange('snippets/toggle.jsx', 'snippet-v2');
-        await until(() => builds >= settled);
+        await untilBuilt(settled);
         expect(await packageSnippets()).toEqual([]);
     });
     it('keeps a package snippet that fails to parse on islands when package.json and docs.json are saved', async () => {
@@ -291,10 +310,12 @@ describe('package snippet membership in the mint dev watcher', () => {
         const initial = await outputs(TOGGLE_OUTPUTS);
         await editSnippet('snippets/toggle.jsx', BROKEN_TOGGLE);
         const broken = await outputs(TOGGLE_OUTPUTS);
+        const manifestBuilt = builds + 1;
         await fse.outputJson(docs('package.json'), { dependencies: { '@acme/ui': '1.0.0' } });
         await dev.onFileChange('package.json', getFileCategory('package.json'));
         await fse.outputJson(docs('docs.json'), { name: 'Docs', packages: {} });
         await dev.onFileChange('docs.json', 'docsConfig');
+        await untilBuilt(manifestBuilt);
         expect(await packageSnippets()).toEqual(islands);
         expect(await outputs(TOGGLE_OUTPUTS)).toEqual(broken);
         expectKept(await output('snippets/wrapper.jsx'), TOGGLE_IMPORT, 'A parsing error occurred');
@@ -361,7 +382,7 @@ describe('package snippet membership in the mint dev watcher', () => {
             expect(island).toEqual({ prefix: `mi-${id}-` });
         }
         releaseBuild();
-        await until(() => builds >= settled);
+        await untilBuilt(settled);
         const built = { ...(await render('toggle-page.mdx')), ...(await render('wrapper-page.mdx')) };
         expect(Object.keys(built)).toEqual(Object.keys(pending));
         for (const [id, island] of Object.entries(built)) {
@@ -384,7 +405,7 @@ describe('package snippet membership in the mint dev watcher', () => {
         const [id] = Object.keys(pending);
         expect(pending).toEqual({ [id ?? '']: { prefix: `mi-${id}-` } });
         releaseBuild();
-        await until(() => builds >= settled);
+        await untilBuilt(settled);
         expect(await render('badge-page.mdx')).toEqual({
             [id ?? '']: { prefix: `mi-${id}-`, html: `<svg data-id="${id}"></svg>` },
         });

package/dist/__tests__/packages-dev.test.js

@@ -1201,7 +1201,7 @@ export const Card = () => <LineChart width={400} height={200} data={[{ v: 1 }]}
                 rebuilt = 0;
                 mode.throwing = true;
                 await duringNextBuild(() => write(css('.card { color: blue; }\n')), () => write({ file, content: 'x', category: 'staticFile' }));
-                await until(() => rebuilt >= 1);
+                await until(() => rebuilt >= builds);
                 await settle();
                 expect(rebuilt).toBe(builds);
                 expect(dev.getStatus().state).toBe('failed');