New issue
Advanced search Search tips

Issue 794906 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Task



Sign in to add a comment

cryptohome_authenticator_unittest should not reimplement the cryptohome client fake in order to verify interactions

Project Member Reported by maajid@chromium.org, Dec 14 2017

Issue description

Currently, cryptohome_authenticator_unittest is set up in such a way that it needs to have its own cryptohome fake which has EXPECT statements built into the MountEx implementation.

The test should be rewritten so that this is not necessary, and only the necessary interactions should be verified. As it stands the test is confusing and there is a lot of indirection between each test case and what is actually being verified.
 

Comment 1 by maajid@chromium.org, Dec 14 2017

Cc: hidehiko@chromium.org satorux@chromium.org hashimoto@chromium.org xiy...@chromium.org

Sign in to add a comment