Avoid getfuncargvalue

getfuncargvalue is deprecated and raises warnings:
https://docs.pytest.org/en/latest/builtin.html#_pytest.fixtures.FixtureRequest.getfuncargvalue

We should stop using it and switch to getfixturevalue instead. getfixturevalue has been around for a while now and I think we can just change it.