improve doctest coverage of databases/conway.py

[aghitza]

As of sage-4.4.3:

----------------------------------------------------------------------
conway.py
ERROR: Please add a `TestSuite(s).run()` doctest.
SCORE conway.py: 0% (0 of 7)

Missing documentation:
 * _init(self):
 * __repr__(self):
 * polynomial(self, p, n):
 * has_polynomial(self, p, n):
 * primes(self):
 * degrees(self, p):

Missing doctests:
 * __init__(self, read_only=True):

----------------------------------------------------------------------

CC: @sagetrac-mvngu

Component: doctest coverage

Keywords: conway polynomial database

Author: Alex Ghitza

Reviewer: David Loeffler

Merged: sage-4.5.2.alpha1

Issue created by migration from https://trac.sagemath.org/ticket/9222

[aghitza]

Author: Alex Ghitza

[aghitza]

comment:2

After the patch:

ERROR: Please add a `TestSuite(s).run()` doctest.
SCORE conway.py: 85% (6 of 7)

Missing documentation:
	 * _init(self):

I'm not convinced a TestSuite test makes sense here (and I have tried to put one in and got a pickling-related error). Also, I'm not sure what _init(self) is meant to be doing, so I'm leaving it alone for now.

[loefflerd]

comment:3

Looks fine to me. Tests pass and the documentation builds OK. I agree that the _init method isn't something one can reasonably doctest!

[loefflerd]

Reviewer: David Loeffler

[dandrake]

comment:5

Please include ticket numbers in the commit strings of your patches!

[aghitza]

comment:6

Attachment: trac_9222.patch.gz

Done.

[dandrake]

Merged: sage-4.5.2.alpha1

[dandrake]

comment:7

Replying to @aghitza:

Done.

Thanks!