)]}'
{"test/probe/test_dark_data.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"de95bfa7ba89fd612b4e6646542a4bd3f2dc11c8","unresolved":true,"context_lines":[{"line_number":177,"context_line":"            self.assertTrue(os.path.exists(quarantine_dir))"},{"line_number":178,"context_line":"            parts \u003d file_path.split(os.path.sep)"},{"line_number":179,"context_line":"            self.assertPathExists(os.path.sep.join("},{"line_number":180,"context_line":"                parts[:4] + [\u0027quarantined\u0027]))"},{"line_number":181,"context_line":"            self.assertPathExists(os.path.sep.join("},{"line_number":182,"context_line":"                parts[:4] + [\u0027quarantined\u0027, parts[4]] + parts[-2:]))"},{"line_number":183,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"4a601326_71ea63b0","line":180,"range":{"start_line":180,"start_character":23,"end_line":180,"end_character":24},"updated":"2020-12-11 01:20:47.000000000","message":"Looks like this sometimes needs to be 5...","commit_id":"55832cd03884ea2839a7b35d2d9be5b99990a673"},{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"fc22fcdae3ea62d443c45b442c36183b807559fa","unresolved":true,"context_lines":[{"line_number":177,"context_line":"            self.assertTrue(os.path.exists(quarantine_dir))"},{"line_number":178,"context_line":"            parts \u003d file_path.split(os.path.sep)"},{"line_number":179,"context_line":"            self.assertPathExists(os.path.sep.join("},{"line_number":180,"context_line":"                parts[:4] + [\u0027quarantined\u0027]))"},{"line_number":181,"context_line":"            self.assertPathExists(os.path.sep.join("},{"line_number":182,"context_line":"                parts[:4] + [\u0027quarantined\u0027, parts[4]] + parts[-2:]))"},{"line_number":183,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"760128b9_0962260a","line":180,"range":{"start_line":180,"start_character":23,"end_line":180,"end_character":24},"in_reply_to":"4a601326_71ea63b0","updated":"2020-12-11 03:31:42.000000000","message":"I do not understand what you\u0027re trying to accomplish here.\nIf you want to make sure that the object was moved to the quarantine directory, then quarantine_dir is it. And then, you append the hash and filename to it. There\u0027s no need to try and guess the length of the prefix.","commit_id":"55832cd03884ea2839a7b35d2d9be5b99990a673"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"273d25b059d955dea569d117de3f60a66b7d6903","unresolved":true,"context_lines":[{"line_number":150,"context_line":"    def check_on_disk_files(self, files):"},{"line_number":151,"context_line":"        for file_path in files:"},{"line_number":152,"context_line":"            # File\u0027s not there"},{"line_number":153,"context_line":"            self.assertFalse(os.path.exists(file_path))"},{"line_number":154,"context_line":"            # And it\u0027s not quaratined, either!"},{"line_number":155,"context_line":"            self.assertPathDoesNotExist(os.path.join("},{"line_number":156,"context_line":"                file_path[:file_path.index(\u0027objects\u0027)], \u0027quarantined\u0027))"}],"source_content_type":"text/x-python","patch_set":4,"id":"59880010_fba7acfa","line":153,"range":{"start_line":153,"start_character":17,"end_line":153,"end_character":28},"updated":"2020-12-11 19:18:20.000000000","message":"Why\u0027d I write the helper if I\u0027m not gonna *use* it? Silly Tim...","commit_id":"d199804a717c668b09f6db984e98d26618a27205"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"273d25b059d955dea569d117de3f60a66b7d6903","unresolved":true,"context_lines":[{"line_number":173,"context_line":"            self.assertPathDoesNotExist(file_path)"},{"line_number":174,"context_line":"            # Got quarantined"},{"line_number":175,"context_line":"            parts \u003d file_path.split(os.path.sep)"},{"line_number":176,"context_line":"            quarantine_dir \u003d parts[:parts.index(\u0027objects\u0027)] + [\u0027quarantined\u0027]"},{"line_number":177,"context_line":"            quarantine_path \u003d os.path.sep.join("},{"line_number":178,"context_line":"                quarantine_dir + [\u0027objects\u0027] + parts[-2:])"},{"line_number":179,"context_line":"            self.assertPathExists(quarantine_path)"}],"source_content_type":"text/x-python","patch_set":4,"id":"570b09b6_a0b87816","line":176,"range":{"start_line":176,"start_character":49,"end_line":176,"end_character":56},"updated":"2020-12-11 19:18:20.000000000","message":"What if your replicated policy is policy-1? I was being a little sneaky in the previous test, where the policy index doesn\u0027t matter...","commit_id":"d199804a717c668b09f6db984e98d26618a27205"}]}