Skip to content

fix(tests): load CoreTest autoload from core vendor - #2414

Merged
Seiger merged 1 commit into
evolution-cms:3.5.xfrom
MiddleSokilAI:middleSokil/ci-coretest-bootstrap
Aug 11, 2026
Merged

fix(tests): load CoreTest autoload from core vendor#2414
Seiger merged 1 commit into
evolution-cms:3.5.xfrom
MiddleSokilAI:middleSokil/ci-coretest-bootstrap

Conversation

@MiddleSokilAI

Copy link
Copy Markdown
Collaborator

Problem

The new CI test job installs Composer dependencies from core/, so test dependencies and autoload.php are available under core/vendor.

CoreTest bootstrapped EVO_BASE_PATH as the repository root but still looked for vendor/autoload.php directly under that root, which makes the getTagsFromContent test group fail with Run: composer install in CI.

Root Cause

The test mixed the public repository root path with the core Composer vendor path. EVO_CORE_PATH already points at core/, so the test should use it for the Composer autoloader.

Change Summary

  • Load CoreTest autoload from EVO_CORE_PATH . 'vendor/autoload.php'.
  • Keep the existing EVO_BASE_PATH, EVO_CORE_PATH, storage, manager, and site URL constants unchanged.

Validation

  • git diff --check

Local runtime checks were limited because PHP/Composer are not available in this Windows shell, WSL failed to start, and Docker Desktop is not running. The change directly matches the CI install layout from .github/workflows/ci.yml.

Risk

Low. The change is scoped to one unit test bootstrap path and does not affect runtime code.

@Seiger
Seiger marked this pull request as ready for review August 11, 2026 23:43
@Seiger
Seiger merged commit 7b65296 into evolution-cms:3.5.x Aug 11, 2026
3 of 6 checks passed
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.

2 participants