Skip to content

BLD/TST/CI: add script to show skipped tests at the end of a build #4262

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

Merged
merged 1 commit into from
Jul 16, 2013
Merged

BLD/TST/CI: add script to show skipped tests at the end of a build #4262

merged 1 commit into from
Jul 16, 2013

Conversation

cpcloud
Copy link
Member

@cpcloud cpcloud commented Jul 16, 2013

closes #4261.

@ghost ghost assigned cpcloud Jul 16, 2013
@cpcloud
Copy link
Member Author

cpcloud commented Jul 16, 2013

also shows skips grouped by class where applicable, for easier reading

@jreback
Copy link
Contributor

jreback commented Jul 16, 2013

nice!

now of course need a sscript to go find nose.SkipTest that don't have any explanation!

@jreback
Copy link
Contributor

jreback commented Jul 16, 2013

complete aside.....I think it might be nice to show off the travis build scripts to the travis guys...maybe they will put up as some examples...(e.g. with the wheel stuff and version printing and such)

@cpcloud
Copy link
Member Author

cpcloud commented Jul 16, 2013

this shows the skips and their message, and the ones that don't have a message have empty space next to the colon so no need for another script for that!

@cpcloud
Copy link
Member Author

cpcloud commented Jul 16, 2013

agree about the travis showcase...wonder if @y-p might want to do that?

@cpcloud
Copy link
Member Author

cpcloud commented Jul 16, 2013

i also need to make sure this isn't skipping function tests

@ghost
Copy link

ghost commented Jul 16, 2013

Not me, I prefer the lack of syndication. But feel free to get in touch with them,
they're really nice guys and their blog has picked up in recent months, they'll
probably be glad to point out the use of wheel files to reduce build time.

I think other pydata projects are hurting there, and making others aware that this is both
simple and effective would be a good service to the community.

Should give mad kudos to the pypa guys and the people behind the new PEPs who are
giving python packaging a much needed kick in the backside on several fronts.

@cpcloud
Copy link
Member Author

cpcloud commented Jul 16, 2013

@jreback i think my exception parsing script would tell you where there are empty SkipTest exceptions. you can pass a regex to search for a raise statement argument

@cpcloud
Copy link
Member Author

cpcloud commented Jul 16, 2013

@jreback u ok with this? it's ready to go

@jtratner
Copy link
Contributor

@cpcloud - this is great, I was mulling over the same thing, glad you added
it! Would it be possible to change this into a nose plugin? Not sure how
much work that would be, but it'd be great to use this for other projects
too. Might also make it less fragile (e.g., if location of tempfile
changes, etc)

On Tue, Jul 16, 2013 at 4:22 PM, Phillip Cloud [email protected]:

@jreback https://github.com/jreback u ok with this? it's ready to go


Reply to this email directly or view it on GitHubhttps://github.com//pull/4262#issuecomment-21070704
.

@jreback
Copy link
Contributor

jreback commented Jul 16, 2013

looks ok to me

@cpcloud
Copy link
Member Author

cpcloud commented Jul 16, 2013

@jtratner you're welcome to do that if you want! I don't really have time to do it ATM, could revisit later if you haven't done it by then.

cpcloud added a commit that referenced this pull request Jul 16, 2013
BLD/TST/CI: add script to show skipped tests at the end of a build
@cpcloud cpcloud merged commit a310890 into pandas-dev:master Jul 16, 2013
@cpcloud cpcloud deleted the print-skipped-tests branch July 16, 2013 20:31
@ghost
Copy link

ghost commented Jul 22, 2013

nice.

@ghost ghost mentioned this pull request Jul 22, 2013
@cpcloud
Copy link
Member Author

cpcloud commented Jul 22, 2013

@y-p thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Print skipped tests at the end of travis runs
3 participants