Skip to content

Fix 'restricted open_basedir' e2e test#5138

Merged
staabm merged 1 commit intophpstan:2.1.xfrom
staabm:fix-tests
Mar 6, 2026
Merged

Fix 'restricted open_basedir' e2e test#5138
staabm merged 1 commit intophpstan:2.1.xfrom
staabm:fix-tests

Conversation

@staabm
Copy link
Contributor

@staabm staabm commented Mar 6, 2026

since c288e7e e2e tests started failling:

e2e/restricted-php-ini
...
Installing dependencies from lock file (including require-dev)
Verifying lock file contents can be installed on current platform.
Nothing to install, update or remove
Generating autoload files
 0/4 [░░░░░░░░░░░░░░░░░░░░░░░░░░░░]   0%
 4/4 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%


In AgentDetector.php line 44:
                                                                               
  file_exists(): open_basedir restriction in effect. File(/opt/.devin) is not  
   within the allowed path(s): (/home/runner/work/phpstan-src/phpstan-src:/tm  
  p)                                                                           
                                                                               

analyse [-c|--configuration CONFIGURATION] [-l|--level LEVEL] [--no-progress] [--debug] [-a|--autoload-file AUTOLOAD-FILE] [--error-format ERROR-FORMAT] [-b|--generate-baseline [GENERATE-BASELINE]] [--allow-empty-baseline] [--memory-limit MEMORY-LIMIT] [--xdebug] [--tmp-file TMP-FILE] [--instead-of INSTEAD-OF] [--fix] [--watch] [--pro] [--fail-without-result-cache] [--] [<paths>...]

the test in question was added with phpstan/phpstan#7518
which was fixed with #1466 - this PR inspired me for the fix here.

@staabm staabm marked this pull request as ready for review March 6, 2026 14:43
@staabm staabm requested a review from VincentLanglet March 6, 2026 14:43
@phpstan-bot
Copy link
Collaborator

This pull request has been marked as ready for review.

@staabm staabm merged commit 945e00c into phpstan:2.1.x Mar 6, 2026
637 of 648 checks passed
@staabm staabm deleted the fix-tests branch March 6, 2026 15:29
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