Remove selfie and liveness check from mock doc_auth client

[jskinne3]

🎫 Ticket

LG-7711

🛠 Summary of changes

• Remove selfie and liveness check from mock doc_auth client and relevant specs.
• Put a NotImplementedError to a function that's needed when spec compares to mock client to Acuant client

[jskinne3]

Are these the right errors to expect here?

[aduth]

This test case appears... contradictory. It's claiming that it's testing a passed result, but then asserting that it's unsuccessful?

I wonder if the input YAML is not set up correctly and is being treated as a failure (e.g. missing properties). Could we try pulling in a passed result like what we document and see if it would turn these values to a successful result?

[jskinne3]

Agreed. The author of the test seems not sure which result they wanted to test. Per @jmhooper I used example .yaml files from our docs to split this test into two:

• returns a passed result
• returns caution result

[zachmargolis]

is there a spec that ensures this method is defined? if not, I would just delete it because we still get a NoMethodError when calling it, almost the same as NotImplementedError

[jskinne3]

Yup there is