)]}'
{"doc/source/devref/testing.rst":[{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"46237653fe1c13f3176db9938e92fd6639453bde","unresolved":false,"context_lines":[{"line_number":11,"context_line":"Unit Tests"},{"line_number":12,"context_line":"~~~~~~~~~~"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Unit tests are tests for individual methods, with at most a small handful of"},{"line_number":15,"context_line":"modules involved. Mock should be used to remove any external dependencies."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"All significant code changes should have unit test coverage validating the code"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3ac371cc_38d3ea05","line":14,"updated":"2016-08-17 15:31:10.000000000","message":"Do we want to change this to \"a single module\"?","commit_id":"bdd2832b5cb2d511b01ff9f2752f6c5237685468"},{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"7133b019880273e6903787f20f7e7de638281583","unresolved":false,"context_lines":[{"line_number":11,"context_line":"Unit Tests"},{"line_number":12,"context_line":"~~~~~~~~~~"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Unit tests are tests for individual methods, with at most a small handful of"},{"line_number":15,"context_line":"modules involved. Mock should be used to remove any external dependencies."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"All significant code changes should have unit test coverage validating the code"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3ac371cc_ad979746","line":14,"in_reply_to":"3ac371cc_38d3ea05","updated":"2016-08-17 16:39:56.000000000","message":"I don\u0027t think that really makes sense as a hard-and-fast rule, since some modules or functions are meaningless without some interaction, and there\u0027s no point writing a complex mock for something that is already in the code, you\u0027re just making your life harder","commit_id":"bdd2832b5cb2d511b01ff9f2752f6c5237685468"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"46237653fe1c13f3176db9938e92fd6639453bde","unresolved":false,"context_lines":[{"line_number":32,"context_line":"set of code targeted for testing. Other than the expanded scope, these tests"},{"line_number":33,"context_line":"work the same way as unit tests. No services should be necessary to run in"},{"line_number":34,"context_line":"order to execute the tests."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"Functional Tests"},{"line_number":38,"context_line":"~~~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3ac371cc_784722bc","line":35,"updated":"2016-08-17 15:31:10.000000000","message":"Should we be explicit: no DB, no messageQ, no keystone?","commit_id":"bdd2832b5cb2d511b01ff9f2752f6c5237685468"},{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"7133b019880273e6903787f20f7e7de638281583","unresolved":false,"context_lines":[{"line_number":32,"context_line":"set of code targeted for testing. Other than the expanded scope, these tests"},{"line_number":33,"context_line":"work the same way as unit tests. No services should be necessary to run in"},{"line_number":34,"context_line":"order to execute the tests."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"Functional Tests"},{"line_number":38,"context_line":"~~~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3ac371cc_ed186f20","line":35,"in_reply_to":"3ac371cc_784722bc","updated":"2016-08-17 16:39:56.000000000","message":"We should definitely be explicit, since there\u0027s already been confusion here","commit_id":"bdd2832b5cb2d511b01ff9f2752f6c5237685468"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"46237653fe1c13f3176db9938e92fd6639453bde","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Functional tests run with a database present and may start Cinder services to"},{"line_number":45,"context_line":"accept requests. These tests should not need to access any other OpenStack"},{"line_number":46,"context_line":"non-Cinder services other than potentially keystone."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Tempest Tests"},{"line_number":49,"context_line":"~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3ac371cc_98313e5d","line":46,"updated":"2016-08-17 15:31:10.000000000","message":"What about messageQ ?","commit_id":"bdd2832b5cb2d511b01ff9f2752f6c5237685468"}]}