> So I think this all looks pretty good, here are just a few more observations:
>
> 565 + static std::string s{"com.ubuntu.trust.store"};
>
> I know it doesn't really matter but the "const" is missing here.
> Should be: "static const std::string s{"com.ubuntu.trust.store"};"
>
Fixed.
> 1493 + (core::trust::mir::cli::option_title,
> boost::program_options::value<std::string>(), "Title of the prompt");
>
> Missing a full-stop at the end of the string (doesn't match the others).
>
> The above methods don't appear in your tests.
>
> 1426 === added file 'src/core/trust/mir/prompt_main.cpp'
> 2786 === added file 'tests/test_prompt.cpp'
>
> These 2 files are pretty much doing the same thing. I get how one is a test
> version of the other but is it not a maintenance nightmare waiting to happen?
> If the app is used for internal use only, why not just check for something
> like a --testargs argument when running testing.
> So I think this all looks pretty good, here are just a few more observations: ubuntu. trust.store" }; ubuntu. trust.store" };"
>
> 565 + static std::string s{"com.
>
> I know it doesn't really matter but the "const" is missing here.
> Should be: "static const std::string s{"com.
>
Fixed.
> 1493 + (core:: trust:: mir::cli: :option_ title, program_ options: :value< std::string> (), "Title of the prompt");
> boost::
>
> Missing a full-stop at the end of the string (doesn't match the others).
>
Fixed.
> 1530 + default:
> 1531 + break;
>
> Redundant "break;"
But the compiler needs it :) ptr<core: :trust: :Store: :Query> a_null_query()
>
> 2514 +std::shared_
>
Keeping it for future convenience.
> 2052 +std::function< core::trust: :Request: :Answer( const :wait:: Result& )> mock_translator _to_functor( const ptr<MockTransla tor>& ptr)
> core::posix:
> std::shared_
>
Removed.
> 2062 +std::shared_ ptr<core: :trust: :mir::PromptPro viderHelper> provider_ calling_ bin_false( )
> a_prompt_
>
Removed.
> 2080 +std::shared_ ptr<core: :trust: :mir::PromptPro viderHelper> provider_ calling_ bin_true( )
> a_prompt_
>
Removed.
> The above methods don't appear in your tests. trust/mir/ prompt_ main.cpp' test_prompt. cpp'
>
> 1426 === added file 'src/core/
> 2786 === added file 'tests/
>
> These 2 files are pretty much doing the same thing. I get how one is a test
> version of the other but is it not a maintenance nightmare waiting to happen?
> If the app is used for internal use only, why not just check for something
> like a --testargs argument when running testing.
Fixed.