)]}'
{"glance/tests/unit/test_data_migration_framework.py":[{"author":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"change_message_id":"527c56bc798eb06eafbc240564da0a903594f173","unresolved":false,"context_lines":[{"line_number":173,"context_line":"        actual \u003d data_migrations._run_migrations(mock.Mock(), migrations)"},{"line_number":174,"context_line":"        self.assertEqual(0, actual)"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"    @mock.patch(\u0027glance.db.migration.CURRENT_RELEASE\u0027, \u0027ocata\u0027)"},{"line_number":177,"context_line":"    @mock.patch(\u0027importlib.import_module\u0027)"},{"line_number":178,"context_line":"    @mock.patch(\u0027pkgutil.iter_modules\u0027)"},{"line_number":179,"context_line":"    def test_migrate(self, mock_iter, mock_import):"}],"source_content_type":"text/x-python","patch_set":1,"id":"da36d5c6_34f819c3","line":176,"range":{"start_line":176,"start_character":4,"end_line":176,"end_character":63},"updated":"2017-02-24 16:22:47.000000000","message":"This looks confusing from a developer perspective. I think we\u0027d either rename the migrations so that they don\u0027t look like real world ones or let\u0027s put a NOTE / docstring that explains how the tests.","commit_id":"7e805bc7740c3b9f2bb4cd4f761d17a37b302b5f"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"192902dc20b4a011003f9e272e9dacd297c41f92","unresolved":false,"context_lines":[{"line_number":173,"context_line":"        actual \u003d data_migrations._run_migrations(mock.Mock(), migrations)"},{"line_number":174,"context_line":"        self.assertEqual(0, actual)"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"    @mock.patch(\u0027glance.db.migration.CURRENT_RELEASE\u0027, \u0027ocata\u0027)"},{"line_number":177,"context_line":"    @mock.patch(\u0027importlib.import_module\u0027)"},{"line_number":178,"context_line":"    @mock.patch(\u0027pkgutil.iter_modules\u0027)"},{"line_number":179,"context_line":"    def test_migrate(self, mock_iter, mock_import):"}],"source_content_type":"text/x-python","patch_set":1,"id":"da36d5c6_0bec948e","line":176,"range":{"start_line":176,"start_character":4,"end_line":176,"end_character":63},"in_reply_to":"da36d5c6_34f819c3","updated":"2017-02-27 17:43:35.000000000","message":"I\u0027ll rename the migrations.  Holding off on the docstring, I think the renaming will be sufficient.","commit_id":"7e805bc7740c3b9f2bb4cd4f761d17a37b302b5f"}]}