Skip to content

test(igxGrid): Add test for actionstrip outlet when added post-init#16874

Merged
MayaKirova merged 2 commits intomkirova/fix-16850from
copilot/sub-pr-16873
Feb 6, 2026
Merged

test(igxGrid): Add test for actionstrip outlet when added post-init#16874
MayaKirova merged 2 commits intomkirova/fix-16850from
copilot/sub-pr-16873

Conversation

Copy link
Contributor

Copilot AI commented Feb 6, 2026

Added test coverage for the scenario where an actionstrip is added to a grid after initialization, verifying that the menuOverlaySettings.outlet is correctly set to the grid's outlet.

Changes

  • Test component: GridDynamicActionStripComponent with conditional actionstrip rendering
  • Test case: Validates outlet assignment when actionstrip is added dynamically via @if directive

The test ensures the fix for dynamically added actionstrips (where actionStripComponents.changes subscription sets the outlet) works correctly.

// Component starts without actionstrip
expect(grid.actionStrip).toBeUndefined();

// Actionstrip added dynamically
fixture.componentInstance.showActionStrip = true;
fixture.detectChanges();

// Outlet is properly assigned
expect(actionStrip.menuOverlaySettings.outlet).toBe(grid.outlet);

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: MayaKirova <10397980+MayaKirova@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix igxGrid: Add handling for outlet in case actionstrip is added post init test(igxGrid): Add test for actionstrip outlet when added post-init Feb 6, 2026
Copilot AI requested a review from MayaKirova February 6, 2026 12:21
@MayaKirova MayaKirova marked this pull request as ready for review February 6, 2026 12:24
@MayaKirova MayaKirova merged commit b31a0ba into mkirova/fix-16850 Feb 6, 2026
@MayaKirova MayaKirova deleted the copilot/sub-pr-16873 branch February 6, 2026 12: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.

2 participants