#7650: Fix sagenb doctesting
------------------------+---------------------------------------------------
Reporter: mpatel | Owner: was
Type: defect | Status: positive_review
Priority: blocker | Milestone: sage-4.3.1
Component: notebook | Keywords:
Work_issues: | Author: Mitesh Patel
Upstream: N/A | Reviewer: Tim Dumol
Merged: |
------------------------+---------------------------------------------------
Changes (by mpatel):
* status: needs_review => positive_review
* reviewer: => Tim Dumol
Old description:
> We have not been doctesting SageNB files as often as we test Sage files.
>
> Please apply
>
> * [attachment:trac_7650-scripts_doctest_force_lib_v5.patch] to the
> scripts repository. This adds two options to `sage -t`: `-sagenb` tests
> all SageNB files and `-force_lib` tests files ''as if'' they live in
> `SAGE_ROOT/devel/sage`, i.e., in the Sage library.
>
> * [attachment:trac_7650-sagenb_doctesting_v5.patch] to the sagenb
> repository. This fixes broken SageNB tests.
>
> and replace `SAGE_ROOT/makefile` with
>
> * [attachment:makefile]. This makes `make test` and `make ptest` test
> all SageNB files, too.
>
> Dependencies: Sage 4.3.1.alpha1, #7269.
New description:
We have not been doctesting SageNB files as often as we test Sage files.
Please apply
* [attachment:trac_7650-scripts_doctest_force_lib_v5.patch] to the
scripts repository. This adds two options to `sage -t`: `-sagenb` tests
all SageNB files and `-force_lib` tests files ''as if'' they live in
`SAGE_ROOT/devel/sage`, i.e., in the Sage library.
* [attachment:trac_7650-sagenb_doctesting_v6.patch] '''and'''
[attachment:trac_7650-reviewer_v2.patch] to the sagenb repository. This
fixes broken SageNB tests.
and replace `SAGE_ROOT/makefile` with
* [attachment:makefile]. This makes `make test` and `make ptest` test
all SageNB files, too.
Dependencies: Sage 4.3.1.alpha1, #7269.
--
Comment:
V2 of the reviewer patch:
* Adds a missing dot (`.`) in `interact.py`.
* Moves `# output depends on timezone` to the input statement in
`worksheet.py`.
* [Pre-existing] `most be defined` --> `must be defined`.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7650#comment:21>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
