)]}'
{"spec/defines/cinder_backend_vmdk_spec.rb":[{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"622b87b1bec31c4bd5cb8f58779553d141532534","unresolved":false,"context_lines":[{"line_number":34,"context_line":"    is_expected.to contain_cinder_config(\u0027hippo/vmware_max_object_retrieval\u0027).with_value(100)"},{"line_number":35,"context_line":"    is_expected.to contain_cinder_config(\u0027hippo/vmware_task_poll_interval\u0027).with_value(5)"},{"line_number":36,"context_line":"    is_expected.to contain_cinder_config(\u0027hippo/vmware_image_transfer_timeout_secs\u0027).with_value(7200)"},{"line_number":37,"context_line":"    is_expected.to contain_cinder_config(\u0027hippo/host\u0027).with_value(\"vmdk:#{params[:host_ip]}-cinder-volumes\")"},{"line_number":38,"context_line":"    is_expected.to_not contain_cinder_config(\u0027hippo/vmware_wsdl_location\u0027)"},{"line_number":39,"context_line":"  end"},{"line_number":40,"context_line":""}],"source_content_type":"text/x-ruby","patch_set":2,"id":"3a50d1a3_73a7700b","line":37,"updated":"2015-07-28 08:11:58.000000000","message":"Shouldn\u0027t the optional_params[:volume_folder] to be used instead of  \u0027cinder-volumes\u0027 hardcode? It seems this test executed with defaults, ignoring customized optional_params (line #17). Please make sure you can test custom folder as appropriate.","commit_id":"370dcc420809b57bd3da21775f7ca5a1538b3f33"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"9efe780f09915f1847291b79e85d6aa590815e16","unresolved":false,"context_lines":[{"line_number":43,"context_line":"               :ensure \u003d\u003e \u0027present\u0027)"},{"line_number":44,"context_line":"  end"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"  context \u0027with optional parameters\u0027 do"},{"line_number":47,"context_line":"    before :each do"},{"line_number":48,"context_line":"      params.merge!(optional_params)"},{"line_number":49,"context_line":"    end"}],"source_content_type":"text/x-ruby","patch_set":2,"id":"3a50d1a3_331fa809","line":46,"updated":"2015-07-28 08:13:53.000000000","message":"So, please add a test for the \u0027hippo/host\u0027 here as well.","commit_id":"370dcc420809b57bd3da21775f7ca5a1538b3f33"}]}