TODO: Implement methods in test classes

silverstripe / silverstripe-mfa

MultiFactor Authentication for Silverstripe CMS

BSD 3-Clause "New" or "Revised" License

11 stars 24 forks source link

Closed sabina-talipova closed 11 months ago

sabina-talipova commented 12 months ago

The following classes have unimplemented methods:

SilverStripe\MFA\Tests\Stub\Null\Method
- getThumbnail
- applyRequirements
- getUnavailableMessage
SilverStripe\MFA\Tests\Stub\Null\RegisterHandler
- start
- register
- getDescription
- getSupportLink
- getSupportText
- getComponent
SilverStripe\MFA\Tests\Stub\Null\VerifyHandler
- start
- verify
- getComponent

We are using these classes only for testing.

GuySartorelli commented 11 months ago

Given the tests are passing, I'd say it's fine to just leave these unimplemented.