fix: nushell plugin list all (#1501)

[acidghost]

Summary

Fixes #1501.

[jthegedus]

@acidghost can you add a regression test to https://github.com/asdf-vm/asdf/blob/master/test/asdf_nu.bats ?

[acidghost]

What should it test? Check that the output of asdf plugin list all is the same for Nushell and bash in terms of repositories?

[jthegedus]

It should test the permutations of this regex you're changing to ensure that the command runs successfully for expected repositories.

[acidghost]

@jthegedus I think this is what you meant. To fully test this, there should be more types of test repository URLs (e.g. https, git, etc.). I can add these in another PR as it would require to modify most of the test cases to update the expected values.

[jthegedus]

Looks good, thanks!

[jthegedus]

@jthegedus I think this is what you meant. To fully test this, there should be more types of test repository URLs (e.g. https, git, etc.). I can add these in another PR as it would require to modify most of the test cases to update the expected values.

Yeah, just submit a follow up with some more test cases. There's probably a way to hack more values into just this test case.