Skip to content

Conversation

@merschformann
Copy link
Member

@merschformann merschformann commented Aug 15, 2025

Description

Logs the DAG of a DAG-Test as a mermaid diagram for improved transparency. This allows the user to better understand in which order test cases will get executed.

Changes

  • Logs DAG as mermaid diagram.

Preview

Preview of a DAG visualized from mermaid representation:

image

@merschformann merschformann marked this pull request as ready for review August 15, 2025 15:11
@merschformann merschformann requested a review from Copilot August 15, 2025 15:11
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds DAG visualization functionality by logging a Mermaid diagram representation of test case dependencies. This enhancement improves transparency by allowing users to visually understand the execution order of test cases in DAG-based tests.

Key changes:

  • Added dagToMermaid function to convert DAG test cases into Mermaid diagram format
  • Integrated diagram logging into the DagTest function to output the visualization during test execution

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

Copy link
Member

@sebastian-quintero sebastian-quintero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice addition. You sure like your 🧜🏻‍♀️ .

@merschformann merschformann merged commit d2e8e46 into develop Aug 15, 2025
29 checks passed
@merschformann merschformann deleted the merschformann/dag-mermaid branch August 15, 2025 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants