#13885: Fix optional magma tests.
----------------------------------------+------------------------
Reporter: robertwb | Owner: was
Type: defect | Status: new
Priority: major | Milestone: sage-7.2
Component: interfaces: optional | Resolution:
Keywords: | Merged in:
Authors: | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
Dependencies: | Stopgaps:
----------------------------------------+------------------------
Comment (by mmasdeu):
I think this is more like a problem with the testing framework. Here is a
sample block that fails the test:
{{{
sage: C = codes.HammingCode(3,GF(2))
sage: Cm = magma(C) # optional - magma, indirect doctest
sage: Cm.MinimumWeight() # optional - magma
3
}}}
The problem with this (and I suspect most of the others) is that when the
``--only-optional=magma`` flag prevents the first line in the block above
to get executed, and thus C remains undefined and causes a failure in the
second line.
I think that having to write the ``# optional - magma`` thing in each line
of a block is quite awful. It would be better that the ``--only-
optional=magma`` flag ran (in full) those blocks containing at least one
``# optional - magma`` line.
I don't know how to do this myself, though...
--
Ticket URL: <http://trac.sagemath.org/ticket/13885#comment:8>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
