why skip_if_no_vault_test_server

[maelle]

This function defined in a test helper file doesn't seem to be used anywhere or am I missing something?

[richfitz]

Looks like it is just stale; now we start the test server with

vault_test_server()

which includes a check for things like the server being present and the user having opted in to using it (details). The if_disabled argument to vault_test_server defaults to testthat::skip which does the actual skipping now.

That function can be deleted safely, will do that next time I do some maintenance on this package :)

[richfitz]

that function is now removed