Feature Request: Support for `.runsettings`

[samwcoulter]

I use .runsettings files in the project root directories to manage env vars, credentials, etc, for each project's integration tests.

Is there a way to get neotest-vstest to pass the appropriate --settings <path-to>/.runsettings argument to dotnet test?

Edit: I'd be happy to make a PR adding support for this if you could point me towards the right part of the code to inject the setting. neotest-dotnet Adds a nice little picker / selection UI/command that we could copy.

Edit2: Looks like MTP has switched config over to testconfig.json, so we would need to support both. I think I will try a PR adding support for runsettings/testconfig in the respective client code. Please let me know if you have any ideas/guidance.

[samwcoulter]

I have basic support for .runsettings working in #57

[sebastianstudniczek]

@samwcoulter After upgrading to 0.7 I got error when trying to run test.
Looks like when using empty (default) config, neotest_vstest_timeout_ms is not propagated and it's nil

[samwcoulter]

@sebastianstudniczek really? I'll take a look, should be a quick fix. Thanks for the report

[samwcoulter]

@sebastianstudniczek I'm unable to reproduce with latest on main, can you share your config? Here is how I am setting up neotest + vstest:

    {
        dir = "/home/sam/dev/neotest-vstest",
        name = "neotest-vstest",
        dev = true,
    },
    {
        'nvim-neotest/neotest',
        dependencies = {
            -- 'Nsidorenco/neotest-vstest',
            -- 'samwcoulter/neotest-vstest',
            'neotest-vstest',
            'mfussenegger/nvim-dap',
            'nvim-treesitter/nvim-treesitter',
        },
        config = function()
            local neotest = require('neotest')
            ---@diagnostic disable: missing-fields
            neotest.setup({
                adapters = {
                    require('neotest-vstest')(),
                },
            })
            ---@diagnostic enable: missing-fields
        end,
    },

Or, what symptoms are you seeing?

[sebastianstudniczek]

@samwcoulter Ok, looks like the issue was with my config, I was not calling neotest-vstest module, just defined it like in README without options. Didn't expect earlier any issues that's why I tought it's a problem with this new option. I was getting

return {
  {
    "nsidorenco/neotest-vstest",
  },
  {
    "nvim-neotest/neotest",
    opts = {
      adapters = {
        -- ["neotest-vstest"] = { },
        require("neotest-vstest"),
      },
      floating = {
        border = "rounded",
      },
    },
  },
}

After changing it to require("neotest-vstest")(), it started working. I tought it will work like a "normal" package - meaning don't have to call setup/pass opts to get default values but looks like with neotest adapters it's different

[Nsidorenco]

You shouldn't need to do it like that @sebastianstudniczek. I think the problem was that the default timeout settings were only applied when the adapter was initialized with a config, rather than always having a default value.

I've tried to unify the settings logic a little in #65. Please let me know if it fixes it for you