refactor: Remove long deprecated IServerContainer usages

[CarlSchwan]

No description provided.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 1d81d0cf-1ce5-4794-9005-d8970619ee4e

📥 Commits

Reviewing files that changed from the base of the PR and between 9d67a65 and 325e38e.

📒 Files selected for processing (5)

• lib/AppInfo/Application.php
• tests/Integration/Framework/ImapTestAccount.php
• tests/Integration/Service/DraftServiceIntegrationTest.php
• tests/Integration/Service/MailTransmissionIntegrationTest.php
• tests/Integration/Service/OutboxServiceIntegrationTest.php

💤 Files with no reviewable changes (1)

• tests/Integration/Service/MailTransmissionIntegrationTest.php

---

📝 Walkthrough

Walkthrough

This PR refactors the application and test suite to use the IRootFolder interface instead of IServerContainer for accessing user folders. The core change updates Application.php to register the userFolder service via IRootFolder::getUserFolder(). Test files are updated to match this pattern, replacing server container lookups with direct IRootFolder resolution. Additionally, test infrastructure receives type safety improvements through explicit PHP return types on trait methods and intersection-typed mock properties.

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 20.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check	❓ Inconclusive	No pull request description was provided by the author, making this check inconclusive as the expected context and rationale for the changes cannot be assessed.	Add a pull request description explaining the motivation for removing IServerContainer and the benefits of using IRootFolder and modern service injection patterns.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'refactor: Remove long deprecated IServerContainer usages' directly and clearly describes the main change throughout the pull request: replacing deprecated IServerContainer with modern alternatives like IRootFolder.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[ChristophWurst]

interface IRootFolder extends Folder, Emitter

trading one tech debt with another 🙈

I don't think we need the emitter. Adjust the Psalm config to ignore the missing dependency

[ChristophWurst]

nextcloud/server#61539