alexeymezenin / laravel-best-practices

Laravel best practices
11.24k stars 2.37k forks source link

Change test case methods to snake_case and mention Pest support #169

Open JohnnyWalkerDigital opened 1 year ago

JohnnyWalkerDigital commented 1 year ago

Update naming convention to match official Laravel documentation. (These "best practices" are currently out-of-step with modern Laravel standards.)

Reasoning:

JohnnyWalkerDigital commented 10 months ago

@alexeymezenin You seem to be engaging with this repo again... How about bringing it up to speed with current Laravel standards?

Tpojka commented 10 months ago

Snake case should be used in tests regardless Laravel.

It differs from regular methods and force developer to pay extreme attention to those. It differs from regular (app) methods that should be written following PSR standards. Personally, I comply/obey 100% PSR standards which means I use camelCase for methods, but for testing methods I use snake_case.

JohnnyWalkerDigital commented 2 days ago

@alexeymezenin Are you ever going to address this issue? I want to use this repo as a reference, but I can't while it's outdated.