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

Implement testing #19

Open
mbmilligan opened this issue May 7, 2018 · 1 comment
Open

Implement testing #19

mbmilligan opened this issue May 7, 2018 · 1 comment
Labels

Comments

@mbmilligan
Copy link
Member

We do not have any automated testing set up, but we should.

Testing will need to be focused on integration, not unit tests, since the major sources of error are going to be around property or method accesses to the wrapped Spawner class, which means running actual Jupyterhub code and verifying those interface points. We can probably pull the necessary logic from the Jupyterhub test suite.

@willingc willingc added the tests label May 23, 2018
@mbmilligan
Copy link
Member Author

Bumping issue. From discussion today in the Jupyterhub in HPC call:

  • Most useful part of batchspawner testing has been integration testing
  • Catches changes in jupyterhub that need to be addressed in batchspawner (e.g. changes to the ORM)
  • Interfaces are pretty stable but not fixed
  • WrapSpawner's use of interfaces from jupyterhub are kind of less documented so it may uncover more useful problems to solve
  • Unit tests would be ok but you need to focus on those special parts of the Hub interface
  • => Make an issue on wrapspawner (bumping this issue) about testing, using the batchspawner testing as a starting point, setting up wrapspawner to wrap a LocalProcessSpawner and hit it with tests that way

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

No branches or pull requests

2 participants