Create separate base classes for shared tests

[mikealfare]

blocks dbt-labs/dbt-postgres#18

Problem

There are some tests in the "adapter zone" which do not have base classes. When these tests are imported, they run automatically due to how pytest works. This makes it impossible to configure the test if changes are required for a specific platform.

Solution

• Update the test class name to not start with "Test" (which is what triggers pytest to execute it)
• Create a new test class with the with same name that inherits the base class and does nothing for backwards compatibility

For example, this:

class TestFoo:
    def test_bar(self):
        assert baz == "baz"

becomes:

class BaseFoo:
    def test_bar(self):
        assert baz == "baz"


class TestFoo(BaseFoo):
    pass

Checklist

• I have read the contributing guide and understand what's expected of me
• I have run this code in development and it appears to resolve the stated issue
• This PR includes tests, or tests are not required/relevant for this PR
• This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

[github-actions]

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide.

[VersusFacit]

Couple nits

[mikealfare]

Couple nits

Good questions. I spent a good 15-30 minutes figuring out why the rename looked like it did. It turns out to be correct, but very misleading. So now we have the why captured in this PR.