)]}'
{"spec/nova-setup_spec.rb":[{"author":{"_account_id":11915,"name":"Jan Klare","email":"klare@osism.tech","username":"jklare"},"change_message_id":"61c6d4a9af9c458c7033854acb43ed5855eb5067","unresolved":false,"context_lines":[{"line_number":12,"context_line":"    include_examples \u0027expect_runs_nova_common_recipe\u0027"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"    it \u0027runs db migrations\u0027 do"},{"line_number":15,"context_line":"      expect(chef_run).to run_execute(\u0027nova-manage db sync\u0027).with(user: \u0027nova\u0027, group: \u0027nova\u0027)"},{"line_number":16,"context_line":"    end"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"    it \u0027runs db migrations with timeout\u0027 do"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"3a961159_773b30d1","line":15,"updated":"2014-12-19 10:55:53.000000000","message":"IMO the timeout should be tested here to avoid duplicating code (since its just one resource with some parameters). Also you could just test the default value 3600 from the attributes file.","commit_id":"393d9f9738411b5d02324d69803e29b91f551186"},{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"c9bd82ba1914c68b49834b479bd737c2d19d9875","unresolved":false,"context_lines":[{"line_number":12,"context_line":"    include_examples \u0027expect_runs_nova_common_recipe\u0027"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"    it \u0027runs db migrations\u0027 do"},{"line_number":15,"context_line":"      expect(chef_run).to run_execute(\u0027nova-manage db sync\u0027).with(user: \u0027nova\u0027, group: \u0027nova\u0027)"},{"line_number":16,"context_line":"    end"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"    it \u0027runs db migrations with timeout\u0027 do"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"3a961159_39ef56eb","line":15,"in_reply_to":"3a961159_773b30d1","updated":"2014-12-27 16:24:54.000000000","message":"ok, will add the tests in two parts, the default and the override. I believe that is the most common way to do it.","commit_id":"393d9f9738411b5d02324d69803e29b91f551186"}]}