AccountForm should read pass-reset-token only from query string

Some change in #5 patch

In the issue summary it says:
- pass-reset-token should be checked in query string
- \Drupal\views\Plugin\views\cache\Time stores request object [3109110]
Can these be broken into two separate issues. They are completely different scopes even though they were detected in the same review.

Here is some test coverage based on the account settings form.

-   * @var \Drupal\Core\Entity\EntityBase|\Drupal\Core\Entity\EntityInterface
+   * @var \Drupal\user\UserInterface

changing the type hint of $user variable

-  protected static $modules = ['system', 'user', 'field'];
+  protected static $modules = ['system', 'user'];

removing field module

-    $this->container->get('entity_type.manager')
-      ->getFormObject($entity_type, $operation)
-      ->setEntity($entity);

About this i am not sure , but we are not using this anywhere . So can you please me to understand this get container use-case in buildAccountForm().

Kindly review a patch.

Sorry forgot to upload interdiff.

#18good catch @Hardik_Patel_12. We might not need field related features given we are testing the entity behaviour. I will clean up.

Created new issue for removing getFormObject call from UserAccountFormFieldsTest file kindly look #3154461: Removing getFormObject call from UserAccountFormFieldsTest

LGTM

Committed and pushed 361ae1cb71 to 9.2.x and a6e7eeef22 to 9.1.x. Thanks!