)]}'
{"cue/tests/base.py":[{"author":{"_account_id":13771,"name":"Davide Agnello","email":"dagnello@hp.com","username":"dagnello"},"change_message_id":"a7a5406987880c23603a365ec5076182d4ae90b8","unresolved":false,"context_lines":[{"line_number":30,"context_line":"from cue.common import context as cue_context"},{"line_number":31,"context_line":"from cue.db.sqlalchemy import api as db_api"},{"line_number":32,"context_line":"from cue.manage import database"},{"line_number":33,"context_line":"from cue.tests.test_fixtures import database as database_fixture"},{"line_number":34,"context_line":"from cue.tests.test_fixtures import policy"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"from oslo.config import cfg"}],"source_content_type":"text/x-python","patch_set":1,"id":"9aa53dc9_64780d90","line":33,"updated":"2015-04-08 20:57:37.000000000","message":"I think the name of fixtures directory should just be fixtures vs. test_fixtures since we are already in the test module","commit_id":"887cda45e1cc5bd4e525993d8e33223c09f98efc"},{"author":{"_account_id":10584,"name":"Min Pae","username":"sputnik13","inactive":true},"change_message_id":"7164a8a9b4e49deef45a87dcd753412f65497471","unresolved":false,"context_lines":[{"line_number":30,"context_line":"from cue.common import context as cue_context"},{"line_number":31,"context_line":"from cue.db.sqlalchemy import api as db_api"},{"line_number":32,"context_line":"from cue.manage import database"},{"line_number":33,"context_line":"from cue.tests.test_fixtures import database as database_fixture"},{"line_number":34,"context_line":"from cue.tests.test_fixtures import policy"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"from oslo.config import cfg"}],"source_content_type":"text/x-python","patch_set":1,"id":"9aa53dc9_2b1bcdcb","line":33,"in_reply_to":"9aa53dc9_27b8c7e4","updated":"2015-04-09 17:31:59.000000000","message":"cue.tests.base shouldn\u0027t include the fixtures module, the current base assumed all tests use a set of default fixtures, these should be moved to a separate fixture under cue/tests/fixtures and that specific fixture should be imported by cue.tests.base if a common set of fixtures are used by all tests.","commit_id":"887cda45e1cc5bd4e525993d8e33223c09f98efc"},{"author":{"_account_id":13771,"name":"Davide Agnello","email":"dagnello@hp.com","username":"dagnello"},"change_message_id":"3f14141c8196bff6cf0c7d0347cdd92fd8ee7513","unresolved":false,"context_lines":[{"line_number":30,"context_line":"from cue.common import context as cue_context"},{"line_number":31,"context_line":"from cue.db.sqlalchemy import api as db_api"},{"line_number":32,"context_line":"from cue.manage import database"},{"line_number":33,"context_line":"from cue.tests.test_fixtures import database as database_fixture"},{"line_number":34,"context_line":"from cue.tests.test_fixtures import policy"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"from oslo.config import cfg"}],"source_content_type":"text/x-python","patch_set":1,"id":"9aa53dc9_27b8c7e4","line":33,"in_reply_to":"9aa53dc9_64648d48","updated":"2015-04-08 21:44:22.000000000","message":"ok, I see that","commit_id":"887cda45e1cc5bd4e525993d8e33223c09f98efc"},{"author":{"_account_id":5390,"name":"Steve Leon","username":"steve-leon","inactive":true},"change_message_id":"d9c132929870d71c7371f0d5b7a5f0d7bb211819","unresolved":false,"context_lines":[{"line_number":30,"context_line":"from cue.common import context as cue_context"},{"line_number":31,"context_line":"from cue.db.sqlalchemy import api as db_api"},{"line_number":32,"context_line":"from cue.manage import database"},{"line_number":33,"context_line":"from cue.tests.test_fixtures import database as database_fixture"},{"line_number":34,"context_line":"from cue.tests.test_fixtures import policy"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"from oslo.config import cfg"}],"source_content_type":"text/x-python","patch_set":1,"id":"9aa53dc9_64648d48","line":33,"in_reply_to":"9aa53dc9_64780d90","updated":"2015-04-08 21:16:43.000000000","message":"I tried doing this but there is already a module named \u0027fixture\u0027 so it gets confused when i try to import it.","commit_id":"887cda45e1cc5bd4e525993d8e33223c09f98efc"}],"cue/tests/test_fixtures/database.py":[{"author":{"_account_id":1925,"name":"Vipul Sabhaya","email":"vipuls@gmail.com","username":"vipuls"},"change_message_id":"282a094ea991b6c03f7e24d3f2f0f3efd0a87742","unresolved":false,"context_lines":[{"line_number":12,"context_line":"# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":13,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"# under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"import os"},{"line_number":17,"context_line":"import shutil"},{"line_number":18,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9aa53dc9_d13e508a","line":15,"updated":"2015-04-09 18:08:24.000000000","message":"Echoing Davide\u0027s comment.. Can we rename this module to \u0027fixtures\u0027 from \u0027test_fixtures\u0027","commit_id":"887cda45e1cc5bd4e525993d8e33223c09f98efc"}]}