https://codereview.appspot.com/51230045/diff/20001/tests/10_basic_deploy_test.py#newcode14
tests/10_basic_deploy_test.py:14: file_path = os.path.join(path,
'rabbit-server-privkey.pem')
You're sending SSL certs but never verify if they're actually working.
It'd be nice if this test also attempted an SSL connection to the
rabbitmq server and verify the certs were installed and setup properly
https://codereview.appspot.com/51230045/diff/20001/tests/30_configuration_test.py#newcode54
tests/30_configuration_test.py:54: message = 'The certificate did not
match!'
Simply being able to check one of the several configuration options
isn't enough for a complete test. You should get the contents of the
configuration file (or if possible, query rabbitmq directly) to verify
the charm is doing the right thing
Great start, see my comments below
https:/ /codereview. appspot. com/51230045/ diff/20001/ tests/10_ basic_deploy_ test.py basic_deploy_ test.py (right):
File tests/10_
https:/ /codereview. appspot. com/51230045/ diff/20001/ tests/10_ basic_deploy_ test.py# newcode14 basic_deploy_ test.py: 14: file_path = os.path.join(path, server- privkey. pem')
tests/10_
'rabbit-
You're sending SSL certs but never verify if they're actually working.
It'd be nice if this test also attempted an SSL connection to the
rabbitmq server and verify the certs were installed and setup properly
https:/ /codereview. appspot. com/51230045/ diff/20001/ tests/20_ deploy_ relations_ test.py deploy_ relations_ test.py (right):
File tests/20_
https:/ /codereview. appspot. com/51230045/ diff/20001/ tests/20_ deploy_ relations_ test.py# newcode39 deploy_ relations_ test.py: 39: unit['rabbitmq- server/ 0'].relation( 'ceph', 'ceph:client')
tests/20_
d.sentry.
While testing the ceph relation is important, you should also test the
amqp relation, as that's the primary purpose of this charm
https:/ /codereview. appspot. com/51230045/ diff/20001/ tests/30_ configuration_ test.py configuration_ test.py (right):
File tests/30_
https:/ /codereview. appspot. com/51230045/ diff/20001/ tests/30_ configuration_ test.py# newcode54 configuration_ test.py: 54: message = 'The certificate did not
tests/30_
match!'
Simply being able to check one of the several configuration options
isn't enough for a complete test. You should get the contents of the
configuration file (or if possible, query rabbitmq directly) to verify
the charm is doing the right thing
https:/ /codereview. appspot. com/51230045/