Skip to content

fix(tests): Fix encryption test isolation between test runs#60000

Merged
skjnldsv merged 1 commit intonextcloud:masterfrom
cuppett:cuppett/test-isolation
Apr 30, 2026
Merged

fix(tests): Fix encryption test isolation between test runs#60000
skjnldsv merged 1 commit intonextcloud:masterfrom
cuppett:cuppett/test-isolation

Conversation

@cuppett
Copy link
Copy Markdown
Contributor

@cuppett cuppett commented Apr 29, 2026

Summary

Add global encryption teardown to TestCase base class so encryption state does not leak between test suites regardless of which tests ran earlier.

Spawned from #57279

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Add global encryption teardown to TestCase base class so encryption state
does not leak between test suites regardless of which tests ran earlier.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Stephen Cuppett <steve@cuppett.com>
@cuppett cuppett requested a review from a team as a code owner April 29, 2026 23:45
@cuppett cuppett requested review from Altahrim, ArtificialOwl, leftybournes and salmart-dev and removed request for a team April 29, 2026 23:45
@cuppett cuppett added this to the Nextcloud 34 milestone Apr 29, 2026
@skjnldsv
Copy link
Copy Markdown
Member

Congrats on the 60000th GitHub ticket! 🎊🎊

Copy link
Copy Markdown
Member

@CarlSchwan CarlSchwan left a comment

Choose a reason for hiding this comment

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

make sense to me

@skjnldsv skjnldsv merged commit 723b441 into nextcloud:master Apr 30, 2026
205 of 225 checks passed
@cuppett cuppett deleted the cuppett/test-isolation branch April 30, 2026 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants