Not for this PR, but the fixtures stuff should be reviewed, as this could probably just be a test helper that handles the skip itself, vs. having the:
if testutil.SkipIfNoAPIKey(providerName) {
t.Skip("FOO_API_KEY not set")
}
everywhere
Originally posted by @peteski22 in #26 (comment)