Skip to content

Add "open group" step to copilot variant tests; revert for flaky adaptive card test#5772

Merged
OEvgeny merged 3 commits intofeat/default-group-state-style-optionfrom
copilot/sub-pr-5771
Mar 6, 2026
Merged

Add "open group" step to copilot variant tests; revert for flaky adaptive card test#5772
OEvgeny merged 3 commits intofeat/default-group-state-style-optionfrom
copilot/sub-pr-5771

Conversation

Copy link
Contributor

Copilot AI commented Mar 6, 2026

With collapsibleGroupingDefaultOpen: false now set for the copilot variant, activity groups render folded by default. Tests that snapshot groups in their open state need to explicitly open the group first.

Changes

  • part-grouping/folded.skip.html: Click .collapsible-grouping__toggle after numActivitiesShown(3) and before the first snapshot, then the existing fold step closes it again.
  • part-grouping/index.html: Same click, guarded with if (variant === 'copilot') so non-copilot variants are unaffected.
  • attachment/adaptiveCard/custom.skip.html: Reverted a similar change added in the same session — the custom.html test failure is unrelated/flaky, so the copilot group-opening step was removed to avoid masking it.

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

Co-authored-by: OEvgeny <2841858+OEvgeny@users.noreply.github.com>
Copilot AI changed the title [WIP] Add style option to configure default group open state Add group opening step to copilot variant test files Mar 6, 2026
Co-authored-by: OEvgeny <2841858+OEvgeny@users.noreply.github.com>
Copilot AI changed the title Add group opening step to copilot variant test files Add "open group" step to copilot variant tests; revert for flaky adaptive card test Mar 6, 2026
@OEvgeny OEvgeny marked this pull request as ready for review March 6, 2026 23:55
@OEvgeny OEvgeny merged commit cc18d61 into feat/default-group-state-style-option Mar 6, 2026
1 check passed
@OEvgeny OEvgeny deleted the copilot/sub-pr-5771 branch March 6, 2026 23:55
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