Files
34aeee9e52 [MM-69691] Fix /secure-connection status ordering and malformed table header (#37347)
* [MM-69691] Fix /secure-connection status table sorting and header

Sort deleted connections to the bottom (active first, ordered by
CreateAt within each group) and remove the spurious empty cell from
the Markdown table header separator so it has 9 cells matching the
header and data rows.

Co-authored-by: mattermost-code <[email protected]>

* [MM-69691] Add test for /secure-connection status output

Verifies deleted connections sort to the bottom (active first, ordered
by CreateAt within each group) and that the Markdown table header
separator has exactly nine cells matching the columns.

Co-authored-by: mattermost-code <[email protected]>

* [MM-69691] Strengthen /secure-connection status test assertions

Assert the separator row equals the exact nine-cell string and has
nine alignment cells, verify data rows have nine cells, and cover
CreateAt ordering (overriding alphabetical) plus stable ordering for
equal CreateAt within a group.

Co-authored-by: mattermost-code <[email protected]>

* [MM-69691] Use strings.SplitSeq in status test to satisfy linter

Co-authored-by: mattermost-code <[email protected]>

* [MM-69691] Retrigger CI after buildenv artifact infra failure

Co-authored-by: mattermost-code <[email protected]>

* [MM-69691] Retrigger Enterprise CI after npm ECONNRESET flake

Co-authored-by: mattermost-code <[email protected]>

* Add missing return in doStatus error path

Address review feedback from @wiggin77: the error branch in doStatus
built an error response but discarded it, so execution fell through to
the empty-list check on a nil list. Return the response instead.

---------

Co-authored-by: mattermost-code <[email protected]>

Co-authored-by: Hanzo Dev <[email protected]>
2026-07-04 16:16:11 -04:00
..
…