)]}'
{"trove_tempest_plugin/tests/api/database/datastores/test_datastores.py":[{"author":{"_account_id":27733,"name":"Krzysztof Opasiak","email":"k.opasiak@samsung.com","username":"kopasiak"},"change_message_id":"5f2adaebe631d9e3e3c67f2c4d279d01ff34a8ba","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    @decorators.idempotent_id(\u0027e4cdcadf-51bc-41ec-8cc6-530a3da08d10\u0027)"},{"line_number":31,"context_line":"    def test_datastores(self):"},{"line_number":32,"context_line":"        datastores \u003d self.client.list_db_datastores()[\u0027datastores\u0027]"},{"line_number":33,"context_line":"        self.assertTrue(len(datastores) \u003e 0, \"No available datastores found\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_56e2e4a1","line":33,"updated":"2018-05-10 19:21:07.000000000","message":"hmmm I\u0027m not sure if we should fail the test because there is no datastores at all. On the other hand why should we pass if there is more than 0?\n\nI think that user should be able to configure what is the expected result for this test on his cloud instance and the we should compare the result of that command with the expected result","commit_id":"90fe8eecaebd954fae71217efdeb4ded872d4fe3"}],"trove_tempest_plugin/tests/api/database/flavors/test_flavors.py":[{"author":{"_account_id":27733,"name":"Krzysztof Opasiak","email":"k.opasiak@samsung.com","username":"kopasiak"},"change_message_id":"5f2adaebe631d9e3e3c67f2c4d279d01ff34a8ba","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":"from tempest.common import utils"},{"line_number":17,"context_line":"from tempest.lib import decorators"},{"line_number":18,"context_line":"from testtools import testcase as testtools"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_02166ba9","line":15,"updated":"2018-05-10 19:21:07.000000000","message":"changes in this file seems to be unrelated","commit_id":"90fe8eecaebd954fae71217efdeb4ded872d4fe3"}],"trove_tempest_plugin/tests/scenario/database/test_basic_instance_ops.py":[{"author":{"_account_id":27733,"name":"Krzysztof Opasiak","email":"k.opasiak@samsung.com","username":"kopasiak"},"change_message_id":"5f2adaebe631d9e3e3c67f2c4d279d01ff34a8ba","unresolved":false,"context_lines":[{"line_number":29,"context_line":"                \"users\": ["},{"line_number":30,"context_line":"                    {"},{"line_number":31,"context_line":"                        \"password\": \"tempest_password\","},{"line_number":32,"context_line":"                        \"name\": \"tempest_username\","},{"line_number":33,"context_line":"                        \"databases\": ["},{"line_number":34,"context_line":"                            {"},{"line_number":35,"context_line":"                                \"name\": \"tempest_test\""}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_a2545754","line":32,"updated":"2018-05-10 19:21:07.000000000","message":"I\u0027m not a specialist but usually in sth called \"users\" I\u0027d expect first field to be user name not password","commit_id":"90fe8eecaebd954fae71217efdeb4ded872d4fe3"}]}