Code review comment for lp:~jassmith/unity/unity.alt-tab-timings

Revision history for this message
David Barth (dbarth) wrote :

At the limit, a unit test could be added to verify that the defined default is what is actually the value that is used. There are practical maintenance issues to adding unit tests at that granularity, since the defaults are not managed in a configuration file, but in a design specification that is not programmatically connected to the code.

Besides, while arguing about the interest of an equivalent AP test, we have agreed that an AP test would be equivalent to such a unit test.

So we concluded that the AP test was not necessary, since the equivalent unit test would be rather trivial and difficult to maintain

review: Approve

« Back to merge proposal