Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add counter to testing #20971

Open
jm58660 mannequin opened this issue Jul 7, 2016 · 6 comments
Open

Add counter to testing #20971

jm58660 mannequin opened this issue Jul 7, 2016 · 6 comments

Comments

@jm58660
Copy link
Mannequin

jm58660 mannequin commented Jul 7, 2016

It would be nice to have a counter showing how many test has already run, or how many is left. For example changing

cmd = "sage -t"

to

cmd = "Test %s/%s: "%(self.sources_completed+1, len(self.controller.sources))+" sage -t"

or something like that in src/sage/doctest/reporting.py. Hopefully someone who knows the framework can catch this.

CC: @tscrim @fchapoton

Component: doctest framework

Author: Jori Mäntysalo

Branch/Commit: u/jmantysalo/counter-testing @ d59f4d1

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

@jm58660 jm58660 mannequin added this to the sage-7.3 milestone Jul 7, 2016
@jm58660 jm58660 mannequin modified the milestones: sage-7.3, sage-7.4 Aug 21, 2016
@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Aug 24, 2016

Branch: u/jmantysalo/counter-testing

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Aug 24, 2016

comment:3

One line patch, but what you think about the idea? I would like to have some progress meter when possible, this adds at least something.


New commits:

d59f4d1Add counter to testing.

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Aug 24, 2016

Commit: d59f4d1

@jm58660 jm58660 mannequin added the s: needs review label Aug 24, 2016
@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Aug 24, 2016

Author: Jori Mäntysalo

@jdemeyer
Copy link

comment:5

See patchbot.

@jm58660
Copy link
Mannequin Author

jm58660 mannequin commented Aug 26, 2016

comment:6

My approach was wrong, [x/N] -prefix is not needed in error reporting. Anybody knows enought of doctesting framework to make this work?

(Or, first: anyone other wants something like this?)

@mkoeppe mkoeppe removed this from the sage-7.4 milestone Dec 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants