-
Notifications
You must be signed in to change notification settings - Fork 374
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
Fix integration tests #327
Conversation
@@ -8,5 +8,8 @@ then | |||
mysql -u root "$@" -e 'CREATE DATABASE test;' | |||
mysql -u root "$@" -e "GRANT ALL ON test.* TO 'test'@'localhost' IDENTIFIED BY 'zaphod';" | |||
mysql -u root "$@" -D test < storage/mysql/storage.sql | |||
#TODO(codeinglama): remove when db connection check doesn't ping log 0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spelt @codingllama
@@ -8,5 +8,8 @@ then | |||
mysql -u root "$@" -e 'CREATE DATABASE test;' | |||
mysql -u root "$@" -e "GRANT ALL ON test.* TO 'test'@'localhost' IDENTIFIED BY 'zaphod';" | |||
mysql -u root "$@" -D test < storage/mysql/storage.sql | |||
#TODO(codeinglama): remove when db connection check doesn't ping log 0 | |||
mysql -u root "$@" -D test -e "INSERT INTO Trees VALUES (0,1,'LOG','SHA256','SHA256',false)" | |||
mysql -u root "$@" -D test -e "INSERT INTO TreeControl VALUES (0,false,false,false,1,1)" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could this be done by just invoking createlog.sh 0
?
Would you expect Travis to work yet? |
This resolves the row missing errors. Travis may still fail due to too many DB connections at once. |
Rebased on @codingllama 's #326 to include the DB connections fix so Travis will pass. |
- Creates connection test tree 0 on db reset - Creates trees for integration tests
Pull #328 seemed to fix things so has gone in.... |
This is a temporary hack to get the integration tests running again while #324 is being worked on.
Create databases before using them