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

tests package installed stupidly #69

Closed
djc opened this issue May 6, 2011 · 3 comments
Closed

tests package installed stupidly #69

djc opened this issue May 6, 2011 · 3 comments

Comments

@djc
Copy link

djc commented May 6, 2011

1.5.170 installs the tests in site-packages/tests. This is obviously not a good idea: at the very least, prone to collisions with some other packages. It seems to happen because setuptools.find_packages() correctly finds the tests package and then installs it.

I think there are two viable options of resolving this:

  • don't install the tests
  • move the tests into the oauth2 package

I'd be happy to contribute a patch, but I don't think that would be the sticky point here.

@joestump
Copy link
Owner

We now run tests via py.test and use virtualenv locally. Additionally, we have Travis-CI running.

@jaitaiwan
Copy link
Contributor

Specifically #173 should resolve this when merged.

@joestump
Copy link
Owner

@jaitaiwan Yeah, I think I totally missed what @djc was asking here. As you mentioned, resolved when #173 is merged.

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

No branches or pull requests

3 participants