Skip to content

Avalonia test runner: UI revamp#3189

Merged
xen2 merged 1 commit into
stride3d:masterfrom
xen2:feature/test-runner-ux
May 27, 2026
Merged

Avalonia test runner: UI revamp#3189
xen2 merged 1 commit into
stride3d:masterfrom
xen2:feature/test-runner-ux

Conversation

@xen2
Copy link
Copy Markdown
Member

@xen2 xen2 commented May 24, 2026

Avalonia test runner: UI revamp

Rebuilds the interactive test runner (Avalonia window shown when a Stride test exe is launched with STRIDE_TESTS_INTERACTIVE=1).

Highlights

  • Filter textbox (auto-focused) + status chips (All / Passed / Failed / Skipped / Not run) with live counts.
  • Per-row run buttons: test mode (gold compare diff) & interactive run (preview only, doesn't close window)
  • Re-run failed + Run filtered/selected with Ctrl+R / F5 shortcuts.
  • Multi-select in the tree.
  • Save image checkbox (forces save on success) and RenderDoc capture-mode dropdown (Never / On error / Always).
  • Inspect pane: failure message + stack trace + live-streamed stdout/stderr with Warning: / Error: tinting.
  • Summary bar with running pass/fail/skip totals and elapsed time.
  • Keyboard nav: / or Ctrl+F to focus filter, Esc to clear, Up/Down/PageUp/PageDown from the filter to navigate the tree, Enter / Shift+Enter to run / preview.
  • Dark theme + responsive layout (toolbar wraps, main split flips vertical on narrow / portrait viewports).
  • Gold image preview + open folder in explorer

A screenshot is worth a thousand words:

image

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My change requires a change to the documentation.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have built and run the editor to try this change out.

@xen2 xen2 marked this pull request as draft May 24, 2026 14:30
@github-actions
Copy link
Copy Markdown

🤖 Draft PR — automatic CI is skipped to save runner minutes.

  • /ci main — what would auto-run if this PR weren't draft
  • /ci allmain + screenshot regression (/ci e2e)
  • /ci help — full list, per-platform and per-suite

Marking ready-for-review re-enables full CI automatically.

Icon="/Assets/avalonia-logo.ico"
Title="XUnit Runner (Stride)">
Title="XUnit Runner (Stride)"
Background="#1E1E1E">
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This should use a color from the theme so that if the users prefers a light theme it picks the corresponding background color. Or no color as I think the theme as one by default.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I remember setting it manually because default was ugly
image
I thought it was the default but it was simply because it didn't properly apply the theme one.
Adjusting now

Copy link
Copy Markdown
Member Author

@xen2 xen2 May 26, 2026

Choose a reason for hiding this comment

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

Fixed in latest version
Somehow I had to add it to App.xaml; {DynamicResource SolidBackgroundFillColorBase} didn't work as is.
I wonder if it was really worth the 10+ new lines vs a hardcoded color for this specific dev tool though?

xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
x:Class="xunit.runner.stride.App"
RequestedThemeVariant="Light">
RequestedThemeVariant="Dark">
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should probably be Default instead so that it follows the system theme.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I will keep it as is, dark only for now so that I can focus other things & simplify maintenance for now.
(I chose dark to save my poor eyes during those long coding nights ^^)

PR welcome of course if somebody think it's important.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

That makes sense.

@xen2 xen2 force-pushed the feature/test-runner-ux branch from 2adbbd3 to 2bb694c Compare May 25, 2026 00:31
@xen2 xen2 marked this pull request as ready for review May 26, 2026 03:24
@xen2 xen2 force-pushed the feature/test-runner-ux branch from 2bb694c to 784321f Compare May 26, 2026 09:52
@bnwa9723
Copy link
Copy Markdown

bnwa9723 commented May 26, 2026

Go Xen2 !!!

@xen2 xen2 force-pushed the feature/test-runner-ux branch from 784321f to 15369b9 Compare May 26, 2026 10:45
@xen2 xen2 merged commit ada2a41 into stride3d:master May 27, 2026
1 check passed
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