Skip to content

Conversation

@marius-mather
Copy link
Collaborator

Description

This was causing intermittent CI failures because approval_status was chosen randomly and it only failed on certain statuses. Test all statuses so we don't get intermittent failures

Changes

  • Fix test of save_history
  • Always test all approval statuses so this doesn't happen again

Copy link
Contributor

@amandazhuyilan amandazhuyilan left a comment

Choose a reason for hiding this comment

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

No issues found

@marius-mather marius-mather merged commit 68ca81d into main Jan 13, 2026
5 checks passed
@marius-mather marius-mather deleted the fix/group-membership-history-test branch January 13, 2026 05:15
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