aelitashen marked an inline comment as done.
aelitashen added inline comments.


================
Comment at: lldb/test/API/tools/lldb-vscode/module/TestVSCode_module.py:55-61
+            active_modules = self.vscode.get_active_modules()
+            program_module = active_modules[program_basename]
+            self.assertEqual(program_basename, program_module['name'])
+            self.assertEqual(program, program_module['path'])
+            self.assertIn('symbolFilePath', program_module)
+            self.assertEqual(symbols_path, program_module['symbolFilePath'])
+            self.assertIn('addressRange', program_module)
----------------
clayborg wrote:
> Unindent everything after the self.waitUntil()? Otherwise we are not testing 
> the program, symbolFilePath and addressRange on symbols with size.
When unindenting these codes, the dsym test for darwin fails as the symbol 
paths don't match. One is using dysm path, one is using a.out.path.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83731/new/

https://reviews.llvm.org/D83731



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to