From 335ba40dd0d37ba82c265c99104626f4ede1637a Mon Sep 17 00:00:00 2001 From: Alois Klink Date: Sat, 16 Sep 2023 00:32:32 +0100 Subject: [PATCH 1/4] test(e2e): wait for marker_unique_id.html e2e test Sometimes, Cypress takes the screenshot for marker_unique_id.html early, before mermaid has finished rendering the diagrams. Fixes: 924c9e913b665f22f9a3401dbf31857579833ca4 Co-authored-by: Sidharth Vinod --- cypress/platform/marker_unique_id.html | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/cypress/platform/marker_unique_id.html b/cypress/platform/marker_unique_id.html index e49169c556..98c2c7f4df 100644 --- a/cypress/platform/marker_unique_id.html +++ b/cypress/platform/marker_unique_id.html @@ -42,7 +42,8 @@

Example

From 17f5052a6f5aee254e8639db6e022349358b862d Mon Sep 17 00:00:00 2001 From: Alois Klink Date: Sat, 16 Sep 2023 00:26:24 +0100 Subject: [PATCH 3/4] test(e2e): wait for theme-directives.html The `theme-directives.html` test currently sometimes takes a screenshot before all of the Mermaid diagrams have completed rendering. We can use the `urlSnapshopTest()` helper function, which waits until a `.rendered` property exists on the page. Co-authored-by: Sidharth Vinod --- cypress/integration/rendering/conf-and-directives.spec.js | 5 ++--- cypress/platform/theme-directives.html | 7 +++++++ 2 files changed, 9 insertions(+), 3 deletions(-) diff --git a/cypress/integration/rendering/conf-and-directives.spec.js b/cypress/integration/rendering/conf-and-directives.spec.js index 401a24894b..d447ea9937 100644 --- a/cypress/integration/rendering/conf-and-directives.spec.js +++ b/cypress/integration/rendering/conf-and-directives.spec.js @@ -1,4 +1,4 @@ -import { imgSnapshotTest } from '../../helpers/util.ts'; +import { imgSnapshotTest, urlSnapshotTest } from '../../helpers/util.ts'; describe('Configuration and directives - nodes should be light blue', () => { it('No config - use default', () => { @@ -206,8 +206,7 @@ graph TD describe('when rendering several diagrams', () => { it('diagrams should not taint later diagrams', () => { const url = 'http://localhost:9000/theme-directives.html'; - cy.visit(url); - cy.matchImageSnapshot('conf-and-directives.spec-when-rendering-several-diagrams-diagram-1'); + urlSnapshotTest(url, {}); }); }); }); diff --git a/cypress/platform/theme-directives.html b/cypress/platform/theme-directives.html index 21711a4fe6..bd91688d79 100644 --- a/cypress/platform/theme-directives.html +++ b/cypress/platform/theme-directives.html @@ -121,7 +121,14 @@

Nothing set, should be Default

fontFamily: '"arial", sans-serif', curve: 'cardinal', securityLevel: 'strict', + startOnLoad: false, }); + + await mermaid.run(); + + if (window.Cypress) { + window.rendered = true; + } From 6bab2f66936b0e5fa4a1cff51b59743d3c42e317 Mon Sep 17 00:00:00 2001 From: "renovate[bot]" <29139614+renovate[bot]@users.noreply.github.com> Date: Tue, 19 Sep 2023 11:27:03 +0000 Subject: [PATCH 4/4] chore(deps): update all patch dependencies --- package.json | 2 +- packages/mermaid/src/docs/package.json | 2 +- pnpm-lock.yaml | 13 +++++++++---- 3 files changed, 11 insertions(+), 6 deletions(-) diff --git a/package.json b/package.json index 7a6a032d72..8bdcce0740 100644 --- a/package.json +++ b/package.json @@ -4,7 +4,7 @@ "version": "10.2.4", "description": "Markdownish syntax for generating flowcharts, sequence diagrams, class diagrams, gantt charts and git graphs.", "type": "module", - "packageManager": "pnpm@8.7.5", + "packageManager": "pnpm@8.7.6", "keywords": [ "diagram", "markdown", diff --git a/packages/mermaid/src/docs/package.json b/packages/mermaid/src/docs/package.json index 759d1ffb17..9d8912b04b 100644 --- a/packages/mermaid/src/docs/package.json +++ b/packages/mermaid/src/docs/package.json @@ -32,7 +32,7 @@ "unplugin-vue-components": "^0.25.0", "vite": "^4.3.9", "vite-plugin-pwa": "^0.16.0", - "vitepress": "1.0.0-rc.12", + "vitepress": "1.0.0-rc.14", "workbox-window": "^7.0.0" } } diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml index 5a04bb353b..ef92c63db8 100644 --- a/pnpm-lock.yaml +++ b/pnpm-lock.yaml @@ -475,8 +475,8 @@ importers: specifier: ^0.16.0 version: 0.16.0(vite@4.3.9)(workbox-build@7.0.0)(workbox-window@7.0.0) vitepress: - specifier: 1.0.0-rc.12 - version: 1.0.0-rc.12(@algolia/client-search@4.19.1)(@types/node@18.16.0)(search-insights@2.6.0) + specifier: 1.0.0-rc.14 + version: 1.0.0-rc.14(@algolia/client-search@4.19.1)(@types/node@18.16.0)(search-insights@2.6.0) workbox-window: specifier: ^7.0.0 version: 7.0.0 @@ -15465,9 +15465,14 @@ packages: - terser dev: true - /vitepress@1.0.0-rc.12(@algolia/client-search@4.19.1)(@types/node@18.16.0)(search-insights@2.6.0): - resolution: {integrity: sha512-mZknN5l9lgbBjXwumwdOQQDM+gPivswFEykEQeenY0tv7eocS+bb801IpFZT3mFV6YRhSddmbutHlFgPPADjEg==} + /vitepress@1.0.0-rc.14(@algolia/client-search@4.19.1)(@types/node@18.16.0)(search-insights@2.6.0): + resolution: {integrity: sha512-yChIeXOAcNvVnSVjhziH1vte0uhKb00PuZf7KdIMfx3ixTMAz73Nn+6gREvCv0SdH+anteGUKz5eljv0ygcgGQ==} hasBin: true + peerDependencies: + markdown-it-mathjax3: ^4.3.2 + peerDependenciesMeta: + markdown-it-mathjax3: + optional: true dependencies: '@docsearch/css': 3.5.2 '@docsearch/js': 3.5.2(@algolia/client-search@4.19.1)(search-insights@2.6.0)