Standard Tests Framework Rollout

[kgpayne]

In anticipation of sdk##1171 merging, this issue is to track the rollout of the new tests to commonly used Taps and Targets in MeltanoLabs:

Key:

• 🚫 Blocked
• ⚠️ In Progress
• ✅ Done

Taps:

• tap-gitlab - PR 🚫
• tap-snowflake - PR 🚫
• tap-github - PR ⚠️
• tap-stackexchange - PR ✅
• tap-google-analytics PR ✅
• tap-csv - PR

Targets:

• target-csv PR ⚠️
• target-snowflake PR ⚠️

[aaronsteers]

@kgpayne - Thanks very much for pre-testing here.

As I noted in the main SDK PR, one thing I wanted to check was to confirm that the old and new test methods both still work. I added back the legacy test with a test_sdk_legacy.py test file name on PR. Looking at the tap-github and tap-stackexchange, it looks like the old method is still in place (at least as of now). Safe to assume we expect legacy tests to keep working at the same time as we are lighting up the new test methods?

[edgarrmondragon]

Safe to assume we expect legacy tests to keep working at the same time as we are lighting up the new test methods?

Might be worth setting a deprecation timeline for the legacy tests. I've ported a docs reorganization from meltano/sdk#1294 into meltano/sdk#1353 that includes a deprecation page: https://meltano-sdk--1353.org.readthedocs.build/en/1353/development/deprecation.html

[aaronsteers]

@edgarrmondragon - Good point. I think in v1.0 we should remove them. I'll log I've logged this as an issue for follow-up as part of the 1.0 milestone.

• Remove the legacy "standard test" methods in v1.0 meltano/sdk#1354