Skip to content

test: Enhance tests for flame_markdown to ensure nested styles are propagated#3439

Merged
luanpotter merged 1 commit intomainfrom
luan.text-test
Jan 9, 2025
Merged

test: Enhance tests for flame_markdown to ensure nested styles are propagated#3439
luanpotter merged 1 commit intomainfrom
luan.text-test

Conversation

@luanpotter
Copy link
Copy Markdown
Member

Description

Enhance tests for flame_markdown to ensure nested styles are propagated.

Specifically adds a second section to each test to assert the element tree (after having asserted the node tree).

And then uses that to add a nested inline style test making sure the "CSS" is propagated and merged correctly.

Checklist

  • I have followed the Contributor Guide when preparing my PR.
  • I have updated/added tests for ALL new/updated/fixed functionality.
  • I have updated/added relevant documentation in docs and added dartdoc comments with ///.
  • I have updated/added relevant examples in examples or docs.

Breaking Change?

  • Yes, this PR is a breaking change.
  • No, this PR is not a breaking change.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 9, 2025

Benchmark Results

Package flame:

Benchmarks Current Branch
[luan.text-test]
Base Branch
[main]
Diff
Render Components Benchmark 1166.489 μs 1167.369 μs 🟢 -0.075 %
Updating Components Benchmark 268472.750 μs 247186.111 μs 🔴 +8.612 %

Benchmarks provided with 💙 by Dart Benchmark Action.

@luanpotter luanpotter marked this pull request as ready for review January 9, 2025 21:08
@luanpotter luanpotter merged commit 6ea5389 into main Jan 9, 2025
@luanpotter luanpotter deleted the luan.text-test branch January 9, 2025 21:09
nickf2k pushed a commit to nickf2k/flame that referenced this pull request Nov 16, 2025
…opagated (flame-engine#3439)

Enhance tests for flame_markdown to ensure nested styles are propagated.

Specifically adds a second section to each test to assert the element
tree (after having asserted the node tree).

And then uses that to add a nested inline style test making sure the
"CSS" is propagated and merged correctly.
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.

2 participants