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

Improve testing speed #986

Open
driesvints opened this issue Dec 29, 2023 · 4 comments
Open

Improve testing speed #986

driesvints opened this issue Dec 29, 2023 · 4 comments
Labels
enhancement help wanted

Comments

@driesvints
Copy link
Member

For some reason, our test suite is super slow. We should look into this and try to decrease memory usage and speed it up.

Parallel run:

  Tests:    316 passed (785 assertions)
  Duration: 13.64s
  Parallel: 10 processes

Regular run:

  Tests:    316 passed (785 assertions)
  Duration: 96.30s
@ash-jc-allen ash-jc-allen mentioned this issue Mar 3, 2024
Closed
@apmedev apmedev mentioned this issue Oct 4, 2024
Closed
@driesvints
Copy link
Member Author

We got it to run much faster now with #1151 but we might want to investigate individual slow tests going forward with vendor/bin/pest --profile

For example, the password reset test takes 3 seconds? 🤔

@Nathan-Cowin
Copy link
Contributor

Apologies not sure best way to link my PR: #1167 - Improves by 1/3, cheers

@Nathan-Cowin Nathan-Cowin mentioned this issue Oct 31, 2024
Merged
driesvints added a commit that referenced this issue Nov 1, 2024
@Nathan-Cowin @driesvints
Speed up tests by using lower bcrypt rounds in testing env #986 (#1167)
51bcb77 
* Fixes#986 use lower BCRYPT_ROUNDS in testing to speed up tests

* database name to testing

* wip

---------

Co-authored-by: Dries Vints <dries@vints.be>
@driesvints
Copy link
Member Author

Thanks @Nathan-Cowin. Also a good improvement. Will leave this open to see if we can get it down even further eventually.

@Nathan-Cowin
Copy link
Contributor

Another easy second here #1169

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement help wanted
Milestone
No milestone
Development

No branches or pull requests
2 participants
@driesvints @Nathan-Cowin