Skip to content

fix: unset encryption.key variables in RunInSeparateProcess tests - #9670

Merged
paulbalandan merged 1 commit into
codeigniter4:developfrom
paulbalandan:test-separate-process
Aug 12, 2025
Merged

fix: unset encryption.key variables in RunInSeparateProcess tests#9670
paulbalandan merged 1 commit into
codeigniter4:developfrom
paulbalandan:test-separate-process

Conversation

@paulbalandan

@paulbalandanpaulbalandan commented Aug 11, 2025

Copy link
Copy Markdown
Member

Description
I have a .env with a generated encryption key. Running vendor/bin/phpunit --group SeparateProcess fails because the two tests pick up the encryption key from my .env instead of the injected .env file.

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@paulbalandanpaulbalandan added the testing Pull requests that changes tests only label Aug 11, 2025
@paulbalandan
paulbalandanforce-pushed the test-separate-process branch 2 times, most recently from 5b58098 to 35e43daCompareAugust 11, 2025 16:27
neznaika0
neznaika0 reviewed Aug 11, 2025
View reviewed changes
Comment threadtests/system/Config/DotEnvTest.php
@paulbalandan
paulbalandan merged commit 343b6f7 into codeigniter4:developAug 12, 2025
47 checks passed
@paulbalandan
paulbalandan deleted the test-separate-process branch August 12, 2025 12:53
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@michalsnmichalsnmichalsn approved these changes
+1 more reviewer
@neznaika0neznaika0neznaika0 left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

testingPull requests that changes tests only

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@paulbalandan@michalsn@neznaika0