Opened 13 years ago

Closed 8 years ago

#298 closed enhancement (wontfix)

autotesting of examples

Reported by: was Owned by: was
Priority: critical Milestone: sage-duplicate/invalid/wontfix
Component: packages: standard Keywords:
Cc: kini, jdemeyer Merged in:
Authors: Reviewers: John Palmieri, Keshav Kini
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #7494 Stopgaps:

Description

  • get Rishi's autotesting of examples directory to work: He developed it under OS X, but it doesn't work on Linux because of differences in how they run scripts, etc. Plus, it's a really hard challenge to create such automated testing, so it needs to be looked over again.

Change History (12)

comment:1 Changed 12 years ago by mabshoff

  • Milestone set to sage-feature

comment:2 Changed 12 years ago by mabshoff

  • Priority changed from major to critical

Where is that code? Does that mean that we want to be able to run the doctests in examples? Then certainly this ought to be of much higher priority since the examples *must* work with the version of Sage that shipped it.

Cheers,

Michael

comment:3 Changed 10 years ago by jason

This looks invalid or maybe even done.

comment:4 Changed 10 years ago by was

wstein:
Why do you think #298 should be closed?
[2:45pm] wstein:
It's more important than ever to fix.
[2:45pm] jason-:
I thought it was talking about doctesting, basically
[2:45pm] wstein:
There is a directory SAGE_ROOT/examples.
[2:45pm] wstein:
That code isn't tested at all.
[2:45pm] jason-:
to quote the comment, "where is that code"?
[2:45pm] wstein:
It's a terrifying disaster.
[2:45pm] jason-:
oh, okay
[2:45pm] jason-:
never mind
[2:46pm] wstein:
I can't believe I still haven't dealt with this... but oh well.
[2:46pm] wstein:
It's one of those things that looks easy until you try it.

comment:5 Changed 10 years ago by was

See the related ticket #7494. If that is closed, then this ticket is certainly invalid.

comment:6 Changed 9 years ago by kini

  • Cc kini added
  • Report Upstream set to N/A

comment:7 Changed 8 years ago by jhpalmieri

Since #7494 has been closed, I think this can be, too.

comment:8 Changed 8 years ago by kini

  • Status changed from new to needs_review

Yes.

comment:9 Changed 8 years ago by kini

  • Status changed from needs_review to positive_review

comment:10 Changed 8 years ago by leif

  • Cc jdemeyer added
  • Milestone changed from sage-feature to sage-duplicate/invalid/wontfix
  • Reviewers set to John Palmieri, Keshav Kini

comment:11 Changed 8 years ago by leif

  • Dependencies set to #7494

comment:12 Changed 8 years ago by jdemeyer

  • Resolution set to wontfix
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.