fix: update nvim-dap-ruby strategy config

[cpb]

Why?

I get an error message:

Config needs the request property which must be one of attach or launch

out-of-the-box.mov

What?

1. References nvim-dap-ruby's base ruby adapter configurations to resolve the above error message by adding request = "attach", to the return of get_strategy_config
2. Suppresses exit code 1 notices from nvim-dap-ruby by configuring error_on_failure = false
3. Copies the localfs = true, and waiting = 1000 configuration to match the nvim-dap-ruby's base run configuration

Demo

1. Setting a breakpoint
2. Running neotest with dap strategy
3. nvim-dap-ui pops up
4. Continue from the break point
5. Observe test failure

neotest-rspec-dap-strategy-demo.mov

Possible with the following Lazy configuration:

return {
  {
    "nvim-neotest/neotest",
    dependencies = {
      {
        "cpb/neotest-rspec",
        branch = "cpb/update-dap-strategy",
      },
      "nvim-neotest/nvim-nio",
      "nvim-lua/plenary.nvim",
      "antoinemadec/FixCursorHold.nvim",
      "nvim-treesitter/nvim-treesitter",
    },
    config = function()
      require("neotest").setup({
        adapters = {
          require("neotest-rspec")({
            rspec_cmd = function()
              return vim.tbl_flatten({
                "bundle",
                "exec",
                "rdbg",
                "--nonstop",
                "-c",
                "--",
                "rspec",
              })
            end,
          }),
        },
      })
    end,
  },
}

[elken]

Do we want to add documentation to clarify the configuration options for getting debug to run?

[smarquez1]

@elken I think it's a good idea!

[olimorris]

Hi All. Apologies for missing this first time around. Does this require any additional updates to work? Can't say I use nvim-dap-ruby these days.