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

Check whether test framework is installed #15

Closed
jitseniesen opened this issue Oct 22, 2016 · 6 comments
Closed

Check whether test framework is installed #15

jitseniesen opened this issue Oct 22, 2016 · 6 comments
Assignees
Milestone

Comments

@jitseniesen
Copy link
Member

so that we can give a good error message if it is not

@goanpeca
Copy link
Member

@jitseniesen since the plan for 4.0 is to have a more language agnostic Spyder, do you think it is worth making this a generic unit test widget? plus the one with Python support?

@jitseniesen
Copy link
Member Author

@jitseniesen since the plan for 4.0 is to have a more language agnostic Spyder, do you think it is worth making this a generic unit test widget?

Yes, at least at first thought. We'll have to see how it works out.

@jitseniesen
Copy link
Member Author

Need also to check for other dependencies, e.g. lxml

@goanpeca
Copy link
Member

Is the check like a safeguard in case someone uninstalled something by accident? cause the plugin installation should define the dependencies, no?

@jitseniesen
Copy link
Member Author

Good point. I haven't really thought about installation because I have never distributed Python modules. Regarding lxml, I guess it is indeed a safeguard. But there are also the test frameworks themselves (py.test, nose, unittest, ...). I don't want the user to have to install all the frameworks, so the plugin should check which ones are installed and prevent the user from running one that is not installed.

@goanpeca
Copy link
Member

Hmmm true, I can help on the packaging side ;-).

unittest comes with python, and nose is an extension to unitests, so I dont think it is a bad idea to install pytest and nose by default. I think it makes sense

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

2 participants