)]}'
{"playbooks/inventory/dynamic_inventory.py":[{"author":{"_account_id":18784,"name":"Michael Carden","email":"mike.carden@gmail.com","username":"mcarden"},"change_message_id":"55507d92e47696756d2f5663e8b7c107f0a379be","unresolved":false,"context_lines":[{"line_number":899,"context_line":""},{"line_number":900,"context_line":""},{"line_number":901,"context_line":"def load_environment(config_path):"},{"line_number":902,"context_line":"    \"\"\"Create an environment dictionary from config files"},{"line_number":903,"context_line":"    :param config_path: ``str``path where the environment files are kept"},{"line_number":904,"context_line":"    \"\"\""},{"line_number":905,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"5a18252c_bdfa68bb","line":902,"updated":"2016-04-14 00:05:02.000000000","message":"I think the linter wants you to put a blank line after this summary.","commit_id":"b5b26ceb10247400256121c9c5c04a359d166aac"}],"tests/test_inventory.py":[{"author":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"change_message_id":"65ffc148ba5be55afa271e16d7e7324561300751","unresolved":false,"context_lines":[{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    def test_loading_environment(self):"},{"line_number":56,"context_line":"        \"\"\"Test that the environment file can be loaded\"\"\""},{"line_number":57,"context_line":"        env_dict \u003d di.load_environment(TARGET_DIR, self.env_file)"},{"line_number":58,"context_line":"        self.assertIsInstance(env_dict, dict)"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    def test_envd_read(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"5aef4532_173dc3dc","line":57,"updated":"2016-03-10 19:54:59.000000000","message":"You could move this into setUp since its shared between the test methods","commit_id":"be04831aa8951f9aac23ea8b5769ed9e9bd2fcd7"},{"author":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"change_message_id":"65ffc148ba5be55afa271e16d7e7324561300751","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    def test_envd_read(self):"},{"line_number":61,"context_line":"        \"\"\"Test that the env.d contents are inserted into the environment"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"        This test is more of a \u0027spot check\u0027, rather than completely verifying"},{"line_number":64,"context_line":"        the entire structure."},{"line_number":65,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5aef4532_d7287a88","line":62,"updated":"2016-03-10 19:54:59.000000000","message":"Can we assert on the presence of component_skel, container_skel and physical_skel and that they are all non-empty as basic structural test","commit_id":"be04831aa8951f9aac23ea8b5769ed9e9bd2fcd7"}]}