Skip to content

test: make test case in MessageTest a bit more realistic#6686

Merged
kenjis merged 1 commit intocodeigniter4:developfrom
kenjis:fix-MessageTest
Oct 15, 2022
Merged

test: make test case in MessageTest a bit more realistic#6686
kenjis merged 1 commit intocodeigniter4:developfrom
kenjis:fix-MessageTest

Conversation

@kenjis
Copy link
Copy Markdown
Member

@kenjis kenjis commented Oct 14, 2022

Description

  • adding string \n does not make sense

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

@kenjis kenjis added the testing Pull requests that changes tests only label Oct 14, 2022
@kenjis kenjis merged commit ca76a09 into codeigniter4:develop Oct 15, 2022
@kenjis kenjis deleted the fix-MessageTest branch October 15, 2022 02:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@paulbalandan paulbalandan paulbalandan approved these changes

Assignees

No one assigned

Labels

testing Pull requests that changes tests only

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@kenjis @paulbalandan