Wrap devtoolslib doctests in a unittest so that the test runner sees them.

Wrap devtoolslib doctests in a unittest so that the test runner sees them.

[ppi, 2015-06-12 17:02:37]

ppi@chromium.org changed reviewers:
+ etiennej@chromium.org, viettrungluu@chromium.org

[ppi, 2015-06-12 17:02:56]

Hi Trung or Etienne,

ptal.

[viettrungluu, 2015-06-12 17:09:53]

On 2015/06/12 17:02:56, ppi wrote:
> Hi Trung or Etienne,
> 
> ptal.

What are you using doctests for and why?

[ppi, 2015-06-12 17:16:04]

For
https://github.com/domokit/mojo/blob/master/mojo/devtools/common/devtoolslib/...
, because it seems a good fit for simple functions without side-effects.

Is there any reason not to use doctests?

[viettrungluu, 2015-06-12 17:24:48]

On 2015/06/12 17:16:04, ppi wrote:
> For
>
https://github.com/domokit/mojo/blob/master/mojo/devtools/common/devtoolslib/...
> , because it seems a good fit for simple functions without side-effects.
> 
> Is there any reason not to use doctests?

Yes, because it adds another framework, necessitating another way of running
tests.

[ppi, 2015-06-12 17:28:33]

Or rather another way of writing tests - as this very CL adds a wrapper that
makes them runnable by the unittest framework.

Fair enough, I'll convert these to a unittest, thanks.