[MAINT] Improve provider tests

[deiga]

Supersedes #3033

---

Before the change?

• It was possible to inadvertently set token and app_auth at the same time
• Some provider test's weren't actually testing anything

After the change?

• Ensures that token (or GITHUB_TOKEN) and app_auth can't be set at the same time
• Updated some provider tests to actually initialize the provider and verify changes

Pull request checklist

• Schema migrations have been created if needed (example)
• Tests for the changes have been added (for bug fixes / features)
• Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

• Yes
• No

---

[github-actions]

👋 Hi! Thank you for this contribution! Just to let you know, our GitHub SDK team does a round of issue and PR reviews twice a week, every Monday and Friday! We have a process in place for prioritizing and responding to your input. Because you are a part of this community please feel free to comment, add to, or pick up any issues/PRs that are labeled with Status: Up for grabs. You & others like you are the reason all of this works! So thank you & happy coding! 🚀

[stevehipwell]

This looks good. It might be worth adding support for app auth via env variable? This is something we require to better be able to run integration tests.

[deiga]

@stevehipwell Yeah, that could be good, but I don't think it's best to do in this PR. There is an open Feature request for that as well

[stevehipwell]

LGTM