\Drupal calls should be avoided in classes, use dependency injection instead

I have implemented the dependency injection and issued an MR. Please review.

Hi @sidharth_soman, I reviewed the MR and found an issue. with UserPasswordFormAlter class. The $container->get('entity_type.manager') container is missing which is causing this error as follows:

ArgumentCountError: Too few arguments to function Drupal\decoupled_auth\Form\UserPasswordFormAlter::__construct(), 2 passed in /app/web/modules/contrib/decoupled_auth-3356179/src/Form/UserPasswordFormAlter.php on line 58 and exactly 3 expected in Drupal\decoupled_auth\Form\UserPasswordFormAlter->__construct() (line 46 of /app/web/modules/contrib/decoupled_auth-3356179/src/Form/UserPasswordFormAlter.php)

There are few more issue reported by phpcs here:

FILE: /home/arpitkayare/drupal_9.5/web/modules/contrib/decoupled_auth-3356179/src/DecoupledAuthSettingsForm.php
---------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------
 117 | WARNING | Role::loadMultiple calls should be avoided in classes, use dependency injection instead

FILE: /home/arpitkayare/drupal_9.5/web/modules/contrib/decoupled_auth-3356179/tests/src/Functional/RegistrationTest.php
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------------------------------
 75 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead

Moving the issue to Needs work and updating the MR.

Thanks!

I have updated the MR with fixes. Please review.

Thanks!