[Lldb-commits] [lldb] [lldb-dap] Adding additional asserts to unit tests. (PR #140107)

[John Harrison via lldb-commits]

================
@@ -67,7 +67,7 @@ def test_core_file_source_mapping_array(self):
         self.create_debug_adapter()
 
         source_map = [["/home/labath/test", current_dir]]
-        self.attach(exe_file, coreFile=core_file, sourceMap=source_map)
+        self.attach(program=exe_file, coreFile=core_file, sourceMap=source_map)
----------------
ashgti wrote:

For `attach` and `launch`, I wanted them to require kwargs because they have so many arguments and almost all of them are optional. `launch` I only left `program` as a positional arg for the simplicity of `self.launch(program)` but the rest, I was using the syntax that requires kwargs.

https://github.com/llvm/llvm-project/pull/140107