Details
-
Task
-
Status: Resolved
-
Major
-
Resolution: Won't Fix
-
1.6.0
-
None
Description
The ObjectApprover::approved() method has the following signature:
virtual Try<bool> approved( const Option<Object>& object) const noexcept = 0;
This is unfortunate since it is impossible to mock a function in google mock with two qualifiers, which reduces the amount of tests we can perform.
Moreover, the noexcept is not even needed in Mesos, since it does not use exception and it is compiled without exception support by default.
The tricky situation here is that this is a public API so it would be tricky to replace since it will break backwards compatibility.