-
Notifications
You must be signed in to change notification settings - Fork 3
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
Alyx needs a test suite #41
Comments
For ibllib the tests are run against a alyx test database that is on an open aws instance. You can have a look here: This has the advantage of having portable integration tests. |
Great, is the code always the same as the dev? |
Also is it repopulated like alyx-dev or do posts remain there for good? |
@oliche One issue I'm having is that my newExp test fails when using the IBL test instance. I believe this is because base sessions are not permitted on the IBL Alyx?
|
So every night the database is re-created from input fixtures to its original state. So no worries about creating/patching/deleting stuff. For the second thing the code is thoroughly identical to the cortexlab Alyx. In this case there wasn't the init fixtures |
Thorough tests required, particularly as the database changes move quite quickly.
The following required adequate tests:
The text was updated successfully, but these errors were encountered: