@@ -47,7 +47,6 @@ public:

         user user (found_user[0]);

         std::string key_string = tokens_.ensure_key (user);

-        // TODO: use vectors in generator input instead of strings to avoid this nonsense

         std::vector<uint8_t> key = base32().decode (key_string);

         std::string token = generator_.generate_token (key);

@@ -21,7 +21,6 @@

 #include "test_util.h"

 #include "generator.h"

-// TODO: audit usage of 'token', sometimes it should be 'key'

 class mock_tokens : public tokens_ifc

 {

 public:

@@ -147,30 +146,6 @@ int installs_token()

     succeed();

 }

-// int unistd_does_not_find_user_name_nullptr_case()

-// {

-//     //given

-//     std::string user_name ("user");

-//     std::string key ("token");

-//     auto  test_tokens = std::make_shared<mock_tokens>(key);

-//     tokens tokens{test_tokens};

-//     unistd unistd (std::make_shared<fail_unistd>());

-//     directory directory (std::make_shared<fake_directory> (user_name));

-//     totp_generator generator (std::make_shared<fake_totp_generator>());

-

-//     installer installer = installer::create (tokens, unistd, directory,

-//                           generator);

-

-//     //when

-//     auto returned = installer.install_key();

-

-//     //then

-//     check (test_tokens->captured_token == "",

-//            "should not have installed a token");

-//     check (returned == "", "did not return empty token");

-//     succeed();

-// }

-

 int unistd_does_not_find_user_name_empty_string_case()

 {

     //given

@@ -226,7 +201,6 @@ int directory_finds_no_user_info()

 int run_tests()

 {

     test (installs_token);

-    // test (unistd_does_not_find_user_name_nullptr_case);

     test (unistd_does_not_find_user_name_empty_string_case);

     test (directory_finds_no_user_info);

     succeed();