Skip to content

test: Increase Email test coverage and enable component in random tests#10411

Open
gr8man wants to merge 1 commit into
codeigniter4:developfrom
gr8man:email-tests-coverage
Open

test: Increase Email test coverage and enable component in random tests#10411
gr8man wants to merge 1 commit into
codeigniter4:developfrom
gr8man:email-tests-coverage

Conversation

@gr8man

@gr8man gr8man commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Description
Ref: #9968

  • Added additional test cases for the Email class (covering header injection prevention, SMTP disconnection handling, UTF-8 filenames, and other edge cases).
  • Resolved global state pollution issues in namespace overrides by implementing cleanup in tearDown().
  • Enabled the Email component in the random tests execution configuration (random-tests-config.txt).
  • Fixed a PHPDoc type typo for the $newline property in Email.php.

Checklist:

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

@gr8man
gr8man force-pushed the email-tests-coverage branch from 71e52b3 to e22a54b Compare July 15, 2026 19:44
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.

1 participant