mirror of https://github.com/icsharpcode/ILSpy.git
You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
595 lines
28 KiB
595 lines
28 KiB
// Copyright (c) 2026 AlphaSierraPapa for the SharpDevelop Team |
|
// |
|
// Permission is hereby granted, free of charge, to any person obtaining a copy of this |
|
// software and associated documentation files (the "Software"), to deal in the Software |
|
// without restriction, including without limitation the rights to use, copy, modify, merge, |
|
// publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons |
|
// to whom the Software is furnished to do so, subject to the following conditions: |
|
// |
|
// The above copyright notice and this permission notice shall be included in all copies or |
|
// substantial portions of the Software. |
|
// |
|
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, |
|
// INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR |
|
// PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE |
|
// FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR |
|
// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER |
|
// DEALINGS IN THE SOFTWARE. |
|
|
|
using System.Linq; |
|
using System.Threading.Tasks; |
|
|
|
using Avalonia.Headless.NUnit; |
|
using Avalonia.Media; |
|
using Avalonia.VisualTree; |
|
|
|
using AwesomeAssertions; |
|
|
|
using Dock.Avalonia.Controls; |
|
|
|
using ILSpy.AppEnv; |
|
using ILSpy.Docking; |
|
using ILSpy.TreeNodes; |
|
using ILSpy.ViewModels; |
|
using ILSpy.Views; |
|
|
|
using NUnit.Framework; |
|
|
|
namespace ICSharpCode.ILSpy.Tests; |
|
|
|
[TestFixture] |
|
public class PreviewTabPromotionTests |
|
{ |
|
[AvaloniaTest] |
|
public void MainTab_Starts_In_Preview_State() |
|
{ |
|
// The persistent MainTab is the preview slot; tree-node clicks should replace its |
|
// Content in place until the user explicitly freezes it. |
|
var window = AppComposition.Current.GetExport<MainWindow>(); |
|
window.Show(); |
|
var vm = (MainWindowViewModel)window.DataContext!; |
|
TestCapture.Step("booted"); |
|
var mainTab = ((ILSpyDockFactory)vm.DockWorkspace.Factory).MainTab!; |
|
mainTab.IsPreview.Should().BeTrue( |
|
"the freshly-created MainTab is the preview slot until the user freezes it"); |
|
} |
|
|
|
[AvaloniaTest] |
|
public async Task Carve_Out_Tabs_Are_Born_Frozen() |
|
{ |
|
// Open-in-new-tab tabs are explicit user intent — they should never be replaced |
|
// by subsequent tree-node selections, so they're born frozen (IsPreview=false). |
|
var (_, vm) = await TestHarness.BootAsync(3); |
|
|
|
var typeNode = vm.AssemblyTreeModel.FindNode<TypeTreeNode>( |
|
"System.Linq", "System.Linq", "System.Linq.Enumerable"); |
|
vm.DockWorkspace.OpenNodeInNewTab(typeNode); |
|
TestCapture.Step("carve-out-tab-opened"); |
|
|
|
var carveOut = vm.DockWorkspace.Documents!.VisibleDockables! |
|
.OfType<ContentTabPage>() |
|
.Last(t => t.SourceNode == typeNode); |
|
carveOut.IsPreview.Should().BeFalse( |
|
"carve-out tabs are explicit user intent and must survive tree-selection changes"); |
|
} |
|
|
|
[AvaloniaTest] |
|
public async Task FreezeCurrentTab_Flips_IsPreview_Without_Spawning_A_New_Tab() |
|
{ |
|
// New semantics: Freeze only flips IsPreview=false on the current MainTab. No new |
|
// preview tab spawns at freeze time. A fresh preview tab opens lazily later, when a |
|
// tree-selection change finds the active tab frozen — see |
|
// Selecting_A_Different_Node_After_Freeze_Opens_A_New_Preview_Tab below. |
|
var (_, vm) = await TestHarness.BootAsync(3); |
|
|
|
var factory = (ILSpyDockFactory)vm.DockWorkspace.Factory; |
|
var previousMainTab = factory.MainTab!; |
|
previousMainTab.IsPreview.Should().BeTrue("baseline"); |
|
|
|
// Load content into MainTab so the freeze has something meaningful to keep. |
|
var typeNode = vm.AssemblyTreeModel.FindNode<TypeTreeNode>( |
|
"System.Linq", "System.Linq", "System.Linq.Enumerable"); |
|
vm.AssemblyTreeModel.SelectNode(typeNode); |
|
vm.DockWorkspace.SettleSelection(); |
|
TestCapture.Step("enumerable-selected"); |
|
ReferenceEquals(previousMainTab.SourceNode, typeNode).Should().BeTrue( |
|
"baseline: tree selection populated MainTab with the chosen node"); |
|
|
|
var tabCountBefore = vm.DockWorkspace.Documents!.VisibleDockables!.Count; |
|
|
|
// Freeze. |
|
vm.DockWorkspace.FreezeCurrentTab(); |
|
TestCapture.Step("tab-frozen"); |
|
|
|
// Tab is now frozen, content preserved. |
|
previousMainTab.IsPreview.Should().BeFalse( |
|
"after freeze, the previously-preview MainTab becomes a regular frozen tab"); |
|
ReferenceEquals(previousMainTab.SourceNode, typeNode).Should().BeTrue( |
|
"freeze must not throw away the tab's content"); |
|
|
|
// factory.MainTab still points at the same (now-frozen) tab — no rotation. |
|
factory.MainTab.Should().BeSameAs(previousMainTab, |
|
"freeze alone must not rotate factory.MainTab — the frozen tab keeps the slot until a selection change spawns a new preview"); |
|
|
|
// And critically: no new tab spawned. |
|
vm.DockWorkspace.Documents!.VisibleDockables!.Count.Should().Be(tabCountBefore, |
|
"freezing alone must not open a new tab; new tabs open lazily on the next selection change"); |
|
} |
|
|
|
[AvaloniaTest] |
|
public async Task Selecting_A_Different_Node_After_Freeze_Opens_A_New_Preview_Tab() |
|
{ |
|
// Freeze holds the current tab's content. Selecting a different tree node while |
|
// the (now-frozen) tab is active must spawn a fresh preview tab beside it and |
|
// route the new content there — never overwrite the frozen tab. |
|
var (_, vm) = await TestHarness.BootAsync(3); |
|
|
|
var factory = (ILSpyDockFactory)vm.DockWorkspace.Factory; |
|
|
|
// Phase 1: load type A into MainTab. |
|
var typeA = vm.AssemblyTreeModel.FindNode<TypeTreeNode>( |
|
"System.Linq", "System.Linq", "System.Linq.Enumerable"); |
|
vm.AssemblyTreeModel.SelectNode(typeA); |
|
vm.DockWorkspace.SettleSelection(); |
|
TestCapture.Step("type-a-selected"); |
|
var frozenTab = factory.MainTab!; |
|
var frozenContent = frozenTab.Content; |
|
|
|
// Phase 2: freeze. factory.MainTab still points at the same (now-frozen) tab — |
|
// no spawn yet (covered by FreezeCurrentTab_Flips_IsPreview_Without_Spawning_A_New_Tab). |
|
var tabCountBeforeSelection = vm.DockWorkspace.Documents!.VisibleDockables!.Count; |
|
vm.DockWorkspace.FreezeCurrentTab(); |
|
TestCapture.Step("tab-frozen"); |
|
factory.MainTab.Should().BeSameAs(frozenTab, "freeze alone keeps the slot"); |
|
|
|
// Phase 3: select a different type — NOW a new preview tab should spawn AND |
|
// receive the new content. |
|
var typeB = vm.AssemblyTreeModel.FindNode<TypeTreeNode>( |
|
"System.Private.Uri", "System", "System.Uri"); |
|
vm.AssemblyTreeModel.SelectNode(typeB); |
|
vm.DockWorkspace.SettleSelection(); |
|
TestCapture.Step("type-b-opens-new-preview"); |
|
|
|
// New tab spawned beside the frozen one. |
|
vm.DockWorkspace.Documents!.VisibleDockables!.Count.Should().Be(tabCountBeforeSelection + 1, |
|
"selecting a different node while the active tab is frozen must spawn a new preview tab"); |
|
|
|
var newMainTab = factory.MainTab!; |
|
newMainTab.Should().NotBeSameAs(frozenTab, |
|
"factory.MainTab must rotate to the freshly-spawned preview tab once a selection change forces it"); |
|
newMainTab.IsPreview.Should().BeTrue( |
|
"the freshly-spawned tab is itself a preview tab (the user can freeze it next)"); |
|
ReferenceEquals(newMainTab.SourceNode, typeB).Should().BeTrue( |
|
"new tree selection must land in the freshly-spawned preview tab"); |
|
ReferenceEquals(frozenTab.SourceNode, typeA).Should().BeTrue( |
|
"the frozen tab must keep type A — not be overwritten by the new selection"); |
|
frozenTab.Content.Should().BeSameAs(frozenContent, |
|
"frozen tab's Content reference must survive subsequent tree selections"); |
|
} |
|
|
|
[AvaloniaTest] |
|
public async Task Preview_MainTab_Header_Renders_Italic() |
|
{ |
|
// Visual contract: the DocumentTabStripItem for the preview MainTab binds its |
|
// FontStyle to ContentTabPage.IsPreview via BoolToFontStyleConverter.Italic. |
|
// Frozen tabs and tool-pane tabs fall through to FontStyle.Normal. |
|
var (window, vm) = await TestHarness.BootAsync(); |
|
|
|
// Wait for the tab strip to realise its items. |
|
await Waiters.WaitForAsync(() => window.GetVisualDescendants().OfType<DocumentTabStripItem>().Any(), |
|
System.TimeSpan.FromSeconds(10)); |
|
|
|
var factory = (ILSpyDockFactory)vm.DockWorkspace.Factory; |
|
var mainTabItem = window.GetVisualDescendants().OfType<DocumentTabStripItem>() |
|
.Single(item => ReferenceEquals(item.DataContext, factory.MainTab)); |
|
TestCapture.Step("before-italic-header-check"); |
|
|
|
mainTabItem.FontStyle.Should().Be(FontStyle.Italic, |
|
"the App.axaml Style for DocumentTabStripItem must apply, italicising the tab title via FontStyle inheritance"); |
|
} |
|
|
|
[AvaloniaTest] |
|
public async Task Preview_MainTab_Header_Has_Left_Edge_Accent_Stripe() |
|
{ |
|
// Border companion to the italic check: the preview MainTab carries a non-null |
|
// BorderBrush and a 3px left-only BorderThickness via the accent-stripe converters. |
|
var window = AppComposition.Current.GetExport<MainWindow>(); |
|
window.Show(); |
|
await ((MainWindowViewModel)window.DataContext!).AssemblyTreeModel.WaitForAssembliesAsync(minimumCount: 1); |
|
TestCapture.Step("booted"); |
|
|
|
await Waiters.WaitForAsync(() => window.GetVisualDescendants().OfType<DocumentTabStripItem>().Any(), |
|
System.TimeSpan.FromSeconds(10)); |
|
|
|
var factory = (ILSpyDockFactory)((MainWindowViewModel)window.DataContext!).DockWorkspace.Factory; |
|
var mainTabItem = window.GetVisualDescendants().OfType<DocumentTabStripItem>() |
|
.Single(item => ReferenceEquals(item.DataContext, factory.MainTab)); |
|
TestCapture.Step("before-accent-stripe-check"); |
|
|
|
((object?)mainTabItem.BorderBrush).Should().NotBeNull( |
|
"the preview MainTab must carry the accent BorderBrush"); |
|
mainTabItem.BorderThickness.Should().Be(new global::Avalonia.Thickness(3, 0, 0, 0), |
|
"the preview MainTab must carry the left-only 3px accent stripe"); |
|
// The accent is purple (#9B59B6) -- deliberately not the selection/toolbar blue, so the |
|
// One stays distinct from a blue-highlighted selected tab. |
|
(mainTabItem.BorderBrush as global::Avalonia.Media.ISolidColorBrush)!.Color |
|
.Should().Be(global::Avalonia.Media.Color.FromRgb(0x9B, 0x59, 0xB6), |
|
"the preview accent stripe must be purple, not blue"); |
|
} |
|
|
|
[AvaloniaTest] |
|
public async Task Freeze_Button_Fits_Inside_The_Tab_When_Close_Button_Is_Visible() |
|
{ |
|
// The single-tab scenario hides the close button (UpdateLastDocumentCanClose sets |
|
// CanClose=false), so the freeze has lots of room. The bug shows up only when the |
|
// close button is visible and competes for horizontal space. Open a carve-out |
|
// tab so both MainTab and carve-out exist; close button becomes visible; verify |
|
// the freeze's right edge stays inside the tab's right edge. |
|
|
|
var (window, vm) = await TestHarness.BootAsync(3); |
|
|
|
// Open a carve-out so close buttons become visible on both tabs. |
|
var typeNode = vm.AssemblyTreeModel.FindNode<TypeTreeNode>( |
|
"System.Linq", "System.Linq", "System.Linq.Enumerable"); |
|
vm.DockWorkspace.OpenNodeInNewTab(typeNode); |
|
TestCapture.Step("carve-out-tab-opened"); |
|
|
|
await Waiters.WaitForAsync(() => window.GetVisualDescendants().OfType<DocumentTabStripItem>().Count() >= 2, |
|
System.TimeSpan.FromSeconds(10)); |
|
global::Avalonia.Threading.Dispatcher.UIThread.RunJobs(); |
|
|
|
var factory = (ILSpyDockFactory)vm.DockWorkspace.Factory; |
|
var mainTabItem = window.GetVisualDescendants().OfType<DocumentTabStripItem>() |
|
.Single(item => ReferenceEquals(item.DataContext, factory.MainTab)); |
|
|
|
// Force layout so Bounds are up to date. |
|
mainTabItem.Measure(global::Avalonia.Size.Infinity); |
|
mainTabItem.Arrange(new global::Avalonia.Rect(mainTabItem.DesiredSize)); |
|
global::Avalonia.Threading.Dispatcher.UIThread.RunJobs(); |
|
|
|
var freezeButton = mainTabItem.GetVisualDescendants() |
|
.OfType<global::Avalonia.Controls.Button>() |
|
.Single(b => (b.Tag as string) == "PreviewTabFreezeButton"); |
|
|
|
// Walk up the visual tree summing each ancestor's bounds origin until we reach |
|
// mainTabItem — gives the freeze's top-left in tab coordinates without depending on |
|
// TranslatePoint (which lives on different namespaces across Avalonia versions). |
|
static global::Avalonia.Point OriginRelativeTo(global::Avalonia.Visual node, global::Avalonia.Visual ancestor) |
|
{ |
|
double x = 0, y = 0; |
|
var current = node; |
|
while (current != null && !ReferenceEquals(current, ancestor)) |
|
{ |
|
x += current.Bounds.X; |
|
y += current.Bounds.Y; |
|
current = current.GetVisualParent(); |
|
} |
|
return new global::Avalonia.Point(x, y); |
|
} |
|
var freezeTopLeft = OriginRelativeTo(freezeButton, mainTabItem); |
|
var freezeRight = freezeTopLeft.X + freezeButton.Bounds.Width; |
|
var tabRight = mainTabItem.Bounds.Width; |
|
|
|
// Also find the close button so we can include it in the failure message for context. |
|
var closeButton = mainTabItem.GetVisualDescendants() |
|
.OfType<global::Avalonia.Controls.Button>() |
|
.Where(b => (b.Tag as string) != "PreviewTabFreezeButton") |
|
.Select(b => new { b, p = OriginRelativeTo(b, mainTabItem) }) |
|
.OrderByDescending(x => x.p.X) |
|
.FirstOrDefault(); |
|
var closeInfo = closeButton == null ? "no close button found" |
|
: $"close at x={closeButton.p.X:0}-{closeButton.p.X + closeButton.b.Bounds.Width:0}"; |
|
|
|
TestContext.WriteLine($"tab width={tabRight:0}; freeze at x={freezeTopLeft.X:0}-{freezeRight:0}; {closeInfo}"); |
|
|
|
freezeRight.Should().BeLessThanOrEqualTo(tabRight, |
|
"freeze button's right edge must fit inside the tab's right edge (cut-off bug)"); |
|
if (closeButton != null && closeButton.b.Bounds.Width > 0) |
|
{ |
|
freezeRight.Should().BeLessThanOrEqualTo(closeButton.p.X + 0.5, |
|
"freeze button must sit to the LEFT of the close button, not overlap it"); |
|
} |
|
} |
|
|
|
[AvaloniaTest] |
|
public async Task Freeze_Button_Fits_Inside_The_Tab_Even_With_A_Long_Title() |
|
{ |
|
// Production stress case: tabs with long titles can exceed the tab strip's available |
|
// width if the title isn't capped. The freeze button must still fit inside the tab's |
|
// right edge regardless of title length. |
|
|
|
var (window, vm) = await TestHarness.BootAsync(3); |
|
|
|
// Open carve-out so close button is visible. |
|
var typeNode = vm.AssemblyTreeModel.FindNode<TypeTreeNode>( |
|
"System.Linq", "System.Linq", "System.Linq.Enumerable"); |
|
vm.DockWorkspace.OpenNodeInNewTab(typeNode); |
|
TestCapture.Step("carve-out-tab-opened"); |
|
|
|
await Waiters.WaitForAsync(() => window.GetVisualDescendants().OfType<DocumentTabStripItem>().Count() >= 2, |
|
System.TimeSpan.FromSeconds(10)); |
|
|
|
var factory = (ILSpyDockFactory)vm.DockWorkspace.Factory; |
|
var mainTab = factory.MainTab!; |
|
// Force an absurdly long title to simulate a long type signature. |
|
mainTab.Title = "This.Is.An.Extremely.Long.Tab.Title.That.Would.Normally.Overflow<Foo, Bar, Baz>"; |
|
|
|
global::Avalonia.Threading.Dispatcher.UIThread.RunJobs(); |
|
TestCapture.Step("long-title-applied"); |
|
var mainTabItem = window.GetVisualDescendants().OfType<DocumentTabStripItem>() |
|
.Single(item => ReferenceEquals(item.DataContext, mainTab)); |
|
// Constrain the tab to 200px (mimics production where the document strip shares |
|
// width among multiple tabs). |
|
mainTabItem.MaxWidth = 200; |
|
mainTabItem.Measure(new global::Avalonia.Size(200, double.PositiveInfinity)); |
|
mainTabItem.Arrange(new global::Avalonia.Rect(0, 0, 200, mainTabItem.DesiredSize.Height)); |
|
global::Avalonia.Threading.Dispatcher.UIThread.RunJobs(); |
|
|
|
var freezeButton = mainTabItem.GetVisualDescendants() |
|
.OfType<global::Avalonia.Controls.Button>() |
|
.Single(b => (b.Tag as string) == "PreviewTabFreezeButton"); |
|
|
|
static global::Avalonia.Point OriginRelativeTo(global::Avalonia.Visual node, global::Avalonia.Visual ancestor) |
|
{ |
|
double x = 0, y = 0; |
|
var current = node; |
|
while (current != null && !ReferenceEquals(current, ancestor)) |
|
{ |
|
x += current.Bounds.X; |
|
y += current.Bounds.Y; |
|
current = current.GetVisualParent(); |
|
} |
|
return new global::Avalonia.Point(x, y); |
|
} |
|
var freezeTopLeft = OriginRelativeTo(freezeButton, mainTabItem); |
|
var freezeRight = freezeTopLeft.X + freezeButton.Bounds.Width; |
|
var tabRight = mainTabItem.Bounds.Width; |
|
TestContext.WriteLine($"long-title scenario: tab={tabRight:0}, freeze x={freezeTopLeft.X:0}-{freezeRight:0}"); |
|
|
|
freezeRight.Should().BeLessThanOrEqualTo(tabRight, |
|
$"freeze must fit inside tab width even with long titles (tab={tabRight:0}, freezeRight={freezeRight:0})"); |
|
|
|
// Lock the rendering fixes in: |
|
// 1. The title StackPanel must clip — otherwise a long title TextBlock measures |
|
// at its full unconstrained width and paints over the freeze and close buttons. |
|
var titleStack = mainTabItem.GetVisualDescendants().OfType<global::Avalonia.Controls.StackPanel>().First(); |
|
titleStack.ClipToBounds.Should().BeTrue( |
|
"the title StackPanel must clip its content so a long title TextBlock doesn't render over the freeze button"); |
|
// 2. The freeze Button itself must NOT clip — Avalonia Buttons default to ClipToBounds=true |
|
// which cuts the right edge of Segoe Fluent Icon glyphs (whose visual extent |
|
// exceeds the reported advance width). |
|
freezeButton.ClipToBounds.Should().BeFalse( |
|
"freeze button must not clip so the glyph's visual extent (often wider than the font advance width) survives"); |
|
} |
|
|
|
[AvaloniaTest] |
|
public async Task Inline_Freeze_Button_Appears_On_Preview_Tab_And_Freezes_When_Clicked() |
|
{ |
|
var (window, vm) = await TestHarness.BootAsync(); |
|
|
|
await Waiters.WaitForAsync(() => window.GetVisualDescendants().OfType<DocumentTabStripItem>().Any(), |
|
System.TimeSpan.FromSeconds(10)); |
|
|
|
var factory = (ILSpyDockFactory)vm.DockWorkspace.Factory; |
|
var mainTabItem = window.GetVisualDescendants().OfType<DocumentTabStripItem>() |
|
.Single(item => ReferenceEquals(item.DataContext, factory.MainTab)); |
|
|
|
// Freeze button is injected via PreviewTabFreezeButtonBehavior — find it by Tag. |
|
await Waiters.WaitForAsync(() => mainTabItem.GetVisualDescendants() |
|
.OfType<global::Avalonia.Controls.Button>() |
|
.Any(b => (b.Tag as string) == "PreviewTabFreezeButton"), |
|
System.TimeSpan.FromSeconds(5)); |
|
|
|
var freezeButton = mainTabItem.GetVisualDescendants() |
|
.OfType<global::Avalonia.Controls.Button>() |
|
.Single(b => (b.Tag as string) == "PreviewTabFreezeButton"); |
|
freezeButton.IsVisible.Should().BeTrue("freeze button must show while the tab is preview"); |
|
|
|
var previousMainTab = factory.MainTab!; |
|
var tabCountBefore = vm.DockWorkspace.Documents!.VisibleDockables!.Count; |
|
|
|
// Simulate a user click on the freeze button. |
|
freezeButton.RaiseEvent(new global::Avalonia.Interactivity.RoutedEventArgs( |
|
global::Avalonia.Controls.Button.ClickEvent)); |
|
TestCapture.Step("freeze-button-clicked"); |
|
|
|
// New semantics: clicking freeze flips IsPreview, doesn't spawn a new tab. |
|
previousMainTab.IsPreview.Should().BeFalse( |
|
"clicking the freeze button must flip IsPreview=false on the current MainTab"); |
|
factory.MainTab.Should().BeSameAs(previousMainTab, |
|
"freeze alone must not rotate factory.MainTab; the new preview tab opens lazily on the next tree selection"); |
|
vm.DockWorkspace.Documents!.VisibleDockables!.Count.Should().Be(tabCountBefore, |
|
"freeze alone must not change tab count"); |
|
} |
|
|
|
[AvaloniaTest] |
|
public async Task Freeze_Entry_Appears_In_Tab_Context_Menu_Only_When_Preview() |
|
{ |
|
// The right-click context-menu Freeze entry must be visible on the preview MainTab |
|
// and hidden (its IsVisible flips) on frozen carve-out tabs. |
|
var (window, vm) = await TestHarness.BootAsync(3); |
|
var typeNode = vm.AssemblyTreeModel.FindNode<TypeTreeNode>( |
|
"System.Linq", "System.Linq", "System.Linq.Enumerable"); |
|
vm.DockWorkspace.OpenNodeInNewTab(typeNode); |
|
TestCapture.Step("carve-out-tab-opened"); |
|
|
|
await Waiters.WaitForAsync(() => window.GetVisualDescendants().OfType<DocumentTabStripItem>().Count() >= 2, |
|
System.TimeSpan.FromSeconds(10)); |
|
|
|
var factory = (ILSpyDockFactory)vm.DockWorkspace.Factory; |
|
var mainTabItem = window.GetVisualDescendants().OfType<DocumentTabStripItem>() |
|
.Single(item => ReferenceEquals(item.DataContext, factory.MainTab)); |
|
var carveOutItem = window.GetVisualDescendants().OfType<DocumentTabStripItem>() |
|
.Single(item => item.DataContext is ContentTabPage t && t.SourceNode == typeNode); |
|
|
|
var mainFreezeEntry = mainTabItem.DocumentContextMenu!.Items.OfType<global::Avalonia.Controls.MenuItem>().Single(); |
|
mainFreezeEntry.Header.Should().Be("Freeze tab"); |
|
mainFreezeEntry.IsVisible.Should().BeTrue("MainTab is preview — Freeze tab entry must be visible"); |
|
|
|
var carveFreezeEntry = carveOutItem.DocumentContextMenu!.Items.OfType<global::Avalonia.Controls.MenuItem>().Single(); |
|
carveFreezeEntry.IsVisible.Should().BeFalse( |
|
"carve-out tabs are already frozen — the Freeze entry must hide"); |
|
} |
|
|
|
[AvaloniaTest] |
|
public async Task Frozen_Carve_Out_Tab_Header_Renders_Upright() |
|
{ |
|
// Regression guard for the bug where every DocumentTabStripItem got italicised |
|
// because the App.axaml setter used a static `FontStyle="Italic"` value instead |
|
// of the binding to IsPreview. A carve-out tab (IsPreview=false) must render |
|
// Normal, even when sitting next to a preview MainTab. |
|
var (window, vm) = await TestHarness.BootAsync(3); |
|
|
|
// Open a carve-out tab via the production code path. |
|
var typeNode = vm.AssemblyTreeModel.FindNode<TypeTreeNode>( |
|
"System.Linq", "System.Linq", "System.Linq.Enumerable"); |
|
vm.DockWorkspace.OpenNodeInNewTab(typeNode); |
|
TestCapture.Step("carve-out-tab-opened"); |
|
|
|
await Waiters.WaitForAsync(() => window.GetVisualDescendants().OfType<DocumentTabStripItem>().Any(), |
|
System.TimeSpan.FromSeconds(10)); |
|
|
|
var carveOutModel = vm.DockWorkspace.Documents!.VisibleDockables! |
|
.OfType<ContentTabPage>() |
|
.Last(t => t.SourceNode == typeNode); |
|
carveOutModel.IsPreview.Should().BeFalse("baseline: carve-out tab is frozen"); |
|
|
|
var carveOutItem = window.GetVisualDescendants().OfType<DocumentTabStripItem>() |
|
.Single(item => ReferenceEquals(item.DataContext, carveOutModel)); |
|
carveOutItem.FontStyle.Should().Be(FontStyle.Normal, |
|
"a frozen (IsPreview=false) document tab must render with upright FontStyle"); |
|
} |
|
|
|
[AvaloniaTest] |
|
public void FreezeCurrentTab_Is_Noop_When_MainTab_Already_Frozen() |
|
{ |
|
// Idempotency: a second freeze against an already-frozen MainTab is a no-op. The new |
|
// "freeze = flip-only, spawn-lazy" semantics make this simpler than the old version: |
|
// FreezeCurrentMainTab() returns null when MainTab.IsPreview is already false, and |
|
// FreezeCurrentTab early-returns. |
|
var window = AppComposition.Current.GetExport<MainWindow>(); |
|
window.Show(); |
|
var vm = (MainWindowViewModel)window.DataContext!; |
|
var factory = (ILSpyDockFactory)vm.DockWorkspace.Factory; |
|
TestCapture.Step("booted"); |
|
|
|
// Manually flip the current MainTab to frozen, simulating the post-freeze state. |
|
factory.MainTab!.IsPreview = false; |
|
var before = factory.MainTab; |
|
|
|
vm.DockWorkspace.FreezeCurrentTab(); |
|
TestCapture.Step("freeze-noop"); |
|
|
|
factory.MainTab.Should().BeSameAs(before, |
|
"with no preview MainTab to flip, FreezeCurrentTab must leave the factory state untouched"); |
|
factory.MainTab.IsPreview.Should().BeFalse( |
|
"the already-frozen tab stays frozen"); |
|
} |
|
|
|
[AvaloniaTest] |
|
public async Task File_Menu_Separates_MenuCategory_Groups() |
|
{ |
|
// Regression: the MainMenu builder must emit separators at MenuCategory boundaries |
|
// (Open / Save / Remove / Exit groups in the File menu, ...). With the menu now living |
|
// as a NativeMenu, separator rendering is the platform's job; this test pins only |
|
// the structural claim that separators are inserted between groups. |
|
var (window, _) = await TestHarness.BootAsync(); |
|
|
|
var nativeMenu = global::Avalonia.Controls.NativeMenu.GetMenu(window) |
|
?? throw new System.InvalidOperationException("MainMenu.Attach should have set NativeMenu on the window"); |
|
var fileMenu = nativeMenu.Items.OfType<global::Avalonia.Controls.NativeMenuItem>() |
|
.Single(m => string.Equals(m.Header, ICSharpCode.ILSpy.Properties.Resources._File, System.StringComparison.Ordinal)); |
|
|
|
var separators = fileMenu.Menu!.Items.OfType<global::Avalonia.Controls.NativeMenuItemSeparator>().ToList(); |
|
separators.Should().HaveCountGreaterThanOrEqualTo(2, |
|
"File menu's MenuCategory groups (Open / Save / Remove / Exit) must produce at least two separators"); |
|
} |
|
|
|
[AvaloniaTest] |
|
public async Task Freeze_Button_Inherits_The_Close_Button_Theme() |
|
{ |
|
// Visual parity: the freeze button should look like the close button — same size, same |
|
// hover background. Both are plain Avalonia.Controls.Button instances; the close |
|
// button gets its visual identity from a ControlTheme applied by Dock's tab |
|
// template. Freeze should copy that Theme rather than carry its own custom style. |
|
var (window, vm) = await TestHarness.BootAsync(3); |
|
|
|
// Open a carve-out so the close button is visible alongside the freeze (single-tab |
|
// scenario hides the close button). |
|
var typeNode = vm.AssemblyTreeModel.FindNode<TypeTreeNode>( |
|
"System.Linq", "System.Linq", "System.Linq.Enumerable"); |
|
vm.DockWorkspace.OpenNodeInNewTab(typeNode); |
|
TestCapture.Step("carve-out-tab-opened"); |
|
|
|
await Waiters.WaitForAsync(() => window.GetVisualDescendants().OfType<DocumentTabStripItem>().Count() >= 2, |
|
System.TimeSpan.FromSeconds(10)); |
|
global::Avalonia.Threading.Dispatcher.UIThread.RunJobs(); |
|
|
|
var factory = (ILSpyDockFactory)vm.DockWorkspace.Factory; |
|
var mainTabItem = window.GetVisualDescendants().OfType<DocumentTabStripItem>() |
|
.Single(item => ReferenceEquals(item.DataContext, factory.MainTab)); |
|
|
|
var freezeButton = mainTabItem.GetVisualDescendants() |
|
.OfType<global::Avalonia.Controls.Button>() |
|
.Single(b => (b.Tag as string) == "PreviewTabFreezeButton"); |
|
var closeButton = mainTabItem.GetVisualDescendants() |
|
.OfType<global::Avalonia.Controls.Button>() |
|
.FirstOrDefault(b => (b.Tag as string) != "PreviewTabFreezeButton"); |
|
closeButton.Should().NotBeNull("baseline: close button exists as a sibling of the freeze"); |
|
|
|
freezeButton.Theme.Should().BeSameAs(closeButton!.Theme, |
|
"freeze button must use the same ControlTheme as the close button so size + hover bg match"); |
|
freezeButton.Classes.Should().NotContain("preview-freeze", |
|
"after the Theme-copy refactor the custom class-based styling is unused; the Theme drives all visuals"); |
|
} |
|
|
|
[AvaloniaTest] |
|
public async Task Tree_Selection_While_Frozen_Tab_Active_Opens_A_New_Preview_Tab() |
|
{ |
|
// "Frozen" covers: (a) user-frozen tabs (IsPreview=false via FreezeCurrentTab), |
|
// (b) carve-out tabs (born IsPreview=false via OpenNodeInNewTab), and (c) static |
|
// content tabs (Options, About) whose Content is a static viewmodel. Tree-node |
|
// selections while any of those are active must spawn a fresh preview tab and |
|
// route the new content there. This test uses a carve-out as a stand-in for the |
|
// frozen-tab family — the IsWritablePreview check returns null for all three. |
|
var (_, vm) = await TestHarness.BootAsync(3); |
|
|
|
var factory = (ILSpyDockFactory)vm.DockWorkspace.Factory; |
|
|
|
// Open a carve-out (born frozen) and let it become the active dockable. |
|
var typeA = vm.AssemblyTreeModel.FindNode<TypeTreeNode>( |
|
"System.Linq", "System.Linq", "System.Linq.Enumerable"); |
|
vm.DockWorkspace.OpenNodeInNewTab(typeA); |
|
var carveOut = vm.DockWorkspace.Documents!.VisibleDockables! |
|
.OfType<ContentTabPage>() |
|
.Last(t => t.SourceNode == typeA); |
|
factory.SetActiveDockable(carveOut); |
|
TestCapture.Step("carve-out-active"); |
|
carveOut.IsPreview.Should().BeFalse("baseline: carve-out tab is frozen"); |
|
ReferenceEquals(vm.DockWorkspace.Documents.ActiveDockable, carveOut).Should().BeTrue( |
|
"baseline: the carve-out is the active document"); |
|
|
|
var tabCountBefore = vm.DockWorkspace.Documents!.VisibleDockables!.Count; |
|
|
|
// Select a different tree node — the carve-out is frozen, so this must NOT |
|
// overwrite it. |
|
var typeB = vm.AssemblyTreeModel.FindNode<TypeTreeNode>( |
|
"System.Private.Uri", "System", "System.Uri"); |
|
vm.AssemblyTreeModel.SelectNode(typeB); |
|
vm.DockWorkspace.SettleSelection(); |
|
TestCapture.Step("type-b-opens-new-preview"); |
|
|
|
vm.DockWorkspace.Documents!.VisibleDockables!.Count.Should().Be(tabCountBefore + 1, |
|
"selecting a tree node while a frozen tab is active must spawn a new preview tab"); |
|
ReferenceEquals(carveOut.SourceNode, typeA).Should().BeTrue( |
|
"the frozen carve-out tab must keep its original content"); |
|
|
|
var newPreview = factory.MainTab!; |
|
newPreview.Should().NotBeSameAs(carveOut, |
|
"the freshly-spawned preview must be a separate tab from the frozen one"); |
|
newPreview.IsPreview.Should().BeTrue( |
|
"the freshly-spawned tab is itself a preview tab"); |
|
ReferenceEquals(newPreview.SourceNode, typeB).Should().BeTrue( |
|
"the new preview tab must host the just-selected node's content"); |
|
} |
|
}
|
|
|