Skip to content

Conversation

@come-nc
Copy link
Contributor

@come-nc come-nc commented May 19, 2025

Summary

Some applications are directly referring to OC::$loader in their tests bootstrap.php to be able to load Test\TestCase class.

This adds an autoload.php file that they can use instead, to allow core to refactor TestCase autoloading without breaking apps.

Pre-requisite for #52945

Checklist

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc come-nc added this to the Nextcloud 32 milestone May 19, 2025
@come-nc come-nc self-assigned this May 19, 2025
@come-nc come-nc requested a review from a team as a code owner May 19, 2025 09:32
@come-nc come-nc removed the request for review from a team May 19, 2025 09:32
@come-nc come-nc added the 3. to review Waiting for reviews label May 19, 2025
@codecov
Copy link

codecov bot commented May 19, 2025

❌ Unsupported file format

Upload processing failed due to unsupported file format. Please review the parser error message:

Error deserializing json

Caused by:
    expected value at line 1 column 1

For more help, visit our troubleshooting guide.

@come-nc
Copy link
Contributor Author

come-nc commented May 19, 2025

/backport to stable31

@come-nc
Copy link
Contributor Author

come-nc commented May 19, 2025

/backport to stable30

@come-nc
Copy link
Contributor Author

come-nc commented May 19, 2025

/backport to stable29

@nickvergessen
Copy link
Member

Should be documented in https://docs.nextcloud.com/server/latest/developer_manual/app_publishing_maintenance/app_upgrade_guide/upgrade_to_32.html

@nickvergessen nickvergessen added pending documentation This pull request needs an associated documentation update tests Related to tests labels May 19, 2025
@come-nc come-nc enabled auto-merge May 19, 2025 15:39
@come-nc come-nc requested review from artonge and susnux May 19, 2025 15:39
@come-nc come-nc merged commit 0b2cf0b into master May 19, 2025
202 of 204 checks passed
@come-nc come-nc deleted the fix/add-autoload.php-for-tests branch May 19, 2025 15:43
@come-nc come-nc removed the pending documentation This pull request needs an associated documentation update label May 22, 2025
ChristophWurst added a commit to ChristophWurst/nextcloud_sentry that referenced this pull request Jun 13, 2025
nextcloud/server#52951

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@skjnldsv skjnldsv mentioned this pull request Aug 19, 2025
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews tests Related to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants