Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

test: improve tests execution time #1041

Merged
merged 3 commits into from
Feb 24, 2024
Merged

test: improve tests execution time #1041

merged 3 commits into from
Feb 24, 2024

Conversation

datamweb
Copy link
Collaborator

@datamweb datamweb commented Feb 23, 2024 •
edited
Loading

Description
This PR tries to speed up the execution of tests by setting a value of $hashCost = 4 for the test environment.
More Info see #916

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

@datamweb datamweb added the refactor Pull requests that refactor code label Feb 23, 2024
@datamweb datamweb mentioned this pull request Feb 23, 2024
Merged
5 tasks
@datamweb @lonnieezell
docs: improved description
0633ec5 
Co-authored-by: Lonnie Ezell <[email protected]>
@kenjis kenjis added the testing Pull requests that changes tests only label Feb 23, 2024
@kenjis kenjis changed the title refactor: improve tests execution time test: improve tests execution time Feb 23, 2024
@datamweb datamweb merged commit d71090d into codeigniter4:develop Feb 24, 2024
25 checks passed
@datamweb datamweb deleted the improve-test-time branch February 24, 2024 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kenjis kenjis kenjis approved these changes

@lonnieezell lonnieezell lonnieezell approved these changes

Assignees
No one assigned
Labels
refactor Pull requests that refactor code testing Pull requests that changes tests only
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@datamweb @lonnieezell @kenjis