Skip to content

Automated tests for listing failed messages - #3140

Merged
afprtclr merged 4 commits into
masterfrom
feature/779-deleting
Sep 29, 2026
Merged

afprtclr merged 4 commits into
masterfrom
feature/779-deleting

Conversation

@afprtclr

@afprtclr afprtclr commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Tests for listing failed messages.
Implementation for https://github.com/Particular/PlatformInternals/issues/779

Reviewer Checklist

  • Components are broken down into sensible and maintainable sub-components.
  • Styles are scoped to the component using it. If multiple components need to share CSS, then a .css file is created containing the shared CSS and imported into component scoped style sections.
  • Naming is consistent with existing code, and adequately describes the component or function being introduced
  • Only functions utilizing Vue state or lifecycle hooks are named as composables (i.e. starting with 'use');
  • No module-level state is being introduced. If so, request the PR author to move the state to the corresponding Pinia store.

This comment was marked as resolved.

afprtclr and others added 2 commits September 29, 2026 19:55
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

This comment was marked as resolved.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The added test infrastructure and scenarios consistently exercise the intended failed-message behavior without identified blocking issues.

Review effort: Balanced
Findings: None

@afprtclr
afprtclr marked this pull request as ready for review September 29, 2026 19:56
@afprtclr
afprtclr enabled auto-merge (squash) September 29, 2026 20:01
@afprtclr
afprtclr merged commit d5a7565 into master Sep 29, 2026
6 checks passed
@afprtclr
afprtclr deleted the feature/779-deleting branch September 29, 2026 21:40
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